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

26294 - Feature - Accelerator Pipeline Adjustments #483

Merged
merged 17 commits into from
Mar 28, 2023

Conversation

oZakari
Copy link
Contributor

@oZakari oZakari commented Mar 27, 2023

Overview/Summary

AB#26294

General refinements to workflows and associated deployments for ALZ Accelerator.

This PR fixes/adds/changes/removes

  1. Updated accelerator workflows for policy assignment deployment to use built-in all parameter file, updated subscription placement deployment to use the orchestration module.
  2. Disabled role assignments deployment within workflow by commenting out to treating as example for end-user to enable as necessary.
  3. Changed all parameter files containing the tag of environment from PoC to Live based upon discussions with team.

Breaking Changes

No breaking changes

Testing Evidence

Policy Assignment Changes Testing
https://github.com/oZakari/ALZ-Bicep-Accelerator/actions/runs/4538894345
image

Subscription Placement Changes Testing
https://github.com/oZakari/ALZ-Bicep-Accelerator/actions/runs/4528809185
image

As part of this Pull Request I have

@ghost ghost added the Needs: Triage 🔍 Needs triaging by the team label Mar 27, 2023
@oZakari oZakari changed the title Accelerator 26294 - Feature - Accelerator Pipeline adjustements Mar 28, 2023
@oZakari oZakari changed the title 26294 - Feature - Accelerator Pipeline adjustements 26294 - Feature - Accelerator Pipeline Adjustments Mar 28, 2023
@oZakari oZakari marked this pull request as ready for review March 28, 2023 03:40
@jtracey93 jtracey93 merged commit ef1466e into Azure:main Mar 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs: Triage 🔍 Needs triaging by the team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants