change service port name to support Istio explicit protocol selection #346
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Kubernetes has a native construct for this called
appProtocol
. Kubernetesexpects either a IANA protocol or a custom protocol prefix with a domain.
I tried setting
appProtocol
tonats.io/client
but that does not effectanything Istio, as istio expects only these protocols:
https://istio.io/latest/docs/ops/configuration/traffic-management/protocol-selection/#explicit-protocol-selection
Setting
appProtocol
totcp
probably does work, but tcp is not a validiana service name. See:
https://www.iana.org/assignments/service-names-port-numbers/service-names-port-numbers.xhtml
Therefor the least intrusive, compatible leaves us with prefixing the service
port name.
Fixes: #88