You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Nov 5, 2021. It is now read-only.
What happened: When using k8s service discovery to discover services, ports do not get discovered.
What you expected to happen: pb.Resource for the discovered k8s service to contain svc.Spec.Ports[0].Port
How to reproduce it (as minimally and precisely as possible): create a k8s cluster and use k8s discovery to discover a service. the returned *pb.Resource has an empty Port
Anything else we need to know?: it looks like ports are not set for k8s services at all, not sure if this is intentional or not. There's also an issue with services potentially containing multiple ports and pb.Resource only has a single Port field. Perhaps iterating over ports and creating a new pb.Resource for each unique one could be the right solution?
The text was updated successfully, but these errors were encountered:
Your assessment is right about services discovery not populating the port. This seems like an oversight. For multiple ports case, I think we need to handle multiple ports the same way as endpoints:
What happened: When using k8s service discovery to discover services, ports do not get discovered.
What you expected to happen: pb.Resource for the discovered k8s service to contain
svc.Spec.Ports[0].Port
How to reproduce it (as minimally and precisely as possible): create a k8s cluster and use k8s discovery to discover a service. the returned *pb.Resource has an empty
Port
Anything else we need to know?: it looks like ports are not set for k8s services at all, not sure if this is intentional or not. There's also an issue with services potentially containing multiple ports and pb.Resource only has a single Port field. Perhaps iterating over ports and creating a new pb.Resource for each unique one could be the right solution?
The text was updated successfully, but these errors were encountered: