Skip to content

feat: Added main.waf.parameters.json with updated parameters and updated deployment.md with waf deployment steps #697

Open
Kanchan-Microsoft wants to merge 2 commits intodevfrom
psl-pk-waf
Open

feat: Added main.waf.parameters.json with updated parameters and updated deployment.md with waf deployment steps #697
Kanchan-Microsoft wants to merge 2 commits intodevfrom
psl-pk-waf

Conversation

@Kanchan-Microsoft
Copy link
Contributor

Purpose

This pull request adds improved documentation and configuration options for deploying the Content Generation Solution Accelerator, with a focus on distinguishing between sandbox/dev and production environments. It introduces a new production-ready parameters file aligned with the Well-Architected Framework (WAF), updates deployment instructions, and clarifies parameter usage for different environments.

Deployment Documentation Improvements:

  • Added a new "Deployment Options" section to DEPLOYMENT.md, explaining the use of the default sandbox (main.parameters.json) and new production (main.waf.parameters.json) parameter files, and providing step-by-step instructions for switching between them.

Production Configuration Support:

  • Introduced main.waf.parameters.json, a production-ready parameters file with WAF-aligned settings, including enabled monitoring, private networking, and scalability options, as well as required VM admin credentials.

Sandbox/Dev Configuration Adjustments:

  • Removed enablePrivateNetworking, enableMonitoring, enableScalability, and enableRedundancy parameters from main.parameters.json to simplify the sandbox/dev configuration and reduce potential confusion between environments.

Does this introduce a breaking change?

  • Yes
  • No

Golden Path Validation

  • I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

  • I have validated the deployment process successfully and all services are running as expected with this change.

What to Check

Verify that the following are valid

  • ...

Other Information

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.

2 participants