-
Notifications
You must be signed in to change notification settings - Fork 2.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
Extend endpoint related interfaces #14743
Conversation
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.
LGTM, minor optional comment if it makes sense
1831c50
to
6c53400
Compare
Embed interface into a mock so that all the methods are inherited, and only the relevant methods can be implemented in tests in order to override real code. Signed-off-by: Aditi Ghag <aditi@cilium.io>
External plugins can use these interfaces to get Endpoint state. Signed-off-by: Aditi Ghag <aditi@cilium.io>
6c53400
to
94c9454
Compare
Travis CI run failed because of the rate limit issue. But one of the runs passed - https://travis-ci.com/github/cilium/cilium/jobs/477348687
|
test-me-please |
GKE test failed due to failure to scale - https://jenkins.cilium.io/job/Cilium-PR-K8s-GKE/4130/ |
test-gke |
Still failing with the same issue - https://jenkins.cilium.io/job/Cilium-PR-K8s-GKE/4149/console |
test-gke |
Blocking merging for now to discuss some fundamentals around this PR. |
See commit description.