KEMBAR78
Service Metadata differs in fabric8 and k8s client in some corner case · Issue #1405 · spring-cloud/spring-cloud-kubernetes · GitHub
Skip to content

Service Metadata differs in fabric8 and k8s client in some corner case #1405

@wind57

Description

@wind57

As part of the ServiceInstance that a discovery client returns, there is a field called metadata, which is some meta information about the service. As part of it, we return a so called port name and port number, it really does not even matter what they mean or where they come from.

The relevant part for this issue is that sometimes port name might not be present. In that case:

  • fabric8 ignores that field
  • k8s takes that field into account

For example, here is how fabric8 would represent a response:

Map.of("k8s_namespace", "namespace1", "type", "ClusterIP")

while k8s:

Map.of("<unset>", "8080", "k8s_namespace", "namespace1", "type", "ClusterIP")

Notice the "<unset>", "8080".


imo, k8s is not doing the correct thing, for a few reasons:

  • in general, we do not care about this metadata too much. We care about the "primary port" of the service, that is returned elsewhere and is computed correctly for both clients.
  • if there are multiple ports without a name, k8s will override each other, because it does:
Map<String, String> ports = ...

for (CoreV1EndpointPorts onePort: allOfServicesPorts) { 
     if (onePort.getName == null) {
           ports.put("<unset>", onePort.getNumber())
     }
}

you can see that if there are more then one port without a name, one will override the other.

I am open to any suggestions, cause this is rather confusing for me too.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions