From: "Daniel P. Berrangé" <berrange@redhat.com>
To: "Philippe Mathieu-Daudé" <philippe.mathieu.daude@gmail.com>
Cc: "Peter Maydell" <peter.maydell@linaro.org>,
"Thomas Huth" <thuth@redhat.com>,
qemu-block@nongnu.org,
"Christian Schoenebeck" <qemu_oss@crudebyte.com>,
qemu-devel@nongnu.org, "Cameron Esfahani" <dirty@apple.com>,
"Philippe Mathieu-Daudé" <f4bug@amsat.org>,
"Roman Bolshakov" <r.bolshakov@yadro.com>,
"Will Cohen" <wwcohen@gmail.com>,
"Gerd Hoffmann" <kraxel@redhat.com>,
"Akihiko Odaki" <akihiko.odaki@gmail.com>,
"Alex Bennée" <alex.bennee@linaro.org>
Subject: Re: [PATCH v7 22/22] gitlab-ci: Support macOS 12 via cirrus-run
Date: Wed, 9 Mar 2022 13:02:05 +0000 [thread overview]
Message-ID: <YiilTXYgeORlVl96@redhat.com> (raw)
In-Reply-To: <ae5f9f7d-e0b3-3d79-1fe1-d8dfdb1ba2b7@gmail.com>
On Wed, Mar 09, 2022 at 01:50:34PM +0100, Philippe Mathieu-Daudé wrote:
> On 9/3/22 13:33, Daniel P. Berrangé wrote:
> > On Wed, Mar 09, 2022 at 12:58:42PM +0100, Thomas Huth wrote:
>
> > > Basically fine for me, but can we really run additional cirrus-ci jobs by
> > > default? IIRC the parallel execution of those were quite limited for the
> > > free tier, so did you look close that we don't run into additional timeouts
> > > yet, due to delayed cirrus-ci jobs?
> >
> > You can run 2 jobs in parallel in Cirrus. Beyond that they
> > get queued/serialized
> >
> > We have a 1 hour job timeout.
> >
> > We have to expect jobs will sometimes run slower than normal.
> >
> > IOW if we have a job on Cirrus taking 30 minutes normally, we
> > expect it will sometimes take 45 minutes.
> >
> > All this means that if we want Cirrus to be reliable and not
> > time out, we can really only have 2 jobs by default, unless
> > we can get the job execution time down to around 20 minutes
> > to allow for serialization.
> >
> > We used to have terrible problems with cirrus timeouts when
> > we were running 4 jobs concurrently (2 on staging and 2 on
> > master). We addressed that in 9968de0a4a5470bd7b98dcd2fae5d5269908f16b
> > by disabling the jobs on master.
> >
> > IOW, we really need to choose 1 macOS job and 1 FreeBSD job
> > and others need to be marked manual.
>
> Not sure which job to choose yet. Per the first google hits we
> still want to cover Catalina first:
> https://www.statista.com/statistics/944559/worldwide-macos-version-market-share/
My general gut feeling is usually to prioritize testing older versions
as they tend to be more widely used, and we want to avoid regresions
on stuff that has been around the longest. Compatibility problems with
new releases tend to get reported by users/maintainers and would not
be regressions, but rather enhancements to support the new platform.
But it is not really an easy decision, as clearly we would prefer to
test both old and new if we had the ability.
Having one as a nightly job is a reasonable compromise.
With 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 :|
next prev parent reply other threads:[~2022-03-09 13:04 UTC|newest]
Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-03-06 23:17 [PATCH v7 00/22] host: Support macOS 12 Philippe Mathieu-Daudé
2022-03-06 23:17 ` [PATCH v7 01/22] configure: Allow passing extra Objective C compiler flags Philippe Mathieu-Daudé
2022-03-06 23:17 ` [PATCH v7 02/22] tests/fp/berkeley-testfloat-3: Ignore ignored #pragma directives Philippe Mathieu-Daudé
2022-03-09 11:17 ` Alex Bennée
2022-03-06 23:17 ` [PATCH v7 03/22] hvf: Use standard CR0 and CR4 register definitions Philippe Mathieu-Daudé
2022-03-06 23:17 ` [PATCH v7 04/22] hvf: Make hvf_get_segments() / hvf_put_segments() local Philippe Mathieu-Daudé
2022-03-06 23:17 ` [PATCH v7 05/22] hvf: Remove deprecated hv_vcpu_flush() calls Philippe Mathieu-Daudé
2022-03-06 23:17 ` [PATCH v7 06/22] hvf: Fix OOB write in RDTSCP instruction decode Philippe Mathieu-Daudé
2022-03-09 10:20 ` Philippe Mathieu-Daudé
2022-03-15 12:58 ` Peter Maydell
2022-03-15 13:00 ` Philippe Mathieu-Daudé
2022-03-06 23:17 ` [PATCH v7 07/22] block/file-posix: Remove a deprecation warning on macOS 12 Philippe Mathieu-Daudé
2022-03-06 23:17 ` [PATCH v7 08/22] audio/coreaudio: " Philippe Mathieu-Daudé
2022-03-06 23:17 ` [PATCH v7 09/22] audio/dbus: Fix building with modules on macOS Philippe Mathieu-Daudé
2022-03-06 23:17 ` [PATCH v7 10/22] audio: Log context for audio bug Philippe Mathieu-Daudé
2022-03-06 23:17 ` [PATCH v7 11/22] coreaudio: Always return 0 in handle_voice_change Philippe Mathieu-Daudé
2022-03-06 23:17 ` [PATCH v7 12/22] audio: Rename coreaudio extension to use Objective-C compiler Philippe Mathieu-Daudé
2022-03-06 23:17 ` [PATCH v7 13/22] osdep: Avoid using Clang-specific __builtin_available() Philippe Mathieu-Daudé
2022-03-07 4:11 ` Akihiko Odaki
2022-03-06 23:17 ` [PATCH v7 14/22] meson: Resolve the entitlement.sh script once for good Philippe Mathieu-Daudé
2022-03-07 4:13 ` Akihiko Odaki
2022-03-06 23:17 ` [PATCH v7 15/22] meson: Log QEMU_CXXFLAGS content in summary Philippe Mathieu-Daudé
2022-03-07 4:13 ` Akihiko Odaki
2022-03-06 23:17 ` [PATCH v7 16/22] configure: Pass filtered QEMU_OBJCFLAGS to meson Philippe Mathieu-Daudé
2022-03-07 4:15 ` Akihiko Odaki
2022-03-06 23:17 ` [PATCH v7 17/22] ui/cocoa: Constify qkeycode translation arrays Philippe Mathieu-Daudé
2022-03-07 4:16 ` Akihiko Odaki
2022-03-06 23:17 ` [PATCH v7 18/22] ui/cocoa: add option to disable left-command forwarding to guest Philippe Mathieu-Daudé
2022-03-06 23:17 ` [PATCH v7 19/22] ui/cocoa: release mouse when user switches away from QEMU window Philippe Mathieu-Daudé
2022-03-06 23:17 ` [PATCH v7 20/22] ui/cocoa: capture all keys and combos when mouse is grabbed Philippe Mathieu-Daudé
2022-03-07 17:01 ` Will Cohen
2022-03-06 23:17 ` [PATCH v7 21/22] ui/cocoa: add option to swap Option and Command Philippe Mathieu-Daudé
2022-03-07 17:00 ` Will Cohen
2022-03-06 23:17 ` [PATCH v7 22/22] gitlab-ci: Support macOS 12 via cirrus-run Philippe Mathieu-Daudé
2022-03-09 10:24 ` Philippe Mathieu-Daudé
2022-03-09 11:58 ` Thomas Huth
2022-03-09 12:33 ` Daniel P. Berrangé
2022-03-09 12:50 ` Philippe Mathieu-Daudé
2022-03-09 12:52 ` Thomas Huth
2022-03-09 12:55 ` Daniel P. Berrangé
2022-03-09 13:02 ` Daniel P. Berrangé [this message]
2022-03-09 13:09 ` Peter Maydell
2022-03-09 13:40 ` Philippe Mathieu-Daudé
2022-03-09 10:28 ` Daniel P. Berrangé
2022-03-15 12:48 ` [PATCH v7 00/22] host: Support macOS 12 Philippe Mathieu-Daudé
2022-05-03 9:40 ` Claudio Fontana
2022-05-09 12:31 ` Philippe Mathieu-Daudé via
2022-05-09 13:08 ` Claudio Fontana
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=YiilTXYgeORlVl96@redhat.com \
--to=berrange@redhat.com \
--cc=akihiko.odaki@gmail.com \
--cc=alex.bennee@linaro.org \
--cc=dirty@apple.com \
--cc=f4bug@amsat.org \
--cc=kraxel@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=philippe.mathieu.daude@gmail.com \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=qemu_oss@crudebyte.com \
--cc=r.bolshakov@yadro.com \
--cc=thuth@redhat.com \
--cc=wwcohen@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.