-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
fix(config): metrics #18936
fix(config): metrics #18936
Conversation
WalkthroughThe configuration structure within the telemetry metrics code has been updated to rename a mapstructure tag and alter the default setting for a metrics sink. The field previously tagged as "type" is now "metrics-sink", and the default value for this field has been maintained as "mem". Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChat with CodeRabbit Bot (
|
@tac0turtle can we backport into 50? |
@Mergifyio backport release/v0.50.x |
✅ Backports have been created
|
(cherry picked from commit 0deaa23)
Co-authored-by: Devon Bear <[email protected]>
upstream: fix(config): metrics (cosmos#18936)
Description
Closes: #XXXX
Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...
!
in the type prefix if API or client breaking changeCHANGELOG.md
Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...