-
Notifications
You must be signed in to change notification settings - Fork 200
Sidecar controller should populate service annotations independently #3021
Copy link
Copy link
Open
Labels
kind/cleanupCategorizes issue or PR as related to cleaning up code, process, or technical debt.Categorizes issue or PR as related to cleaning up code, process, or technical debt.priority/important-longtermImportant over the long term, but may not be staffed and/or may need multiple releases to complete.Important over the long term, but may not be staffed and/or may need multiple releases to complete.triage/acceptedIndicates an issue or PR is ready to be actively worked on.Indicates an issue or PR is ready to be actively worked on.
Milestone
Metadata
Metadata
Assignees
Labels
kind/cleanupCategorizes issue or PR as related to cleaning up code, process, or technical debt.Categorizes issue or PR as related to cleaning up code, process, or technical debt.priority/important-longtermImportant over the long term, but may not be staffed and/or may need multiple releases to complete.Important over the long term, but may not be staffed and/or may need multiple releases to complete.triage/acceptedIndicates an issue or PR is ready to be actively worked on.Indicates an issue or PR is ready to be actively worked on.
Type
Fields
Give feedbackNo fields configured for issues without a type.
Context
Currently, the sidecar controller populates both node Service annotations (hostID and token ring hash) at once. That means that even if it could already populate the hostID annotation, but there's no token ring hash available yet, it doesn't populate either of those.
The goal of this issue is to make the logic responsible for populating hostID and token ring hash annotations independent so that they do not block each other.
Acceptance criteria
internal.scylla-operator.scylladb.com/host-idannotation once the host ID is available, without waiting for the token ringinternal.scylla-operator.scylladb.com/current-token-ring-hashannotation once the token ring for the local node is available