Skip to content

[codex] add tenant status condition reconciliation#117

Merged
GatewayJ merged 2 commits intorustfs:mainfrom
GatewayJ:feature/status-conditions-events
May 9, 2026
Merged

[codex] add tenant status condition reconciliation#117
GatewayJ merged 2 commits intorustfs:mainfrom
GatewayJ:feature/status-conditions-events

Conversation

@GatewayJ
Copy link
Copy Markdown
Member

@GatewayJ GatewayJ commented May 7, 2026

Type of Change

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

Related Issues

N/A

Summary of Changes

Add a Tenant status condition framework for RustFS Operator and wire it into reconcile and Console APIs.

This PR introduces canonical condition types, reason values, condition upsert/sorting helpers, currentState summarization, and a runtime StatusBuilder. Reconcile now patches status before known early-return failures, records Events using the same stable reason as the condition, distinguishes direct pool deletion from pool rename, and avoids repeated no-op status patches.

Console list/detail/topology now read a shared status summary derived from status.conditions, including ready/reconciling/degraded flags, primary reason, stale generation detection, and next action hints.

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: Tenant status conditions and Console tenant responses include additional status summary fields.

Verification

make pre-commit

Additional Notes

  • PR is opened as draft for maintainer review.
  • Console next_actions are generated from the status reason registry and are not written back to CR status.
  • Status patch failures record a Warning Event with reason StatusPatchFailed without recursively patching status.

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 force-pushed the feature/status-conditions-events branch from 1127bdb to 67bc255 Compare May 8, 2026 17:40
@GatewayJ GatewayJ marked this pull request as ready for review May 9, 2026 02:39
@GatewayJ GatewayJ added this pull request to the merge queue May 9, 2026
Merged via the queue into rustfs:main with commit 9c24a37 May 9, 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