-
Notifications
You must be signed in to change notification settings - Fork 217
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
Update csi snapshotter sidecar image to 4.2.1 #325
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: xing-yang 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 |
The test are failing because k8s.gcr.io/sig-storage/csi-snapshotter:v4.2.1 cannot be pulled (latest published version is v4.2.0). What does that have to do with kubernetes-csi/csi-release-tools#170? |
39195ec
to
66a7c83
Compare
/test ci-kubernetes-csi-canary-test-on-kubernetes-master |
@xing-yang: The specified target(s) for
The following commands are available to trigger optional jobs:
Use
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Looks like only the snapshot-controller image was updated when I submitted this PR: kubernetes/k8s.io#2613 |
I was just trying to update the image to v4.2.1. The failure I'm talking about is referring to the flaky tests that 170 is trying to address, not related to this image issue. |
/retest |
/lgtm |
What type of PR is this?
/kind flake
What this PR does / why we need it:
This updates the CSI-Snapshotter sidecar image from v4.2.0 to v4.2.1.
Note that the CI test failure is from snapshot-controller images so we need kubernetes-csi/csi-release-tools#170 for the test to pass.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?: