Skip to content

Migrate healthcare-assistant 2-app-with-instrumentation from galileo to splunk-ao SDK - #732

Closed
etserend wants to merge 1 commit into
splunk:mainfrom
etserend:HYBIM-977-healthcare-assistant-splunk-ao-migration
Closed

Migrate healthcare-assistant 2-app-with-instrumentation from galileo to splunk-ao SDK#732
etserend wants to merge 1 commit into
splunk:mainfrom
etserend:HYBIM-977-healthcare-assistant-splunk-ao-migration

Conversation

@etserend

Copy link
Copy Markdown

Summary

  • Replace galileo SDK with splunk-ao across app code, k8s config, and workshop content pages
  • splunk-ao is a 1:1 rebrand — same callback architecture, direct import path rename only
  • Validated locally against lab0 (O11y Alpha) and Galileo staging with three-way trace comparison (galileo SDK / splunk-ao standalone / splunk-ao O11y)

Changes

  • requirements.txt: galileosplunk-ao
  • setup_env.py: GALILEO_*SPLUNK_AO_REALM / SPLUNK_AO_O11Y_TOKEN / SPLUNK_AO_PROJECT / SPLUNK_AO_AGENT_STREAM
  • agent.py / agent-with-instrumentation.py: galileo_contextsplunk_ao_context, GalileoAsyncCallbackSplunkAOAsyncCallback, log_stream=agent_stream=
  • helpers/hallucination_helpers.py: GalileoLoggerSplunkAOLogger
  • app.py: session state keys renamed
  • k8s.yaml: galileo-configsplunk-ao-config; single GALILEO_API_KEY secret → splunk-ao-secret with realm, o11y-token, o11y-api-token
  • Workshop content pages: remove "Note about SDK" boxes, update kubectl commands, imports, and log output examples

Test plan

  • Ran validate_traces.py — all 3 envs produced traces with correct RAG answers
  • Confirmed span tree, token counts, and model badge in AO console (Galileo staging + O11y Alpha)

@dmitchsplunk

Copy link
Copy Markdown
Collaborator

Thank you for the PR but we can't make these changes until after the workshop is delivered at GSX on August 25th. Closing for now to ensure it doesn't get merged inadvertently.

cc @timhard-splunk @billg-splunk @rcastley

@etserend

Copy link
Copy Markdown
Author

Sure, I've filed ticket for tracking purpose.

Thank you for the PR but we can't make these changes until after the workshop is delivered at GSX on August 25th. Closing for now to ensure it doesn't get merged inadvertently.

cc @timhard-splunk @billg-splunk @rcastley

@satish-m

Copy link
Copy Markdown

@dmitchsplunk sorry about this, we will make changes in a fork and can update here after the GSX. Have a great demo.
cc: @etserend

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