Skip to content

HYBIM-948 refresh README and changelog for first Splunk AO release - #195

Merged
pradystar merged 5 commits into
mainfrom
docs/first-release-readme-changelog
Jul 31, 2026
Merged

HYBIM-948 refresh README and changelog for first Splunk AO release#195
pradystar merged 5 commits into
mainfrom
docs/first-release-readme-changelog

Conversation

@pradystar

@pradystar pradystar commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

Summary

Refresh the README and changelog for the first Splunk AO Python SDK release, covering standalone and O11y configuration and the SDK’s OTLP based telemetry behavior.

Why

The existing documentation described legacy Galileo behavior and did not accurately explain current routing, lifecycle, batching, authentication, diagnostics, and OpenTelemetry integration requirements.

What changed

  • Documented standalone and O11y installation, authentication, and routing.
  • Added Agent Stream name/ID guidance and routing precedence.
  • Documented service naming and reserved routing Resource attributes.
  • Clarified @log, handler, manual logger, flush(), terminate(), and provider shutdown behavior.
  • Added standard OpenTelemetry instrumentation and custom exporter examples.
  • Documented export diagnostics and session behavior for both deployments.
  • Corrected dataset and experiment examples.
  • Reset the changelog for the first Splunk AO release.

Documentation corrections

  • Updated the main, A2A, and ADK setup guidance to use SPLUNK_AO_O11Y_TOKEN and SPLUNK_AO_O11Y_API_TOKEN.
  • Simplified general routing documentation to use Project and Agent Stream names; ID routing remains documented only where Agent Control requires it.
  • Reset the A2A and ADK changelogs for their first Splunk AO releases.
  • Clarified that the generated low-level API client does not read SDK environment variables or perform deployment-aware configuration.

Tests

Validated 23 Python examples, 9 shell examples, 76 focused tests.

@pradystar
pradystar requested a review from Copilot July 31, 2026 00:02
@pradystar
pradystar requested review from fercor-cisco and sheilattran8 and removed request for Copilot July 31, 2026 00:02

@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: request_changes — The new "Export diagnostics" docs describe a public attribute (processor.export_health) and rejection/recovery/logger behavior that do not exist in the source.

Comment thread README.md
Comment thread README.md
Comment thread README.md
Comment thread CHANGELOG.md
@pradystar

Copy link
Copy Markdown
Collaborator Author

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

Verdict: request_changes — The new "Export diagnostics" docs describe a public attribute (processor.export_health) and rejection/recovery/logger behavior that do not exist in the source.

@fercor-cisco This branch now includes all missing changes from other PRs. Please check. Thank you :).

@pradystar
pradystar requested a review from fercor-cisco July 31, 2026 15:18

@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 — Documentation-only PR; all documented APIs, env vars, and behaviors verified accurate against the source.

Follow-ups

Suggested follow-up work that could be tracked as Jira tickets:

  • README.md:295-311: The OpenAI streaming example (pre-existing, outside this PR's edits) does not call flush() or terminate(). In a finite script the completed span may never be exported before the process exits, unlike the other examples in this refresh which now consistently show explicit flush()/terminate(). Consider aligning it with the new guidance in a later pass.
  • src/splunk_ao/decorator.py:1404-1415: The docstring for splunk_ao_context.init() still documents a 'log_stream' parameter that no longer exists (the parameter is 'agent_stream'). This source docstring is out of scope for a docs PR but is now inconsistent with the README's Agent Stream terminology and could be updated for consistency.

fercor-cisco
fercor-cisco previously approved these changes Jul 31, 2026
@fercor-cisco

Copy link
Copy Markdown
Collaborator

Filed follow-up stories under epic HYBIM-697 for the review items deferred from this PR:

  • HYBIM-953 — Align the OpenAI streaming README example (README.md:295-311) with the new flush()/terminate() guidance so all examples are consistent.
  • HYBIM-954 — Fix the stale log_stream parameter in the splunk_ao_context.init() docstring (src/splunk_ao/decorator.py:1404-1415); it should reference agent_stream.

…adme-changelog

# Conflicts:
#	README.md
#	splunk-ao-a2a/README.md
@pradystar
pradystar requested a review from fercor-cisco July 31, 2026 18:34
@pradystar
pradystar merged commit acc2d24 into main Jul 31, 2026
21 checks passed
@pradystar
pradystar deleted the docs/first-release-readme-changelog branch July 31, 2026 18:38
@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