-
Notifications
You must be signed in to change notification settings - Fork 55
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
Added support for additional tls certs to custom services #2642
base: main
Are you sure you want to change the base?
Conversation
chore: some internal docs for helm repo and auth
Codecov ReportAttention:
... and 28 files with indirect coverage changes 📢 Thoughts on this report? Let us know!. |
… `certs` field) also store pem data in memory, not a file
a bit of lifetime trickery for the async path
Testing:
|
See PR mittwald/go-helm-client#182 for a possible fix in the mittwald client to expose the necessary information to change by the caller. The upstream PR is approved and waiting for merging since Aug 30, i.e. 1.5 months as of the time of writing this here. Decision: Suspending work on this PR and associated ticket until the upstream PR is merged and usable. |
fix #2555
Similar to the support for certs by export to OCI registries.
Difference:
might be better to adjust code here to use a separate secret as well, allow for sharing with export, and between services