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

[deprecation] Remove support for expvar metrics #4722

Open
1 of 2 tasks
yurishkuro opened this issue Sep 4, 2023 · 1 comment · May be fixed by #5437
Open
1 of 2 tasks

[deprecation] Remove support for expvar metrics #4722

yurishkuro opened this issue Sep 4, 2023 · 1 comment · May be fixed by #5437
Labels
good first issue Good for beginners help wanted Features that maintainers are willing to accept but do not have cycles to implement

Comments

@yurishkuro
Copy link
Member

yurishkuro commented Sep 4, 2023 •
edited

Currently most Jaeger components (including HotROD) accept a CLI flag that allows selecting between prometheus, expvar, and none as metrics backends. Most production use cases utilize prometheus metrics, which is the default in Jaeger.

Proposal

  • Deprecate the corresponding CLI flags and remove the ability to configure metrics backends at all
  • Still support the endpoint name CLI flag, and if explicitly set to empty string do not mount the endpoint

Rollout plan

  1. Add CLI flag deprecation notice, but do not change anything -- Deprecate expvar metrics backend #4740
  2. After three releases, remove the CLI flag and the other backends

Comments

  • Removing expvar will remove the last dependency on go-kit
  • I am working on a new version of HotROD walkthrough that will reflect this change, as well as the OTEL upgrade
@yurishkuro yurishkuro mentioned this issue Sep 9, 2023
Merged
yurishkuro added a commit that referenced this issue Sep 9, 2023
@yurishkuro
Deprecate expvar metrics backend (#4740)
770b60e 
## Which problem is this PR solving?
- Step 1 of #4722

## Description of the changes
- add deprecation notice for `expvar` as metrics backend
- change HotROD default backend to `prometheus`

## How was this change tested?
- run binaries locally to observe the messages
```
$ go run ./cmd/all-in-one help
      --metrics-backend string                                    Defines which metrics backend to use for metrics reporting: prometheus, none, or expvar (deprecated, will be removed after 2024-01-01 or in release v1.53.0, whichever is later)  (default "prometheus")

$ go run ./cmd/all-in-one --metrics-backend=expvar
2023/09/09 19:09:01 using expvar as metrics backend (deprecated, will be removed after 2024-01-01 or in release v1.53.0, whichever is later)
```

---------

Signed-off-by: Yuri Shkuro <github@ysh.us>
@yurishkuro yurishkuro added help wanted Features that maintainers are willing to accept but do not have cycles to implement good first issue Good for beginners labels May 10, 2024
@joeyyy09
Copy link

I'd like to take this up, Thanks!

@joeyyy09 joeyyy09 linked a pull request May 10, 2024 that will close this issue
Open
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good for beginners help wanted Features that maintainers are willing to accept but do not have cycles to implement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove support for expvar metrics joeyyy09/jaeger
2 participants
@yurishkuro @joeyyy09