-
Notifications
You must be signed in to change notification settings - Fork 828
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
kubernetes-csi: promote first multi-arch mock-driver release #940
kubernetes-csi: promote first multi-arch mock-driver release #940
Conversation
/assign @msau42 |
bd658da
to
3ddf6d8
Compare
The OWNERS file should have been in the previous PR #934 but I accidentally dropped it while reworking that PR. Now we need approval from one of the repo owners one more (and hopefully then the last) time. |
k8s.gcr.io/images/OWNERS
Outdated
@@ -0,0 +1,9 @@ | |||
# See the OWNERS docs at https://go.k8s.io/owners |
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.
should this be one level down in "k8s-staging-csi"?
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.
Oops, yes. Fixed.
csi-test v3.1.1 produced the first multi-arch image. We probably want to promote only the combined image (tag: v3.1.1) and not the individual ones for each architecture (tags: amd64-linux-v3.1.1, ppc64le-linux-v3.1.1, s390x-linux-v3.1.1). This showed that the `tags~` regex matching does a sub-string match and not a full match, so the script gets updated a bit to simplify some terms and tighten the one that matches versions.
3ddf6d8
to
1d7d651
Compare
/lgtm |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: pohly, spiffxp 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 |
csi-test v3.1.1 produced the first multi-arch image. We probably want
to promote only the combined image (tag: v3.1.1) and not the
individual ones for each architecture (tags: amd64-linux-v3.1.1,
ppc64le-linux-v3.1.1, s390x-linux-v3.1.1).
This showed that the
tags~
regex matching does a sub-string matchand not a full match, so the script gets updated a bit to simplify
some terms and tighten the one that matches versions.
/cc @msau42