`azurerm_monitor_diagnostic_setting` - `retention_policy` deprecated in favor of `azurerm_storage_management_policy` #23260

teowa · 2023-09-14T02:32:47Z

tombuildsstuff

If we can fix up the documentation this should otherwise be good to go 👍

tombuildsstuff · 2023-09-14T09:45:42Z

website/docs/r/monitor_diagnostic_setting.html.markdown

@@ -159,6 +159,8 @@ A `retention_policy` block supports the following:
 * `enabled` - (Required) Is this Retention Policy enabled?

 * `days` - (Optional) The number of days for which this Retention Policy should apply.
+
+!> **NOTE:** `retention_policy` has been deprecated in favor of `azurerm_storage_management_policy` resource - to learn more information on the deprecation [in the Azure documentation](https://aka.ms/diagnostic_settings_log_retention).


this should be moved up to line 157 and also copied under line 151

fixed, thanks.

stephybun

Thanks @teowa LGTM 👍

<Actions> <action id="4a39167e811ac038e4a588362092472c27cfbe9e4929ae61d035f708a093a669"> <h3>Bump Terraform `azurerm` provider version</h3> <details id="1d9343c012f5434ac9fe8a98135bae3667b399259be16d9b14302ea3bd424a24"> <summary>Update Terraform lock file</summary> <p>"hashicorp/azurerm" updated from "3.73.0" to "3.74.0" in file ".terraform.lock.hcl"</p> <details> <summary>3.74.0</summary> <pre>Changelog retrieved from:
	https://github.com/hashicorp/terraform-provider-azurerm/releases/tag/v3.74.0
NOTES:

* `azurerm_synapse_sql_pool` - users that have imported `azurerm_synapse_sql_pool` resources that were created outside of Terraform using an `LRS` storage account type will need to use `ignore_changes` to avoid the resource from being destroyed and recreated.

FEATURES:

* **New Resource**: `azurerm_arc_resource_bridge_appliance` ([#23108](https://github.com/hashicorp/terraform-provider-azurerm/issues/23108))
* **New Resource**: `azurerm_data_factory_dataset_azure_sql_table` ([#23264](https://github.com/hashicorp/terraform-provider-azurerm/issues/23264))
* **New Resource**: `azurerm_function_app_connection` ([#23127](https://github.com/hashicorp/terraform-provider-azurerm/issues/23127))

ENHANCEMENTS:

* dependencies: updating to `v0.20230918.1115907` of `github.com/hashicorp/go-azure-sdk` ([#23337](https://github.com/hashicorp/terraform-provider-azurerm/issues/23337))
* dependencies: downgrading to `v1.12.5` of `github.com/rickb777/date` ([#23296](https://github.com/hashicorp/terraform-provider-azurerm/issues/23296))
* `mysql`: updating to use API Version `2022-01-01` ([#23320](https://github.com/hashicorp/terraform-provider-azurerm/issues/23320))
* `azurerm_app_configuration` - support for the `replica` block ([#22452](https://github.com/hashicorp/terraform-provider-azurerm/issues/22452))
* `azurerm_bot_channel_directline` - support for `user_upload_enabled`, `endpoint_parameters_enabled`, and `storage_enabled` ([#23149](https://github.com/hashicorp/terraform-provider-azurerm/issues/23149))
* `azurerm_container_app` - support for scale rules ([#23294](https://github.com/hashicorp/terraform-provider-azurerm/issues/23294))
* `azurerm_container_app_environment` - support for zone redundancy ([#23313](https://github.com/hashicorp/terraform-provider-azurerm/issues/23313))
* `azurerm_container_group` - support for the `key_vault_user_identity_id` property for Customer Managed Keys ([#23332](https://github.com/hashicorp/terraform-provider-azurerm/issues/23332))
* `azurerm_cosmosdb_account` - support for MongoDB connection strings ([#23331](https://github.com/hashicorp/terraform-provider-azurerm/issues/23331))
* `azurerm_data_factory_dataset_delimited_text` - support for the `dynamic_file_system_enabled`, `dynamic_path_enabled`, and `dynamic_filename_enabled` properties ([#23261](https://github.com/hashicorp/terraform-provider-azurerm/issues/23261))
* `azurerm_data_factory_dataset_parquet` - support for the `azure_blob_fs_location` block ([#23261](https://github.com/hashicorp/terraform-provider-azurerm/issues/23261))
* `azurerm_monitor_diagnostic_setting` - validation to ensure either `category` or `category_group` are supplied in `enabled_log` and `log` blocks ([#23308](https://github.com/hashicorp/terraform-provider-azurerm/issues/23308))
* `azurerm_network_interface` - support for the `auxiliary_mode` and `auxiliary_sku` properties ([#22979](https://github.com/hashicorp/terraform-provider-azurerm/issues/22979))
* `azurerm_postgresql_flexible_server` - increased the maximum supported value for `storage_mb` ([#23277](https://github.com/hashicorp/terraform-provider-azurerm/issues/23277))
* `azurerm_shared_image_version` - support for the `replicated_region_deletion_enabled` and `target_region.exclude_from_latest_enabled` properties ([#23147](https://github.com/hashicorp/terraform-provider-azurerm/issues/23147))
* `azurerm_storage_account` - support for setting `domain_name` and `domain_guid` for `AADKERB` ([#22833](https://github.com/hashicorp/terraform-provider-azurerm/issues/22833))
* `azurerm_storage_account_customer_managed_key` - support for cross-tenant customer-managed keys with the `federated_identity_client_id`, and `key_vault_uri` properties ([#20356](https://github.com/hashicorp/terraform-provider-azurerm/issues/20356))
* `azurerm_web_application_firewall_policy` - support for the `rate_limit_duration`, `rate_limit_threshold`, `group_rate_limit_by`, and `request_body_inspect_limit_in_kb` properties ([#23239](https://github.com/hashicorp/terraform-provider-azurerm/issues/23239))

BUG FIXES:

* Data Source: `azurerm_container_app_environment`: fix `log_analytics_workspace_name` output to correct value ([#23298](https://github.com/hashicorp/terraform-provider-azurerm/issues/23298))
* `azurerm_api_management_api` - set the `service_url` property when importing the resource ([#23011](https://github.com/hashicorp/terraform-provider-azurerm/issues/23011))
* `azurerm_app_configuration` - prevent crash by nil checking the encryption configuration ([#23302](https://github.com/hashicorp/terraform-provider-azurerm/issues/23302))
* `azurerm_app_configuration_feature` - update `percentage_filter_value` to accept correct type of float ([#23263](https://github.com/hashicorp/terraform-provider-azurerm/issues/23263))
* `azurerm_container_app` - fix an issue with `commands` and `args` being overwritten when using multiple containers ([#23338](https://github.com/hashicorp/terraform-provider-azurerm/issues/23338))
* `azurerm_key_vault_certificate` - fix issue where certificates couldn't be recovered anymore ([#23204](https://github.com/hashicorp/terraform-provider-azurerm/issues/23204))
* `azurerm_key_vault_key` - the ForceNew when `expiration_date` is removed from the config file ([#23327](https://github.com/hashicorp/terraform-provider-azurerm/issues/23327))
* `azurerm_linux_function_app` - fix a bug in setting the storage settings when using Elastic Premium plans ([#21212](https://github.com/hashicorp/terraform-provider-azurerm/issues/21212))
* `azurerm_linux_web_app` - fix docker app stack update ([#23303](https://github.com/hashicorp/terraform-provider-azurerm/issues/23303))
* `azurerm_linux_web_app` - fix crash in auto heal expansion ([#21328](https://github.com/hashicorp/terraform-provider-azurerm/issues/21328))
* `azurerm_linux_web_app_slot` - fix docker app stack update ([#23303](https://github.com/hashicorp/terraform-provider-azurerm/issues/23303))
* `azurerm_linux_web_app_slot` - fix crash in auto heal expansion ([#21328](https://github.com/hashicorp/terraform-provider-azurerm/issues/21328))
* `azurerm_log_analytics_solution` - fix bug where the resource wasn't handling successful creation on subsequent applies ([#23312](https://github.com/hashicorp/terraform-provider-azurerm/issues/23312))
* `azurerm_management_group_subscription_association` - fix bug to correctly mark resource as gone if not found during read ([#23335](https://github.com/hashicorp/terraform-provider-azurerm/issues/23335))
* `azurerm_mssql_elasticpool` - remove check that prevents `license_type` from being set for certain skus ([#23262](https://github.com/hashicorp/terraform-provider-azurerm/issues/23262))
* `azurerm_servicebus_queue` - fixing an issue where `auto_delete_on_idle` couldn't be set to `P10675199DT2H48M5.4775807S` ([#23296](https://github.com/hashicorp/terraform-provider-azurerm/issues/23296))
* `azurerm_servicebus_topic` - fixing an issue where `auto_delete_on_idle` couldn't be set to `P10675199DT2H48M5.4775807S` ([#23296](https://github.com/hashicorp/terraform-provider-azurerm/issues/23296))
* `azurerm_storage_account` - prevent sending unsupported blob properties in payload for `Storage` account kind ([#23288](https://github.com/hashicorp/terraform-provider-azurerm/issues/23288))
* `azurerm_synapse_sql_pool` - expose `storage_account_type` ([#23217](https://github.com/hashicorp/terraform-provider-azurerm/issues/23217))
* `azurerm_windows_function_app` - fix a bug in setting the storage settings when using Elastic Premium plans ([#21212](https://github.com/hashicorp/terraform-provider-azurerm/issues/21212))
* `azurerm_windows_web_app` - fix docker app stack update ([#23303](https://github.com/hashicorp/terraform-provider-azurerm/issues/23303))
* `azurerm_windows_web_app_slot` - fix docker app stack update ([#23303](https://github.com/hashicorp/terraform-provider-azurerm/issues/23303))

DEPRECATIONS:

* `azurerm_application_gateway` - deprecate `Standard` and `WAF` skus ([#23310](https://github.com/hashicorp/terraform-provider-azurerm/issues/23310))
* `azurerm_bot_channel_web_chat` - deprecate `site_names` in favour of `site` block ([#23161](https://github.com/hashicorp/terraform-provider-azurerm/issues/23161))
* `azurerm_monitor_diagnostic_setting` - deprecate `retention_policy` in favour of `azurerm_storage_management_policy` ([#23260](https://github.com/hashicorp/terraform-provider-azurerm/issues/23260))


</pre> </details> </details> </action> </Actions> --- <table> <tr> <td width="77"> <img src="https://www.updatecli.io/images/updatecli.png" alt="Updatecli logo" width="50" height="50"> </td> <td> <p> Created automatically by <a href="https://www.updatecli.io/">Updatecli</a> </p> <details><summary>Options:</summary> <br /> <p>Most of Updatecli configuration is done via <a href="https://www.updatecli.io/docs/prologue/quick-start/">its manifest(s)</a>.</p> <ul> <li>If you close this pull request, Updatecli will automatically reopen it, the next time it runs.</li> <li>If you close this pull request and delete the base branch, Updatecli will automatically recreate it, erasing all previous commits made.</li> </ul> <p> Feel free to report any issues at <a href="https://github.com/updatecli/updatecli/issues">github.com/updatecli/updatecli</a>.<br /> If you find this tool useful, do not hesitate to star <a href="https://github.com/updatecli/updatecli/stargazers">our GitHub repository</a> as a sign of appreciation, and/or to tell us directly on our <a href="https://matrix.to/#/#Updatecli_community:gitter.im">chat</a>! </p> </details> </td> </tr> </table> Co-authored-by: Jenkins Infra Bot (updatecli) <60776566+jenkins-infra-bot@users.noreply.github.com>

Panzerbjrn · 2023-10-19T12:54:49Z

It would be really great to have some documentation to clearly show how "azurerm_storage_management_policy" should be used in place of a simple

    retention_policy {
      days    = 0 
      enabled = false
    }

block in azurerm_monitor_diagnostic_setting. Looking at azurerm_storage_management_policy doesn't really help much.

nkol2307 · 2023-12-26T10:32:13Z

any update on this ?
is there any example how to replace this block inside azurerm_monitor_diagnostic_settings

Flo2388 · 2024-04-18T09:18:47Z

There is also something additional missing in my opinion in the documentation:

In the documentation for the retention_policy argument you can read:

Setting this to 0 will retain the events indefinitely.

So our configuration looks like the following to disable the rule (enabled by default according to documentation) and set the storing to infinite:

  enabled_log {
    category = "logs"
    retention_policy {
      days    = 0
      enabled = false
    }
  }

What I can't find nor from the provider neither from the MS migration guide is, if the retention_policy block is deprecated, do I have to just remove the blocks to get infinite storing or do I have to implement the policy resource. And if the latter: What is the value to set for infinite logging? Just leave it to the defaults (-1), set it to 0 or what do I have to do?

From documentation for the day related settings:

(Optional) The age in days after last modification to tier blobs to archive storage. Supports blob currently at Hot or Cool tier. Must be between 0 and 99999. Defaults to -1.

Thanks for your help in advance.

github-actions · 2024-05-19T02:09:18Z

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

enhance deprecation message

138b9fb

github-actions bot added documentation service/monitor size/XS labels Sep 14, 2023

tombuildsstuff suggested changes Sep 14, 2023

View reviewed changes

github-actions bot added the waiting-response label Sep 14, 2023

add more copy

3312227

github-actions bot removed the waiting-response label Sep 14, 2023

rcskosir mentioned this pull request Sep 14, 2023

azurerm_monitor_diagnostic_setting retention_policy deprecated without clear resolution #23250

Closed

1 task

stephybun approved these changes Sep 18, 2023

View reviewed changes

stephybun merged commit a8aef80 into hashicorp:main Sep 18, 2023
24 checks passed

github-actions bot added this to the v3.74.0 milestone Sep 18, 2023

stephybun added a commit that referenced this pull request Sep 20, 2023

Update for #23011 #23277 #23260 #22833 #23288 #23147 #22979

e266595

HorizonNet mentioned this pull request Sep 22, 2023

Remove azurerm_monitor_diagnostic_setting configuration ultratendency/terraform-azurerm-secure-storage-account#67

Merged

sheeeng mentioned this pull request Sep 26, 2023

fix: deprecated retention policy days EnergiMidt/terraform-azurerm-diagnosticsetting#3

Merged

BillCoateARS mentioned this pull request Oct 25, 2023

azurerm_monitor_diagnostic_setting - retention_policy deprecated in favor of azurerm_storage_management_policy Azure/missionlz#787

Closed

github-actions bot locked as resolved and limited conversation to collaborators May 19, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

`azurerm_monitor_diagnostic_setting` - `retention_policy` deprecated in favor of `azurerm_storage_management_policy` #23260

`azurerm_monitor_diagnostic_setting` - `retention_policy` deprecated in favor of `azurerm_storage_management_policy` #23260

teowa commented Sep 14, 2023

tombuildsstuff left a comment

tombuildsstuff Sep 14, 2023

teowa Sep 14, 2023

stephybun left a comment

Panzerbjrn commented Oct 19, 2023

nkol2307 commented Dec 26, 2023 •

edited

Loading

Flo2388 commented Apr 18, 2024 •

edited

Loading

github-actions bot commented May 19, 2024

azurerm_monitor_diagnostic_setting - retention_policy deprecated in favor of azurerm_storage_management_policy #23260

azurerm_monitor_diagnostic_setting - retention_policy deprecated in favor of azurerm_storage_management_policy #23260

Conversation

teowa commented Sep 14, 2023

tombuildsstuff left a comment

Choose a reason for hiding this comment

tombuildsstuff Sep 14, 2023

Choose a reason for hiding this comment

teowa Sep 14, 2023

Choose a reason for hiding this comment

stephybun left a comment

Choose a reason for hiding this comment

Panzerbjrn commented Oct 19, 2023

nkol2307 commented Dec 26, 2023 • edited Loading

Flo2388 commented Apr 18, 2024 • edited Loading

github-actions bot commented May 19, 2024

`azurerm_monitor_diagnostic_setting` - `retention_policy` deprecated in favor of `azurerm_storage_management_policy` #23260

`azurerm_monitor_diagnostic_setting` - `retention_policy` deprecated in favor of `azurerm_storage_management_policy` #23260

nkol2307 commented Dec 26, 2023 •

edited

Loading

Flo2388 commented Apr 18, 2024 •

edited

Loading