Message ID | 20210902142150.3389673-1-i.maximets@ovn.org |
---|---|
State | Accepted |
Headers | show |
Series | [ovs-dev] ci: ovn-kubernetes: Avoid unnecessary builds. | expand |
Context | Check | Description |
---|---|---|
ovsrobot/apply-robot | success | apply and check: success |
ovsrobot/github-robot-_Build_and_Test | success | github build: passed |
ovsrobot/github-robot-_ovn-kubernetes | fail | github build: failed |
On Thu, Sep 2, 2021 at 10:22 AM Ilya Maximets <i.maximets@ovn.org> wrote: > > There is no need to build OVS and OVN separately before building RPMs. > > This saves 3-5 minutes of the build. > > Signed-off-by: Ilya Maximets <i.maximets@ovn.org> Thanks for the patch. I applied this to the main branch. Numan > --- > .ci/ovn-kubernetes/Dockerfile | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/.ci/ovn-kubernetes/Dockerfile b/.ci/ovn-kubernetes/Dockerfile > index 43c2bfbb5..9dde47259 100644 > --- a/.ci/ovn-kubernetes/Dockerfile > +++ b/.ci/ovn-kubernetes/Dockerfile > @@ -24,14 +24,14 @@ WORKDIR /tmp/ovn/ovs > > RUN ./boot.sh > RUN ./configure -v > -RUN make && make rpm-fedora > +RUN make rpm-fedora > RUN rm rpm/rpmbuild/RPMS/x86_64/*debug* > RUN rm rpm/rpmbuild/RPMS/x86_64/*devel* > > WORKDIR /tmp/ovn > RUN ./boot.sh > RUN ./configure > -RUN make && make rpm-fedora > +RUN make rpm-fedora > RUN rm rpm/rpmbuild/RPMS/x86_64/*debug* > RUN rm rpm/rpmbuild/RPMS/x86_64/*docker* > > -- > 2.31.1 > > _______________________________________________ > dev mailing list > dev@openvswitch.org > https://mail.openvswitch.org/mailman/listinfo/ovs-dev >
diff --git a/.ci/ovn-kubernetes/Dockerfile b/.ci/ovn-kubernetes/Dockerfile index 43c2bfbb5..9dde47259 100644 --- a/.ci/ovn-kubernetes/Dockerfile +++ b/.ci/ovn-kubernetes/Dockerfile @@ -24,14 +24,14 @@ WORKDIR /tmp/ovn/ovs RUN ./boot.sh RUN ./configure -v -RUN make && make rpm-fedora +RUN make rpm-fedora RUN rm rpm/rpmbuild/RPMS/x86_64/*debug* RUN rm rpm/rpmbuild/RPMS/x86_64/*devel* WORKDIR /tmp/ovn RUN ./boot.sh RUN ./configure -RUN make && make rpm-fedora +RUN make rpm-fedora RUN rm rpm/rpmbuild/RPMS/x86_64/*debug* RUN rm rpm/rpmbuild/RPMS/x86_64/*docker*
There is no need to build OVS and OVN separately before building RPMs. This saves 3-5 minutes of the build. Signed-off-by: Ilya Maximets <i.maximets@ovn.org> --- .ci/ovn-kubernetes/Dockerfile | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)