-
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
bump Kubernetes packaged CNI to v0.5.1 #43536
bump Kubernetes packaged CNI to v0.5.1 #43536
Conversation
/release-note-none |
@ixdy: you can only set release notes if you are the author or an assignee. In response to this comment:
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. I understand the commands that are listed here. |
@@ -18,7 +18,7 @@ | |||
# [ARCH=amd64] [CNI_RELEASE=v0.3.0] make (build|push) | |||
|
|||
ARCH?=amd64 | |||
CNI_RELEASE?=07a8a28637e97b22eb8dfe710eeae1344f69d16e | |||
CNI_RELEASE?=0799f5732f2a11b329d9e3d51b9c8f2e3759f2ff |
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'm not sure why we don't just use the tag, but I guess we can think about that at a later date.
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.
@freehan do you know why we use a sha instead of a tag?
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 at some point it was not tagged
/assign |
/release-note-none |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ethernetdan, ixdy
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
If we really do this, we should bump godeps and update the debs/rpms as well |
...and there's a lot of places where commit 07a8a28637e97b22eb8dfe710eeae1344f69d16e is referenced, we have to update those |
Automatic merge from submit-queue |
I was working on the Godeps stuff, LMK if I should continue |
I see the following differences between the vendor'ed 3.x.x and 5.x.x:
Both the NetConfList field and the Version field are marked omitIfEmpty. |
Update version of packaged CNI to v0.5.1 in order as part of a fix for #43488.
Next steps compiling these versions and updating references to artifacts.
/cc @ixdy