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

fully move to bicep #249

Open
fmms opened this issue Dec 8, 2021 · 4 comments · May be fixed by #282
Open

fully move to bicep #249

fmms opened this issue Dec 8, 2021 · 4 comments · May be fixed by #282
Assignees
Labels
committed Committed item feature Feature Requests
Milestone

Comments

@fmms
Copy link

fmms commented Dec 8, 2021

Hi,

currently the repository contains the bicep templates AND the arm template.

Indeed the compile step is not part of the build pipeline and thus just the pre-compiled ARM template is executed:

csmFile: "$(System.DefaultWorkingDirectory)/infra/main.json"

Shouldn't

az bicep build --file .\main.bicep

be added to first compile bicep to ARM and then deploy that ...

Thanks for the great repo!

@marvinbuss
Copy link
Collaborator

We have this on our backlog. The GitHub Actions actually already support the deployment of bicep templates. This is not the case for the ADO Task, but I reached out to the team to get clarity on this to have a consistent experience across ADO and GH.
I will keep you posted.

@marvinbuss marvinbuss added backlog Backlog item feature Feature Requests labels Dec 8, 2021
@marvinbuss marvinbuss added this to the v1.3.0 milestone Dec 8, 2021
@marvinbuss marvinbuss self-assigned this Dec 8, 2021
@fmms
Copy link
Author

fmms commented Dec 8, 2021

@marvinbuss my idea was pretty much based on https://blog.johnnyreilly.com/2021/03/23/bicep-meet-azure-pipelines-2 .

Adding a

- bash: az bicep build --file infra/main.json
  displayName: 'Compile Bicep to ARM'

which then would remove the redundant file already from the repo.

@marvinbuss
Copy link
Collaborator

I understand what you meant. We will most likely have to do that, but optimally, this is something that would be supported by the task itself, so that we do not have to compile the main.bicep.
Here is the issue, where this gets tracked: microsoft/azure-pipelines-tasks#15337

@marvinbuss
Copy link
Collaborator

Team is working on implementing the above. Once that is done, we will switch to deploying bicep files instead of ARM.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
committed Committed item feature Feature Requests
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants