-
Notifications
You must be signed in to change notification settings - Fork 935
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
Add Kubernetes v1.29 conformance results for EKS Anywhere #3049
Conversation
ca40ac6
to
dc7494a
Compare
13 of 15 requirements have passed. Please review the following:
for a full list of requirements, please refer to these sections of the docs: content of the PR, and requirements. |
Please fix the issues with this PR so we can review and merge. |
We are using Cilium as our CNI and are currently facing this issue (closed as duplicate but original issue not fixed) in our conformance testing. I saw a comment in another PR which also discussed this same known issue with Cilium. The test report files in this submission were generated by skipping this single test in our |
There's a PR under way for this fix in Cilium |
Yes, we are waiting for that change to get shipped in a new Cilium release, but it's hard to say when that will be, because there hasn't been any activity on the PR for some time. |
@abhay-krishna It's been identified (if you are running with kube-proxy) that helm flag
|
Thanks for the suggestion @tnorlin! I shall try that and update this PR after seeing how it goes. Before trying it out though, I have a doubt. After reading the Cilium documentation, I have come to the understanding that the service proxy name acts as a kind of selector for services that Cilium should manage, based on the |
@tnorlin Your suggestion worked! The test suite is passing now. Thanks for the guidance!
|
Signed-off-by: Abhay Krishna Arunachalam <arnchlm@amazon.com>
All requirements (15) have passed for the submission! |
@abhay-krishna that was great news! No problem, we're in this together! I haven't tried deeper on the constraints, but I'm afraid that what's mentioned in the other repo holds true and we'll have to wait for the other PR to get the full feature set. |
You are now Certified Kubernetes |
Signed-off-by: Abhay Krishna Arunachalam <arnchlm@amazon.com>
Pre-submission checklist:
Please check each of these after submitting your pull request:
repo_url
?For a full list of requirements, please refer to these sections of the docs: Contents of the PR, and Requirements.