Skip to content

fix(console): use console service account in login hint#178

Merged
GatewayJ merged 1 commit into
rustfs:mainfrom
GatewayJ:agent/fix-console-login-token-hint
Jul 11, 2026
Merged

fix(console): use console service account in login hint#178
GatewayJ merged 1 commit into
rustfs:mainfrom
GatewayJ:agent/fix-console-login-token-hint

Conversation

@GatewayJ

Copy link
Copy Markdown
Member

Type of Change

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

Related Issues

Fixes #160

Summary of Changes

  • Update the Console login help command to create a token for the chart-managed Console ServiceAccount instead of the operator controller ServiceAccount.
  • Add Helm install notes that print the exact token command using the release namespace and consoleServiceAccountName helper.
  • Clarify the chart README and operator user guides to follow Helm install notes when namespace or Console ServiceAccount names are customized.

Root cause: the Console login form documented rustfs-operator, but Console login expects a Kubernetes bearer token with the Console RBAC permissions granted to rustfs-operator-console by default.

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

No runtime RBAC or authentication behavior changes are included; this only aligns user-facing token guidance with the existing Console ServiceAccount boundary.


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.

@GatewayJ GatewayJ marked this pull request as ready for review July 11, 2026 09:03
@GatewayJ GatewayJ added this pull request to the merge queue Jul 11, 2026
Merged via the queue into rustfs:main with commit c781fe3 Jul 11, 2026
3 checks passed
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.

Console login hint uses wrong ServiceAccount (rustfs-operator vs rustfs-operator-console)

1 participant