Skip to content

Updated internal statsbeat routing to follow the customer's effective ingestion route after an accepted redirect so EU SDK statistics are reported to the EU statsbeat destination. - #48817

Draft
Radhika Gupta (rads-1996) wants to merge 6 commits into
Azure:mainfrom
rads-1996:fix-eu-sdkstats-routing
Draft

Radhika Gupta (rads-1996) wants to merge 6 commits into
Azure:mainfrom
rads-1996:fix-eu-sdkstats-routing

Conversation

@rads-1996

@rads-1996 Radhika Gupta (rads-1996) commented Aug 31, 2026

Copy link
Copy Markdown
Member

All SDK Contribution checklist:

  • The pull request does not introduce [breaking changes]
  • CHANGELOG is updated for new features, bug fixes or other significant changes.
  • I have read the contribution guidelines.

General Guidelines and Best Practices

  • Title of the pull request is clear and informative.
  • There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

  • Pull request includes test coverage for the included changes.

@github-actions github-actions Bot added the Monitor - Exporter Monitor OpenTelemetry Exporter label Aug 31, 2026
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
9 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates Azure Monitor redirect handling so Statsbeat follows the customer’s effective regional ingestion route.

Changes:

  • Groups trusted redirect domains by Azure cloud.
  • Dynamically updates Statsbeat routing and host dimensions.
  • Adds redirect, region-classification, and reconfiguration tests.

Reviewed changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
CHANGELOG.md Documents the redirect-routing fix.
_constants.py Groups trusted suffixes and adds Germany North.
export/_base.py Handles cross-suffix redirects and notifies Statsbeat.
statsbeat/_manager.py Reconfigures Statsbeat after redirects.
statsbeat/_statsbeat.py Exposes the endpoint-update hook.
statsbeat/_statsbeat_metrics.py Refreshes the host metric dimension.
statsbeat/_utils.py Extracts regions and selects Statsbeat destinations.
tests/test_base_exporter.py Tests redirect handling and propagation.
tests/statsbeat/test_manager.py Tests Statsbeat route updates.
tests/statsbeat/test_metrics.py Tests host-dimension updates.
tests/statsbeat/test_utils.py Tests region and boundary selection.

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

@rads-1996
Radhika Gupta (rads-1996) force-pushed the fix-eu-sdkstats-routing branch 2 times, most recently from bd3e916 to 72ab7ea Compare September 15, 2026 20:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Monitor - Exporter Monitor OpenTelemetry Exporter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants