Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

decrease optional var logging level and allow to pass extra opts to pavics-compose.sh up -d #426

Merged
merged 11 commits into from
Mar 6, 2024
Merged

decrease optional var logging level and allow to pass extra opts to pavics-compose.sh up -d #426

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
d69088b
logging: decrease logging level for empty optional vars from WARN to …
tlvu Feb 23, 2024
7567e8d
compose script: allow to pass extra options to `up` operation
tlvu Feb 23, 2024
bfd402d
docs: fix typo and correctness (review feedback from mishaschwartz)
tlvu Feb 24, 2024
dead9ab
Revert "compose script: allow to pass extra options to `up` operation"
tlvu Feb 27, 2024
4757ed0
config: add sample config to configure docker-compose to remove orphans
tlvu Feb 27, 2024
21ace57
Revert "Revert "compose script: allow to pass extra options to `up` o…
tlvu Feb 28, 2024
5870611
compose script: exit early when any errors occurred during invocation
tlvu Feb 28, 2024
5cf9838
compose script: wrong order for COMPOSE_EXTRA_OPTS (mishaschwartz fee…
tlvu Feb 28, 2024
e0a776f
compose script: update log to match COMPOSE_EXTRA_OPTS ordering (mish…
tlvu Feb 28, 2024
6c28a86
Merge remote-tracking branch 'origin/master' into fix-optional-vars-l…
tlvu Mar 6, 2024
77926f2
Bump version: 2.1.0 → 2.1.1
tlvu Mar 6, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion birdhouse/pavics-compose.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,7 @@ fi

log INFO "Executing docker-compose with extra options: ${COMPOSE_EXTRA_OPTS} $*"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And we should switch the log message to match too

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh geez ! Thanks !!!

# the PROXY_SECURE_PORT is a little trick to make the compose file invalid without the usage of this wrapper script
PROXY_SECURE_PORT=443 HOSTNAME=${PAVICS_FQDN} docker-compose ${COMPOSE_CONF_LIST} ${COMPOSE_EXTRA_OPTS} $*
PROXY_SECURE_PORT=443 HOSTNAME=${PAVICS_FQDN} docker-compose ${COMPOSE_CONF_LIST} $* ${COMPOSE_EXTRA_OPTS}
ERR=$?
if [ ${ERR} -gt 0 ]; then
log ERROR "docker-compose error, exit code ${ERR}"
Expand Down
Loading