-
Notifications
You must be signed in to change notification settings - Fork 176
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: receive router deployment selector labels #295
Conversation
The pod selector labels on the receive router deployment include the version label. Selector labels are immutable and this prevents updating the receive router. ``` Deployment/monitoring/thanos-receive-router dry-run failed, reason: Invalid, error: Deployment.apps "thanos-receive-router" is invalid: spec.selector: Invalid value: v1.LabelSelector{MatchLabels:map[string]string{"app.kubernetes.io/component":"thanos-receive-router", "app.kubernetes.io/instance":"thanos-receive", "app.kubernetes.io/name":"thanos-receive", "app.kubernetes.io/version":"v0.30.2"}, MatchExpressions:[]v1.LabelSelectorRequirement(nil)}: field is immutable ``` Signed-off-by: Daniel Schaaff <daniel@danielschaaff.com>
Signed-off-by: Daniel Schaaff <daniel@danielschaaff.com>
Hi @philipgough, @douglascamata, @metalmatze Would it be possible to get this PR reviewed and merged? @dschaaff would you be able to resolve conflicts? |
@michael-burt out of all the people you pinged, only @metalmatze is a maintainer, but he hasn't been active in the project. The others are just contributors (like @philipgough) or triagers (like me), without merge rights. Please ask for a review in the #thanos-dev channel in the CNCF slack, that's the right place. |
I can review and approve, but I can't merge. |
Also, Github is telling me that there are conflicts in this branch that must be solved. |
Signed-off-by: Daniel Schaaff <daniel@danielschaaff.com>
I've resolved the conflicts |
Thanks |
The pod selector labels on the receive router deployment include the version label. Selector labels are immutable and this prevents updating the receive router.
Signed-off-by: Daniel Schaaff daniel@danielschaaff.com
Changes
Verification