From: Anthony PERARD <anthony.perard@citrix.com>
To: Stefano Stabellini <sstabellini@kernel.org>
Cc: <xen-devel@lists.xenproject.org>, <iwj@xenproject.org>,
<cardoe@cardoe.com>, <wl@xen.org>, <andrew.cooper3@citrix.com>,
"Stefano Stabellini" <stefano.stabellini@xilinx.com>
Subject: Re: [PATCH 3/3] automation: add a QEMU based x86_64 Dom0/DomU test
Date: Wed, 27 Oct 2021 14:59:22 +0100 [thread overview]
Message-ID: <YXlbOjiphjN/XqMz@perard> (raw)
In-Reply-To: <alpine.DEB.2.21.2110251754210.4586@sstabellini-ThinkPad-T480s>
On Mon, Oct 25, 2021 at 06:33:53PM -0700, Stefano Stabellini wrote:
> On Mon, 25 Oct 2021, Anthony PERARD wrote:
> > There is something I'm missing, how is it a problem to have a container
> > that is a bit bigger? What sort of problem could we have to deal with?
>
> It takes time to clone the container in the gitlab-ci, the bigger the
> container the more time it takes. It is fetched over the network. Now we
> are fetching qemu (as part of the container) 10 times during the build
> although it is not needed.
I guess the issue would be more like we don't do enough caching with our
gitlab runners. So package installation is just a workaround.
> But we do have a severe problem at the moment with external sources: our
> "git clones" keep failing during the build on x86. That is definitely
> something worth improving (see my other email thread on the subject) and
> it is the main problem affecting gitlab-ci at the moment, I keep having
> to restart jobs almost daily to get the overall pipeline to "pass".
>
> If you have any ideas on how to stop fetching things using "git" from
> external repositories in gitlab-ci that would be fantastic :-)
> The only thing I could think of to "fix it" is moving all external repos
> to gitlab repositories mirrors.
I don't think that would work, I've seen the initial clone/fetch of a
job fail as well, so from gitlab. If we could have a cache of those
external resources closer to the runners, that would be better.
> > > I am not entirely sure what is the best solution overall, but for this
> > > series at this stage I would prefer to keep the same strategy used for
> > > the ARM tests (i.e. reuse the debian unstable build container and
> > > apt-get the few missing packages.) If we do change the way we do it, I
> > > would rather change both x86 and ARM at the same time.
> >
> > I'm pretty sure the best strategy would be to do as little as possible
> > during a job, download as little as possible and possibly cache as much
> > as possible and do as much as possible ahead of time. Feel free to
> > change the Arm test, but I don't think it is necessary to change the Arm
> > test at the same time as introducing an x86 test.
>
> I agree.
>
> At the same time it would be nice to follow the same strategy between
> x86 and ARM going forward: if one optimization is made for one, it is
> also made for the other.
Probably better, yes.
Thanks,
--
Anthony PERARD
next prev parent reply other threads:[~2021-10-27 14:00 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-10-21 23:08 [PATCH 0/3] automation: introduce an x86_64 Dom0/DomU test Stefano Stabellini
2021-10-21 23:08 ` [PATCH 1/3] automation: add x86_64 alpine 3.12 test-artifact Stefano Stabellini
2021-10-22 12:37 ` Anthony PERARD
2021-10-22 12:54 ` Andrew Cooper
2021-10-22 20:01 ` Stefano Stabellini
2021-10-21 23:08 ` [PATCH 2/3] automation: Linux 5.10.74 test-artifact Stefano Stabellini
2021-10-22 12:38 ` Anthony PERARD
2021-10-22 20:02 ` Stefano Stabellini
2021-10-22 13:00 ` Andrew Cooper
2021-10-22 19:41 ` Stefano Stabellini
2021-10-25 5:15 ` Juergen Gross
2021-10-26 0:54 ` Stefano Stabellini
2021-10-27 7:34 ` Juergen Gross
2021-10-27 22:44 ` Stefano Stabellini
2021-10-27 23:24 ` Stefano Stabellini
2021-10-28 7:00 ` Juergen Gross
2021-10-28 16:41 ` Stefano Stabellini
2021-10-29 5:43 ` Juergen Gross
2021-11-02 0:12 ` Stefano Stabellini
2021-10-21 23:08 ` [PATCH 3/3] automation: add a QEMU based x86_64 Dom0/DomU test Stefano Stabellini
2021-10-22 13:03 ` Anthony PERARD
2021-10-22 20:05 ` Stefano Stabellini
2021-10-25 16:13 ` Anthony PERARD
2021-10-26 1:33 ` Stefano Stabellini
2021-10-27 13:59 ` Anthony PERARD [this message]
2021-10-27 21:43 ` Solving the gitlab-ci git fetch issue, was: " Stefano Stabellini
2021-10-28 9:48 ` Anthony PERARD
2021-10-28 14:19 ` Stefano Stabellini
2021-10-22 9:25 ` [PATCH 0/3] automation: introduce an " Ian Jackson
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=YXlbOjiphjN/XqMz@perard \
--to=anthony.perard@citrix.com \
--cc=andrew.cooper3@citrix.com \
--cc=cardoe@cardoe.com \
--cc=iwj@xenproject.org \
--cc=sstabellini@kernel.org \
--cc=stefano.stabellini@xilinx.com \
--cc=wl@xen.org \
--cc=xen-devel@lists.xenproject.org \
/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.