[ovs-dev] Travis build failures due to base image change to Xenial.
i.maximets at ovn.org
Mon Oct 28 14:04:56 UTC 2019
On 09.08.2019 12:23, Ilya Maximets wrote:
> On 01.08.2019 20:48, Aaron Conole wrote:
>> Ilya Maximets <i.maximets at samsung.com> writes:
>>> Hi, everyone.
>>> I'm trying to fix TravisCI build failures on older branches. Recently they
>>> started to change default images from Trusty to Xenial. Not all the repositories
>>> affected so far, but main openvswitch github repo already builds with Ubuntu
>>> Xenial as a base image.
>>> This caused few issues on older OVS branches due to more recent compilers:
>>> There are patches that needs to be backported:
>>> * Followiong patch needed for all branches from 2.9 to 2.5:
>>> a7021b08b 2018-07-09 | configure: Disable -Wnull-pointer-arithmetic Clang warning. [Ben Pfaff]
>>> * Branches 2.7 - 2.5 additionally needs:
>>> 1e78e3085 2017-01-26 | libX.pc: use the correct output directory [Aaron Conole]
>> Ack to that backport (if it matters).
> OK. I went ahead and backported/pushed above patches to branches 2.9 - 2.6.
> Travis CI should be green now on these branches.
>>> Above makes branches 2.9 - 2.6 to build correctly.
>>> I prepared backports for these branches and could push them.
>>> However, there is an additional issue with branch-2.5:
>>> branch-2.5 has kernel 2.6.32 in the .travis.yml, but gcc >= 5 is not
>>> able to build this kernel. This kernel reached its EOL few years ago
>>> already and will never be fixed. So, there are few options for this issue:
>>> 1. Simply remove 2.6.32 kernel from the build matrix.
>>> (I have a simple patch for this.)
> For branch-2.5 I'm waiting for more comments/Ack on the patch:
> Backports to this branch will go along with applying above patch.
This discussion happened while you were out of office, but I'd like to
hear your opinion on this issue with branch-2.5 that still exists.
Best regards, Ilya Maximets.
>>> 2. Officially drop support of this (or maybe some other too) kernel on branch-2.5.
>>> Note: Starting from branch-2.6 OVS officially supports only kernels >= 3.10.
>>> This will, probably, require documentation updates on all newer branches.
>>> 3. Drop support of branch-2.5, i.e. stop backporting patches and preparing
>>> stable releases.
>>> This might be not that easy as ovs-2.5 claimed as an LTS release right now.
>>> Does anybody use it?
>> I don't have an opinion on what the right thing to do is. However, we
>> don't have any users from Red Hat for OvS 2.5 that I'm aware. We have
>> some users of OvS 2.6, but most have already migrated to 2.11 AFAIK.
>>> 4. Add 'dist: trusty' to .travis.yml on branch-2.5 so Travis will keep using
>>> Ubuntu Trusty image with old compilers that able to build kernel 2.6.32.
>>> Would like to hear some opinions.
>>> Best regards, Ilya Maximets.
More information about the dev