Skip to content

Conversation

Weil0ng
Copy link
Contributor

@Weil0ng Weil0ng commented Apr 28, 2020

Auto-detects EndpointSlice support via apimachinery instead of
version-matching.

For some clusters (e.g., gke-1.17.5), EndpointSlice is disabled although
the cluster version is >= 1.17. In those cases, version-matching would
turn on endpointslice controller while that api is not available,
resulting in cilium agent not functioning.

Signed-off-by: Weilong Cui cuiwl@google.com

Auto detect EndpointSlice support by checking enabled APIs in Kubernetes.

version-matching.

For some clusters (e.g., gke-1.17.5), EndpointSlice is disabled although
the cluster version is >= 1.17. In those cases, version-matching would
turn on endpointslice controller while that api is not available,
resulting in cilium agent not functioning.

Signed-off-by: Weilong Cui <cuiwl@google.com>
@Weil0ng Weil0ng requested a review from a team April 28, 2020 22:30
@maintainer-s-little-helper
Copy link

Please set the appropriate release note label.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.007%) to 44.627% when pulling 48fde7d on Weil0ng:eps-detect into ebe156f on cilium:master.

@pchaigno pchaigno added the release-note/bug This PR fixes an issue in a previous release of Cilium. label Apr 29, 2020
@pchaigno
Copy link
Member

test-me-please

Copy link
Member

@aanm aanm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💯

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/bug This PR fixes an issue in a previous release of Cilium.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants