Skip to content

feat: add SMB scenarios 10 and 11, load scenarios from JSON config#532

Merged
jaredfholgate merged 2 commits intomainfrom
feat/add-smb-scenarios-10-11
Apr 10, 2026
Merged

feat: add SMB scenarios 10 and 11, load scenarios from JSON config#532
jaredfholgate merged 2 commits intomainfrom
feat/add-smb-scenarios-10-11

Conversation

@jaredfholgate
Copy link
Copy Markdown
Member

@jaredfholgate jaredfholgate commented Apr 10, 2026

Summary

Adds missing SMB scenarios 10 and 11 to the accelerator folder structure creation, and refactors scenario management to use the existing TerraformScenarios.json config file as the single source of truth.

Closes Azure/Azure-Landing-Zones#4105

Changes

TerraformScenarios.json

  • Added path property to each scenario entry, mapping scenario numbers to their tfvars file paths
  • Scenarios 10 and 11 were already present (labels only) — now they also have paths:
    • 10: smb-single-region/hub-and-spoke-vnet.tfvars
    • 11: smb-single-region/virtual-wan.tfvars

New-AcceleratorFolderStructure.ps1

  • Replaced hardcoded `` hashtable with dynamic loading from TerraformScenarios.json
  • Fixed int64/int32 type mismatch when looking up scenario paths (JSON parser returns int64, parameter is [int]/int32 — without the cast, hashtable lookup returns `` and Copy-Item creates a directory instead of copying the file)

- Add path property to TerraformScenarios.json for all scenarios
- Add scenario 10 (SMB Single-Region Hub and Spoke VNet)
- Add scenario 11 (SMB Single-Region Virtual WAN)
- Replace hardcoded scenario hashtable with dynamic JSON loading
- Fix int64/int32 type mismatch when looking up scenario paths
@jaredfholgate jaredfholgate merged commit 7832440 into main Apr 10, 2026
7 checks passed
@jaredfholgate jaredfholgate deleted the feat/add-smb-scenarios-10-11 branch April 10, 2026 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs: Triage 🔍 Needs triaging by the team

Projects

Development

Successfully merging this pull request may close these issues.

[Bug]: platform-landing-zone.tfvars is a directory

2 participants