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

Add nightly and PR validation pipelines in Azure DevOps #437

Merged
merged 136 commits into from
Nov 8, 2021

Conversation

vidyambala
Copy link
Contributor

@vidyambala vidyambala commented Oct 4, 2021

Description

Adds Azure DevOps pipelines to:

  • nightly deploy the Bicep solution into AzureCloud and AzureUsGovernment
  • nightly deploy the Terraform solution into AzureCloud and AzureUsGovernment
  • on PR into /main run a --what-if AZ CLI deployment to ensure the Bicep templates are deployable
  • on PR into /main run a terraform plan to ensure the Terraform templates are deployable
  • adds a resourcePrefix tag to the Terraform resources

Issue reference

The issue this PR will close #49 close #94 close #466

Checklist

Please make sure you've completed the relevant tasks for this PR out of the following list:

  • All acceptance criteria in the backlog item are met
  • The documentation is updated to cover any new or changed features
  • Manual tests have passed
  • Relevant issues are linked to this PR

@vidyambala vidyambala marked this pull request as draft October 5, 2021 17:45
Copy link
Contributor

@glennmusa glennmusa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's get the failed checks resolved.

Want to look at these two:

https://github.com/Azure/missionlz/runs/4117640642?check_suite_focus=true#step:4:462

and I'll take a look at why the Terraform validation check is failing -- the error message is cryptic:

https://github.com/Azure/missionlz/runs/4117640677?check_suite_focus=true

README.md Outdated Show resolved Hide resolved
src/build/delete.sh Show resolved Hide resolved
Copy link
Contributor

@glennmusa glennmusa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @vidyambala 🚀

@glennmusa glennmusa merged commit bf33406 into main Nov 8, 2021
@glennmusa glennmusa deleted the mlz-azure-pipelines branch November 8, 2021 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants