Skip to content

fix: preserve status timestamps across transitions - #633

Merged
zdtsw merged 2 commits into
llm-d:mainfrom
sudiptob2:fix/preserve-status-transition-timestamps
Aug 10, 2026
Merged

fix: preserve status timestamps across transitions#633
zdtsw merged 2 commits into
llm-d:mainfrom
sudiptob2:fix/preserve-status-transition-timestamps

Conversation

@sudiptob2

@sudiptob2 sudiptob2 commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Why is this PR needed?

Fixes #632

What does this PR do?

How was this tested?

  • Unit tests added/updated/verified
  • Integration/e2e tests added/updated/verified
  • Manual testing performed

Checklist

  • Commits are signed off (git commit -s) per DCO
  • Code follows project contributing guidelines
  • CI checks pass (make ci)
  • E2E tests pass (make test-e2e)

Related Issues

@github-actions github-actions Bot added the bug Fixes incorrect behavior label Aug 2, 2026
Signed-off-by: Sudipto Baral <sudiptobaral.me@gmail.com>
@sudiptob2
sudiptob2 force-pushed the fix/preserve-status-transition-timestamps branch from 991eb5a to ea18402 Compare August 7, 2026 22:27
@sudiptob2
sudiptob2 requested a review from zdtsw as a code owner August 7, 2026 22:27

@zdtsw zdtsw left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/LGTM

@zdtsw

zdtsw commented Aug 10, 2026

Copy link
Copy Markdown
Member

@yizhaodev wanna take a look?

@zdtsw
zdtsw merged commit 46b536b into llm-d:main Aug 10, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Fixes incorrect behavior

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: in_progress_at and finalizing_at are null on every completed batch

3 participants