Set topologySpreadConstraints type to array

[gjacquet]

Set type to array in order to match type from Kubernetes API specs: https://kubernetes.io/docs/concepts/scheduling-eviction/topology-spread-constraints/#topologyspreadconstraints-field

Addresses #746

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: gjacquet / name: Guillaume Jacquet (437a2c8)

[Allex1]

@gjacquet pls sign the cla

[gjacquet]

@Allex1 done!

[TylerHelmuth]

@gjacquet can you provide a render of the new values.yaml in action? I want to make sure the deployment template doesn't need changed.

[gjacquet]

In my case it gave something like this:

  topologySpreadConstraints:
  - labelSelector:
      matchLabels:
        app.kubernetes.io/instance: opentelemetry-collector
        app.kubernetes.io/name: opentelemetry-collector
        component: standalone-collector
    maxSkew: 1
    topologyKey: topology.kubernetes.io/zone
    whenUnsatisfiable: ScheduleAnyway
  - labelSelector:
      matchLabels:
        app.kubernetes.io/instance: opentelemetry-collector
        app.kubernetes.io/name: opentelemetry-collector
        component: standalone-collector
    maxSkew: 1
    topologyKey: kubernetes.io/hostname
    whenUnsatisfiable: DoNotSchedule

[gjacquet]

I would not expect any change to be required as toYaml is used to render this value and it works on all sort of types, including lists and objects.
https://github.com/open-telemetry/opentelemetry-helm-charts/blob/main/charts/opentelemetry-collector/templates/_pod.tpl#L191