Skip to content
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

Automated backport of #983: Extend CreateOrUpdate to allow mutating before create #1001: Add IdentfyingLabels to CreateOrUpdateWithOptions #1004

Conversation

tpantelis
Copy link
Contributor

Backport of #983 #1001 on release-0.17.

#983: Extend CreateOrUpdate to allow mutating before create
#1001: Add IdentfyingLabels to CreateOrUpdateWithOptions

For details on the backport process, see the backport requests page.

The user may want to dynamically modify the resource before creating
instead of populating all fields up front. This is useful if populating
a field incurs some expense that should only be done if needed.

Signed-off-by: Tom Pantelis <[email protected]>
If the GenerateName field is set in the target resource, the
existing resource is searched for by the target's labels. However
this assumes the labels remain static for the lifetime of the
resource. If the labels are updated then the existing resource is
not found and a new one is created. To avoid this, allow the user
to specify which labels uniquely identify the resource by adding
an IdentfyingLabels field to CreateOrUpdateOptions.

Signed-off-by: Tom Pantelis <[email protected]>
@submariner-bot
Copy link
Contributor

🤖 Created branch: z_pr1004/tpantelis/automated-backport-of-#983-#1001-upstream-release-0.17

@tpantelis tpantelis merged commit 390697e into submariner-io:release-0.17 Oct 7, 2024
16 checks passed
@submariner-bot
Copy link
Contributor

🤖 Closed branches: [z_pr1004/tpantelis/automated-backport-of-#983-#1001-upstream-release-0.17]

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

Successfully merging this pull request may close these issues.

4 participants