Skip to content

feat: update telemetry for RevTel dashboard compatibility#728

Open
vbpaws wants to merge 1 commit into
mainfrom
feat/revtel-telemetry
Open

feat: update telemetry for RevTel dashboard compatibility#728
vbpaws wants to merge 1 commit into
mainfrom
feat/revtel-telemetry

Conversation

@vbpaws

@vbpaws vbpaws commented Apr 24, 2026

Copy link
Copy Markdown
Collaborator

Change Description

What this change does:

Updates RevTel telemetry User-Agent to genai-chatbot-on-aws for RevTel dashboard compatibility.

  • Updates boto_config.py to use correct solution ID format
  • Adds explicit import genai_core.boto_config to Lambda handlers that were missing genai_core imports
  • Adds unit tests for telemetry verification

How does this change contribute to the story:

Ensures chatbot AWS API usage is correctly attributed in the RevTel dashboard with the proper solution ID.

Documentation/Wiki

N/A

Related Issues

N/A

Testing Performed

Unit Testing

  • Unit tests verify botocore User-Agent injection with correct format

Integration Tests

N/A - telemetry is passive and does not affect functionality

How was this tested?

  • Unit tests pass
  • Log messages (RevTel telemetry initialized: genai-chatbot-on-aws) can be verified in CloudWatch after deployment

Reviewer Checklist

  • Reviewed linked issues to understand scope
  • Unit tests exist where applicable
  • Code is production-ready
  • Build succeeds
  • Documentation is updated if needed

Updates User-Agent format from app-name/version to aws-solutions/<solution-id>
for RevTel dashboard tracking:
- Solution ID: aws-genai-llm-chatbot
- User-Agent: aws-solutions/aws-genai-llm-chatbot
- Adds per-request logging for visibility

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant