Skip to content

Match Upstream Changes: Update scope value to support new exporter path#62

Merged
rads-1996 merged 2 commits intomicrosoft:mainfrom
rads-1996:update-per-upstream-changes
Apr 24, 2026
Merged

Match Upstream Changes: Update scope value to support new exporter path#62
rads-1996 merged 2 commits intomicrosoft:mainfrom
rads-1996:update-per-upstream-changes

Conversation

@rads-1996
Copy link
Copy Markdown
Member

@rads-1996 rads-1996 commented Apr 24, 2026

Include changes in upstream PR - microsoft/Agent365-python#235

Copilot AI review requested due to automatic review settings April 24, 2026 00:02
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

Updates the Agent365 observability export configuration to match upstream changes by adjusting the export URL path structure and updating the production auth scope, plus adding tests to pin these values together.

Changes:

  • Updated build_export_url() to include the /otlp/ segment in both standard and S2S export paths.
  • Updated production observability scope constants to a new Agent365-specific scope value.
  • Added snapshot-style tests to keep production endpoint, scope, and URL path structure from drifting independently.

Reviewed changes

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

File Description
tests/a365/test_utils.py Updates the expected standard export URL to include /otlp/.
tests/a365/runtime/test_export_config_consistency.py Adds snapshot tests pinning production endpoint, auth scope, and export URL path patterns.
src/microsoft/opentelemetry/a365/runtime/environment_utils.py Updates PROD_OBSERVABILITY_SCOPE constant to the new scope value.
src/microsoft/opentelemetry/a365/core/exporters/utils.py Updates build_export_url() paths and changes _A365_DEFAULT_SCOPE used by token resolvers.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/microsoft/opentelemetry/a365/runtime/environment_utils.py
Comment thread src/microsoft/opentelemetry/a365/core/exporters/utils.py
Comment thread tests/a365/runtime/test_export_config_consistency.py
@rads-1996 rads-1996 force-pushed the update-per-upstream-changes branch from 0578279 to 1ef6073 Compare April 24, 2026 14:47
@rads-1996 rads-1996 merged commit dd3e997 into microsoft:main Apr 24, 2026
8 checks passed
@rads-1996 rads-1996 deleted the update-per-upstream-changes branch April 24, 2026 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants