Skip to content

Commit

Permalink
Update comment
Browse files Browse the repository at this point in the history
  • Loading branch information
jukie committed Apr 4, 2024
1 parent c5b1603 commit 91cf02f
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion control-plane/catalog/to-consul/resource.go
Original file line number Diff line number Diff line change
Expand Up @@ -742,7 +742,8 @@ func (t *ServiceResource) registerServiceInstance(
}
}
for _, endpoint := range endpointSlice.Endpoints {
// From Kubernetes docs nil is an unknown state that can be interpreted as the endpoint being in a ready state
// From Kubernetes docs nil is an unknown state that can be interpreted as
// the endpoint being in a non-terminating state (assume ready or at least a state we shouldn't ignore)
// Ref: https://github.com/kubernetes/api/blob/5147c1a32f6a0b9b155bb84e59f933e0ff8a3792/discovery/v1/types.go#L129-L137
if endpoint.Conditions.Terminating == nil || !*endpoint.Conditions.Terminating {
for _, endpointAddr := range endpoint.Addresses {
Expand Down

0 comments on commit 91cf02f

Please sign in to comment.