Add http- prefix to port names in collector and agent services

[cnvergence]

Signed-off-by: Karol Szwaj [email protected]

Which problem is this PR solving?

• Add option to change port names helm-charts#344 (comment) (partially)

Short description of the changes

Adding a prefix to service port names, resolves issues with istioctl

Info [IST0118] (Service tracing-jaeger-agent.kyma-system) Port name config-rest (port: 5778, targetPort: 5778) doesn't follow the naming convention of Istio port.
Info [IST0118] (Service tracing-jaeger-collector-headless.kyma-system) Port name c-tchan-trft (port: 14267, targetPort: 14267) doesn't follow the naming convention of Istio port.
Info [IST0118] (Service tracing-jaeger-collector.kyma-system) Port name c-tchan-trft (port: 14267, targetPort: 14267) doesn't follow the naming convention of Istio port.

Potentially possible problems similar to this old issue.

[frzifus]

thanks!

[rubenvp8510]

LGTM

[codecov]

Codecov Report

Merging #1862 (c79b056) into main (06bcef6) will not change coverage.
The diff coverage is 100.00%.

❗ Current head c79b056 differs from pull request most recent head f5dcc97. Consider uploading reports for the commit f5dcc97 to get more accurate results

@@           Coverage Diff           @@
##             main    #1862   +/-   ##
=======================================
  Coverage   89.42%   89.42%           
=======================================
  Files         100      100           
  Lines        6118     6118           
=======================================
  Hits         5471     5471           
  Misses        471      471           
  Partials      176      176           

Impacted Files	Coverage Δ
pkg/service/agent.go	100.00% <100.00%> (ø)
pkg/service/collector.go	100.00% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 06bcef6...f5dcc97. Read the comment docs.

[frzifus]

hi @cnvergence, could you tell me how you produced those lines using istioctl?

Info [IST0118] (Service tracing-jaeger-agent.kyma-system) Port name config-rest (port: 5778, targetPort: 5778) doesn't follow the naming convention of Istio port.
Info [IST0118] (Service tracing-jaeger-collector-headless.kyma-system) Port name c-tchan-trft (port: 14267, targetPort: 14267) doesn't follow the naming convention of Istio port.
Info [IST0118] (Service tracing-jaeger-collector.kyma-system) Port name c-tchan-trft (port: 14267, targetPort: 14267) doesn't follow the naming convention of Istio port.

update: seems to be istioctl analyze :)

[cnvergence]

hey @frzifus, yes, it is istioctl analyze :)