[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 36/39] .gitlab-ci.d/windows.yml: Increase the timeout to 9
From: |
Bin Meng |
Subject: |
Re: [PATCH v2 36/39] .gitlab-ci.d/windows.yml: Increase the timeout to 90 minutes |
Date: |
Sat, 24 Sep 2022 09:13:12 +0800 |
On Sat, Sep 24, 2022 at 12:24 AM Alex Bennée <alex.bennee@linaro.org> wrote:
>
>
> Bin Meng <bmeng.cn@gmail.com> writes:
>
> > From: Bin Meng <bin.meng@windriver.com>
> >
> > commit 9f8e6cad65a6 ("gitlab-ci: Speed up the msys2-64bit job by using
> > --without-default-devices"
> > changed to compile QEMU with the --without-default-devices switch for
> > the msys2-64bit job, due to the build could not complete within the
> > project timeout (1h), and also mentioned that a bigger timeout was
> > getting ignored on the shared Gitlab-CI Windows runners.
> >
> > However as of today it seems the shared Gitlab-CI Windows runners does
> > honor the job timeout, and the runner has the timeout limit of 2h, so
> > let's increase the timeout to 90 minutes and drop the configure switch
> > "--without-default-devices" to get a larger build coverage.
>
> I'd like to push back lightly against increasing the time because it
> lengthens the total run time before we can merge a branch. Ideally we
> could come up with a combination of build and tests that exercises all
> the Windows code without exhaustively testing code paths that are tested
> elsewhere. For device emulation are there any host specific things we
> are testing?
>
Yes, the upcoming virtio-9p Windows support will update the qtests to
test the Windows specific 9p implementation.
Besides that, during the development of this patch series, several
QEMU code bugs were exposed only when we run qtests on Windows, or
even Windows 32-bit. I still see benefits of running qtests on
Windows.
The thing is that we need to find a more powerful machine to do the
CI. Current GitLab shared Windows runners are just too slow.
Regards,
Bin
- Re: [PATCH v2 22/39] tests/qtest: migration-test: Disable IO redirection for win32, (continued)
- [PATCH v2 29/39] tests/qtest: microbit-test: Fix socket access for win32, Bin Meng, 2022/09/20
- [PATCH v2 25/39] chardev/char-file: Add FILE_SHARE_WRITE when openning the file for win32, Bin Meng, 2022/09/20
- [PATCH v2 31/39] tests/qtest: libqtest: Correct the timeout unit of blocking receive calls for win32, Bin Meng, 2022/09/20
- [PATCH v2 33/39] io/channel-watch: Drop the unnecessary cast, Bin Meng, 2022/09/20
- [PATCH v2 36/39] .gitlab-ci.d/windows.yml: Increase the timeout to 90 minutes, Bin Meng, 2022/09/20
- [PATCH v2 24/39] tests/qtest: virtio-net-failover: Disable migration tests for win32, Bin Meng, 2022/09/20
- [PATCH v2 30/39] tests/qtest: libqtest: Replace the call to close a socket with closesocket(), Bin Meng, 2022/09/20
- [PATCH v2 32/39] io/channel-watch: Drop a superfluous '#ifdef WIN32', Bin Meng, 2022/09/20
- [PATCH v2 34/39] io/channel-watch: Fix socket watch on Windows, Bin Meng, 2022/09/20
- [PATCH v2 37/39] .gitlab-ci.d/windows.yml: Display meson test logs, Bin Meng, 2022/09/20
- [PATCH v2 35/39] tests/qtest: migration-test: Skip running some TLS cases for win32, Bin Meng, 2022/09/20