Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

1779 single subscription scenario fails #1782

Merged
merged 4 commits into from
Oct 9, 2024
Merged

1779 single subscription scenario fails #1782

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
9195745
Update management group values to use platform variable for single pl…
arjenhuitema Oct 2, 2024
4ffc581
Update managementSubscriptionId to use singlePlatformSubscriptionId p…
arjenhuitema Oct 2, 2024
3505c9d
Whats New: Fix bug in Portal Accelerator for single platform subscrip…
arjenhuitema Oct 2, 2024
f189473
Merge branch 'main' into 1779-single-subscription-scenario-fails
jtracey93 Oct 9, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions docs/wiki/Whats-new.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@

- [Updates](#updates)
- [🔃 Policy Refresh Q1 FY25](#-policy-refresh-q1-fy25)
- [October 2024](#october-2024)
- [September 2024](#september-2024)
- [August 2024](#august-2024)
- [July 2024](#july-2024)
Expand Down Expand Up @@ -68,6 +69,12 @@ Here's what's changed in Enterprise Scale/Azure Landing Zones:

>Note: there a known issue when re-deploying ALZ using the portal accelerator with CMK enabled. For details please see [Known Issues](ALZ-Known-Issues).

### October 2024

#### Tooling

- Resolved a bug in the Portal Accelerator related to deploying the single platform subscription setup. Incorrect parameter settings led to the failure of AMBA, as it erroneously attempted to deploy to a standard management group structure instead of a single platform management group as needed.

### September 2024

#### Documentation
Expand Down
12 changes: 6 additions & 6 deletions eslzArm/eslzArm.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2346,13 +2346,13 @@
"value": "[variables('mgmtGroups').platform]"
},
"IdentityManagementGroup": {
"value": "[variables('mgmtGroups').identity]"
"value": "[variables('mgmtGroups').platform]"
},
"managementManagementGroup": {
"value": "[variables('mgmtGroups').management]"
"value": "[variables('mgmtGroups').platform]"
},
"connectivityManagementGroup": {
"value": "[variables('mgmtGroups').connectivity]"
"value": "[variables('mgmtGroups').platform]"
},
"LandingZoneManagementGroup": {
"value": "[variables('mgmtGroups').lzs]"
Expand Down Expand Up @@ -2391,7 +2391,7 @@
"value": "[array(parameters('ambaAgEmailContact'))]"
},
"managementSubscriptionId": {
"value": "[parameters('managementSubscriptionId')]"
"value": "[parameters('singlePlatformSubscriptionId')]"
},
"enableAMBALoadBalancing": {
"value": "[parameters('enableAMBALoadBalancing')]"
Expand Down Expand Up @@ -8861,7 +8861,7 @@
}
},
/*
Note: ES Lite only: assign policy for identity to deny subnet without NSG
Note: ES Lite only: assign policy for identity to deny subnet without NSG
*/
{
// Assigning deny subnet without nsg policy to identity management group if condition is true
Expand Down Expand Up @@ -8892,7 +8892,7 @@
}
},
/*
Note: ES Lite only: assign policy to deny management ports from internet to platform MG
Note: ES Lite only: assign policy to deny management ports from internet to platform MG
*/
{
// Assigning deny management ports from internet policy landing zones management group if condition is true
Expand Down
Loading