-
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_log_analytics_workspace_table -- added to allow retention_in_days to be configured for a given table. Fix for issue 10573 #15462
Conversation
Hi Everyone, I realize there are a lot of pull requests to go through, but if anyone has a couple minutes, would you mind looking over the azurerm_log_analytics_workspace_table resource in this PR. Just want to make sure I'm on the right track for configuring individual tables included in log analytics workspaces. Thanls, |
Hi @markrzasa, Thanks for this PR and apologies for the delayed review here. I’ve taken a look through and noticed that the new resource Could you go ahead and move your bug fix out into a separate PR and I will open a PR in the coming days with the generated SDK. Thanks! |
Absolutely. No problem. Should I abandon this pull request or use it to integrate with the new version of the Azure SDK? |
It will probably be easiest to abandon this PR. Thanks again! |
Will revisit this once the Azure SDK for Go is updated. |
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. |
This pull request attempts to resolve the following issues:
Please let me know if you would prefer separate pull requests for each issue.