Skip to content

Commit

Permalink
Remove debug and add comments
Browse files Browse the repository at this point in the history
  • Loading branch information
badsyntax committed Dec 18, 2020
1 parent af52162 commit 8fde7be
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 6 deletions.
3 changes: 2 additions & 1 deletion .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,4 +31,5 @@ jobs:
- name: Run tests
run: sudo make test
env:
HOSTNAME: "an-engine.com"
# "an-engine" is used to test the env var parsing logic (-e), see issue #13
HOSTNAME: "an-engine.discourse.dokku.me"
6 changes: 1 addition & 5 deletions internal-functions
Original file line number Diff line number Diff line change
Expand Up @@ -191,16 +191,12 @@ fn-configure-app() {
dokku_log_info1 "Generating docker config..."

local RUN_ARGS MAC_ADDRESS SHM_SIZE ENV_VARS
RUN_ARGS=$(fn-get-run-args "$APP_NAME" 2>&1)

echo "# RUN_ARGS: $RUN_ARGS"

RUN_ARGS=$(fn-get-run-args "$APP_NAME" 2>&1)
MAC_ADDRESS=$(echo "$RUN_ARGS" | sed -r 's/.*--mac-address(=| )([^ ]+).*/\2/')
SHM_SIZE=$(echo "$RUN_ARGS" | sed -r 's/.*--shm-size(=| )([^ ]+).*/\2/')
ENV_VARS=$(fn-get-env-vars-from-run-args "$RUN_ARGS")

echo "# ENV_VARS: $ENV_VARS"

domains_set "$APP_NAME" "$HOSTNAME"

## TODO: call plugin functions instead of dokku
Expand Down

0 comments on commit 8fde7be

Please sign in to comment.