Skip to content

fix(examples): correct pool name in simple-tenant.yaml RUSTFS_VOLUMES…#132

Merged
GatewayJ merged 1 commit into
rustfs:mainfrom
GatewayJ:fix/simple-tenant-docs-pool-name
Jun 5, 2026
Merged

fix(examples): correct pool name in simple-tenant.yaml RUSTFS_VOLUMES…#132
GatewayJ merged 1 commit into
rustfs:mainfrom
GatewayJ:fix/simple-tenant-docs-pool-name

Conversation

@GatewayJ

@GatewayJ GatewayJ commented Jun 5, 2026

Copy link
Copy Markdown
Member

… comment

The comment referenced 'pool-0' but the actual pool name in the manifest is 'primary'. Update the RUSTFS_VOLUMES example to match the real StatefulSet naming pattern: {tenant}-{pool}-{ordinal}.

Type of Change

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

Related Issues

Summary of Changes

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.

… comment

The comment referenced 'pool-0' but the actual pool name in the manifest
is 'primary'. Update the RUSTFS_VOLUMES example to match the real
StatefulSet naming pattern: {tenant}-{pool}-{ordinal}.
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@GatewayJ GatewayJ added this pull request to the merge queue Jun 5, 2026
Merged via the queue into rustfs:main with commit e25ce64 Jun 5, 2026
2 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.

1 participant