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

Logging of EDGEX_CONF_DIR override is incorrect #264

Closed
lenny-goodell opened this issue Jul 20, 2021 · 0 comments · Fixed by #266
Closed

Logging of EDGEX_CONF_DIR override is incorrect #264

lenny-goodell opened this issue Jul 20, 2021 · 0 comments · Fixed by #266
Assignees
@lenny-goodell
Labels
1-low priority denoting isolated changes bug Something isn't working
Projects
Ireland known bugs documented on Wiki

Comments

@lenny-goodell
Copy link
Member

The log statement is:

logEnvironmentOverride(lc, "-c/-confdir", envFile, envValue)

when it should be:

logEnvironmentOverride(lc, "-c/-confdir", envConfDir, envValue)

Located at:
https://github.com/edgexfoundry/go-mod-bootstrap/blob/main/bootstrap/environment/variables.go#L300
@lenny-goodell lenny-goodell added 1-low priority denoting isolated changes bug Something isn't working labels Jul 20, 2021
@lenny-goodell lenny-goodell added this to New Issues in Core WG via automation Jul 20, 2021
@JamesKButcher JamesKButcher moved this from New Issues to Release Backlog in Core WG Aug 5, 2021
lenny-goodell referenced this issue in lenny-goodell/go-mod-bootstrap Aug 5, 2021
fix: Use correct name when logging EDGEX_CONF_DIR override
96e528d 
fixes #264

Signed-off-by: lenny <leonard.goodell@intel.com>
@lenny-goodell lenny-goodell mentioned this issue Aug 5, 2021
Merged
6 tasks
@lenny-goodell lenny-goodell linked a pull request Aug 5, 2021 that will close this issue
fix: Use correct name when logging EDGEX_CONF_DIR override #266
Merged
6 tasks
@lenny-goodell lenny-goodell moved this from Release Backlog to QA/Code Review in Core WG Aug 5, 2021
Core WG automation moved this from QA/Code Review to Done Aug 5, 2021
lenny-goodell pushed a commit that referenced this issue Aug 5, 2021
fix: Use correct name when logging EDGEX_CONF_DIR override (#266)
2a375e7 
fixes #264

Signed-off-by: lenny <leonard.goodell@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lenny-goodell lenny-goodell

Labels
1-low priority denoting isolated changes bug Something isn't working
Projects
No open projects
Ireland known bugs documented on Wiki
Documented on Wiki
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: Use correct name when logging EDGEX_CONF_DIR override lenny-goodell/go-mod-bootstrap
1 participant
@lenny-goodell