-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
azurerm_security_center_auto_provisioning
- deprecate resource
#28030
azurerm_security_center_auto_provisioning
- deprecate resource
#28030
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @ziyeqf, this looks good but the link in the description doesn't mention the deprecation, maybe I missed it. Could you please double check?
### `azurerm_security_center_auto_provisioning` | ||
|
||
* This deprecated resources has been removed from the Azure Provider. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be under the ### Removed Resource
section and it would be good to provide a link to the deprecation message as well
### `azurerm_security_center_auto_provisioning` | |
* This deprecated resources has been removed from the Azure Provider. | |
### `azurerm_security_center_auto_provisioning` | |
* This deprecated resource has been removed from the Azure Provider. | |
Somehow I pasted a wrong link... it's here |
@@ -34,6 +34,9 @@ Please follow the format in the example below for adding removed resources: | |||
This deprecated resource has been superseded/retired and has been removed from the Azure Provider. | |||
``` | |||
|
|||
### `azurerm_security_center_auto_provisioning` | |||
|
|||
* This deprecated resource has been removed from the Azure Provider. [document](https://learn.microsoft.com/en-us/azure/defender-for-cloud/prepare-deprecation-log-analytics-mma-agent#log-analytics-agent-autoprovisioning-experience---deprecation-plan) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* This deprecated resource has been removed from the Azure Provider. [document](https://learn.microsoft.com/en-us/azure/defender-for-cloud/prepare-deprecation-log-analytics-mma-agent#log-analytics-agent-autoprovisioning-experience---deprecation-plan) | |
* This deprecated resource has been removed from the Azure Provider. Please see the [documention for more details](https://learn.microsoft.com/en-us/azure/defender-for-cloud/prepare-deprecation-log-analytics-mma-agent#log-analytics-agent-autoprovisioning-experience---deprecation-plan). |
@@ -36,7 +36,7 @@ This deprecated resource has been superseded/retired and has been removed from t | |||
|
|||
### `azurerm_security_center_auto_provisioning` | |||
|
|||
* This deprecated resource has been removed from the Azure Provider. [document](https://learn.microsoft.com/en-us/azure/defender-for-cloud/prepare-deprecation-log-analytics-mma-agent#log-analytics-agent-autoprovisioning-experience---deprecation-plan) | |||
* This deprecated resource has been removed from the Azure Provider. [Please see the [documention for more details]](https://learn.microsoft.com/en-us/azure/defender-for-cloud/prepare-deprecation-log-analytics-mma-agent#log-analytics-agent-autoprovisioning-experience---deprecation-plan). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is now linked inside a link?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kind of carelessness, sry for that
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @ziyeqf LGTM 👍
Community Note
Description
The capability is deprecated per the document
PR Checklist
For example: “
resource_name_here
- description of change e.g. adding propertynew_property_name_here
”Changes to existing Resource / Data Source
Testing
The resource is deprecated on the service side and the tests can not pass any more with following error:
Change Log
Below please provide what should go into the changelog (if anything) conforming to the Changelog Format documented here.
azurerm_security_center_auto_provisioning
- deprecate resource [azurerm_security_center_auto_provisioning
- deprecate resource #28030]This is a (please select all that apply):
Related Issue(s)
Fixes #27670
Note
If this PR changes meaningfully during the course of review please update the title and description as required.