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.

Sign up for GitHub

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

Jump to bottom

💡 Feature Request - Change deployment scope for MG Diagnostics #338

Closed
chrisking81 opened this issue Oct 13, 2022 · 2 comments
Closed

💡 Feature Request - Change deployment scope for MG Diagnostics #338

chrisking81 opened this issue Oct 13, 2022 · 2 comments
Assignees
@lachaves

Comments

@chrisking81
Copy link
Contributor

Thanks for the recently added modules to set the diagnostic settings on the MG hierarchy.

My customer would like to use these but is currently blocked, due to the fact that the orchestration module is a 'tenant' scoped deployment.

My customer has no permissions to \ or Tenant Root Group, due to segregation of duties.

Could we please investigate whether we can change the orchestration module to be a 'management group' scoped deployment?
@ghost ghost added the Needs: Triage 🔍 Needs triaging by the team label Oct 13, 2022
@jtracey93
Copy link
Collaborator

Great ask @chrisking81 and thanks for raising this following our chat this afternoon.

We will triage and prioritise this on our scrum call today.

cc: @lachaves

@jtracey93 jtracey93 removed the Needs: Triage 🔍 Needs triaging by the team label Oct 20, 2022
@lachaves lachaves mentioned this issue Nov 2, 2022
Merged
14 tasks
jtracey93 added a commit that referenced this issue Nov 3, 2022
@lachaves @jtracey93
Change deployment scope for MG Diagnostics #338 (#372)
d2accf8 
* Updated README to show new scope at mg level

* Changes to mgDiagSettingsAll.bicep with new mg deployment scope

* Added mgDiagSettingsAll as a new orchestration module to the list

* Fixed a URL tfor step 4.1 to point to the corret module

* Update infra-as-code/bicep/orchestration/mgDiagSettingsAll/mgDiagSettingsAll.bicep

Co-authored-by: Jack Tracey <41163455+jtracey93@users.noreply.github.com>

* Changed the test-pipelines to reflec the deployment at the mg level now

Co-authored-by: Luis Chaves <luchaves@microsoft.com>
Co-authored-by: Jack Tracey <41163455+jtracey93@users.noreply.github.com>
@jtracey93
Copy link
Collaborator

#372 merged so closing

@ghost ghost locked as resolved and limited conversation to collaborators Dec 3, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@lachaves lachaves

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@chrisking81 @lachaves @jtracey93