Skip to content

Conversation

@codefromthecrypt
Copy link
Contributor

@codefromthecrypt codefromthecrypt commented Nov 9, 2025

Description

updates image generation request tracing to match latest OpenInference

  • Span name changed to ImagesResponse
  • Add output.mime_type and output.value attributes
  • Remove llm.model_name from request attributes (now in invocation_parameters)
  • Fix missing RecordResponse call in translator

Related Issues/PRs (if applicable)

#1519

@codefromthecrypt codefromthecrypt requested a review from a team as a code owner November 9, 2025 03:17
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Nov 9, 2025
@codefromthecrypt codefromthecrypt marked this pull request as draft November 9, 2025 03:22
@codefromthecrypt
Copy link
Contributor Author

will work on this more when online again (long flight ahead)

@codefromthecrypt codefromthecrypt force-pushed the fix/image-generation-tracing branch from 277ac56 to 9dd7bf0 Compare November 9, 2025 05:07
@codecov-commenter
Copy link

codecov-commenter commented Nov 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.93%. Comparing base (e584dff) to head (8d1ffed).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1520   +/-   ##
=======================================
  Coverage   83.93%   83.93%           
=======================================
  Files         144      144           
  Lines       12667    12670    +3     
=======================================
+ Hits        10632    10635    +3     
  Misses       1419     1419           
  Partials      616      616           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

- Span name changed to ImagesResponse
- Add output.mime_type and output.value attributes
- Add llm.invocation_parameters attribute
- Remove llm.model_name from request attributes
- Fix missing RecordResponse call in translator
- Refactor tests to match chat/embeddings style

Signed-off-by: Adrian Cole <[email protected]>
@codefromthecrypt codefromthecrypt force-pushed the fix/image-generation-tracing branch from 9dd7bf0 to 8fff806 Compare November 9, 2025 10:55
@codefromthecrypt
Copy link
Contributor Author

note I also polished test style, since codecov is angry anyway

@codefromthecrypt codefromthecrypt marked this pull request as ready for review November 9, 2025 11:34
@nacx nacx enabled auto-merge (squash) November 10, 2025 08:57
@nacx nacx merged commit 5cd5f1b into envoyproxy:main Nov 10, 2025
31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants