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

Microsoft.Insights/components - retentionInDays incorrectly marked readonly #14801

Open
anthony-c-martin opened this issue Jun 14, 2021 · 5 comments
Labels
Monitor - ApplicationInsights Service Attention Workflow: This issue is responsible by Azure service team.

Comments

@anthony-c-martin
Copy link
Member

Originally reported under Azure/bicep#3212. On testing, the service actually allows this value to be modified via a PUT.

See

"RetentionInDays": {
"type": "integer",
"description": "Retention period in days.",
"readOnly": true
},

This may also affect other apiVersions.

@ghost ghost added the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Jun 14, 2021
@leniatgh leniatgh added Monitor - ApplicationInsights Service Attention Workflow: This issue is responsible by Azure service team. labels Jun 15, 2021
@ghost ghost removed the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Jun 15, 2021
@ghost
Copy link

ghost commented Jun 15, 2021

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @azmonapplicationinsights.

Issue Details

Originally reported under Azure/bicep#3212. On testing, the service actually allows this value to be modified via a PUT.

See

"RetentionInDays": {
"type": "integer",
"description": "Retention period in days.",
"readOnly": true
},

This may also affect other apiVersions.

Author: anthony-c-martin
Assignees: leni-msft
Labels:

Monitor - ApplicationInsights, Service Attention, needs-triage

Milestone: -

@timja
Copy link

timja commented Jul 21, 2021

This is blocking the azurerm terraform provider adoption of the workspace model with application insights

@leniatgh
Copy link
Contributor

@zakimaksyutov @igorse would you be able to take a look at this issue or route to correct person? Thanks!

@dylanmorley
Copy link

Fixed by #15247

@Jaroslaw-Roman-Cudzich
Copy link

Hello,

Thank you for the fix.

For testing purposes, please keep in mind that as per https://learn.microsoft.com/en-us/azure/azure-monitor/logs/data-retention-archive?tabs=portal-1%2Cportal-2 , a change in the RetentionInDays value can only be tested after 30 days time after the change as "Azure Monitor waits 30 days before removing the data, so you can revert the change and prevent data loss in the event of an error in configuration."

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Monitor - ApplicationInsights Service Attention Workflow: This issue is responsible by Azure service team.
Projects
None yet
Development

No branches or pull requests

6 participants
@dylanmorley @leniatgh @timja @anthony-c-martin @Jaroslaw-Roman-Cudzich and others