Skip to content

Conversation

@mattdot
Copy link
Collaborator

@mattdot mattdot commented Oct 15, 2025

This pull request improves the documentation for infrastructure resilience, testing, and advanced deployment scenarios. It refactors and expands the disaster recovery and resilience guidance, moves detailed testing instructions to a dedicated guide, and clarifies networking requirements for multi-region Power Platform geographies. The changes are primarily documentation updates to help users better understand deployment options, security, and operational considerations.

Documentation Restructuring and Content Relocation:

  • The detailed testing instructions have been removed from README.md and replaced with a concise pointer to a new Testing Guide, streamlining the main documentation and making test execution guidance easier to find and maintain.

Infrastructure Resilience and Disaster Recovery Guidance:

  • The former docs/bcdr_considerations.md has been renamed to docs/infrastructure_resilience.md and significantly rewritten. The new guide clearly defines three deployment tiers (Basic, Zone-redundant, Regional failover ready), provides prescriptive recommendations for each, and clarifies that the template provides only the foundational networking and identity scaffolding for regional failover, not full automation.
  • The guide now includes explicit guidance about dual-region virtual network requirements for multi-region Power Platform geographies, ensuring users do not remove necessary networking scaffolding in those scenarios. [1] [2]

Clarifications and Known Limitations:

  • The documentation now explicitly states that the template does not automate cross-region failover, data replication, or DNS/traffic management for regional disaster recovery; these must be handled manually by the user. [1] [2]
  • Known issues and workarounds regarding the inability to disable secondary-region networking scaffolding in development scenarios are documented, helping users understand current template limitations and cost implications.

Additional Improvements:

  • References to supporting guides (security, CI/CD, custom networking, custom resource groups) have been updated for clarity and consistency, with corrected links and improved explanations of advanced scenarios.
  • The last updated date has been refreshed to reflect the new guidance.

Resolves #304

Copilot AI review requested due to automatic review settings October 15, 2025 17:56
@mattdot mattdot requested a review from a team as a code owner October 15, 2025 17:56
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Clarifies mandatory dual-region networking requirements in multi-region Power Platform geographies and updates disaster recovery documentation to remove prior guidance about disabling secondary-region networking. Emphasizes that dual virtual networks are a compliance prerequisite and adds authoritative reference links.

  • Clarifies that dual-region networking is mandatory (cannot be disabled) in multi-region geographies.
  • Replaces prior workaround/limitation section with forward-looking status note.
  • Adds important notice about lack of automated cross-region failover plus new reference links.

@devorekristen devorekristen self-requested a review October 15, 2025 21:56
devorekristen
devorekristen previously approved these changes Oct 16, 2025
@mattdot mattdot self-assigned this Oct 16, 2025
@mattdot mattdot added the 📖 documentation Improvements or additions to documentation label Oct 16, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 7 comments.

mattdot and others added 2 commits October 16, 2025 06:47
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
@devorekristen devorekristen merged commit dd637bf into main Oct 16, 2025
8 checks passed
@devorekristen devorekristen deleted the 304-failover-network-documenation branch October 16, 2025 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📖 documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Task] Update BCDR Documentation to explain requirements for primary/failover network

4 participants