workflows.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Michel Dänzer" <michel.daenzer@mailbox.org>
To: Linus Torvalds <torvalds@linuxfoundation.org>,
	Nikolai Kondrashov <spbnick@gmail.com>
Cc: Maxime Ripard <mripard@kernel.org>,
	Helen Koike <helen.koike@collabora.com>,
	linuxtv-ci@linuxtv.org, dave.pigott@collabora.com,
	linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,
	linux-kselftest@vger.kernel.org, gustavo.padovan@collabora.com,
	pawiecz@collabora.com, tales.aparecida@gmail.com,
	workflows@vger.kernel.org, kernelci@lists.linux.dev,
	skhan@linuxfoundation.org, kunit-dev@googlegroups.com,
	nfraprado@collabora.com, davidgow@google.com, cocci@inria.fr,
	Julia.Lawall@inria.fr, laura.nao@collabora.com,
	ricardo.canuelo@collabora.com, kernel@collabora.com,
	gregkh@linuxfoundation.org
Subject: Re: [PATCH 1/3] kci-gitlab: Introducing GitLab-CI Pipeline for Kernel Testing
Date: Tue, 5 Mar 2024 12:54:01 +0100	[thread overview]
Message-ID: <2c828aa1-d9bb-4b27-844b-a098e206318f@mailbox.org> (raw)
In-Reply-To: <CAHk-=wixVy3WYvjbt43ZSrCqPDsS76QJQSkXFbbPsAOs1MCSAQ@mail.gmail.com>

On 2024-02-29 21:21, Linus Torvalds wrote:
> On Thu, 29 Feb 2024 at 01:23, Nikolai Kondrashov <spbnick@gmail.com> wrote:
>>
>> However, I think a better approach would be *not* to add the .gitlab-ci.yaml
>> file in the root of the source tree, but instead change the very same repo
>> setting to point to a particular entry YAML, *inside* the repo (somewhere
>> under "ci" directory) instead.
> 
> I really don't want some kind of top-level CI for the base kernel project.
> 
> We already have the situation that the drm people have their own ci
> model. II'm ok with that, partly because then at least the maintainers
> of that subsystem can agree on the rules for that one subsystem.
> 
> I'm not at all interested in having something that people will then
> either fight about, or - more likely - ignore, at the top level
> because there isn't some global agreement about what the rules are.
> 
> For example, even just running checkpatch is often a stylistic thing,
> and not everybody agrees about all the checkpatch warnings.
> 
> I would suggest the CI project be separate from the kernel.

That would be missing a lot of the point / benefit of CI.

A CI system which is separate from the kernel will tend to be out of sync, so it can't gate the merging of changes and thus can't prevent regressions from propagating.


-- 
Earthling Michel Dänzer            |                  https://redhat.com
Libre software enthusiast          |         Mesa and Xwayland developer


  parent reply	other threads:[~2024-03-05 11:54 UTC|newest]

Thread overview: 58+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-02-28 22:55 [PATCH 0/3] " Helen Koike
2024-02-28 22:55 ` [PATCH 1/3] " Helen Koike
2024-02-29  2:44   ` Bird, Tim
2024-02-29 16:15     ` Nicolas Dufresne
2024-02-29  9:02   ` Maxime Ripard
2024-02-29  9:23     ` Nikolai Kondrashov
2024-02-29  9:56       ` Maxime Ripard
2024-02-29 10:05         ` Laurent Pinchart
2024-02-29 20:21       ` Linus Torvalds
2024-03-01 10:27         ` Nikolai Kondrashov
2024-03-01 14:07           ` Mark Brown
2024-03-01 14:21             ` Nikolai Kondrashov
2024-03-01 20:10           ` Linus Torvalds
2024-03-04 21:45             ` Helen Koike
2024-03-07 22:43               ` Leonardo Brás
2024-05-23 13:21               ` Daniel Vetter
2024-03-02 22:10         ` Guenter Roeck
2024-03-03  0:01           ` Randy Dunlap
2024-03-03  9:30             ` Geert Uytterhoeven
2024-03-04  8:12               ` Geert Uytterhoeven
2024-03-04  9:15                 ` Maxime Ripard
2024-03-04 10:07                   ` Geert Uytterhoeven
2024-03-04 10:19                     ` Maxime Ripard
2024-03-04 11:12                       ` Geert Uytterhoeven
2024-03-04 11:28                         ` Maxime Ripard
2024-03-04  9:24           ` Maxime Ripard
2024-03-04 15:46             ` Guenter Roeck
2024-03-04 16:05               ` Maxime Ripard
2024-03-04 16:17                 ` Guenter Roeck
2024-03-04 17:09                   ` Maxime Ripard
2024-03-04 17:22                     ` Guenter Roeck
2024-03-04 19:44                     ` Guenter Roeck
2024-03-05 11:54         ` Michel Dänzer [this message]
2024-03-07 18:05     ` Nicolas Dufresne
2024-03-11  8:40       ` Maxime Ripard
2024-02-28 22:55 ` [PATCH 2/3] kci-gitlab: Add documentation Helen Koike
2024-02-28 22:55 ` [PATCH 3/3] kci-gitlab: docs: Add images Helen Koike
2024-02-28 23:07 ` [PATCH 0/3] kci-gitlab: Introducing GitLab-CI Pipeline for Kernel Testing Laurent Pinchart
2024-02-29  8:43   ` Maxime Ripard
2024-02-29  9:26   ` Nikolai Kondrashov
2024-02-29  9:34     ` Laurent Pinchart
2024-02-29 11:10       ` Nikolai Kondrashov
2024-02-29 11:19         ` Laurent Pinchart
2024-02-29 11:22           ` Nikolai Kondrashov
2024-02-29 11:41           ` Mark Brown
2024-02-29 11:53             ` Guillaume Tucker
2024-02-29 12:20               ` Laurent Pinchart
2024-02-29 12:25                 ` Laurent Pinchart
2024-02-29 14:12                   ` Nikolai Kondrashov
2024-02-29  9:39   ` Sakari Ailus
2024-02-29 11:09     ` Mark Brown
2024-02-29 12:20 ` Guillaume Tucker
2024-02-29 14:16   ` Nikolai Kondrashov
2024-02-29 16:28     ` Nicolas Dufresne
2024-03-01 21:56       ` Guillaume Tucker
2024-03-02 21:48         ` Gustavo Padovan
2024-03-04  8:33           ` Guillaume Tucker
2024-05-21  9:28 ` Jarkko Sakkinen

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=2c828aa1-d9bb-4b27-844b-a098e206318f@mailbox.org \
    --to=michel.daenzer@mailbox.org \
    --cc=Julia.Lawall@inria.fr \
    --cc=cocci@inria.fr \
    --cc=dave.pigott@collabora.com \
    --cc=davidgow@google.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=gustavo.padovan@collabora.com \
    --cc=helen.koike@collabora.com \
    --cc=kernel@collabora.com \
    --cc=kernelci@lists.linux.dev \
    --cc=kunit-dev@googlegroups.com \
    --cc=laura.nao@collabora.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-kselftest@vger.kernel.org \
    --cc=linuxtv-ci@linuxtv.org \
    --cc=mripard@kernel.org \
    --cc=nfraprado@collabora.com \
    --cc=pawiecz@collabora.com \
    --cc=ricardo.canuelo@collabora.com \
    --cc=skhan@linuxfoundation.org \
    --cc=spbnick@gmail.com \
    --cc=tales.aparecida@gmail.com \
    --cc=torvalds@linuxfoundation.org \
    --cc=workflows@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox