Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[WIP] Forward ClusterIP traffic to inter-node communication port before CQL is ready #1614

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

[WIP] Forward ClusterIP traffic to inter-node communication port before CQL is ready #1614

wants to merge 2 commits into from

Commits on Jul 10, 2024

  1. Reconcile per-container Endpoints and EndpointSlices for member Services 

    Kubernetes reconciles Endpoints and EndpointSlices for Services having
selector by taking entire Pod readiness into account. Because we need to
allow inter-node communication before CQL traffic we used
PublishNonReadyEndpoints on Services to overcome this.
But at the same time, we would like stop accepting new connections when
Pod is tearing down. These two requirements contradicts with each other.

To satisfy both requirements, Operator will reconcile Endpoints and
EndpointSlice resources in per-container way. If Pod container specifies
port and has its own Readiness probe, Endpoint/EndpointSlice
will become ready for this port when given container is ready. If these
two conditions aren't met, given port becomes ready when entire Pod is
ready.
Controller logic will consider Pod being deleted (non-nil
deletionTimestamp) as fully non-ready so that all endpoints for all
ports
and containers will removed.

Additional container added to Scylla Pod controls readiness of ports
used for inter-node communication.
    @zimnx
    zimnx committed Jul 10, 2024
    Configuration menu
    Copy the full SHA
    e7d2fe1 View commit details
    Browse the repository at this point in the history

  2. Update autogenerated

    @zimnx
    zimnx committed Jul 10, 2024
    Configuration menu
    Copy the full SHA
    b14579a View commit details
    Browse the repository at this point in the history