qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] cirrus.yml: Exclude some targets in the FreeBSD job to speed


From: Thomas Huth
Subject: Re: [PATCH] cirrus.yml: Exclude some targets in the FreeBSD job to speed up the build
Date: Mon, 31 Aug 2020 08:13:00 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.6.0

On 28/08/2020 11.54, Thomas Huth wrote:
> On 28/08/2020 10.59, Daniel P. Berrangé wrote:
>> On Mon, Aug 24, 2020 at 11:44:10AM +0200, Thomas Huth wrote:
>>> The FreeBSD jobs currently hit the 1h time limit in the Cirrus-CI.
>>> We've got to exclude some build targets here to make sure that the job
>>> finishes in time again. The targets that are excluded should not hurt
>>> much, since e.g. all the code from i386-softmmu is covered again by
>>> x86_64-softmmu.
>>>
>>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>>> ---
>>>  Not sure why the FreeBSD builds suddenly got slower and hit the 1h
>>>  time limit now. Looking at https://cirrus-ci.com/github/qemu/qemu
>>>  it seems like the FreeBSD jobs were already taking ca. 48 minutes
>>>  since quite a while, but since the Meson build system has been merged,
>>>  they now always hit the 1h limit. Could it be that Meson is slow on
>>>  FreeBSD?
>>>  
>>>  I hope disabling some of the "redundant" targets is ok for everybody
>>>  to get this working again. Alternatively, we could also introduce a
>>>  second FreeBSD job and run half of the targets in one job, and half
>>>  of the targets in the other job if that's preferred?
>>
>> I'd probably suggest we go for two jobs, as there doesn't look like
>> any constraint on running multiple jobs that we'll hit any time
>> soon.
> 
> I gave it a try, but either I'm doing something wrong, or it is not
> possible to run multiple FreeBSD tasks on cirrus-ci for one commit 

I apparently needed a weekend in between to get a clean head again. I
was doing indeed something wrong... apparently you have to name your
tasks on cirrus-ci with "_task" at the end. If you use "_task1" and
"_task2", it does now work anymore. Sigh.

So I'll try to cook a proper patch to split the job instead.

 Thomas




reply via email to

[Prev in Thread] Current Thread [Next in Thread]