[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v8 16/27] cirrus: Building freebsd in a single short
From: |
Thomas Huth |
Subject: |
Re: [PATCH v8 16/27] cirrus: Building freebsd in a single short |
Date: |
Mon, 14 Sep 2020 09:27:29 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.6.0 |
On 13/09/2020 00.44, 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(-)
I tried this a couple of times now, and currently the problem seems to
be gone, indeed. I'd still prefer to understand first why we have seen
the slowdown a couple of weeks ago, but if people agree that we should
revert the patch now, that's ok for me now, too.
Tested-by: Thomas Huth <thuth@redhat.com>
- [PATCH v8 08/27] tests: Fixes test-replication.c on msys2/mingw., (continued)
- [PATCH v8 08/27] tests: Fixes test-replication.c on msys2/mingw., Yonggang Luo, 2020/09/12
- [PATCH v8 12/27] gcrypt: test_tls_psk_init should write binary file instead text file., Yonggang Luo, 2020/09/12
- [PATCH v8 13/27] tests: Enable crypto tests under msys2/mingw, Yonggang Luo, 2020/09/12
- [PATCH v8 14/27] meson: remove empty else and duplicated gio deps, Yonggang Luo, 2020/09/12
- [PATCH v8 15/27] vmstate: Fixes test-vmstate.c on msys2/mingw, Yonggang Luo, 2020/09/12
- [PATCH v8 16/27] cirrus: Building freebsd in a single short, Yonggang Luo, 2020/09/12
- Re: [PATCH v8 16/27] cirrus: Building freebsd in a single short,
Thomas Huth <=
- [PATCH v8 17/27] tests: Convert g_free to g_autofree macro in test-logging.c, Yonggang Luo, 2020/09/12
- [PATCH v8 18/27] tests: Fixes test-io-channel-socket.c tests under msys2/mingw, Yonggang Luo, 2020/09/12
- [PATCH v8 19/27] tests: fixes aio-win32 about aio_remove_fd_handler, get it consistence with aio-posix.c, Yonggang Luo, 2020/09/12
- [PATCH v8 20/27] tests: Fixes test-io-channel-file by mask only owner file state mask bits, Yonggang Luo, 2020/09/12
- [PATCH v8 21/27] tests: fix test-util-sockets.c, Yonggang Luo, 2020/09/12
- [PATCH v8 22/27] tests: Fixes test-qdev-global-props.c, Yonggang Luo, 2020/09/12
- [PATCH v8 23/27] rcu: fixes test-logging.c by call drain_call_rcu before rmdir_full, Yonggang Luo, 2020/09/12