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

Update Dockerfile base image to Ubuntu 22.04 #1527

Merged
merged 1 commit into from
Feb 17, 2024
Merged

Conversation

basisbit
Copy link
Contributor

@basisbit basisbit commented Feb 17, 2024

The OME Dockerfiles currently are based on Ubuntu 20.04. This is becoming a security issue because of Ubuntu 20.04 not getting properly supplied with security updates any more. Anything that is not core Ubuntu or Linux kernel is not covered by the LTS promise and package maintainers have largely started dropping the upkeep work for the 20.04 package repository in the past year.

Thus I suggest updating the docker image to 22.04.
This also requires some changes to the misc/prerequisites.sh (especially the check_version + proceed_yn functions) and the scripts for installing the GPU acceleration stuff, as well as testing OME.

I started testing with Dockerfile updated to 22.04 and compiling works, starting as origin works, starting as edge works, rtmp provider works, srt provider works, OVT publisher works, ovt provider works, WebRTC provider works over udp and over tcp, LLHLS provider works, HLS provider works, x264 transcoding works and AAC -> OPUS transcoding works. So far I didn't find any new issues after changing the docker base image to 22.04 and rebuilding the docker images.

However, I don't have any hardware that I could use for testing the GPU acceleration stuff, thus I did not update the Dockerfile.cuda and the Dockerfile.cuda.local.

This would probably also be a good opportunity to remove any mentions of Ubuntu 18, which is not really usable nowadays anymore anyways, especially once OME also upgrades to ffmpeg 6.1 (which might be needed for the support of Enhanced RTMP for being able to ingest+forward HEVC/AV1 over RTMP). However I decided to exclude that change from this pull request in order to not slow down merging of this change because of fear of breaking any old automated setups.

@getroot getroot merged commit 7d8f7e0 into AirenSoft:master Feb 17, 2024
@getroot
Copy link
Member

getroot commented Feb 17, 2024

Thank you!

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