Remove TODO about API proxy checking for svc

This commit is contained in:
Tim Hockin 2022-12-23 12:32:17 -08:00
parent d2504c94a0
commit ed3ebbaaa7
No known key found for this signature in database

View File

@ -464,7 +464,7 @@ func (r *REST) ResourceLocation(ctx context.Context, id string) (*url.URL, http.
// but in the expected case we'll only make one.
for try := 0; try < len(ss.Addresses); try++ {
addr := ss.Addresses[(addrSeed+try)%len(ss.Addresses)]
// TODO(thockin): do we really need this check?
// We only proxy to addresses that are actually pods.
if err := isValidAddress(ctx, &addr, r.pods); err != nil {
utilruntime.HandleError(fmt.Errorf("Address %v isn't valid (%v)", addr, err))
continue