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.

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

added parAutomationAccountUseManagedIdentity parameter #433

Merged
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
25 changes: 0 additions & 25 deletions .ps-rule/ManagedIdentityForAutomation.Rule.yaml

This file was deleted.

12 changes: 12 additions & 0 deletions infra-as-code/bicep/modules/logging/generateddocs/logging.bicep.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ parLogAnalyticsWorkspaceLogRetentionInDays | No | Number of days of log re
parLogAnalyticsWorkspaceSolutions | No | Solutions that will be added to the Log Analytics Workspace.
parAutomationAccountName | No | Automation account name.
parAutomationAccountLocation | No | Automation Account region name. - Ensure the regions selected is a supported mapping as per: https://docs.microsoft.com/azure/automation/how-to/region-mappings.
parAutomationAccountUseManagedIdentity | No | Automation Account - use managed identity.
parTags | No | Tags you would like to be applied to all resources in this module.
parAutomationAccountTags | No | Tags you would like to be applied to Automation Account.
parLogAnalyticsWorkspaceTags | No | Tags you would like to be applied to Log Analytics Workspace.
Expand Down Expand Up @@ -78,6 +79,14 @@ Automation Account region name. - Ensure the regions selected is a supported map

- Default value: `[resourceGroup().location]`

### parAutomationAccountUseManagedIdentity

![Parameter Setting](https://img.shields.io/badge/parameter-optional-green?style=flat-square)

Automation Account - use managed identity.

- Default value: `True`

### parTags

![Parameter Setting](https://img.shields.io/badge/parameter-optional-green?style=flat-square)
Expand Down Expand Up @@ -164,6 +173,9 @@ outAutomationAccountId | string |
"parAutomationAccountLocation": {
"value": "[resourceGroup().location]"
},
"parAutomationAccountUseManagedIdentity": {
"value": true
},
"parTags": {
"value": {}
},
Expand Down
6 changes: 6 additions & 0 deletions infra-as-code/bicep/modules/logging/logging.bicep
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,9 @@ param parAutomationAccountName string = 'alz-automation-account'
@sys.description('Automation Account region name. - Ensure the regions selected is a supported mapping as per: https://docs.microsoft.com/azure/automation/how-to/region-mappings.')
param parAutomationAccountLocation string = resourceGroup().location

@sys.description('Automation Account - use managed identity.')
param parAutomationAccountUseManagedIdentity bool = true

@sys.description('Tags you would like to be applied to all resources in this module.')
param parTags object = {}

Expand All @@ -78,6 +81,9 @@ resource resAutomationAccount 'Microsoft.Automation/automationAccounts@2021-06-2
name: parAutomationAccountName
location: parAutomationAccountLocation
tags: parAutomationAccountTags
identity: parAutomationAccountUseManagedIdentity ? {
type: 'SystemAssigned'
} : null
properties: {
sku: {
name: 'Basic'
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,9 @@
"parAutomationAccountLocation": {
"value": "eastus2"
},
"parAutomationAccountUseManagedIdentity": {
"value": true
},
"parTags": {
"value": {
"Environment": "POC"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,9 @@
"parAutomationAccountLocation": {
"value": "chinaeast2"
},
"parAutomationAccountUseManagedIdentity": {
"value": true
},
"parTags": {
"value": {
"Environment": "POC"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@ module baseline_logging '../logging.bicep' = {
'VMInsights'
]
parAutomationAccountName: 'alz-automation-account'
parAutomationAccountUseManagedIdentity: true
parTelemetryOptOut: false
}
}