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

[SDK] Remove old metrics collection and REST /metrics endpoint #1281

Closed
lenny-goodell opened this issue Jan 25, 2023 · 0 comments · Fixed by #1287
Closed

[SDK] Remove old metrics collection and REST /metrics endpoint #1281

lenny-goodell opened this issue Jan 25, 2023 · 0 comments · Fixed by #1287
Labels
breaking-change enhancement New feature or request
Milestone
Minnesota

Comments

@lenny-goodell
Copy link
Member

🚀 Feature Request

Relevant Package [REQUIRED]

Obsolete old Metrics collection and /metrics endpoints

Description [REQUIRED]

Obsolete old Metrics collection and /metrics endpoint are no longer needed and should be removed.

Describe the solution you'd like

Remove the code for collecting the old metrics, the endpoints to get the metrics and remove /metrics the swagger files

Describe alternatives you've considered

Have you considered any alternative solutions or workarounds?
@lenny-goodell lenny-goodell added enhancement New feature or request breaking-change labels Jan 25, 2023
@lenny-goodell lenny-goodell added this to the Minnesota milestone Jan 25, 2023
vli11 added a commit to vli11/app-functions-sdk-go that referenced this issue Feb 2, 2023
@vli11
feat!: [SDK] Remove old metrics collection and REST /metrics endpoint
a99287e 
BREAKING CHANGE: /metrics endpoint no longer available for any service

Closes: edgexfoundry#1281
Signed-off-by: Valina Li <valina.li@intel.com>
@vli11 vli11 mentioned this issue Feb 2, 2023
Merged
5 tasks
lenny-goodell pushed a commit that referenced this issue Feb 6, 2023
@vli11
feat!: [SDK] Remove old metrics collection and REST /metrics endpoint (…
34fb173 
…#1287)

* feat!: [SDK] Remove old metrics collection and REST /metrics endpoint

BREAKING CHANGE: /metrics endpoint no longer available for any service

Closes: #1281
Signed-off-by: Valina Li <valina.li@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
breaking-change enhancement New feature or request
Projects
No open projects
Application Services
Archived in project
Milestone
Minnesota
Development

Successfully merging a pull request may close this issue.

feat!: [SDK] Remove old metrics collection and REST /metrics endpoint vli11/app-functions-sdk-go
1 participant
@lenny-goodell