Skip to content

workflows/gke: Re-add missing UID in cluster name#44605

Merged
pchaigno merged 1 commit intomainfrom
pr/pchaigno/ci-gke-readd-uid
Mar 3, 2026
Merged

workflows/gke: Re-add missing UID in cluster name#44605
pchaigno merged 1 commit intomainfrom
pr/pchaigno/ci-gke-readd-uid

Conversation

@pchaigno
Copy link
Copy Markdown
Member

@pchaigno pchaigno commented Mar 3, 2026

Commit a3b5307 ("gke: Remove repository_owner and UID from clusterName") removed the UID from the cluster name by mistake, probably while fixing a merge conflict. As a consequence, because Conformance KPR is calling Conformance GKE multiple times, it ends up trying to create multiple clusters with the same name.

cc @YutaroHayakawa

Fixes: #44547.

Commit a3b5307 ("gke: Remove repository_owner and UID from
clusterName") removed the UID from the cluster name by mistake, probably
while fixing a merge conflict. As a consequence, because Conformance
KPR is calling Conformance GKE multiple times, it ends up trying to
create multiple clusters with the same name.

Fixes: a3b5307 ("gke: Remove repository_owner and UID from clusterName")
Signed-off-by: Paul Chaignon <paul.chaignon@gmail.com>
@pchaigno pchaigno requested review from a team as code owners March 3, 2026 09:42
@pchaigno pchaigno added release-note/ci This PR makes changes to the CI. integration/cloud Related to integration with cloud environments such as AKS, EKS, GKE, etc. labels Mar 3, 2026
@pchaigno pchaigno requested review from ldelossa and nbusseneau March 3, 2026 09:42
@pchaigno
Copy link
Copy Markdown
Member Author

pchaigno commented Mar 3, 2026

/test

@pchaigno pchaigno enabled auto-merge March 3, 2026 09:42
@pchaigno pchaigno added this pull request to the merge queue Mar 3, 2026
Merged via the queue into main with commit 4b79591 Mar 3, 2026
93 checks passed
@pchaigno pchaigno deleted the pr/pchaigno/ci-gke-readd-uid branch March 3, 2026 14:25
Copy link
Copy Markdown
Contributor

@ldelossa ldelossa left a comment

Choose a reason for hiding this comment

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

LGTM

@YutaroHayakawa
Copy link
Copy Markdown
Member

Oops, sorry about that...

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

Labels

integration/cloud Related to integration with cloud environments such as AKS, EKS, GKE, etc. release-note/ci This PR makes changes to the CI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants