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 Endpoint Slice API Test #100045
Add Endpoint Slice API Test #100045
Conversation
@swetharepakula: GitHub didn't allow me to assign the following users: liggit. Note that only kubernetes members, repo collaborators and people who have commented on this issue/PR can be assigned. Additionally, issues/PRs can only have 10 assignees at the same time. 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. |
@swetharepakula: This issue is currently awaiting triage. If a SIG or subproject determines this is a relevant issue, they will accept it by applying the The 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. |
Hi @swetharepakula. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. 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. |
Thanks! /ok-to-test |
Thanks! /lgtm |
/assign @thockin |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: swetharepakula, thockin 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 |
Since the PR was lgtm'ed and approved before the Code Freeze was in effect and is just blocked on Prow, I am adding it to the milestone. /milestone v1.21 |
Thank you @swetharepakula for this e2e tests. |
@Riaankl do you know why that seems to conflict with what was actually added by this test? This test seems to pretty clearly include |
@robscott I do not see any obvious reason why the endpoints do not show up in the logs. And I have little knowledge about Network endpoints. |
all namespace list and watch should be there too. it's very odd. It would seem to be an apisnoop reporting issue - we should double check the logs and see |
Thank you for the quick review @johnbelamaric |
Just verified that all the endpoints are being tracked now. Thanks @Riaankl & @johnbelamaric ! |
@swetharepakula @robscott |
This PR added coverage for some of the endpoints. We are waiting for the 2 week soak policy to promote to conformance. |
@swetharepakula I can not find the EndpointSlice tests on the test grid. |
@johnbelamaric Thank you for your advice, it turned out to be the way APISnoop consumes these specific endpoints logs. |
@Riaankl I am not exactly sure what the difference in the links are but this testgrid link shows the tests: https://testgrid.k8s.io/sig-release-master-blocking#gce-cos-master-default&include-filter-by-regex=EndpointSlice&width=5. The test started running 3/10/2021 , so it will qualify for conformance on the 24th. One clarification, is the test freeze on March 23 or March 24th? The release docs say March 24. |
@swetharepakula testgrid is looking good. (got my link wrong) |
What type of PR is this?
/kind feature
What this PR does / why we need it:
This PR adds a E2E test that tests all the EndpointSlice APIs.
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:
KEP: EndpointSlice KEP
Enhancement Issue: kubernetes/enhancements#752
/cc @aojea @andrewsykim @thockin @robscott @wojtek-t
/assign @liggitt
/sig network
/sig testing
/priority important-soon