From: Baurzhan Ismagulov <ibr@radix50.net>
To: isar-users@googlegroups.com
Subject: Re: [PATCH 0/3] use local repo for multistrap and drop "dpkg -i"
Date: Fri, 11 Aug 2017 17:36:39 +0200 [thread overview]
Message-ID: <20170811153638.GA3809@yssyq.radix50.net> (raw)
In-Reply-To: <20170811101346.3f5e67d5@md1em3qc>
On Fri, Aug 11, 2017 at 10:13:46AM +0200, Henning Schild wrote:
> > Regarding the increase of build time, I'd really like to avoid that.
>
> Me too, that is why i started the discussion on caching. Giving up on
> the gatekeeper idea would IMHO not be the way to go.
>
> > Building all configurations in one run already takes up to 40 min.
>
> CI is patient and for basic testing one does not need to build all of
> them.
I'm not patient ;) , since I test every commit during every rebase... We should
automate that, too. But even then, build time will still be developer time.
Performance is the last but not the least requirement.
> > The problem should be easily mitigatable by splitting rootfs
> > multistrap and apt-getting the built packages into separate recipes.
>
> Yes but that is what i mean with "giving up the gatekeeper". We could
> implement both ways controlled by a variable in a config. That would
> speed up the builds during development and allow switching to strict
> gatekeeper mode for releases.
What is the disadvantage of splitting into two recipes? An image would still
require apt-get install, so this step would not be omitted. Implemented in this
way, the second recipe would be your gatekeeper, no?
Adding one boolean variable doubles the amount of testing required. It's
configuration permutations that kill. That is why I'd prefer settling on one
way.
With kind regards,
Baurzhan.
prev parent reply other threads:[~2017-08-11 15:36 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-08-01 15:24 Henning Schild
2017-08-01 15:24 ` [PATCH 1/3] classes: images: remove cyclic "inherit" Henning Schild
2017-08-01 15:24 ` [PATCH 2/3] classes: move rootfs into image class and split into tasks Henning Schild
2017-08-01 15:24 ` [PATCH 3/3] classes: image: remove populate and replace it with a custom repo Henning Schild
2017-08-09 6:21 ` Henning Schild
2017-08-02 6:16 ` [PATCH 0/3] use local repo for multistrap and drop "dpkg -i" Jan Kiszka
2017-08-02 7:04 ` Claudius Heine
2017-08-02 8:15 ` Henning Schild
2017-08-02 8:11 ` Henning Schild
2017-08-02 8:12 ` Henning Schild
2017-08-03 19:17 ` Henning Schild
2017-08-10 12:46 ` Baurzhan Ismagulov
2017-08-11 8:13 ` Henning Schild
2017-08-11 15:36 ` Baurzhan Ismagulov [this message]
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=20170811153638.GA3809@yssyq.radix50.net \
--to=ibr@radix50.net \
--cc=isar-users@googlegroups.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox