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 relabelings for ScrapeConfig #5611
Fix relabelings for ScrapeConfig #5611
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.
LGTM, thanks for the report and fix!
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.
lgtm
@EtienneBarbier @xiu - if we want to include this fix as a patch release it should target release-0.65 branch instead of main
Ok, only one PR in the release branch (with backport to main) or two PR ? |
@EtienneBarbier just this one and we can cherry-pick back after |
Ok Great 👍 |
@EtienneBarbier I think it will need a rebase when you change the target branch btw |
73e2ddc
to
cae5447
Compare
@philipgough Rebase done. |
@EtienneBarbier I think we might want to limit this PR to just commit cae5447 as opposed to the unrelated tests too! |
Signed-off-by: EtienneBarbier <etienne.barbier@atos.net>
cae5447
to
f42fa79
Compare
6ae0250
into
prometheus-operator:release-0.65
* Fix relabelings for ScrapeConfig (#5611) Signed-off-by: EtienneBarbier <etienne.barbier@atos.net> * chore: Prep for release 0.65.2 (#5622) * test: Add tests for ScrapeConfig relabelling --------- Signed-off-by: EtienneBarbier <etienne.barbier@atos.net> Co-authored-by: Etienne Barbier <etienne.barbier@atos.net>
* Fix relabelings for ScrapeConfig (prometheus-operator#5611) Signed-off-by: EtienneBarbier <etienne.barbier@atos.net> * chore: Prep for release 0.65.2 (prometheus-operator#5622) * test: Add tests for ScrapeConfig relabelling --------- Signed-off-by: EtienneBarbier <etienne.barbier@atos.net> Co-authored-by: Etienne Barbier <etienne.barbier@atos.net>
Description
Fix relabelings issue in ScrapeConfig CRD, generate them correctly.
Fix #5610
Type of change
What type of changes does your code introduce to the Prometheus operator? Put an
x
in the box that apply.CHANGE
(fix or feature that would cause existing functionality to not work as expected)FEATURE
(non-breaking change which adds functionality)BUGFIX
(non-breaking change which fixes an issue)ENHANCEMENT
(non-breaking change which improves existing functionality)NONE
(if none of the other choices apply. Example, tooling, build system, CI, docs, etc.)Changelog entry
Please put a one-line changelog entry below. This will be copied to the changelog file during the release process.