-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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.
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
Fix bug in CLI Output if user has overridden CHE_HOST #3749
Conversation
info "${CHE_PRODUCT_NAME}_HOST = ${CHE_HOST}" | ||
local CHE_HOST_LOCAL=${CHE_HOST} | ||
if is_initialized; then | ||
CHE_HOST_LOCAL=$(docker_run --env-file="${REFERENCE_CONTAINER_ENVIRONMENT_FILE}" \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe create function that retrieves variable by name? It is possible that other variables in different places will be also needed at some point
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, that is a good idea. Let me think about how to do such a thing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sweeet!
Build success. https://ci.codenvycorp.com/job/che-pullrequests-build/1649/ |
* added utility function
What does this PR do?
Fixes issue uncovere by @garagatyi where the CLI output in two locations shows the wrong CHE_HOST value if the user has overridden CHE_HOST in the che.env file.
This implements a printout mechanism that uses Docker to print out the value and passes in
che.env
as an input into that.