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

Flatten Consul key-value structure #775

Closed
ejlee3 opened this issue Nov 30, 2022 · 1 comment · Fixed by #776
Closed

Flatten Consul key-value structure #775

ejlee3 opened this issue Nov 30, 2022 · 1 comment · Fixed by #776

Comments

@ejlee3
Copy link
Contributor

ejlee3 commented Nov 30, 2022

Fix Consul paths so version is not duplicated:
Existing:
edgex/core/2.0/<service_key>
edgex/appservices/2.0/<service_key>
edgex/devices/2.0/<service_key>

Proposed:
edgex/3.0/<service_key>

@lenny-goodell lenny-goodell added this to the Levski milestone Nov 30, 2022
@bnevis-i
Copy link
Collaborator

Added to security WG as this also affects the consul ACL hardening.

@lenny-goodell lenny-goodell transferred this issue from edgexfoundry/go-mod-configuration Dec 1, 2022
@lenny-goodell lenny-goodell moved this from New Issues to In Progress in Technical WG Dec 1, 2022
lenny-goodell pushed a commit to lenny-goodell/go-mod-core-contracts that referenced this issue Dec 1, 2022
closes edgexfoundry#775

BREAKING CHANGE: Location of service configuartion in Config Provider (Consul) has changed

Signed-off-by: Leonard Goodell <[email protected]>
lenny-goodell pushed a commit to lenny-goodell/go-mod-core-contracts that referenced this issue Dec 2, 2022
BREAKING CHANGE: Location of service configuartion in Config Provider (Consul) has changed

closes edgexfoundry#775

Signed-off-by: Leonard Goodell <[email protected]>
Repository owner moved this from In Progress to Minnesota Done in Technical WG Dec 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants