fix: Removal of a default TLS for the Keycloak ingress on values file

[HujinoKun]

Hello,

When managing certificates through ingress annotations, with the current version it is impossible to remove the default tls section on the Keycloak ingress.

In my case, I manage the certificate part through my Azure Application Gateway and its controller on AKS.

Bad output :

# Source: keycloakx/templates/ingress.yaml
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
  name: keycloak
  namespace: default
  annotations:
    appgw.ingress.kubernetes.io/cookie-based-affinity: "true"
    appgw.ingress.kubernetes.io/ssl-redirect: "true"
    appgw.ingress.kubernetes.io/appgw-ssl-certificate: "cert-xx"
  labels:
    helm.sh/chart: keycloakx-7.1.4
    app.kubernetes.io/name: keycloakx
    app.kubernetes.io/instance: keycloak
    app.kubernetes.io/version: "26.4.2"
    app.kubernetes.io/managed-by: Helm
spec:
  ingressClassName: azure-application-gateway
  tls: <-- BAD : Default values
    - hosts:
        - "keycloak.example.com"
  rules:
    - host: "keycloak.domain.com"
      http:
        paths:
          - path: "/"
            pathType: Prefix
            backend:
              service:
                name: keycloak-http
                port:
                  name: http

Good output :

# Source: keycloakx/templates/ingress.yaml
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
  name: keycloak
  namespace: default
  annotations:
    appgw.ingress.kubernetes.io/cookie-based-affinity: "true"
    appgw.ingress.kubernetes.io/ssl-redirect: "true"
    appgw.ingress.kubernetes.io/appgw-ssl-certificate: "cert-xx"
  labels:
    helm.sh/chart: keycloakx-7.1.4
    app.kubernetes.io/name: keycloakx
    app.kubernetes.io/instance: keycloak
    app.kubernetes.io/version: "26.4.2"
    app.kubernetes.io/managed-by: Helm
spec:
  ingressClassName: azure-application-gateway
  rules:
    - host: "keycloak.domain.com"
      http:
        paths:
          - path: "/"
            pathType: Prefix
            backend:
              service:
                name: keycloak-http
                port:
                  name: http

This fix allows us to completely bypass the tls section on ingress by removing the default values.

I also bumped the chart version by incrementing the number assigned to the fix, from 7.1.4 to 7.1.5.

[HujinoKun]

@stefangries Hello,

Could you spare a few minutes for this PR?

[stefangries]

I'm fine with the changes. Please remove the version bump as we only bump versions once we release a new one and not with every pull request.

[HujinoKun]

Hello @stefangries :)

I have just made the amend and rebase commands in order to remove the change to the Chart.yaml file.