Skip to content

feat(telemetry, extensibility): add joule_studio_gsid to extension context and span - #293

Merged
LucasAlvesSoares merged 6 commits into
SAP:mainfrom
rishikunnath2747:gsid_impl
Sep 3, 2026
Merged

feat(telemetry, extensibility): add joule_studio_gsid to extension context and span#293
LucasAlvesSoares merged 6 commits into
SAP:mainfrom
rishikunnath2747:gsid_impl

Conversation

@rishikunnath2747

@rishikunnath2747 rishikunnath2747 commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Disclaimer: Do not include SAP-internal or customer-specific information in this PR (e.g. internal system URLs, customer names, tenant IDs, or confidential configurations). This is a public repository.

Description

Adds Joule Studio GSID (joule_studio_gsid) as a first-class field alongside solution_id throughout the extension telemetry pipeline.

  • Define ATTR_JOULE_STUDIO_GSID constant in extensions.py and export it from the telemetry package init
  • Add joule_studio_gsid parameter to extension_context(), with baggage propagation and log filter support matching solution_id behaviour
  • Add joule_studio_gsid to build_extension_span_attributes() as a conditional span attribute (omitted when empty)
  • Add joule_studio_gsid field to ExtensionSourceInfo dataclass and parse it from the UMS GraphQL response (jouleStudioGsid)
  • Rename ATTR_GLOBAL_SOLUTION_ID → ATTR_JOULE_STUDIO_GSID for consistency with the field name
  • Extend test_extensions.py to cover joule_studio_gsid in all tests that already cover solution_id

Type of Change

Please check the relevant option:

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Code refactoring
  • Dependency update

Checklist

Before submitting your PR, please review and check the following:

  • I have read the Contributing Guidelines
  • I have verified that my changes solve the issue
  • I have added/updated automated tests to cover my changes
  • All tests pass locally
  • I have verified that my code follows the Code Guidelines
  • I have updated documentation (if applicable)
  • I have added type hints for all public APIs
  • My code does not contain sensitive information (credentials, tokens, etc.)
  • I have followed Conventional Commits for commit messages

…attributes

Adds Joule Studio GSID (joule_studio_gsid) as a first-class field
alongside solution_id throughout the extension telemetry pipeline.

- Define ATTR_JOULE_STUDIO_GSID constant in extensions.py and export
  it from the telemetry package __init__
- Add joule_studio_gsid parameter to extension_context(), with baggage
  propagation and log filter support matching solution_id behaviour
- Add joule_studio_gsid to build_extension_span_attributes() as a
  conditional span attribute (omitted when empty)
- Add joule_studio_gsid field to ExtensionSourceInfo dataclass and
  parse it from the UMS GraphQL response (jouleStudioGsid)
- Rename ATTR_GLOBAL_SOLUTION_ID → ATTR_JOULE_STUDIO_GSID for
  consistency with the field name
- Extend test_extensions.py to cover joule_studio_gsid in all tests
  that already cover solution_id
@rishikunnath2747
rishikunnath2747 requested a review from a team as a code owner August 31, 2026 11:07
@LucasAlvesSoares

Copy link
Copy Markdown
Contributor

Missing changes in uv.lock. Run uv sync to sync the lock file with the project descriptor.

@LucasAlvesSoares
LucasAlvesSoares merged commit 635c3c7 into SAP:main Sep 3, 2026
11 checks passed
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.

2 participants