Skip to content

fix: add missing 'Microsoft' prefix to bare 'Entra ID' references in Azure Monitor fundamentals docs#259

Open
EoinTrial wants to merge 4 commits intoMicrosoftDocs:mainfrom
EoinTrial:fix/azure-monitor-bare-entra-references
Open

fix: add missing 'Microsoft' prefix to bare 'Entra ID' references in Azure Monitor fundamentals docs#259
EoinTrial wants to merge 4 commits intoMicrosoftDocs:mainfrom
EoinTrial:fix/azure-monitor-bare-entra-references

Conversation

@EoinTrial
Copy link
Copy Markdown

What does this PR change?

Adds the missing "Microsoft" prefix to two instances of bare "Entra ID" in the Azure Monitor fundamentals documentation.

overview.md
Before: > including audit logs from Entra ID
After: > including audit logs from Microsoft Entra ID

whats-new.md
Before: > the application must also authenticate with Entra ID or telemetry will be rejected
After: > the application must also authenticate with Microsoft Entra ID or telemetry will be rejected

Why?

Per Microsoft's naming guidance, the correct product name is always "Microsoft Entra ID", never "Entra ID" alone. Both surrounding sentences already use the full "Microsoft Entra ID" name correctly. These were the two remaining inconsistent references.

References

https://learn.microsoft.com/en-us/entra/fundamentals/new-name

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 8fe7a81:

✅ Validation status: passed

File Status Preview URL Details
articles/azure-monitor/fundamentals/overview.md ✅Succeeded
articles/azure-monitor/fundamentals/whats-new.md ✅Succeeded

For more details, please refer to the build report.

@ttorble
Copy link
Copy Markdown
Contributor

ttorble commented Apr 6, 2026

reopen for CLA check

@ttorble ttorble closed this Apr 6, 2026
@ttorble ttorble reopened this Apr 6, 2026
@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 8fe7a81:

❌ Validation status: errors

Please follow instructions here which may help to resolve issue.

File Status Preview URL Details
❌Error Details

  • [Error: CannotMergeCommit] Cannot merge commit 8fe7a815301a1306f592158f16b0e5e7192a6956 in branch fix/azure-monitor-bare-entra-references of repository https://github.com/EoinTrial/azure-monitor-docs into branch main (commit 66484ea45c367e5eb39a824bbb802241c9f8a9d1). Please follow this documentation: https://help.github.com/articles/resolving-a-merge-conflict-using-the-command-line/ to use git.exe to resolve you content conflicts locally and then push to remote.

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.

@ttorble
Copy link
Copy Markdown
Contributor

ttorble commented Apr 6, 2026

@EoinTrial

Thank you for your contribution. Would you take a moment to sign the Contributor License Agreement (CLA)? After the CLA is signed, someone can review your pull request. Thanks!

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

@ttorble ttorble added the aq-pr-triaged C+L Pull Request Review Team label label Apr 6, 2026
@EoinTrial
Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree

@prmerger-automator
Copy link
Copy Markdown
Contributor

@EoinTrial : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit eff520e:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/azure-monitor/fundamentals/whats-new.md ⚠️Warning Details
articles/azure-monitor/fundamentals/overview.md ✅Succeeded

articles/azure-monitor/fundamentals/whats-new.md

Multiple H1s(H1 '|Subservice | Article | Description |
|---|---|---|
|Alerts|Customize log search alert email subjects (preview)|Log search alert email subjects can now be customized.|
<<<<<<< fix/azure-monitor-bare-entra-references
|Application-Insights|Application Insights for ASP.NET and ASP.NET Core applications|Classic API SDK fundamentals for both ASP.NET and ASP.NET Core are now merged in one place.|
|Application-Insights|Metrics in Application Insights|The documentation now clarifies that Azure Monitor Metrics and Workspaces ingest custom metrics at a fixed 60-second interval, regardless of how often they're sent, while Log Analytics records them at the actual send interval. We also introduced the OTEL_METRIC_EXPORT_INTERVAL environment variable, allowing configuration of the OpenTelemetry metric export interval, with a default of 60 seconds.|
|Application-Insights|Troubleshoot missing application telemetry in Azure Monitor Application Insights|Clarified that when using Microsoft Entra ID for the Application Insights data collection endpoint, the application must also authenticate with Microsoft Entra ID or telemetry will be rejected.|
|Application-Insights|Application Insights availability tests|We published a query to discover deprecated URL ping tests with Azure Resource Graph Explorer.|') are not allowed. You can only have one top-level heading.
  • Line 166, Column 1: [Suggestion: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.

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.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 33fc22b:

✅ Validation status: passed

File Status Preview URL Details
articles/azure-monitor/fundamentals/overview.md ✅Succeeded
articles/azure-monitor/fundamentals/whats-new.md ✅Succeeded

For more details, please refer to the build report.

@v-dirichards v-dirichards requested a review from Copilot April 6, 2026 20:25
@v-dirichards
Copy link
Copy Markdown
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"
@MicrosoftDocs/public-repo-pr-review-team

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR aims to align Azure Monitor fundamentals documentation with Microsoft naming guidance by changing bare “Entra ID” references to “Microsoft Entra ID”.

Changes:

  • Updated the Azure Monitor overview to refer to “Microsoft Entra ID”.
  • Updated the “What’s new” entry text related to Application Insights endpoint authentication terminology (but currently via duplicated table rows).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
articles/azure-monitor/fundamentals/whats-new.md Attempts to update “Entra ID” wording in the July 2025 changelog table, but introduces duplicated rows and leaves one bare “Entra ID” instance behind.
articles/azure-monitor/fundamentals/overview.md Replaces “Entra ID” with “Microsoft Entra ID” in the “Monitor your cloud resources” section.
Comments suppressed due to low confidence (1)

articles/azure-monitor/fundamentals/whats-new.md:167

  • The July 2025 table now contains duplicate rows for the same Application Insights articles (asp-net, metrics-overview, missing-telemetry, availability). This looks like the PR added new entries instead of editing the existing ones, which makes the changelog confusing and inconsistent. Consider updating the existing rows in-place (or removing the older duplicates) so each article appears once with a single description.
|Application-Insights|[Application Insights for ASP.NET and ASP.NET Core applications](../app/asp-net.md)|Classic API SDK fundamentals for both ASP.NET and ASP.NET Core are now merged in one place.|
|Application-Insights|[Metrics in Application Insights](../app/metrics-overview.md)|The documentation now clarifies that Azure Monitor Metrics and Workspaces ingest custom metrics at a fixed 60-second interval, regardless of how often they're sent, while Log Analytics records them at the actual send interval. We also introduced the OTEL_METRIC_EXPORT_INTERVAL environment variable, allowing configuration of the OpenTelemetry metric export interval, with a default of 60 seconds.|
|Application-Insights|[Troubleshoot missing application telemetry in Azure Monitor Application Insights](/troubleshoot/azure/azure-monitor/app-insights/telemetry/investigate-missing-telemetry)|Clarified that when using Microsoft Entra ID for the Application Insights data collection endpoint, the application must also authenticate with Microsoft Entra ID or telemetry will be rejected.|
|Application-Insights|[Application Insights availability tests](../app/availability.md)|We published a query to discover deprecated URL ping tests with [Azure Resource Graph Explorer](/azure/governance/resource-graph/first-query-portal).|
|Application-Insights|[Application Insights for ASP.NET and ASP.NET Core applications](../app/asp-net.md)|[Classic API](/previous-versions/azure/azure-monitor/app/classic-api) SDK fundamentals for both ASP.NET and ASP.NET Core are now merged in one place.|
|Application-Insights|[Metrics in Application Insights](../app/metrics-overview.md)|The documentation now clarifies that Azure Monitor Metrics and Workspaces ingest custom metrics at a fixed 60-second interval, regardless of how often they're sent, while Log Analytics records them at the actual send interval. Introduced the OTEL_METRIC_EXPORT_INTERVAL environment variable, allowing configuration of the OpenTelemetry metric export interval, with a default of 60 seconds.|
|Application-Insights|[Troubleshoot missing application telemetry in Azure Monitor Application Insights](/troubleshoot/azure/azure-monitor/app-insights/telemetry/investigate-missing-telemetry)|Clarified that when using Microsoft Entra ID for the Application Insights data collection endpoint, the application must also authenticate with Entra ID or telemetry is rejected.|
|Application-Insights|[Application Insights availability tests](../app/availability.md)|Published a query to discover deprecated URL ping tests with [Azure Resource Graph Explorer](/azure/governance/resource-graph/first-query-portal).|

Comment thread articles/azure-monitor/fundamentals/whats-new.md Outdated
@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 1c3a42e:

✅ Validation status: passed

File Status Preview URL Details
articles/azure-monitor/fundamentals/overview.md ✅Succeeded
articles/azure-monitor/fundamentals/whats-new.md ✅Succeeded

For more details, please refer to the build report.

@github-actions
Copy link
Copy Markdown

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 Apr 21, 2026
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 azure-monitor/svc Change sent to author do-not-merge fundamentals/subsvc inactive This PR is inactive for more than 14 days

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants