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

Add Bicep cleanup instructions for diagnostic settings #564

Merged
merged 1 commit into from
Dec 7, 2021

Conversation

brooke-hamilton
Copy link
Contributor

Description

Update the Bicep README.md to include cleanup instructions, especially for deleting subscription-level diagnostic settings.

Issue reference

The issue this PR will close: #423

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

@brooke-hamilton brooke-hamilton enabled auto-merge (squash) December 7, 2021 21:52

The Bicep/ARM deployment of Mission Landing Zone can be deleted with two steps:

1. Delete all resource groups.
Copy link
Contributor

Choose a reason for hiding this comment

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

I usually delete in bulk via the command line and let ARM figure it out. Will users deleting from the Portal have specific order of operations considerations?

Copy link
Contributor

Choose a reason for hiding this comment

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

^ we can address this in another PR, of course. I assume it will take some investigation to work out.

@brooke-hamilton brooke-hamilton merged commit 2c3182f into main Dec 7, 2021
@brooke-hamilton brooke-hamilton deleted the brooke/monitor branch December 7, 2021 23:45
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.

Update documentation with instructions to purge unused diagnostic logging settings
2 participants