Skip to content

Update create-diagnostic-settings.md #179

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

Closed
wants to merge 1 commit into from

Conversation

miguel-parada
Copy link
Contributor

Adding update to reflect the following known issue and considerations to avoid unexpected billing due loop ingestion issues.

From Azure portal when creating or updating Diagnostic Settings for an Azure Storage account or Azure Event Hub namespace, you could be unable to select itself as a destination for the resource logs or metrics data.
This is by design as it is possible to get into a state where resource logs or metrics being sent from a resource to the same resource would generate an infinite loop of generating and writing data.
This design is only applied at the Azure portal UX layer, if there is truly a need to write data to the same resource and you are willing to accept the associated risks, you can create the Diagnostic Setting using Azure PowerShell, Azure CLI, REST API, ARM Template or other supported Microsoft SDK.

Adding update to reflect the following known issue and considerations to avoid unexpected billing due loop ingestion issues.

> From Azure portal when creating or updating Diagnostic Settings for an Azure Storage account or Azure Event Hub namespace, you could be unable to select itself as a destination for the resource logs or metrics data.
> This is by design as it is possible to get into a state where resource logs or metrics being sent from a resource to the same resource would generate an infinite loop of generating and writing data.  
> This design is only applied at the Azure portal UX layer, if there is truly a need to write data to the same resource and you are willing to accept the associated risks, you can create the Diagnostic Setting using Azure PowerShell, Azure CLI, REST API, ARM Template or other supported Microsoft SDK.
Copy link
Contributor

Learn Build status updates of commit 1097398:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/azure-monitor/platform/create-diagnostic-settings.md ⚠️Warning Details

articles/azure-monitor/platform/create-diagnostic-settings.md

  • Line 56, Column 1: [Warning: code-block-indented - See documentation] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```).

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@v-dirichards
Copy link
Contributor

@bwren

Can you review the proposed changes?

Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
#assign: @bwren
@MicrosoftDocs/public-repo-pr-review-team

@v-dirichards v-dirichards added the aq-pr-triaged C+L Pull Request Review Team label label Jul 16, 2025
Copy link

This pull request has been inactive for at least 14 days. If you are finished with your changes, don't forget to sign off. See the contributor guide for instructions.
Get Help
Docs Support Teams Channel
Resolve Merge Conflict

@github-actions github-actions bot added the inactive This PR is inactive for more than 14 days label Jul 30, 2025
@miguel-parada
Copy link
Contributor Author

@bwren, do you have any update maybe?

@github-actions github-actions bot removed the inactive This PR is inactive for more than 14 days label Aug 6, 2025
@v-dirichards
Copy link
Contributor

@miguel-parada The article has been deleted and now redirects to Diagnostic settings in Azure Monitor, and this change no longer applies.

If you'd still like to update the article, please open a new PR. Or if you believe this PR was closed by mistake, email the PublicPRs alias.

#please-close
@MicrosoftDocs/public-repo-pr-review-team

@miguel-parada
Copy link
Contributor Author

done :) #185

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aq-pr-triaged C+L Pull Request Review Team label do-not-merge review-team-triage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants