The helm chart default values has ingress annotations

[robvanmieghem]

Since providing a custom values file merges with the default one and does not override, kubernetes.io/ingress.class: traefik is always present

[sameh-farouk]

@robvanmieghem Can you add more context or explain what need to be done here?

[robvanmieghem]

Looks like this issue is transferred from the tfchain_activation_service repo where the deafult values file of the helm chart had kubernetes.io/ingress.class: traefik in the ingress annotations. As I explained above,a custom values file merges with the default one and does not override so this the ingress class annotation can not be overidden. Currently the default values file has kubernetes.io/ingress.class: nginx set: https://github.com/threefoldtech/tfchain/blob/38ea78797ca2bb2c1ee27efa43b607c86b322e6c/activation-service/helm/tfchainactivationservice/values.yaml#L43 It's best to get rid of that, do mind that this needs to be communicated to the people using that chart since they will have to provide the annotation themselves if they need it.