jp-gouin/helm-openldap

init-tls-secret image exec /bin/sh: exec format error

Closed this issue · 2 comments

Describe the bug
A clear and concise description of what the bug is.
init-tls-secret image does not appear to be multi-arch by default.

To Reproduce
Steps to reproduce the behavior:

  1. Attempt helm install on an arm64 cluster

Expected behavior
Helm chart installs without issue

Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
Add any other context about the problem here.

The default image appears to be alpine/openssl, which not only is not multiarch, but has not been built in more than 3 years.

If you need a prebuilt container with openssl installed, the docker nginx container is relatively small, is multiarch, has openssl, and is updated frequently.

Alternatively I'd be happy to submit a PR that uses GitHub actions to regularly build a multi-arch alpine-based openssl container and stores it in GitHub container registry.

Yeah I know , I noticed it when I implemented the feature that's why you can configure this under customTls.image.repository
I was a bit lazy 😁

Feel free to maybe create a repo with what you suggest , if you don't want to own it you can transfer the ownership to me.

I think it best to separate the container creation from this chart and have them into separate repo

stale commented

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.