-
Notifications
You must be signed in to change notification settings - Fork 15
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
refactor!: Remove legacy node selector on RoleGroup #652
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.
Thanks, LGTM! But i would bet we have it in some other operators / examples (e.g. https://github.com/stackabletech/opa-operator/blob/9c8c3bbdcf731b2f2d4f6af67bec1044c64ce56d/tests/templates/kuttl/smoke/10-install-opa.yaml.j2#L37C9-L38C23). Might be worth doing a grep on all repos?
Regarding the grep: Totally agreed, did that on the docs repo: stackabletech/documentation#453. |
Lets wait for Arch meeting |
@sbernauer Can you tell me what happened with this one? It has the 23.11 label but is not actually included. |
Maybe @maltesander can chime in? |
Yeah we can remove the label since this is not in 23.11. |
I'll add it to the board as it seems to be basically done |
Yeah, PR is good to merge. I wasn't allowed to merge before because of priority reasons ;) |
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.
I approve the intention 👍
Description
Remove deprecated code from #556.
Users had enough time to migrate
Definition of Done Checklist
Author
Reviewer
Acceptance