----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/75088/#review226680 -----------------------------------------------------------
Ship it! 3rdparty/libprocess/src/tests/ssl_tests.cpp Line 126 (original), 126 (patched) <https://reviews.apache.org/r/75088/#comment314946> split this libprocess commit into a follow up one (we try to keep commits self contained in stout and libprocess so that we can upstream changes to these libraries) configure.ac Lines 2321-2324 (original), 2326-2346 (patched) <https://reviews.apache.org/r/75088/#comment314945> you can remove these two, we don't use them we use the following FWIW: inflateInit2 inflate inflateEnd deflateInit2 deflate deflateEnd but any reason you split them? we should put a comment in the code if there's a need to split these, or just not check any specific functions since that can get very verbose if we start checking every function we need from every library? - Benjamin Mahler On July 15, 2024, 3:46 p.m., Jason Zhou wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/75088/ > ----------------------------------------------------------- > > (Updated July 15, 2024, 3:46 p.m.) > > > Review request for mesos and Benjamin Mahler. > > > Repository: mesos > > > Description > ------- > > For in review #75080, we made use of replace_bpf_fd and BPF_F_REPLACE > which were added in kernel 5.6. Our current ubuntu 20.04 base image > uses kernel 5.4. > > As such we will be upgrading the ubuntu version used > in Jenkins to 22.04, whose base image uses kernel 5.15, so that we can > make mesos on the updated pipeline, enabling reviewbot, tidybot, and > coverity. > > > Diffs > ----- > > 3rdparty/libprocess/src/tests/ssl_tests.cpp > 10e609d8a056fa0efe25b1d3a49912d1893f6029 > configure.ac 7c0597f56a11cbaf6874aba2e910cb36debd20a7 > support/docker-build.sh cb0b05b6effef246a88efedd5dd5f4cd15231397 > support/jenkins/reviewbot.sh 03ebcd6007d6da9bfb5ef0d2b3f84b1f83219127 > support/mesos-build/ubuntu-20.04-arm.dockerfile > a3b321fb740a5b0a0add12f31511ed2ff6b03691 > support/mesos-build/ubuntu-20.04.dockerfile > e183ff742b034009f814372998a596899a3e3b7b > support/mesos-tidy/Dockerfile 769a22d9e0dde4cf6518ad5900c5b316c48d925c > support/verify-reviews.py 8e7c54faae0f728ef2138995c86a4a4ef4b5df47 > > > Diff: https://reviews.apache.org/r/75088/diff/1/ > > > Testing > ------- > > Tested build with 22.04 base image using both cmake and autotools, builds > were able to complete. > Tested using docker-build, build was able to complete. > Mesos-tidy will need to be tested on jenkins but I expect it should also > complete as we can now build on 22.04 > > > Thanks, > > Jason Zhou > >
