Helm Support service.spec.loadBalancerClass

[rodrigosandrin]

Hi,

We use Antrea as our CNI, and we need to include the field loadbalancerClass: antrea in our services of type loadbalance.

Can you include it in helm templates/controller-service.yaml

Service Sample:

apiVersion: v1
kind: Service
metadata:
  name: nginx-ingress
  annotations:
    service.antrea.io/external-ip-pool: "myexternalpool"
spec:
  externalTrafficPolicy: Local
  type: LoadBalancer
  loadBalancerIP: 192.168.0.1
  loadBalancerClass: antrea

this can be achieved in this block here:

  {{- if .Values.controller.service.loadBalancerClass }}
  loadBalancerClass: {{ .Values.controller.service.loadBalancerClass }}
  {{- end }}

I think that the values.schema.json also should be changed:

            "loadBalancerClass": {
              "default": "",
              "title": "The loadBalancerClass",
              "$ref": "???"
            },

Should something be added on ref? If you want, i can open a PR with those changes

[pdabelf5]

@rodrigosandrin thank you for contacting us about this. Could you create a new feature request issue, the team will review the issue in our next Triage meeting. If the issue is something the team are happy to proceed with, a pull request would be welcomed 😄

Regarding your question about the ref, something along these lines would be needed https://raw.githubusercontent.com/nginxinc/kubernetes-json-schema/master/v1.33.1/_definitions.json#/definitions/io.k8s.api.core.v1.ServiceSpec/properties/loadBalancerClass

[rodrigosandrin]

Thanks @pdabelf5 , I opened the feature request:

#8441

[pdabelf5]

Discussion has been moved to the issue #8441