Skip to content

Update - Microsoft.Management missing from Subscription during Initialize - #871

Merged
Jesper Fajers (Jefajers) merged 7 commits into
Azure:mainfrom
SebastianClaesson:main
Apr 3, 2024
Merged

Update - Microsoft.Management missing from Subscription during Initialize#871
Jesper Fajers (Jefajers) merged 7 commits into
Azure:mainfrom
SebastianClaesson:main

Conversation

@SebastianClaesson

Copy link
Copy Markdown
Contributor

Overview/Summary

  • Added Microsoft.Management Resource Provider to the prerequisites list.
  • Added to FAQ if Azure Mangement groups are not populated correctly.
  • Added new log message 'Initialize-AzOpsEnvironment.ManagementGroup.InsufficientPermissions'
  • Added logic to match if the error is due to insufficient permissions to register the Microsoft.Management resource provider.

I'm a bit unsure of how you like to have the catch logic & message(s), as it's maybe not such a clean fix and can be improved.
Please come with any feedback of suggestions and I'll happily implement them!

This PR fixes/adds/changes/removes

  1. RP Microsoft.Management missing from Subscription during Initialize #863

Testing Evidence

image

As part of this Pull Request I have

  • Checked for duplicate Pull Requests
  • Associated it with relevant issues, for tracking and closure.
  • Ensured my code/branch is up-to-date with the latest changes in the mainbranch
  • Performed testing and provided evidence.
  • Updated relevant and associated documentation.

@Jefajers

Copy link
Copy Markdown
Member

Sebastian Claesson (@SebastianClaesson), thanks for this contribution. I have made slight wording changes and a tradeoff in regards to the error message output by combining the current "simple" message and "logic" but always appending the full error message.

What do you think?

@JefajersJesper Fajers (Jefajers) changed the title PR - Microsoft.Management missing from Subscription during InitializeUpdate - Microsoft.Management missing from Subscription during InitializeApr 2, 2024
@JefajersJesper Fajers (Jefajers) added this to the v2.6.1 milestone Apr 2, 2024
@JefajersJesper Fajers (Jefajers) added enhancement New feature or request wiki-doc Improvements or additions to documentation labels Apr 2, 2024
Changing from "Registered" to "Register".
@SebastianClaesson

Copy link
Copy Markdown
ContributorAuthor

Thanks! Looks really good!
I've changed from "Registered" to "Register" in prerequisties.

@JefajersJesper Fajers (Jefajers) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good :)

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancementNew feature or requestwiki-docImprovements or additions to documentation

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

RP Microsoft.Management missing from Subscription during Initialize

2 participants

@SebastianClaesson@Jefajers