skip internal stream hottier for suspended tenant

[nikhilsinhaparseable]

Summary by CodeRabbit

• Bug Fixes
  • Fixed hot-tier metadata provisioning to properly exclude suspended workspaces from background operations.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 07b1b06e-f116-4e66-a5dd-d905c9978d64

📥 Commits

Reviewing files that changed from the base of the PR and between f761b2d and ce57818.

📒 Files selected for processing (2)

• src/hottier.rs
• src/tenants/mod.rs

---

Walkthrough

The changes add suspension checks to hot-tier operations. A new is_workspace_suspended method is introduced in TenantMetadata to detect suspended workspaces. This method is then integrated into put_internal_stream_hot_tier and create_pstats_hot_tier to prevent hot-tier provisioning for suspended tenants.

Changes

Cohort / File(s)	Summary
Tenant Metadata src/tenants/mod.rs	Added public method is_workspace_suspended(&self, tenant_id: &str) -> bool to check if a tenant's workspace service is in the suspended services set.
Hot Tier Manager src/hottier.rs	Added dependency on tenants::TENANT_METADATA. Integrated suspension checks in put_internal_stream_hot_tier and create_pstats_hot_tier to skip hot-tier operations for tenants with suspended workspaces.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

• chore: add mandatory hottier for pstats dataset #1414: Modifies create_pstats_hot_tier and put_internal_stream_hot_tier with suspended tenant checks—directly overlaps with the main PR's changes to these same functions.
• fix hot tier path #1562: Changes hot-tier metadata handling with path resolution checks for tenant-scoped files—related to the same hot-tier operations being modified.
• fix: proper listing of hottier #1565: Introduces tenant-aware hot-tier handling with signature changes—contextually related to tenant-scoped hot-tier behavior.

Poem

🐰 Hop, hop, suspended tenants fade,
No hot-tier paths shall now be made,
With is_workspace_suspended check so keen,
Cleanest hot-tier flows ever seen! 🔥

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The pull request description is empty, missing all required sections including description of goals, rationale, key changes, and testing/documentation checkboxes.	Add a comprehensive description explaining the purpose of skipping hot-tier operations for suspended tenants, the rationale, key changes made, and complete the testing and documentation checklist.
Docstring Coverage	⚠️ Warning	Docstring coverage is 20.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately summarizes the main change: skipping hot-tier operations for suspended tenants in internal stream handling.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}