Repository navigation
fix: [Cosmos] Rename HA recommendation and extend it to cover PPAF feature - #848
Open
Tomas Kovarik (tomtronic) wants to merge 3 commits into
Open
Tomas Kovarik (tomtronic) wants to merge 3 commits into
Tomas Kovarik (tomtronic) wants to merge 3 commits into
Conversation
Achieving 5 9's availability previously required enabling multi-region writes. Enabling it has implications for application design, and not all applications are eligible. Recently, Cosmos DB GA'ed Per-Partition Automatic Failover (PPAF) which provides same high SLAs in single-region writes set up opening up path to very high availability for many more applications. This PR does two things: - rename the APRL recommendation to Enable Business Critical BC tier, which covers scenarios where either feature is enabled - update the relevant KQL to include check for PPAF disabled
Tomas Kovarik (tomtronic)
changed the title
Tomas Kovarik (tomtronic)
marked this pull request as ready for review
October 6, 2026 00:51
Contributor
There was a problem hiding this comment.
Copilot review overview
🔵 Needs a closer look
The PPAF property's availability and disabled-state representation in Azure Resource Graph need human validation.
Review effort: Balanced
Findings: None
What changed in this PR
Updates the Cosmos DB high-availability recommendation to cover both multi-region writes and Per-Partition Automatic Failover (PPAF).
Changes:
- Renames the recommendation to Business Critical tier and explains both options.
- Adds a PPAF-disabled condition to the detection query.
| File | Description |
|---|---|
| azure-resources/DocumentDB/databaseAccounts/recommendations.yaml | Updates the recommendation title and explanation. |
| azure-resources/DocumentDB/databaseAccounts/kql/9ce78192-74a0-104c-b5bb-9a443f941649.kql | Filters for accounts with PPAF explicitly disabled. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Achieving 5 9's availability previously required enabling multi-region writes capability. However, doing so has profound implications for application design, and not all applications are eligible/ready.
Recently, Cosmos DB GA'ed Per-Partition Automatic Failover (PPAF) which provides same high SLAs but in a single-region writes set up, opening up path to very high availability for many more applications.
This PR does two things:
Overview/Summary
Related Issues/Work Items
Breaking Changes
As part of this pull request I have
mainbranch