search

jp-gouin / helm-openldap

Helm chart of Openldap in High availability with multi-master replication and PhpLdapAdmin and Ltb-Passwd
Apache License 2.0
199 stars 118 forks source link

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

Closed e3b0c442 closed 11 months ago

e3b0c442 commented 1 year ago

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.

jp-gouin commented 1 year ago

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[bot] commented 11 months ago

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.