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

Microsoft.Automation/automationAccounts/configurations require location property however Bicep does not include ti #5631

Open
nazakathussain opened this issue Jan 13, 2022 · 3 comments

Comments

@nazakathussain
Copy link

Bicep version
Bicep CLI version 0.4.1124 (66c84c8)

Describe the bug
To deploy Microsoft.Automation/automationAccounts/configurations a number of properties are required including location, even though it is a read only property. Bicep does not support the property however in ARM this is a property and during deployment this is expected.

Example code:

resource automationAccountsConfiguration 'Microsoft.Automation/automationAccounts/configurations@2019-06-01' = { parent: automationAccounts name: 'ExampleDSCConfiguration' }

Error on the portal:

{ "status": "Failed", "error": { "code": "LocationRequired", "message": "The location property is required for this definition." } }

To Reproduce
Deploy the below code using Bicep:

resource automationAccounts 'Microsoft.Automation/automationAccounts@2021-06-22' = {
  name: 'ExampleAutomation'
  location: location
  identity: {
    type: 'SystemAssigned'
  }
  properties: {
    encryption: {
      identity: {}
      keySource: 'Microsoft.Automation'
    }
    sku: {
      name: 'Basic'
    }
  }
  tags: {}
}

resource automationAccountsConfiguration 'Microsoft.Automation/automationAccounts/configurations@2019-06-01' = {
  parent: automationAccounts
  name: 'ExampleDSC'
}
@ghost ghost added the Needs: Triage 🔍 label Jan 13, 2022
@alex-frankel
Copy link
Collaborator

In addition to the swagger issue, you should be unblocked once #5509 is merged. cc @miqm as FYI

@alex-frankel
Copy link
Collaborator

FYI - as of today's 0.4.1272 release, this error should not be a warning. I will leave the issue open to track the swagger specification issue.

@Afsalmc
Copy link

Afsalmc commented Jul 18, 2023

I'm facing the same issue with Microsoft.Automation/automationAccounts/credentials.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants