[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 12:43:29 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.6.0 |
On 14/09/2020 10.50, Philippe Mathieu-Daudé wrote:
> On 9/14/20 9:27 AM, Thomas Huth wrote:
>> 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.
>
> Not sure if related (probably not as failure and not timeout),
> I hit this during the night:
>
> TEST iotest-qcow2: 030 [fail]
No, that's a different (known) issue. See my patch on the list:
[PATCH v2] iotests: Skip test_stream_parallel in test 030 when doing
"make check"
Thomas
- Re: [PATCH v8 13/27] tests: Enable crypto tests under msys2/mingw, (continued)
- Re: [PATCH v8 13/27] tests: Enable crypto tests under msys2/mingw, Yonggang Luo, 2020/09/14
- Re: [PATCH v8 13/27] tests: Enable crypto tests under msys2/mingw, Thomas Huth, 2020/09/14
- Re: [PATCH v8 13/27] tests: Enable crypto tests under msys2/mingw, Yonggang Luo, 2020/09/14
- Re: [PATCH v8 13/27] tests: Enable crypto tests under msys2/mingw, Daniel P . Berrangé, 2020/09/14
- Re: [PATCH v8 13/27] tests: Enable crypto tests under msys2/mingw, Yonggang Luo, 2020/09/14
[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
[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 23/27] rcu: fixes test-logging.c by call drain_call_rcu before rmdir_full, Yonggang Luo, 2020/09/12
[PATCH v8 22/27] tests: Fixes test-qdev-global-props.c, Yonggang Luo, 2020/09/12
[PATCH v8 24/27] ci: Enable msys2 ci in cirrus, Yonggang Luo, 2020/09/12