k8s/watchers: Add missing v1 EndpointSlice group on init#17778
k8s/watchers: Add missing v1 EndpointSlice group on init#17778pchaigno merged 1 commit intocilium:masterfrom
Conversation
joestringer
left a comment
There was a problem hiding this comment.
nice catch, looks like this was introduced by one of my refactoring PRs during v1.11 development cycle. v1.10 looks fine in this regard.
Curious also how you noticed this, since CI doesn't appear to catch this condition? (at least not in a reliable way; for all I know this was causing a flake but I don't know).
|
@joestringer My guess for why we don't see it in CI is because we probably don't disable the original I found it through regular code inspection while working on #17714. |
e907017 to
da0fba1
Compare
|
/test Job 'Cilium-PR-K8s-1.20-kernel-5.4' failed and has not been observed before, so may be related to your PR: Click to show.Test NameFailure OutputIf it is a flake, comment Job 'Cilium-PR-K8s-1.21-kernel-4.19' failed and has not been observed before, so may be related to your PR: Click to show.Test NameFailure OutputIf it is a flake, comment Job 'Cilium-PR-K8s-1.16-net-next' failed and has not been observed before, so may be related to your PR: Click to show.Test NameFailure OutputIf it is a flake, comment |
The K8sWatcher's InitK8sSubsystem() calls resourceGroups() which returns a list of resources that should be waited on via WaitForCacheSync(). The v1 version of EndpointSlice was missing from this list. This commit adds it. Signed-off-by: Chris Tarazi <chris@isovalent.com>
da0fba1 to
527a598
Compare
|
/test |
|
Reviews are in. Multicluster has the usual failure and k8s-1.21-kernel-4.19 hit known flake #13552. Marking ready to merge and merging. |
The K8sWatcher's InitK8sSubsystem() calls resourceGroups() which returns
a list of resources that should be waited on via WaitForCacheSync(). The
v1 version of EndpointSlice was missing from this list. This commit adds
it.
Signed-off-by: Chris Tarazi chris@isovalent.com