Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Helm chart server requires cert-manager clusterissuer only, issuer cannot be used #4130

Open
flashpixx opened this issue Jan 10, 2025 · 0 comments

Comments

@flashpixx
Copy link

Hello,

Within the Helm chart, the ingress needs a ClusterIssues of the cert-manager
https://github.com/armadaproject/armada/blob/master/deployment/armada/templates/ingress.yaml#L16-L17

Expected behavior

In my case I'm using an Issuer resource of the cert-manager and the cert-manager is installed within the namespace, not globally within the cluster. So it should be possible to use both settings a ClutserIssuer (cert-manager.io/cluster-issuer) or Issuer (cert-manager.io/issuer)

Actual behavior

It is not possible to use the ingress setup with an Issuer resource.

Additional context

  1. IMHO it could be a better solution to remove the cert-manager/* configuration lines from the template and append them into the values.yaml as default annotation list, it does not beak the current behavior but it make it possible to overwrite the configuration with local settings (see https://github.com/armadaproject/armada/blob/master/deployment/armada/templates/ingress.yaml#L19 ).
  2. Another solution is to support both possibilities ClusterIssuer and Issuer, but the first suggestion is more flexible

Thanks a lot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant