Skip to content

feat(helm): Allow a user provided jwt secret - #268

Open
dacbd wants to merge 6 commits into
rustfs:mainfrom
dacbd:main
Open

dacbd wants to merge 6 commits into
rustfs:mainfrom
dacbd:main

Conversation

@dacbd

@dacbd dacbd commented Oct 4, 2026 •

Copy link
Copy Markdown

Type of Change

  • New Feature
  • Bug Fix
  • Documentation
  • Performance Improvement
  • Test/CI
  • Refactor
  • Other:

Related Issues

NA

Summary of Changes

Allow for passing in a pre existing secret for the console jwt-secret.

Checklist

  • I have read and followed the CONTRIBUTING.md guidelines
  • Passed make pre-commit (fmt-check + clippy + test + console-lint + console-fmt-check)
  • Added/updated necessary tests
  • Documentation updated (if needed)
  • CHANGELOG.md updated under [Unreleased] (if user-visible change)
  • CI/CD passed (if applicable)

Impact

  • Breaking change (CRD/API compatibility)
  • Requires doc/config/deployment update
  • Other impact:

Verification

make pre-commit

Additional Notes


Thank you for your contribution! Please ensure your PR follows the community standards (CODE_OF_CONDUCT.md) and sign the CLA if this is your first contribution.

@dacbd
dacbd marked this pull request as ready for review October 4, 2026 21:08
@dacbd

dacbd commented Oct 4, 2026 •

Copy link
Copy Markdown
Author

Hey rustfs team,

Essentially I want to populate the jwt-secret via external-secrets and the chart doesn't allow for that secret to be externally managed, this allows one to pass a secretName through the chart instead of the secret directly through the values.yaml.

If you want comments to be less verbose I can trim them down.

(commits are unverifed because edits were made from an old laptop and I don't have my yubikey for code signing handy)

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.

1 participant