cilium-cli icon indicating copy to clipboard operation
cilium-cli copied to clipboard

connectivity: add endpointslice clustermesh sync test

Open MrFreezeex opened this issue 2 years ago • 2 comments

This adds endpointslice synchronization testing inside a clustermesh. It does that by checking that we can get a DNS answer with a global headless service with endpointslice synchronization enabled across two clusters.

Testing it via DNS ensures that the created EndpointSlice are correctly linked to the headless service.

Related to https://github.com/cilium/cilium/pull/28440 e2e testing

MrFreezeex avatar Jan 28 '24 16:01 MrFreezeex

I am converting this to draft as I guess we would need to merge https://github.com/cilium/cilium/pull/28440 first? Unsure of the exact timing here :sweat_smile: and not sure if I added the correct pre-requirements to launch this test, checking 1.16.0 + that we have clustermesh seems logical to me but maybe there would be more check before launching this?

I tested that locally and it seems to work fine but not sure how to trigger this in the CI as well as this test an ongoing PR...

[=] Test [endpointslice-mesh] [66/66]
..

✅ All 1 tests (2 actions) successful, 65 tests skipped, 0 scenarios skipped.

MrFreezeex avatar Jan 28 '24 16:01 MrFreezeex

as the newly introduce headless service should be skipped in a few more places.

Ah yes now I need to ignore this from some tests and not only from the "deployment" phase, will look into it

MrFreezeex avatar Mar 18 '24 19:03 MrFreezeex

This should be ready :tada: (I didn't change any code since the reviews, just rebasing for most of the gh actions to pass), the Multicluster action failing seems to be an unrelated flake that could be retried I believe

MrFreezeex avatar Apr 11 '24 20:04 MrFreezeex

the Multicluster action failing seems to be an unrelated flake that could be retried I believe

Yep, I've triggered a retry

giorio94 avatar Apr 12 '24 10:04 giorio94