Skip to content

feat: add Kubernetes secret resolver and documentation#933

Merged
burningalchemist merged 4 commits intoburningalchemist:masterfrom
zofer1zohargo:k8s-secrets-code
Mar 22, 2026
Merged

feat: add Kubernetes secret resolver and documentation#933
burningalchemist merged 4 commits intoburningalchemist:masterfrom
zofer1zohargo:k8s-secrets-code

Conversation

@zofer1zohargo
Copy link
Contributor

  • Implement k8ssecret URL provider for reading DSN from Kubernetes secrets
  • Support both full DSN and partial DSN with template placeholders (DSN_VALUE)
  • Add comprehensive examples and documentation for k8s-secret usage
  • Register k8ssecret provider in secret resolver
  • Update main README to include Kubernetes secrets documentation
  • Bump application version to 0.20.1 and chart appVersion

ZOHARGO and others added 2 commits March 21, 2026 21:14
- Implement k8ssecret URL provider for reading DSN from Kubernetes secrets
- Support both full DSN and partial DSN with template placeholders (DSN_VALUE)
- Add comprehensive examples and documentation for k8s-secret usage
- Register k8ssecret provider in secret resolver
- Update main README to include Kubernetes secrets documentation
- Bump application version to 0.20.1 and chart appVersion

Made-with: Cursor
@zofer1zohargo
Copy link
Contributor Author

Hi @burningalchemist,
I am not sure such developments can be handled.
I think I tried all options :-(
It is now in two PRs that need to be handled together, per your request.
We should really understand how to fix the CI to handle this case.
I assume that after approving the first one the second one should work but when I bump the appVersion I also need to submit the Chart.yaml which triggers a helm chart update.
No idea how you are handling it.
Waiting for your instructions

@burningalchemist
Copy link
Owner

burningalchemist commented Mar 21, 2026

Hey @zofer1zohargo, just remove all the helm stuff from the PR (Chart.yaml), leave only the go-related code as a feature for k8ssecret provider.

I'll do the release next week (so we have the container versions published), and with the separate pr we include the helm-related template changes and the rest.

This should work 🙌

P.S. Also, please revert VERSION file back to 0.20.0. The release version bump is done in a separate commit to tag and cut off the release. I'll handle it.

Reset versions
reset version
@zofer1zohargo
Copy link
Contributor Author

Done. One day will get the hang of it :-)

@burningalchemist burningalchemist changed the title Add Kubernetes secret DSN provider and documentation feat: add Kubernetes secret resolver and documentation Mar 22, 2026
@burningalchemist burningalchemist merged commit 64a8888 into burningalchemist:master Mar 22, 2026
4 checks passed
@burningalchemist
Copy link
Owner

@zofer1zohargo Looks good, thanks! 🎉 Yeah, I'll tag you to demo the procedure. 🙌 I'll reply in the other PR shortly. 👍

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

Successfully merging this pull request may close these issues.

2 participants