Skip to content

Conversation

lenny-goodell
Copy link
Member

@lenny-goodell lenny-goodell commented Mar 19, 2021

Dependent on edgexfoundry/app-functions-sdk-go#747 to be merged first.

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)

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Other... Please describe:

What is the current behavior?

Clients configuration in profiles use old names for services.

Issue Number: #212

What is the new behavior?

Clients configuration in profiles now use the standard service keys for the names for the services.

Does this PR introduce a breaking change?

  • Yes
  • No

BREAKING CHANGE: Clients configuration has changed

Are there any new imports or modules? If so, what are they used for and why?

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

Other information

Copy link

@jim-wang-yutsung jim-wang-yutsung left a comment

Choose a reason for hiding this comment

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

LGTM

closes edgexfoundry#212

BREAKING CHANGE: Clients configuration has changed

Signed-off-by: lenny <leonard.goodell@intel.com>
@lenny-goodell lenny-goodell merged commit be356fe into edgexfoundry:master Mar 19, 2021
@lenny-goodell lenny-goodell deleted the service-key branch March 19, 2021 19:09
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.

2 participants