Skip to content

Allow resource updates to omit an owner refs UID#17423

Merged
mkeeler merged 1 commit intomainfrom
resource-update-uid
May 22, 2023
Merged

Allow resource updates to omit an owner refs UID#17423
mkeeler merged 1 commit intomainfrom
resource-update-uid

Conversation

@mkeeler
Copy link
Member

@mkeeler mkeeler commented May 22, 2023

Description

This change enables workflows where you are reapplying a resource that should have an owner ref to publish modifications to the resources data without performing a read to figure out the current owner resource incarnations UID.

Basically we want workflows similar to kubectl apply or consul config write to be able to work seamlessly even for owned resources.

In these cases the users intention is to have the resource owned by the “current” incarnation of the owner resource.

PR Checklist

  • updated test coverage
  • external facing docs updated
  • appropriate backport labels added
  • not a security concern

This change enables workflows where you are reapplying a resource that should have an owner ref to publish modifications to the resources data without performing a read to figure out the current owner resource incarnations UID.

Basically we want workflows similar to `kubectl apply` or `consul config write` to be able to work seamlessly even for owned resources.

In these cases the users intention is to have the resource owned by the “current” incarnation of the owner resource.
@mkeeler mkeeler requested review from analogue and boxofrad May 22, 2023 14:08
@mkeeler mkeeler added pr/no-changelog PR does not need a corresponding .changelog entry pr/no-backport labels May 22, 2023
@mkeeler mkeeler merged commit 93bad3e into main May 22, 2023
@mkeeler mkeeler deleted the resource-update-uid branch May 22, 2023 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr/no-backport pr/no-changelog PR does not need a corresponding .changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants