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
Treat LabelSelectors as atomic by Server Side Apply to fix #92913 #93901
Conversation
/priority important-soon |
/cc @liggitt |
7c9947d
to
3e55005
Compare
@apelisse I've picked up kubernetes-sigs/structured-merge-diff#170 to reconcile existing fieldset data with atomic<->granular schema changes. This is ready for review. cc @liggitt for version bumps |
That's exciting! |
Let's make sure we update the documentation along with that. I was also tracking a few tasks relative to that, let me look them up. |
Mostly tracked under this umbrella comment: |
@liggitt Could I get you to approve just the version bump? I've checked the the pin-dependency and bazel-update changes look clean. |
/lgtm |
/hold cancel |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jpbetz, liggitt The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest Review the full test history for this PR. Silence the bot with an |
What type of PR is this?
/kind bug
What this PR does / why we need it:
+structType=atomic
tag onLabelSelector
to fix Server Side Apply - PodSelector empty dict not applied #92913.Which issue(s) this PR fixes:
Fixes #92913
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
/sig api-machinery
/wg api-expression
/cc @apelisse