Skip to content

HYBIM-951 rename O11y token environment variables - #199

Merged
pradystar merged 2 commits into
mainfrom
refactor/o11y-token-env-vars
Jul 31, 2026
Merged

HYBIM-951 rename O11y token environment variables#199
pradystar merged 2 commits into
mainfrom
refactor/o11y-token-env-vars

Conversation

@pradystar

@pradystar pradystar commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

Summary

Rename the O11y SDK token environment variables to make their purpose explicit:

  • SPLUNK_AO_SF_TOKEN → SPLUNK_AO_O11Y_TOKEN
  • SPLUNK_AO_SF_API_TOKEN → SPLUNK_AO_O11Y_API_TOKEN

Why

The previous names exposed internal SF terminology and did not clearly indicate that these tokens configure O11y Cloud authentication. Since the SDK has not been released, the old names are removed without compatibility aliases.

What changed

  • Updated O11y deployment detection, configuration, validation, and authentication.
  • Updated the Galileo-core configuration bridge.
  • Updated tests, README documentation, A2A examples, and ADK references.
  • Kept wire authentication unchanged: O11y requests continue using the X-SF-Token
    header.
  • Standalone authentication behavior is unchanged.

Additional fixes

A2A

  • Updated A2A compatibility tests to reflect that splunk_ao.* attribute duplication is disabled by default while preserving native gen_ai.* attributes.

ADK

  • Restored ADK legacy ingestion_hook compatibility with recursive trace completion.
  • Added coverage for nested ADK trace completion.

Testing

2130 passed, 4 skipped; ADK 20 passed; live @log smoke test successfully exported to O11y.

@pradystar
pradystar requested a review from fercor-cisco July 31, 2026 16:56

@fercor-cisco fercor-cisco left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

🤖 This review was generated by the Astra agent (claude-opus-4-8). It may contain mistakes.

Verdict: approve — Clean, complete env-var rename with no stray references; two additional fixes (ADK conclude_all override, A2A test isolation) are correct and well-tested.

@pradystar
pradystar merged commit 083ff7f into main Jul 31, 2026
21 checks passed
@pradystar
pradystar deleted the refactor/o11y-token-env-vars branch July 31, 2026 17:50
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 31, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants