Skip to content

Update LPMS to https://github.com/livepeer/lpms/pull/445#3863

Merged
j0sh merged 3 commits intomasterfrom
ja/lpms-transcoder-crash-hotfix
Mar 10, 2026
Merged

Update LPMS to https://github.com/livepeer/lpms/pull/445#3863
j0sh merged 3 commits intomasterfrom
ja/lpms-transcoder-crash-hotfix

Conversation

@j0sh
Copy link
Collaborator

@j0sh j0sh commented Jan 27, 2026

Pulls in a number of fixes and mitigations in LPMS for out-of-order timestamps, runaway encodes, etc: livepeer/lpms#445

@github-actions github-actions bot added the dependencies Pull requests that update a dependency file label Jan 27, 2026
@j0sh j0sh marked this pull request as ready for review March 5, 2026 20:41
@j0sh j0sh requested a review from Shih-Yu March 5, 2026 20:51
Copy link

@Shih-Yu Shih-Yu left a comment

Choose a reason for hiding this comment

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

LGTM!

@codecov
Copy link

codecov bot commented Mar 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 32.81737%. Comparing base (48c5d86) to head (f0e556f).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@                 Coverage Diff                 @@
##              master       #3863         +/-   ##
===================================================
+ Coverage   32.79594%   32.81737%   +0.02143%     
===================================================
  Files            171         171                 
  Lines          41993       41993                 
===================================================
+ Hits           13772       13781          +9     
+ Misses         27189       27180          -9     
  Partials        1032        1032                 

see 4 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 48c5d86...f0e556f. Read the comment docs.
see 4 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@j0sh j0sh changed the title Update LPMS to https://github.com/livepeer/lpms/pull/443 Update LPMS to https://github.com/livepeer/lpms/pull/445 Mar 10, 2026
@j0sh
Copy link
Collaborator Author

j0sh commented Mar 10, 2026

From livepeer/lpms#445

Ran a test with a go-livepeer B+O with around 12 hours' worth of customer segments being sent in alternating order, checked timestamps for all, and and things check out OK.

Merging this so we can cut a release and get the fix into the wild.

@j0sh j0sh merged commit c1b70eb into master Mar 10, 2026
18 checks passed
@j0sh j0sh deleted the ja/lpms-transcoder-crash-hotfix branch March 10, 2026 04:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants