Sidecar exposes all ports as TCP, but some are expected to be UDP

[garyfritz]

Analogous to #577, when injecting agent containers, the container ports of injected sidecar containers are exposed as TCP, but three out of four should be UDP.

This is from an agent pod from a daemon set (which is correct according to #577):

  jaeger-agent-daemonset:
    Image:         jaegertracing/jaeger-agent:1.15.0
    Ports:         5775/UDP, 5778/TCP, 6831/UDP, 6832/UDP, 14271/TCP
    Host Ports:    5775/UDP, 5778/TCP, 6831/UDP, 6832/UDP, 14271/TCP
    Args:
      --reporter.grpc.host-port=dns:///izjaeger-collector-headless.default:14250
      --reporter.type=grpc

While this is from an injected agent container in some of my pods:

  jaeger-agent:
    Image:         jaegertracing/jaeger-agent:1.15.0
    Ports:         5775/TCP, 5778/TCP, 6831/TCP, 6832/TCP
    Host Ports:    0/TCP, 0/TCP, 0/TCP, 0/TCP
    Args:
      --reporter.grpc.host-port=dns:///izjaeger-collector-headless.default:14250
      --reporter.type=grpc

I'd suggest adding the missing protocol specifications to

jaeger-operator/pkg/inject/sidecar.go

Lines 158 to 175 in 058f4fa

	Ports: []corev1.ContainerPort{
	{
	ContainerPort: zkCompactTrft,
	Name: "zk-compact-trft",
	},
	{
	ContainerPort: configRest,
	Name: "config-rest",
	},
	{
	ContainerPort: jgCompactTrft,
	Name: "jg-compact-trft",
	},
	{
	ContainerPort: jgBinaryTrft,
	Name: "jg-binary-trft",
	},
	},

[garyfritz]

Closing since MR #773 is now merged.