-
Notifications
You must be signed in to change notification settings - Fork 516
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
Adding support for new config name #2101
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for this!
Since this change will be released in Mimir 2.2, we can only remove old field in Mimir 2.4. Let's keep only single TODO (next to ActiveSeriesCustomTrackersConfigOld
field) for removal, and drop other TODOs. Compilation errors will show us what to change.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you.
Co-authored-by: Peter Štibraný <pstibrany@gmail.com>
* Adding support for new config name * Hiding documentation of active_series_custom_trackers_config, but still supporting it at deserialization time * Clearing old config value, and adding a test case for this behaviour * Running make reference-help * Adding changelog describing change * Addressing Peter's comments * Tidying up testcases * Removing omitempty, as its behaviour is unexpected (not serializing the object at all) * Update CHANGELOG.md Co-authored-by: Peter Štibraný <pstibrany@gmail.com> Co-authored-by: Peter Štibraný <pstibrany@gmail.com>
What this PR does
In #1188 the config name of
active_series_custom_trackers
has been changed toactive_series_custom_trackers_config
by accident. The aim of this PR is to change it back toąctive_series_custom_trackers
while supporting the suffixed version for 2 releases.When both are specified, the old config takes precedence.
Which issue(s) this PR fixes or relates to
Fixes #
Checklist
CHANGELOG.md
updated - the order of entries should be[CHANGE]
,[FEATURE]
,[ENHANCEMENT]
,[BUGFIX]