remove TODO item about NodeAddresses

that indicates it is only used for getting addresses of calling instance,
which is no longer true for most cloud providers.

Original PR: #8899
This commit is contained in:
Indeed 2020-05-04 14:59:40 -07:00
parent e8d6d3768e
commit 3af3ed8e7d

View File

@ -157,9 +157,6 @@ type LoadBalancer interface {
// Instances is an abstract, pluggable interface for sets of instances.
type Instances interface {
// NodeAddresses returns the addresses of the specified instance.
// TODO(roberthbailey): This currently is only used in such a way that it
// returns the address of the calling instance. We should do a rename to
// make this clearer.
NodeAddresses(ctx context.Context, name types.NodeName) ([]v1.NodeAddress, error)
// NodeAddressesByProviderID returns the addresses of the specified instance.
// The instance is specified using the providerID of the node. The