diff mbox series

[v10,15/26] cirrus: Building freebsd in a single short

Message ID 20200915171234.236-16-luoyonggang@gmail.com
State Superseded
Headers show
Series W32, W64 msys2/mingw patches | expand

Commit Message

罗勇刚(Yonggang Luo) Sept. 15, 2020, 5:12 p.m. UTC
This reverts commit 45f7b7b9f38f5c4d1529a37c93dedfc26a231bba
("cirrus.yml: Split FreeBSD job into two parts").

freebsd 1 hour limit not hit anymore

I think we going to a wrong direction, I think there is some tests a stall the test runner,
please look at
https://cirrus-ci.com/task/5110577531977728
When its running properly, the consumed time are little, but when tests running too long,
look at the cpu usage, the cpu usage are nearly zero. doesn't consuming time.

And look at
https://cirrus-ci.com/task/6119341601062912

If the tests running properly, the time consuming are little
We should not hide the error by split them

Signed-off-by: Yonggang Luo <luoyonggang@gmail.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Reviewed-by: Ed Maste <emaste@FreeBSD.org>
---
 .cirrus.yml | 35 ++++++++---------------------------
 1 file changed, 8 insertions(+), 27 deletions(-)

Comments

Philippe Mathieu-Daudé Sept. 15, 2020, 5:48 p.m. UTC | #1
Typo "single shot" in subject?

On 9/15/20 7:12 PM, Yonggang Luo wrote:
> This reverts commit 45f7b7b9f38f5c4d1529a37c93dedfc26a231bba

> ("cirrus.yml: Split FreeBSD job into two parts").

> 

> freebsd 1 hour limit not hit anymore

> 

> I think we going to a wrong direction, I think there is some tests a stall the test runner,

> please look at

> https://cirrus-ci.com/task/5110577531977728

> When its running properly, the consumed time are little, but when tests running too long,

> look at the cpu usage, the cpu usage are nearly zero. doesn't consuming time.

> 

> And look at

> https://cirrus-ci.com/task/6119341601062912

> 

> If the tests running properly, the time consuming are little

> We should not hide the error by split them

> 

> Signed-off-by: Yonggang Luo <luoyonggang@gmail.com>

> Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>

> Reviewed-by: Ed Maste <emaste@FreeBSD.org>

> ---

>  .cirrus.yml | 35 ++++++++---------------------------

>  1 file changed, 8 insertions(+), 27 deletions(-)

> 

> diff --git a/.cirrus.yml b/.cirrus.yml

> index 3dd9fcff7f..25fb4add9b 100644

> --- a/.cirrus.yml

> +++ b/.cirrus.yml

> @@ -1,38 +1,19 @@

>  env:

>    CIRRUS_CLONE_DEPTH: 1

>  

> -freebsd_1st_task:

> +freebsd_12_task:

>    freebsd_instance:

>      image_family: freebsd-12-1

> -    cpu: 4

> -    memory: 4G

> -  install_script: ASSUME_ALWAYS_YES=yes pkg bootstrap -f ; pkg install -y

> -    bash curl cyrus-sasl git glib gmake gnutls gsed

> -    nettle perl5 pixman pkgconf png usbredir

> +    cpu: 8

> +    memory: 8G

> +  install_script:

> +    - ASSUME_ALWAYS_YES=yes pkg bootstrap -f ;

> +    - pkg install -y bash curl cyrus-sasl git glib gmake gnutls gsed

> +          nettle perl5 pixman pkgconf png usbredir

>    script:

>      - mkdir build

>      - cd build

> -    - ../configure --disable-user --target-list-exclude='alpha-softmmu

> -        ppc64-softmmu ppc-softmmu riscv32-softmmu riscv64-softmmu s390x-softmmu

> -        sparc64-softmmu sparc-softmmu x86_64-softmmu i386-softmmu'

> -        --enable-werror || { cat config.log; exit 1; }

> -    - gmake -j$(sysctl -n hw.ncpu)

> -    - gmake -j$(sysctl -n hw.ncpu) check

> -

> -freebsd_2nd_task:

> -  freebsd_instance:

> -    image_family: freebsd-12-1

> -    cpu: 4

> -    memory: 4G

> -  install_script: ASSUME_ALWAYS_YES=yes pkg bootstrap -f ; pkg install -y

> -    bash curl cyrus-sasl git glib gmake gnutls gtk3 gsed libepoxy mesa-libs

> -    nettle perl5 pixman pkgconf png SDL2 usbredir

> -  script:

> -    - ./configure --enable-werror --target-list='alpha-softmmu ppc64-softmmu

> -        ppc-softmmu riscv32-softmmu riscv64-softmmu s390x-softmmu

> -        sparc64-softmmu sparc-softmmu x86_64-softmmu i386-softmmu

> -        sparc-bsd-user sparc64-bsd-user x86_64-bsd-user i386-bsd-user'

> -        || { cat config.log; exit 1; }

> +    - ../configure --enable-werror || { cat config.log; exit 1; }

>      - gmake -j$(sysctl -n hw.ncpu)

>      - gmake -j$(sysctl -n hw.ncpu) check

>  

>
罗勇刚(Yonggang Luo) Sept. 15, 2020, 5:49 p.m. UTC | #2
On Wed, Sep 16, 2020 at 1:48 AM Philippe Mathieu-Daudé <f4bug@amsat.org>
wrote:
>

> Typo "single shot" in subject?


Yeap, single shot, hope the maintainer fix it
>

>

> On 9/15/20 7:12 PM, Yonggang Luo wrote:

> > This reverts commit 45f7b7b9f38f5c4d1529a37c93dedfc26a231bba

> > ("cirrus.yml: Split FreeBSD job into two parts").

> >

> > freebsd 1 hour limit not hit anymore

> >

> > I think we going to a wrong direction, I think there is some tests a

stall the test runner,
> > please look at

> > https://cirrus-ci.com/task/5110577531977728

> > When its running properly, the consumed time are little, but when tests

running too long,
> > look at the cpu usage, the cpu usage are nearly zero. doesn't consuming

time.
> >

> > And look at

> > https://cirrus-ci.com/task/6119341601062912

> >

> > If the tests running properly, the time consuming are little

> > We should not hide the error by split them

> >

> > Signed-off-by: Yonggang Luo <luoyonggang@gmail.com>

> > Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>

> > Reviewed-by: Ed Maste <emaste@FreeBSD.org>

> > ---

> >  .cirrus.yml | 35 ++++++++---------------------------

> >  1 file changed, 8 insertions(+), 27 deletions(-)

> >

> > diff --git a/.cirrus.yml b/.cirrus.yml

> > index 3dd9fcff7f..25fb4add9b 100644

> > --- a/.cirrus.yml

> > +++ b/.cirrus.yml

> > @@ -1,38 +1,19 @@

> >  env:

> >    CIRRUS_CLONE_DEPTH: 1

> >

> > -freebsd_1st_task:

> > +freebsd_12_task:

> >    freebsd_instance:

> >      image_family: freebsd-12-1

> > -    cpu: 4

> > -    memory: 4G

> > -  install_script: ASSUME_ALWAYS_YES=yes pkg bootstrap -f ; pkg install

-y
> > -    bash curl cyrus-sasl git glib gmake gnutls gsed

> > -    nettle perl5 pixman pkgconf png usbredir

> > +    cpu: 8

> > +    memory: 8G

> > +  install_script:

> > +    - ASSUME_ALWAYS_YES=yes pkg bootstrap -f ;

> > +    - pkg install -y bash curl cyrus-sasl git glib gmake gnutls gsed

> > +          nettle perl5 pixman pkgconf png usbredir

> >    script:

> >      - mkdir build

> >      - cd build

> > -    - ../configure --disable-user --target-list-exclude='alpha-softmmu

> > -        ppc64-softmmu ppc-softmmu riscv32-softmmu riscv64-softmmu

s390x-softmmu
> > -        sparc64-softmmu sparc-softmmu x86_64-softmmu i386-softmmu'

> > -        --enable-werror || { cat config.log; exit 1; }

> > -    - gmake -j$(sysctl -n hw.ncpu)

> > -    - gmake -j$(sysctl -n hw.ncpu) check

> > -

> > -freebsd_2nd_task:

> > -  freebsd_instance:

> > -    image_family: freebsd-12-1

> > -    cpu: 4

> > -    memory: 4G

> > -  install_script: ASSUME_ALWAYS_YES=yes pkg bootstrap -f ; pkg install

-y
> > -    bash curl cyrus-sasl git glib gmake gnutls gtk3 gsed libepoxy

mesa-libs
> > -    nettle perl5 pixman pkgconf png SDL2 usbredir

> > -  script:

> > -    - ./configure --enable-werror --target-list='alpha-softmmu

ppc64-softmmu
> > -        ppc-softmmu riscv32-softmmu riscv64-softmmu s390x-softmmu

> > -        sparc64-softmmu sparc-softmmu x86_64-softmmu i386-softmmu

> > -        sparc-bsd-user sparc64-bsd-user x86_64-bsd-user i386-bsd-user'

> > -        || { cat config.log; exit 1; }

> > +    - ../configure --enable-werror || { cat config.log; exit 1; }

> >      - gmake -j$(sysctl -n hw.ncpu)

> >      - gmake -j$(sysctl -n hw.ncpu) check

> >

> >

>



--
         此致
礼
罗勇刚
Yours
    sincerely,
Yonggang Luo
<div dir="ltr"><br><br>On Wed, Sep 16, 2020 at 1:48 AM Philippe Mathieu-Daudé &lt;<a href="mailto:f4bug@amsat.org">f4bug@amsat.org</a>&gt; wrote:<br>&gt;<br>&gt; Typo &quot;single shot&quot; in subject?<br><br>Yeap, single shot, hope the maintainer fix it <br>&gt;<br>&gt;<br>&gt; On 9/15/20 7:12 PM, Yonggang Luo wrote:<br>&gt; &gt; This reverts commit 45f7b7b9f38f5c4d1529a37c93dedfc26a231bba<br>&gt; &gt; (&quot;cirrus.yml: Split FreeBSD job into two parts&quot;).<br>&gt; &gt;<br>&gt; &gt; freebsd 1 hour limit not hit anymore<br>&gt; &gt;<br>&gt; &gt; I think we going to a wrong direction, I think there is some tests a stall the test runner,<br>&gt; &gt; please look at<br>&gt; &gt; <a href="https://cirrus-ci.com/task/5110577531977728">https://cirrus-ci.com/task/5110577531977728</a><br>&gt; &gt; When its running properly, the consumed time are little, but when tests running too long,<br>&gt; &gt; look at the cpu usage, the cpu usage are nearly zero. doesn&#39;t consuming time.<br>&gt; &gt;<br>&gt; &gt; And look at<br>&gt; &gt; <a href="https://cirrus-ci.com/task/6119341601062912">https://cirrus-ci.com/task/6119341601062912</a><br>&gt; &gt;<br>&gt; &gt; If the tests running properly, the time consuming are little<br>&gt; &gt; We should not hide the error by split them<br>&gt; &gt;<br>&gt; &gt; Signed-off-by: Yonggang Luo &lt;<a href="mailto:luoyonggang@gmail.com">luoyonggang@gmail.com</a>&gt;<br>&gt; &gt; Reviewed-by: Daniel P. Berrangé &lt;<a href="mailto:berrange@redhat.com">berrange@redhat.com</a>&gt;<br>&gt; &gt; Reviewed-by: Ed Maste &lt;emaste@FreeBSD.org&gt;<br>&gt; &gt; ---<br>&gt; &gt;  .cirrus.yml | 35 ++++++++---------------------------<br>&gt; &gt;  1 file changed, 8 insertions(+), 27 deletions(-)<br>&gt; &gt;<br>&gt; &gt; diff --git a/.cirrus.yml b/.cirrus.yml<br>&gt; &gt; index 3dd9fcff7f..25fb4add9b 100644<br>&gt; &gt; --- a/.cirrus.yml<br>&gt; &gt; +++ b/.cirrus.yml<br>&gt; &gt; @@ -1,38 +1,19 @@<br>&gt; &gt;  env:<br>&gt; &gt;    CIRRUS_CLONE_DEPTH: 1<br>&gt; &gt; <br>&gt; &gt; -freebsd_1st_task:<br>&gt; &gt; +freebsd_12_task:<br>&gt; &gt;    freebsd_instance:<br>&gt; &gt;      image_family: freebsd-12-1<br>&gt; &gt; -    cpu: 4<br>&gt; &gt; -    memory: 4G<br>&gt; &gt; -  install_script: ASSUME_ALWAYS_YES=yes pkg bootstrap -f ; pkg install -y<br>&gt; &gt; -    bash curl cyrus-sasl git glib gmake gnutls gsed<br>&gt; &gt; -    nettle perl5 pixman pkgconf png usbredir<br>&gt; &gt; +    cpu: 8<br>&gt; &gt; +    memory: 8G<br>&gt; &gt; +  install_script:<br>&gt; &gt; +    - ASSUME_ALWAYS_YES=yes pkg bootstrap -f ;<br>&gt; &gt; +    - pkg install -y bash curl cyrus-sasl git glib gmake gnutls gsed<br>&gt; &gt; +          nettle perl5 pixman pkgconf png usbredir<br>&gt; &gt;    script:<br>&gt; &gt;      - mkdir build<br>&gt; &gt;      - cd build<br>&gt; &gt; -    - ../configure --disable-user --target-list-exclude=&#39;alpha-softmmu<br>&gt; &gt; -        ppc64-softmmu ppc-softmmu riscv32-softmmu riscv64-softmmu s390x-softmmu<br>&gt; &gt; -        sparc64-softmmu sparc-softmmu x86_64-softmmu i386-softmmu&#39;<br>&gt; &gt; -        --enable-werror || { cat config.log; exit 1; }<br>&gt; &gt; -    - gmake -j$(sysctl -n hw.ncpu)<br>&gt; &gt; -    - gmake -j$(sysctl -n hw.ncpu) check<br>&gt; &gt; -<br>&gt; &gt; -freebsd_2nd_task:<br>&gt; &gt; -  freebsd_instance:<br>&gt; &gt; -    image_family: freebsd-12-1<br>&gt; &gt; -    cpu: 4<br>&gt; &gt; -    memory: 4G<br>&gt; &gt; -  install_script: ASSUME_ALWAYS_YES=yes pkg bootstrap -f ; pkg install -y<br>&gt; &gt; -    bash curl cyrus-sasl git glib gmake gnutls gtk3 gsed libepoxy mesa-libs<br>&gt; &gt; -    nettle perl5 pixman pkgconf png SDL2 usbredir<br>&gt; &gt; -  script:<br>&gt; &gt; -    - ./configure --enable-werror --target-list=&#39;alpha-softmmu ppc64-softmmu<br>&gt; &gt; -        ppc-softmmu riscv32-softmmu riscv64-softmmu s390x-softmmu<br>&gt; &gt; -        sparc64-softmmu sparc-softmmu x86_64-softmmu i386-softmmu<br>&gt; &gt; -        sparc-bsd-user sparc64-bsd-user x86_64-bsd-user i386-bsd-user&#39;<br>&gt; &gt; -        || { cat config.log; exit 1; }<br>&gt; &gt; +    - ../configure --enable-werror || { cat config.log; exit 1; }<br>&gt; &gt;      - gmake -j$(sysctl -n hw.ncpu)<br>&gt; &gt;      - gmake -j$(sysctl -n hw.ncpu) check<br>&gt; &gt; <br>&gt; &gt;<br>&gt;<br><br><br>--<br>         此致<br>礼<br>罗勇刚<br>Yours<br>    sincerely,<br>Yonggang Luo</div>
diff mbox series

Patch

diff --git a/.cirrus.yml b/.cirrus.yml
index 3dd9fcff7f..25fb4add9b 100644
--- a/.cirrus.yml
+++ b/.cirrus.yml
@@ -1,38 +1,19 @@ 
 env:
   CIRRUS_CLONE_DEPTH: 1
 
-freebsd_1st_task:
+freebsd_12_task:
   freebsd_instance:
     image_family: freebsd-12-1
-    cpu: 4
-    memory: 4G
-  install_script: ASSUME_ALWAYS_YES=yes pkg bootstrap -f ; pkg install -y
-    bash curl cyrus-sasl git glib gmake gnutls gsed
-    nettle perl5 pixman pkgconf png usbredir
+    cpu: 8
+    memory: 8G
+  install_script:
+    - ASSUME_ALWAYS_YES=yes pkg bootstrap -f ;
+    - pkg install -y bash curl cyrus-sasl git glib gmake gnutls gsed
+          nettle perl5 pixman pkgconf png usbredir
   script:
     - mkdir build
     - cd build
-    - ../configure --disable-user --target-list-exclude='alpha-softmmu
-        ppc64-softmmu ppc-softmmu riscv32-softmmu riscv64-softmmu s390x-softmmu
-        sparc64-softmmu sparc-softmmu x86_64-softmmu i386-softmmu'
-        --enable-werror || { cat config.log; exit 1; }
-    - gmake -j$(sysctl -n hw.ncpu)
-    - gmake -j$(sysctl -n hw.ncpu) check
-
-freebsd_2nd_task:
-  freebsd_instance:
-    image_family: freebsd-12-1
-    cpu: 4
-    memory: 4G
-  install_script: ASSUME_ALWAYS_YES=yes pkg bootstrap -f ; pkg install -y
-    bash curl cyrus-sasl git glib gmake gnutls gtk3 gsed libepoxy mesa-libs
-    nettle perl5 pixman pkgconf png SDL2 usbredir
-  script:
-    - ./configure --enable-werror --target-list='alpha-softmmu ppc64-softmmu
-        ppc-softmmu riscv32-softmmu riscv64-softmmu s390x-softmmu
-        sparc64-softmmu sparc-softmmu x86_64-softmmu i386-softmmu
-        sparc-bsd-user sparc64-bsd-user x86_64-bsd-user i386-bsd-user'
-        || { cat config.log; exit 1; }
+    - ../configure --enable-werror || { cat config.log; exit 1; }
     - gmake -j$(sysctl -n hw.ncpu)
     - gmake -j$(sysctl -n hw.ncpu) check