Skip to content

Avoid propagating ApplySet labels to owned resources - #171

Open
BjarkeTornager wants to merge 1 commit into
restatedev:mainfrom
BjarkeTornager:fix/do-not-propagate-applyset-label
Open

Avoid propagating ApplySet labels to owned resources#171
BjarkeTornager wants to merge 1 commit into
restatedev:mainfrom
BjarkeTornager:fix/do-not-propagate-applyset-label

Conversation

@BjarkeTornager

Copy link
Copy Markdown

Summary

  • exclude labels in the applyset.kubernetes.io/ namespace when propagating RestateDeployment labels
  • apply the filter to versioned Services and Knative Configurations/Routes
  • retain all ordinary user labels
  • document the behavior change

Why

kubectl adds applyset.kubernetes.io/part-of to resources applied with --applyset. Copying it to an operator-owned versioned Service makes that Service look like a direct ApplySet member. When the same ApplySet contains the Service kind, a later prune can delete the versioned Service before Restate has drained that deployment.

Because the children are server-side applied by the operator, a reconciliation after upgrade also removes labels that older operator versions propagated.

Validation

  • cargo fmt --check
  • cargo test --lib (83 passed)
  • cargo clippy --all-targets --all-features -- -D warnings

cargo test also runs five existing doctests that fail on current main; all 83 unit tests pass.

Closes #170.

@github-actions

github-actions Bot commented Jul 27, 2026

Copy link
Copy Markdown

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@pcholakov
pcholakov self-requested a review August 10, 2026 16:40
@BjarkeTornager

Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA

@BjarkeTornager

Copy link
Copy Markdown
Author

@pcholakov I signed the CLA now. Anything else needed from my side to get this merged in?

@lukebond

Copy link
Copy Markdown
Contributor

apologies for being slow to get back to you! i've fallen a bit behind on my maintenance here but catching up, and will get this reviewed shortly. nothing more you need to do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

RestateDeployment propagates ApplySet membership to owned resources

2 participants