Skip to content
This repository was archived by the owner on Apr 17, 2023. It is now read-only.

Moved customizable env variables to .env file #1366

Merged
merged 1 commit into from
Aug 4, 2017
Merged

Conversation

soedar
Copy link
Contributor

@soedar soedar commented Aug 3, 2017

See #1360

Copy link
Collaborator

@mssola mssola left a comment

Choose a reason for hiding this comment

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

LGTM. I just have one doubt about it.


# Since we have no nginx in insecure mode, portus have to
# serve the static files
RAILS_SERVE_STATIC_FILES: 'true'
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why have you turned this into a string ? Same in the other file

@soedar
Copy link
Contributor Author

soedar commented Aug 4, 2017

It is due to a quirk in the yaml file. YAML file by default will turn a set of boolean value into "True"/"False", and docker-compose will warn (and possibly error) if there is a boolean value in the YAML. See docker/compose#1788

It was okay previous because the environment variables was passed in as an array of environmental strings, which gets interpreted directly. The issue arises when we use the object syntax to pass in env variables.

PS: I realized that I've missed out changing PORTUS_CHECK_SSL_USAGE_ENABLED to use a string instead. Will update this PR 😅

Signed-off-by: Soedarsono <soedar.sg@gmail.com>
Copy link
Collaborator

@mssola mssola left a comment

Choose a reason for hiding this comment

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

LGTM

@mssola mssola merged commit 4e4329b into SUSE:master Aug 4, 2017
@mssola
Copy link
Collaborator

mssola commented Aug 4, 2017

Thanks for taking care of this 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants