-
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
clustermesh: add hostname endpointslice sync #31814
Conversation
4742dca
to
c6ab182
Compare
/test |
c6ab182
to
c862d23
Compare
/test |
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.
Thanks for your contribution! Could you elaborate more about the problem this PR tries to solve or use case you want to realize?
Sure! So this is linked to the new EndpointSlice sync mechanism that was recently merged (https://github.com/cilium/cilium/pull/28440). And this PR aim to resolve one of the limitation of endpointslice sync about not being able to sync the "hostname" field in an EndpointSlice. For instance, the hostname field is used by StatefulSets to resolve with DNS using ${podName}.${svcName} (instead of just ${svcName}). Hope this helps! |
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.
I left a few docs style suggestions, but the docs structure good.
c862d23
to
f30c26c
Compare
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 with a nit.
9ba92be
to
095f4b2
Compare
/test |
15d63d9
to
ab6290b
Compare
/test |
29b113f
to
f22b241
Compare
(I rebased and run |
/test |
This adds hostname synchronization in EndpointSlice. This commit adds the Hostname field in the slims kubernetes struct and propagate that up to the clustermesh's ClusterService struct. Signed-off-by: Arthur Outhenin-Chalandre <arthur@cri.epita.fr>
f22b241
to
eea4c1e
Compare
/test |
/ci-ingress |
/ci-l4lb |
/ci-ipsec-upgrade |
/ci-ingress |
1 similar comment
/ci-ingress |
Please ensure your pull request adheres to the following guidelines:
description and a
Fixes: #XXX
line if the commit addresses a particularGitHub issue.
Fixes: <commit-id>
tag, thenplease add the commit author[s] as reviewer[s] to this issue.
This adds hostname synchronization in EndpointSlice. This PR adds the Hostname field in the slims kubernetes struct and propagate that up to the clustermesh's ClusterService struct.