On Mon, 2020-04-06 at 13:28 +0200, Erik Skultety wrote:
On Fri, Apr 03, 2020 at 05:08:59PM +0200, Andrea Bolognani wrote:
> Other than that, storing the token in ~/.config/lcitool is fine I
> guess. The number of files in that directory is getting fairly
> ridiculous, though, so Someoneā¢ should really look into throwing
> away the current, extremely crude configuration system that's been
> inherited from the original shell implementation and replace it
> with something more sensible like a config.toml.
Let me spend some time on that improvement :).
It was not my intention to volunteer you for this effort, but okay,
I'll take it :)
> Anyway, it's simpler than that: all of our Linux jobs
currently
> expect to run in container images, and we can only have that using
> the Docker executor.
With the functional tests in mind, the Docker executor doesn't really solve
anything, for builds, there are already a bunch of containers taking care of
that, now we need to cover cases where container technology cannot be utilized
as well as environments for functional tests - of course, for that, these
patches are only sort of a prequel.
Okay, so the target for the VMs built this way is
* functional tests, for all of them;
* unit tests, for the FreeBSD ones.
Is that correct? Because I was under the impression that, aside of
filling the FreeBSD-sized gap in our GitLab CI coverage, we were
aiming at increasing the capacity of the existing container-based
unit tests beyond what the shared runners provide. If that's not
what we're aiming for, then of course using the Docker executor
makes little sense and would probably only get in the way.
--
Andrea Bolognani / Red Hat / Virtualization