* [PATCH v3 14/21] cirrus: Building freebsd in a single short
@ 2020-09-09 10:08 Yonggang Luo
2020-09-09 13:07 ` Daniel P. Berrangé
2020-09-09 13:12 ` Philippe Mathieu-Daudé
0 siblings, 2 replies; 3+ messages in thread
From: Yonggang Luo @ 2020-09-09 10:08 UTC (permalink / raw)
To: qemu-devel
Cc: Kevin Wolf, Ed Maste, Michael Roth, qemu-block, Stefan Weil,
Xie Changlong, Richard Henderson, Markus Armbruster, Max Reitz,
Yonggang Luo, Gerd Hoffmann, Wen Congyang, Li-Wen Hsu,
Peter Lieven
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. does'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>
---
.cirrus.yml | 35 ++++++++---------------------------
1 file changed, 8 insertions(+), 27 deletions(-)
diff --git a/.cirrus.yml b/.cirrus.yml
index 49335e68c9..b0004273bb 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
--
2.28.0.windows.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v3 14/21] cirrus: Building freebsd in a single short
2020-09-09 10:08 [PATCH v3 14/21] cirrus: Building freebsd in a single short Yonggang Luo
@ 2020-09-09 13:07 ` Daniel P. Berrangé
2020-09-09 13:12 ` Philippe Mathieu-Daudé
1 sibling, 0 replies; 3+ messages in thread
From: Daniel P. Berrangé @ 2020-09-09 13:07 UTC (permalink / raw)
To: Yonggang Luo
Cc: Kevin Wolf, Ed Maste, qemu-block, Stefan Weil, Xie Changlong,
Richard Henderson, qemu-devel, Michael Roth, Gerd Hoffmann,
Wen Congyang, Max Reitz, Li-Wen Hsu, Markus Armbruster,
Peter Lieven
On Wed, Sep 09, 2020 at 06:08:42PM +0800, Yonggang Luo wrote:
> 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. does'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>
> ---
> .cirrus.yml | 35 ++++++++---------------------------
> 1 file changed, 8 insertions(+), 27 deletions(-)
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v3 14/21] cirrus: Building freebsd in a single short
2020-09-09 10:08 [PATCH v3 14/21] cirrus: Building freebsd in a single short Yonggang Luo
2020-09-09 13:07 ` Daniel P. Berrangé
@ 2020-09-09 13:12 ` Philippe Mathieu-Daudé
1 sibling, 0 replies; 3+ messages in thread
From: Philippe Mathieu-Daudé @ 2020-09-09 13:12 UTC (permalink / raw)
To: Yonggang Luo, qemu-devel
Cc: Kevin Wolf, Ed Maste, qemu-block, Stefan Weil, Xie Changlong,
Richard Henderson, Michael Roth, Markus Armbruster, Gerd Hoffmann,
Wen Congyang, Max Reitz, Li-Wen Hsu, Peter Lieven
On 9/9/20 12:08 PM, Yonggang Luo wrote:
> 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. does'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
>
Can you add:
This reverts commit 45f7b7b9f38f5c4d1529a37c93dedfc26a231bba
("cirrus.yml: Split FreeBSD job into two parts").
(no need to repost right know, let other review your patches.)
> Signed-off-by: Yonggang Luo <luoyonggang@gmail.com>
> ---
> .cirrus.yml | 35 ++++++++---------------------------
> 1 file changed, 8 insertions(+), 27 deletions(-)
>
> diff --git a/.cirrus.yml b/.cirrus.yml
> index 49335e68c9..b0004273bb 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
>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-09-09 13:13 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-09-09 10:08 [PATCH v3 14/21] cirrus: Building freebsd in a single short Yonggang Luo
2020-09-09 13:07 ` Daniel P. Berrangé
2020-09-09 13:12 ` Philippe Mathieu-Daudé
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).