Message ID | 20221212155846.35753-1-david.marchand@redhat.com |
---|---|
State | Rejected |
Headers | show |
Series | [ovs-dev] travis: Fix remaining trace of kernel build tests. | expand |
Context | Check | Description |
---|---|---|
ovsrobot/apply-robot | success | apply and check: success |
ovsrobot/github-robot-_Build_and_Test | success | github build: passed |
ovsrobot/intel-ovs-compilation | success | test: success |
On 12/12/22 16:58, David Marchand wrote: > We stopped building OVS kernel module in the CI but forgot to update the > (unused?) Travis configuration. I'd say, we should just remove the yml. Nobody seems to use it anyway. What do you think? > > Fixes: b6941ca7b8c9 ("ci: Stop building OVS kernel module.") > Signed-off-by: David Marchand <david.marchand@redhat.com> > --- > .travis.yml | 6 ------ > 1 file changed, 6 deletions(-) > > diff --git a/.travis.yml b/.travis.yml > index c7aeede06e..1fae2ea7ac 100644 > --- a/.travis.yml > +++ b/.travis.yml > @@ -36,12 +36,6 @@ matrix: > - arch: arm64 > compiler: gcc > env: TESTSUITE=1 DPDK=1 > - - arch: arm64 > - compiler: gcc > - env: KERNEL_LIST="5.5 4.19" > - - arch: arm64 > - compiler: gcc > - env: KERNEL_LIST="4.9 3.16" > - arch: arm64 > compiler: gcc > env: DPDK_SHARED=1
On Mon, Dec 12, 2022 at 5:14 PM Ilya Maximets <i.maximets@ovn.org> wrote: > > On 12/12/22 16:58, David Marchand wrote: > > We stopped building OVS kernel module in the CI but forgot to update the > > (unused?) Travis configuration. > > I'd say, we should just remove the yml. Nobody seems to use it anyway. > What do you think? I don't mind dropping it, but just in case, I Cc'ed some ARM people. > > > > > Fixes: b6941ca7b8c9 ("ci: Stop building OVS kernel module.") > > Signed-off-by: David Marchand <david.marchand@redhat.com>
David Marchand <david.marchand@redhat.com> writes: > On Mon, Dec 12, 2022 at 5:14 PM Ilya Maximets <i.maximets@ovn.org> wrote: >> >> On 12/12/22 16:58, David Marchand wrote: >> > We stopped building OVS kernel module in the CI but forgot to update the >> > (unused?) Travis configuration. >> >> I'd say, we should just remove the yml. Nobody seems to use it anyway. >> What do you think? > > I don't mind dropping it, but just in case, I Cc'ed some ARM people. I am certainly in favor of dropping it. The credits system they implemented has made it unusable for most projects without some real $$ being spent. >> >> > >> > Fixes: b6941ca7b8c9 ("ci: Stop building OVS kernel module.") >> > Signed-off-by: David Marchand <david.marchand@redhat.com>
On 12/13/22 14:29, Aaron Conole wrote: > David Marchand <david.marchand@redhat.com> writes: > >> On Mon, Dec 12, 2022 at 5:14 PM Ilya Maximets <i.maximets@ovn.org> wrote: >>> >>> On 12/12/22 16:58, David Marchand wrote: >>>> We stopped building OVS kernel module in the CI but forgot to update the >>>> (unused?) Travis configuration. >>> >>> I'd say, we should just remove the yml. Nobody seems to use it anyway. >>> What do you think? >> >> I don't mind dropping it, but just in case, I Cc'ed some ARM people. > > I am certainly in favor of dropping it. The credits system they > implemented has made it unusable for most projects without some real $$ > being spent. +1 For the ARM CI, we should implement new ARM jobs in Cirrus CI. > >>> >>>> >>>> Fixes: b6941ca7b8c9 ("ci: Stop building OVS kernel module.") >>>> Signed-off-by: David Marchand <david.marchand@redhat.com> >
On Tue, Dec 13, 2022 at 2:34 PM Ilya Maximets <i.maximets@ovn.org> wrote: > > On 12/13/22 14:29, Aaron Conole wrote: > > David Marchand <david.marchand@redhat.com> writes: > > > >> On Mon, Dec 12, 2022 at 5:14 PM Ilya Maximets <i.maximets@ovn.org> wrote: > >>> > >>> On 12/12/22 16:58, David Marchand wrote: > >>>> We stopped building OVS kernel module in the CI but forgot to update the > >>>> (unused?) Travis configuration. > >>> > >>> I'd say, we should just remove the yml. Nobody seems to use it anyway. > >>> What do you think? > >> > >> I don't mind dropping it, but just in case, I Cc'ed some ARM people. > > > > I am certainly in favor of dropping it. The credits system they > > implemented has made it unusable for most projects without some real $$ > > being spent. > > +1 > > For the ARM CI, we should implement new ARM jobs in Cirrus CI. Ok, I marked this patch as rejected and submitted a patch to drop Travis config.
diff --git a/.travis.yml b/.travis.yml index c7aeede06e..1fae2ea7ac 100644 --- a/.travis.yml +++ b/.travis.yml @@ -36,12 +36,6 @@ matrix: - arch: arm64 compiler: gcc env: TESTSUITE=1 DPDK=1 - - arch: arm64 - compiler: gcc - env: KERNEL_LIST="5.5 4.19" - - arch: arm64 - compiler: gcc - env: KERNEL_LIST="4.9 3.16" - arch: arm64 compiler: gcc env: DPDK_SHARED=1
We stopped building OVS kernel module in the CI but forgot to update the (unused?) Travis configuration. Fixes: b6941ca7b8c9 ("ci: Stop building OVS kernel module.") Signed-off-by: David Marchand <david.marchand@redhat.com> --- .travis.yml | 6 ------ 1 file changed, 6 deletions(-)