Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

install_ffmpeg: Add FLV muxer #428

Merged
merged 1 commit into from
Dec 3, 2024
Merged

install_ffmpeg: Add FLV muxer #428

merged 1 commit into from
Dec 3, 2024

Conversation

j0sh
Copy link
Collaborator

@j0sh j0sh commented Dec 2, 2024

This should make RTMP output work on production (non-debug) builds of LPMS

@j0sh j0sh requested review from thomshutt, leszko and mjh1 December 2, 2024 22:09
@j0sh
Copy link
Collaborator Author

j0sh commented Dec 2, 2024

Tests are still broken in CI but pass locally

josh@josh-gpu:~/lpms$ time go test --tags=nvidia -timeout 1h ./ffmpeg
ok      github.com/livepeer/lpms/ffmpeg 1575.155s

real    26m16.094s
user    36m18.451s
sys     1m49.643s

Copy link
Member

@victorges victorges left a comment

Choose a reason for hiding this comment

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

LGTM!

@@ -205,7 +205,7 @@ fi

if [[ $BUILD_TAGS == *"debug-video"* ]]; then
echo "video debug mode, building ffmpeg with tools, debug info and additional capabilities for running tests"
DEV_FFMPEG_FLAGS="--enable-muxer=md5,flv --enable-demuxer=hls --enable-filter=ssim,tinterlace --enable-encoder=wrapped_avframe,pcm_s16le "
Copy link
Member

Choose a reason for hiding this comment

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

Why remove this one? Are those only extra from the non-dev ones?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, the base configuration includes flv now, so specifying it (again) with the dev flags isn't necessary (trying to keep things tidy)

@j0sh j0sh merged commit b33cac6 into master Dec 3, 2024
2 of 3 checks passed
@j0sh j0sh deleted the ja/enable-flv-muxer branch December 3, 2024 00:05
j0sh added a commit to livepeer/go-livepeer that referenced this pull request Dec 3, 2024
Updates LPMS to use the FLV muxer introduced in livepeer/lpms#428
which should resolve issues with getting RTMP output.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants