Skip to content

Resource deletion - exemptions#564

Merged
Jefajers merged 6 commits intoAzure:mainfrom
Jefajers:resourceDeletionExemptions
Feb 18, 2022
Merged

Resource deletion - exemptions#564
Jefajers merged 6 commits intoAzure:mainfrom
Jefajers:resourceDeletionExemptions

Conversation

@Jefajers
Copy link
Copy Markdown
Member

@Jefajers Jefajers commented Feb 18, 2022

Overview/Summary

Adds logic to expand on AzOps lifecycle handling of policyExemptions to now include Deletion.
Fixes #488

This PR fixes/adds/changes/removes

  1. Adds logic to support deletion of policyExemptions
  2. Changes the Wiki docs for resource deletion
  3. Updates internal tests to include automated tests of policyExemptions deletions

Breaking Changes

N/A

Testing Evidence

Tested Pull and Push of Policy Exemption at subscription scope
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 main branch
  • Performed testing and provided evidence.
  • Updated relevant and associated documentation.

@Jefajers Jefajers requested a review from a team as a code owner February 18, 2022 15:43
@Jefajers Jefajers temporarily deployed to test February 18, 2022 15:43 Inactive
daltondhcp
daltondhcp reviewed Feb 18, 2022
View reviewed changes
Comment thread src/functions/Invoke-AzOpsPush.ps1 Outdated
@Jefajers @daltondhcp
Update src/functions/Invoke-AzOpsPush.ps1
a598bef 
Co-authored-by: Johan Dahlbom <johan@dahlbom.eu>
@Jefajers Jefajers temporarily deployed to test February 18, 2022 16:45 Inactive
@Jefajers Jefajers temporarily deployed to test February 18, 2022 16:58 Inactive
daltondhcp
daltondhcp approved these changes Feb 18, 2022
View reviewed changes
Copy link
Copy Markdown
Contributor

@daltondhcp daltondhcp left a comment

Choose a reason for hiding this comment

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

Awesome! 🚢

@Jefajers Jefajers merged commit de9fad9 into Azure:main Feb 18, 2022
@Jefajers Jefajers deleted the resourceDeletionExemptions branch March 18, 2022 09:56
@daltondhcp daltondhcp mentioned this pull request May 9, 2022
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@daltondhcp daltondhcp daltondhcp approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Extend functionality of resource deletion to include exemptions

2 participants

@Jefajers @daltondhcp