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

ALZ-Bicep-Accelerator Register missing resource provider #595

Merged
merged 4 commits into from
Aug 1, 2023

Conversation

sebassem
Copy link
Contributor

@sebassem sebassem commented Aug 1, 2023

Overview/Summary

Fixes issue #594

This PR fixes/adds/changes/removes

  1. Adds logic to check for the 'Microsoft.Insights' resource provider

Breaking Changes

N/A

Testing Evidence

Cannot unregister the Microsoft.Insights provider so used another provider just for testing

image

As part of this Pull Request I have

@sebassem sebassem requested a review from a team as a code owner August 1, 2023 13:52
@jtracey93 jtracey93 merged commit ad6fbf6 into Azure:main Aug 1, 2023
@jtracey93 jtracey93 linked an issue Aug 1, 2023 that may be closed by this pull request
@sebassem sebassem deleted the alz_bicep_mgDiag_providers branch August 1, 2023 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🪲 Bug Report - Resource provider registration error
2 participants