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

tier3 deploy policy #727 #728

Merged

Conversation

LManning-Dev
Copy link
Contributor

@LManning-Dev LManning-Dev commented Jul 19, 2022

Tier 3 does not deploy policy like the core (mlz.bicep)

Note: The Scope is dependent on issue #724 as it will reference the resource group in the tier 3 subscription

Description

Please explain the changes you've made
Added 2 new outputs to the Core, so that the Tier 3's can sync to the core, and all Tier 3's can be configured identically.
Added the ability to deploy Policy to Tier 3 (Pulling most from the Core (mlz.bicep))

Note: This is dependent on #724 as this references the resource group in the tier 3 subscription

scope: az.resourceGroup(workloadSubscriptionId, resourceGroup.name)

Issue reference

The issue this PR will close: #727

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

Tier 3 does not deploy policy.

Note: The Scope is dependent on issue Azure#724 as it will reference the the resource group in the tier3 subscription
@LManning-Dev LManning-Dev requested a review from a team as a code owner July 19, 2022 16:59
LManning-Dev and others added 3 commits July 27, 2022 15:10
Adding optional parameters deployPolicy and policy to the tier 3 readme
Adding new outputs to MLZ Core Readme
@lisamurphy-msft
Copy link
Contributor

Thank you for submitting your PR!
Please proceed with the requested change and I should be able to test this out soon

@lisamurphy-msft
Copy link
Contributor

documentation change can be pushed shortly in a separate PR.
Approving as testing was confirmed :)

@lisamurphy-msft lisamurphy-msft merged commit 283aab7 into Azure:main Sep 7, 2022
@lisamurphy-msft lisamurphy-msft added the core New feature or request label Sep 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tier 3 Does Not Deploy Policy Like the Core (mlz.bicep)
2 participants