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.

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

refactor: Set the Config Version when creating Config Client #201

Merged
merged 1 commit into from
Apr 6, 2021

Conversation

lenny-goodell
Copy link
Member

closes #200

Signed-off-by: lenny [email protected]

PR Checklist

Please check if your PR fulfills the following requirements:

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/go-mod-bootstrap/blob/master/.github/Contributing.md.

What is the current behavior?

Config version set separately by Core/Support, App and Device services.

Issue Number: #200

What is the new behavior?

This centralizes the config version for all Go services.

Does this PR introduce a breaking change?

  • Yes
  • No

BREAKING CHANGE: Configuration in Consul now under the /2.0/ path

New Imports

  • Yes
  • No

Specific Instructions

Are there any specific instructions or things that should be known prior to reviewing?

Other information

@lenny-goodell lenny-goodell requested a review from jpwhitemn April 6, 2021 20:52
This centralizes the config version for all services.

closes #200

BREAKING CHANGE: Configuration in Consul now under the `/2.0/` path

Signed-off-by: lenny <[email protected]>
@lenny-goodell lenny-goodell merged commit 615e600 into edgexfoundry:master Apr 6, 2021
@lenny-goodell lenny-goodell deleted the config-version branch April 6, 2021 22:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Centralize setting the Config Version used in path for Consul in go-mod-bootstrap
2 participants