-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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 pd csi driver images to use v1 images (CSINode, CSIDriver, etc) #102212
Conversation
I noticed the controller gce pd driver was at v1.0.1 and the node was at v0.7.0, any reason for the skew or is that just an oversight? At any rate these manifests will soon go away as we are going to be installing the driver by default in kube-up. |
/triage accepted |
/priority important-soon |
/lgtm |
I think it was just oversight |
volumeMounts: | ||
- name: socket-dir | ||
mountPath: /csi | ||
- name: gce-pd-driver | ||
image: gcr.io/gke-release/gcp-compute-persistent-disk-csi-driver:v1.0.1-gke.0 | ||
image: gcr.io/gke-release/gcp-compute-persistent-disk-csi-driver:v1.2.1-gke.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We haven't fixed the k8s image building issue yet?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, I just copied this from stable-master on the upstream driver. But that's out of date, we've moved to v1.2.2-gke.2 on GKE already.
/hold |
I missed the leader election rbac.
…On Fri, May 21, 2021 at 1:25 PM Kubernetes Prow Robot < ***@***.***> wrote:
@mattcary <https://github.com/mattcary>: The following tests *failed*,
say /retest to rerun all failed tests:
Test name Commit Details Rerun command
pull-kubernetes-e2e-gce-storage-slow 4b5e027
<4b5e027>
link
<https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/102212/pull-kubernetes-e2e-gce-storage-slow/1395797744899067904/> /test
pull-kubernetes-e2e-gce-storage-slow
pull-kubernetes-e2e-gce-storage-snapshot 4b5e027
<4b5e027>
link
<https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/102212/pull-kubernetes-e2e-gce-storage-snapshot/1395797745737928704/> /test
pull-kubernetes-e2e-gce-storage-snapshot
pull-kubernetes-e2e-gce-csi-serial 4b5e027
<4b5e027>
link
<https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/102212/pull-kubernetes-e2e-gce-csi-serial/1395797746576789504/> /test
pull-kubernetes-e2e-gce-csi-serial
Full PR test history
<https://prow.k8s.io/pr-history?org=kubernetes&repo=kubernetes&pr=102212>.
Your PR dashboard
<https://prow.k8s.io/pr?query=is%3Apr%20state%3Aopen%20author%3Amattcary>.
Please help us cut down on flakes by linking to
<https://git.k8s.io/community/contributors/devel/sig-testing/flaky-tests.md#filing-issues-for-flaky-tests>
an open issue
<https://github.com/kubernetes/kubernetes/issues?q=is:issue+is:open> when
you hit one in your PR.
Instructions for interacting with me using PR comments are available here
<https://git.k8s.io/community/contributors/guide/pull-requests.md>. If
you have questions or suggestions related to my behavior, please file an
issue against the kubernetes/test-infra
<https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:>
repository. I understand the commands that are listed here
<https://go.k8s.io/bot-commands>.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#102212 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AIJCBAA5AFXXKNS3P5BTLQLTO26TFANCNFSM45JTBY5Q>
.
|
Darn it, 1.2.2 is still in staging. Switching back to 1.2.1 until I figure
out when the promotion will happen.
…On Fri, May 21, 2021 at 4:14 PM Kubernetes Prow Robot < ***@***.***> wrote:
@mattcary <https://github.com/mattcary>: The following tests *failed*,
say /retest to rerun all failed tests:
Test name Commit Details Rerun command
pull-kubernetes-e2e-gce-storage-snapshot b36cece
<b36cece>
link
<https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/102212/pull-kubernetes-e2e-gce-storage-snapshot/1395840144149319680/> /test
pull-kubernetes-e2e-gce-storage-snapshot
pull-kubernetes-e2e-gce-csi-serial b36cece
<b36cece>
link
<https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/102212/pull-kubernetes-e2e-gce-csi-serial/1395840144988180480/> /test
pull-kubernetes-e2e-gce-csi-serial
Full PR test history
<https://prow.k8s.io/pr-history?org=kubernetes&repo=kubernetes&pr=102212>.
Your PR dashboard
<https://prow.k8s.io/pr?query=is%3Apr%20state%3Aopen%20author%3Amattcary>.
Please help us cut down on flakes by linking to
<https://git.k8s.io/community/contributors/devel/sig-testing/flaky-tests.md#filing-issues-for-flaky-tests>
an open issue
<https://github.com/kubernetes/kubernetes/issues?q=is:issue+is:open> when
you hit one in your PR.
Instructions for interacting with me using PR comments are available here
<https://git.k8s.io/community/contributors/guide/pull-requests.md>. If
you have questions or suggestions related to my behavior, please file an
issue against the kubernetes/test-infra
<https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:>
repository. I understand the commands that are listed here
<https://go.k8s.io/bot-commands>.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#102212 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AIJCBAA4XTKX5L7CRUKAPXTTO3SMHANCNFSM45JTBY5Q>
.
|
Oh, wait, we should be using the upstream cloud build jobs now.
…On Fri, May 21, 2021 at 4:56 PM Matthew Cary ***@***.***> wrote:
Darn it, 1.2.2 is still in staging. Switching back to 1.2.1 until I figure
out when the promotion will happen.
On Fri, May 21, 2021 at 4:14 PM Kubernetes Prow Robot <
***@***.***> wrote:
> @mattcary <https://github.com/mattcary>: The following tests *failed*,
> say /retest to rerun all failed tests:
> Test name Commit Details Rerun command
> pull-kubernetes-e2e-gce-storage-snapshot b36cece
> <b36cece>
> link
> <https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/102212/pull-kubernetes-e2e-gce-storage-snapshot/1395840144149319680/> /test
> pull-kubernetes-e2e-gce-storage-snapshot
> pull-kubernetes-e2e-gce-csi-serial b36cece
> <b36cece>
> link
> <https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/102212/pull-kubernetes-e2e-gce-csi-serial/1395840144988180480/> /test
> pull-kubernetes-e2e-gce-csi-serial
>
> Full PR test history
> <https://prow.k8s.io/pr-history?org=kubernetes&repo=kubernetes&pr=102212>.
> Your PR dashboard
> <https://prow.k8s.io/pr?query=is%3Apr%20state%3Aopen%20author%3Amattcary>.
> Please help us cut down on flakes by linking to
> <https://git.k8s.io/community/contributors/devel/sig-testing/flaky-tests.md#filing-issues-for-flaky-tests>
> an open issue
> <https://github.com/kubernetes/kubernetes/issues?q=is:issue+is:open>
> when you hit one in your PR.
>
> Instructions for interacting with me using PR comments are available here
> <https://git.k8s.io/community/contributors/guide/pull-requests.md>. If
> you have questions or suggestions related to my behavior, please file an
> issue against the kubernetes/test-infra
> <https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:>
> repository. I understand the commands that are listed here
> <https://go.k8s.io/bot-commands>.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <#102212 (comment)>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AIJCBAA4XTKX5L7CRUKAPXTTO3SMHANCNFSM45JTBY5Q>
> .
>
|
ae6c6c8
to
cb3d42a
Compare
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mattcary, msau42 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 |
9ed3cd6
to
df2b726
Compare
/test pull-kubernetes-e2e-gce-csi-serial |
Change-Id: I51ddf0014dcfe476495e324ee33aa658e7199477
Looks like a metricsgrabber flake /test pull-kubernetes-e2e-gce-ubuntu-containerd |
SigStorageRegistry: "k8s.gcr.io/sig-storage", | ||
PrivateRegistry: "gcr.io/k8s-authenticated-test", | ||
SampleRegistry: "gcr.io/google-samples", | ||
GcrReleaseRegistry: "gcr.io/gke-release", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can actually remove this one too. But let's leave it to a separate PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 thanks. I meant to ask if I was doing this manifest stuff correctly, I don't understand how it works, I was just adding stuff until the build worked.
/lgtm |
/hold cancel |
/kind bug
/kind failing-test
/sig storage
Fixes #102077
/assign @msau42