Skip to content

Pin GCP CCM image to v32.2.4 #17348

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Apr 9, 2025
Merged

Pin GCP CCM image to v32.2.4 #17348

merged 2 commits into from
Apr 9, 2025

Conversation

rifelpet
Copy link
Member

@rifelpet rifelpet commented Apr 9, 2025

The GCP CCM Image builds included a breaking change in kubernetes/cloud-provider-gcp#825
until we can update our manifest to account for the change, lets pin to the last stable tag from https://console.cloud.google.com/artifacts/docker/k8s-staging-cloud-provider-gcp/us/gcr.io/cloud-controller-manager?invt=AbuPzg

ref: #17347

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Apr 9, 2025
@k8s-ci-robot k8s-ci-robot requested review from hakman and olemarkus April 9, 2025 00:01
@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Apr 9, 2025
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: hakman

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

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 9, 2025
@k8s-ci-robot k8s-ci-robot merged commit 4ba82bb into kubernetes:master Apr 9, 2025
26 checks passed
@k8s-ci-robot k8s-ci-robot added this to the v1.32 milestone Apr 9, 2025
This was referenced Apr 10, 2025
Merged
Merged
Merged
k8s-ci-robot added a commit that referenced this pull request Apr 10, 2025
@k8s-ci-robot
Merge pull request #17351 from rifelpet/automated-cherry-pick-of-#173…
29664d9 
…48-origin-release-1.30

Automated cherry pick of #17348: Pin GCP CCM image to v32.2.4
k8s-ci-robot added a commit that referenced this pull request Apr 10, 2025
@k8s-ci-robot
Merge pull request #17349 from rifelpet/automated-cherry-pick-of-#173…
49a1a52 
…48-origin-release-1.32

Automated cherry pick of #17348: Pin GCP CCM image to v32.2.4
k8s-ci-robot added a commit that referenced this pull request Apr 10, 2025
@k8s-ci-robot
Merge pull request #17350 from rifelpet/automated-cherry-pick-of-#173…
2d0e428 
…48-origin-release-1.31

Automated cherry pick of #17348: Pin GCP CCM image to v32.2.4
@marseel marseel mentioned this pull request Apr 15, 2025
Open
@marseel
Copy link
Member

marseel commented Apr 15, 2025

Thanks for the fix, I was wondering if there is a kops release planned any time soon?
Current released kops versions seems to fail with bringing cluster up on GCP due to the mentioned change and having image pinned to master tag.

@hakman
Copy link
Member

hakman commented Apr 15, 2025

Thanks for the fix, I was wondering if there is a kops release planned any time soon? Current released kops versions seems to fail with bringing cluster up on GCP due to the mentioned change and having image pinned to master tag.

Hey @marseel, we plan do a release this week or next week.

marseel added a commit to marseel/scale-tests-action that referenced this pull request Apr 17, 2025
@marseel
Fix not pinned ccm image
b254562 
Until new kops version is released, current versions are affected by the
bug that CCM image is not pinned to specific version, but to latest
build version instead.
Related PR: kubernetes/kops#17348

Signed-off-by: Marcel Zieba <marcel.zieba@isovalent.com>
@marseel marseel mentioned this pull request Apr 17, 2025
Merged
marseel added a commit to cilium/scale-tests-action that referenced this pull request Apr 17, 2025
@marseel
Fix not pinned ccm image
b83e200 
Until new kops version is released, current versions are affected by the
bug that CCM image is not pinned to specific version, but to latest
build version instead.
Related PR: kubernetes/kops#17348

Signed-off-by: Marcel Zieba <marcel.zieba@isovalent.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hakman hakman hakman approved these changes

@olemarkus olemarkus Awaiting requested review from olemarkus

Assignees

@hakman hakman

Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Milestone
v1.32
Development

Successfully merging this pull request may close these issues.
4 participants
@rifelpet @k8s-ci-robot @marseel @hakman