Skip to content

Remove Extension Validation for Provider in ETCD-Backup#139

Merged
ske-prow[bot] merged 2 commits intoske-v1.114from
skip-s3-validation
Mar 24, 2025
Merged

Remove Extension Validation for Provider in ETCD-Backup#139
ske-prow[bot] merged 2 commits intoske-v1.114from
skip-s3-validation

Conversation

@Duciwuci
Copy link
Copy Markdown

@Duciwuci Duciwuci commented Mar 21, 2025

/kind enhancement

With Gardener v1.114+ the operator extensions are a required resource for the upgrade.
In our use case, we need to deactivate this validation since this is a breaking change.
We need to do a backup-s3 migration.

Drop this commit, after the migration of the extension was performed.

@ske-prow ske-prow Bot added kind/enhancement Enhancement, improvement, extension size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Mar 21, 2025
@Duciwuci Duciwuci added the tide/merge-method-rebase Denotes a PR that should be rebased by tide when it merges. label Mar 21, 2025
@Duciwuci Duciwuci force-pushed the skip-s3-validation branch 3 times, most recently from 7c25aa9 to 2d7364d Compare March 21, 2025 13:24
@Duciwuci
Copy link
Copy Markdown
Author

/hold let's test this first

@ske-prow ske-prow Bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Mar 21, 2025
With Gardener v1.114+ the operator extensions are a required resource for the upgrade.
In our use case, we need to deactivate this validation since this is a breaking change.
We need to do a backup-s3 migration.

Drop this commit, after the migration of the extension was performed.
@Duciwuci Duciwuci force-pushed the skip-s3-validation branch from 2d7364d to b824eff Compare March 21, 2025 14:07
@Duciwuci
Copy link
Copy Markdown
Author

/hold cancel

@ske-prow ske-prow Bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Mar 21, 2025
Copy link
Copy Markdown
Member

@timebertt timebertt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

Comment thread pkg/utils/gardener/garden.go
@ske-prow ske-prow Bot added the lgtm Indicates that a PR is ready to be merged. label Mar 24, 2025
@ske-prow
Copy link
Copy Markdown

ske-prow Bot commented Mar 24, 2025

LGTM label has been added.

DetailsGit tree hash: f16ab6b338583576de9a65e46c6f5b9c4aec76c5

@ske-prow ske-prow Bot removed the lgtm Indicates that a PR is ready to be merged. label Mar 24, 2025
@ske-prow ske-prow Bot requested a review from timebertt March 24, 2025 09:03
@Duciwuci Duciwuci force-pushed the skip-s3-validation branch from 5f1b37d to 897273a Compare March 24, 2025 10:06
Copy link
Copy Markdown
Member

@timebertt timebertt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you :)

@ske-prow ske-prow Bot added the lgtm Indicates that a PR is ready to be merged. label Mar 24, 2025
@ske-prow
Copy link
Copy Markdown

ske-prow Bot commented Mar 24, 2025

LGTM label has been added.

DetailsGit tree hash: fb64c75a0ac45082ddce4c3ebf1cfe27dbad9329

@ske-prow
Copy link
Copy Markdown

ske-prow Bot commented Mar 24, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: timebertt

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ske-prow ske-prow Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 24, 2025
@ske-prow ske-prow Bot merged commit e3a7c6a into ske-v1.114 Mar 24, 2025
4 checks passed
@ske-prow ske-prow Bot deleted the skip-s3-validation branch March 24, 2025 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. kind/enhancement Enhancement, improvement, extension lgtm Indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. tide/merge-method-rebase Denotes a PR that should be rebased by tide when it merges.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants