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

feat: Immutability policy storage account [#IOPID-752] #159

Merged
merged 8 commits into from
Oct 5, 2023

Conversation

BurnedMarshal
Copy link
Contributor

@BurnedMarshal BurnedMarshal commented Sep 14, 2023

List of changes

Add variables to set the immutability policy params for an Azure Storage.
Optional immutability policy into the module.

Motivation and context

The Blob storage that contains audit logs for legal reason must have an immutability policy to prevent data deletion until the end of the retention policy.

Type of changes

  • Add new module
  • Update existing module
  • Remove existing module

Does this introduce a breaking change?

  • Yes
  • No

Other information

The actual version of the hashicorp/azurerm module when we change from Unlocked or Disabled state of the immutability policy to Locked update the terraform state file but doesn't update the resource into the cloud.

Run checks

Useful commands to run checks on local machine

bash .utils/terraform_run_all.sh init local
pre-commit run -a

@BurnedMarshal BurnedMarshal changed the title [#IOPID-752] Immutability policy storage account feat: [#IOPID-752] Immutability policy storage account Sep 14, 2023
@BurnedMarshal BurnedMarshal changed the title feat: [#IOPID-752] Immutability policy storage account feat: Immutability policy storage account [#IOPID-752] Sep 14, 2023
@BurnedMarshal BurnedMarshal force-pushed the IOPID-752-immutability-storage-account branch from 4a1dc0f to cdcac09 Compare September 19, 2023 10:19
@BurnedMarshal BurnedMarshal marked this pull request as ready for review September 19, 2023 10:20
@BurnedMarshal BurnedMarshal requested a review from a team September 19, 2023 10:20
@Krusty93
Copy link
Contributor

@BurnedMarshal I confirm you found a Terraform bug and we are going to open an issue. In the meanwhile, you can use az cli in the Terraform configuration to make the necessary changes

@BurnedMarshal BurnedMarshal requested a review from a team as a code owner October 4, 2023 14:45
@BurnedMarshal BurnedMarshal force-pushed the IOPID-752-immutability-storage-account branch from f1c71bd to 5282ae0 Compare October 4, 2023 14:46
storage_account/main.tf Outdated Show resolved Hide resolved
pasqualedevita
pasqualedevita previously approved these changes Oct 4, 2023
Copy link
Member

@pasqualedevita pasqualedevita left a comment

Choose a reason for hiding this comment

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

LGTM, only a little fix on null_resource name

storage_account/main.tf Outdated Show resolved Hide resolved
Co-authored-by: Andrea Grillo <andrea.grillo@outlook.com>
Krusty93
Krusty93 previously approved these changes Oct 5, 2023
@Krusty93 Krusty93 merged commit 64a6c7b into main Oct 5, 2023
3 checks passed
@Krusty93 Krusty93 deleted the IOPID-752-immutability-storage-account branch October 5, 2023 12:55
@github-actions
Copy link

github-actions bot commented Oct 5, 2023

🎉 This PR is included in version 7.9.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants