-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
chore(networking): bump aws-vpc-cni version to 1.15.5 #16191
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest |
1 similar comment
/retest |
/retest |
Please ignore pull-kops-e2e-aws-upgrade-k127-ko127-to-klatest-kolatest-many-addons. |
@hakman I can't seem to figure out from the logs why is the |
So, I have a feeling something with the newly introduced network policy support is causing a cascading failure that begins with (probably)
Seems like it can't access the k8s api, and from there it all goes south. It's weird, because according to the docs, the enforcements of netpols is disabled by default, but there might be something else that is eluding me at this point. I'll have to keep digging and understand what's the root cause. |
@hakman any chance this is related? aws/amazon-vpc-cni-k8s#2103 |
/test pull-kops-e2e-cni-amazonvpc |
Due to that issue, we run AWS VPC CNI tests with Ubuntu 20.04. From what I see, failures look like flakes in the tests, but I may be wrong. |
/test pull-kops-e2e-cni-amazonvpc |
1 similar comment
/test pull-kops-e2e-cni-amazonvpc |
@moshevayner: The following tests failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. 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. |
I think these are legit errors, based on this comment (just making sure you saw it, since it might've been lost in the pile of comments here). I created a cluster locally using a binary built from this branch, and got the mentioned results. |
I think you are right 😄... |
via https://github.com/awslabs/amazon-eks-ami/blob/master/scripts/install-worker.sh#L104 # Temporary fix for https://github.com/aws/amazon-vpc-cni-k8s/pull/2118
sudo sed -i "s/^MACAddressPolicy=.*/MACAddressPolicy=none/" /usr/lib/systemd/network/99-default.link || true how could this be added? |
Please create a separate issue for this. |
@moshevayner could we try updating to 1.16.2? |
Sure, I'll give that a try asap and update! |
This PR is preceded by #16297 /close |
@moshevayner: Closed this PR. 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. |
What this PR does / why we need it:
Bump aws cni to version 1.15.5.
Once 1.29 release branch is created, I'll bump
master
to 1.16.0 which was released a couple of days ago, to allow an additional buffer between the versions, if that makes sense.Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Special notes for your reviewer:
https://github.com/aws/amazon-vpc-cni-k8s/releases/tag/v1.15.5