Skip to content

Conversation

gardener-ci-robot
Copy link
Contributor

@gardener-ci-robot gardener-ci-robot commented Feb 21, 2025

This is an automated cherry-pick of #11479

/assign rfranzke

A bug which prevented usage of labels with `seed.gardener.cloud/` prefix on `Seed`, `ManagedSeed`, `BackupEntry`, and `Shoot` resources has been fixed.

due to potential naming conflicts when users also use
`seed.gardener.cloud/` as prefix for their labels (not ideal, they
should actually use a unique prefix (e.g., `seed.my.cool.domain.com/`),
but we cannot prevent it for historic reasons
@gardener-ci-robot gardener-ci-robot added area/usability Usability related kind/regression Bug that hit us already in the past and that is reappearing/requires a proper solution labels Feb 21, 2025
@gardener-prow gardener-prow bot requested a review from ialidzhikov February 21, 2025 14:49
@gardener-prow gardener-prow bot added the cla: yes Indicates the PR's author has signed the cla-assistant.io CLA. label Feb 21, 2025
@gardener-prow gardener-prow bot requested a review from timuthy February 21, 2025 14:49
@gardener-prow gardener-prow bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Feb 21, 2025
@plkokanov
Copy link
Contributor

/test pull-gardener-integration-release-v1-112

@shafeeqes
Copy link
Contributor

/lgtm
/approve
/retest

@gardener-prow gardener-prow bot added the lgtm Indicates that a PR is ready to be merged. label Feb 22, 2025
Copy link
Contributor

gardener-prow bot commented Feb 22, 2025

LGTM label has been added.

Git tree hash: 9da51c912faa1a1874ce7bc9e784f8e756d1d70e

Copy link
Contributor

gardener-prow bot commented Feb 22, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: shafeeqes

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@gardener-prow gardener-prow bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 22, 2025
@gardener-prow gardener-prow bot merged commit 62e3df3 into gardener:release-v1.112 Feb 22, 2025
18 checks passed
@gardener-ci-robot gardener-ci-robot deleted the cherry-pick-11479-to-release-v1.112 branch February 22, 2025 22:00
Duciwuci pushed a commit to stackitcloud/gardener that referenced this pull request Feb 25, 2025
…ardener.cloud/` (gardener#11486)

* Rename `seed.gardener.cloud/` prefix to `.name.seed.gardener.cloud/`

due to potential naming conflicts when users also use
`seed.gardener.cloud/` as prefix for their labels (not ideal, they
should actually use a unique prefix (e.g., `seed.my.cool.domain.com/`),
but we cannot prevent it for historic reasons

* Migration code in GCM to re-compute seed labels

* Address PR review feedback

---------

Co-authored-by: rfranzke <rafael.franzke@sap.com>
ske-prow bot pushed a commit to stackitcloud/gardener that referenced this pull request Feb 25, 2025
* Prepare next Development Cycle v1.112.3-dev

* Prevent concurrent map read/writes in `Extension` deployer component (gardener#11467)

Co-authored-by: rfranzke <rafael.franzke@sap.com>

* [release-v1.112] Rename `seed.gardener.cloud/` prefix to `name.seed.gardener.cloud/` (gardener#11486)

* Rename `seed.gardener.cloud/` prefix to `.name.seed.gardener.cloud/`

due to potential naming conflicts when users also use
`seed.gardener.cloud/` as prefix for their labels (not ideal, they
should actually use a unique prefix (e.g., `seed.my.cool.domain.com/`),
but we cannot prevent it for historic reasons

* Migration code in GCM to re-compute seed labels

* Address PR review feedback

---------

Co-authored-by: rfranzke <rafael.franzke@sap.com>

* Release v1.112.3

---------

Co-authored-by: gardener-robot-ci-1 <gardener.ci.user@gmail.com>
Co-authored-by: Gardener Prow Robot <gardener.ci.robot@gmail.com>
Co-authored-by: rfranzke <rafael.franzke@sap.com>
Co-authored-by: gardener-robot-ci-2 <gardener.ci.user2@gmail.com>
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. area/usability Usability related cla: yes Indicates the PR's author has signed the cla-assistant.io CLA. kind/regression Bug that hit us already in the past and that is reappearing/requires a proper solution lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants