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

Commits with a bicep file and ARM template parameter file generates 2 deployments #504

Closed
StefanIvemo opened this issue Dec 21, 2021 · 2 comments · Fixed by #516
Closed
Assignees

Comments

@StefanIvemo
Copy link
Collaborator

StefanIvemo commented Dec 21, 2021

Describe the bug
If I commit a bicep file and a matching ARM template parameter file two deployments will be created by the push pipeline. One deployment for the bicep file and one for the parameter file.

Steps to reproduce

  1. Create a bicep template, storage.bicep
  2. Create an ARM template parameter file for the bicep template, storage.parameters.json
  3. Open a Pull request. (This will generate a PR comment with the WhatIf result duplicated since two deployments are triggered)
  4. Approve PR and merge to main branch
  5. Two deployments will be created:
    • AzOps-storage
    • AzOps-storage.parameters

Screenshots
image

Screenshot shows deployment history for a commit with a bicep template named test1.bicep and a parameter file named test1.parameters.json.

@daltondhcp
Copy link
Contributor

Great find! Right now, your parameters file (even for bicep deployments) will need to end with the extension .parameters.json similar to the examples in at https://docs.microsoft.com/en-us/azure/azure-resource-manager/bicep/parameter-files.

@StefanIvemo
Copy link
Collaborator Author

Great find! Right now, your parameters file (even for bicep deployments) will need to end with the extension .parameters.json similar to the examples in at https://docs.microsoft.com/en-us/azure/azure-resource-manager/bicep/parameter-files.

Of course, I made a typo in the issue. 😊

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 a pull request may close this issue.

2 participants