Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

Steffen911
Copy link
Member

@Steffen911 Steffen911 commented Sep 30, 2025

Fixes #9341

Important

Skip JSON parsing for completion_start_time in OtelIngestionProcessor.ts and add a test to verify correct handling.

  • Behavior:
    • Skip JSON parsing for completion_start_time in extractCompletionStartTime() in OtelIngestionProcessor.ts.
  • Tests:
    • Add test case in otelMapping.servertest.ts to verify completion_start_time is correctly cast from string.

This description was created by Ellipsis for 2526bc7. You can customize this summary. It will automatically update as commits are pushed.

@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. bug Something isn't working labels Sep 30, 2025
@Steffen911 Steffen911 requested a review from hassiebp September 30, 2025 08:54
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Sep 30, 2025
@dosubot dosubot bot added the auto-merge This PR is set to be merged label Sep 30, 2025
@Steffen911 Steffen911 added this pull request to the merge queue Sep 30, 2025
Merged via the queue into main with commit c084d19 Sep 30, 2025
29 checks passed
@Steffen911 Steffen911 deleted the steffen/parse-completion-start-time-otel branch September 30, 2025 11:44
@dosubot dosubot bot removed the auto-merge This PR is set to be merged label Sep 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working lgtm This PR has been approved by a maintainer size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug: Time to First Token (completion_start_time) not displayed in UI despite correct ISO 8601 value on generation observation
2 participants