all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Leo Famulari <leo@famulari.name>
To: Mathieu Othacehe <othacehe@gnu.org>
Cc: 60373@debbugs.gnu.org
Subject: [bug#60373] Kernel job for Cuirass
Date: Wed, 28 Dec 2022 17:50:13 -0500	[thread overview]
Message-ID: <Y6zIJb4Pk2mbDQ/6@jasmine.lan> (raw)
In-Reply-To: <87r0wjsijd.fsf_-_@gnu.org>


[-- Attachment #1.1: Type: text/plain, Size: 1097 bytes --]

On Wed, Dec 28, 2022 at 03:35:18PM +0100, Mathieu Othacehe wrote:
> What you could do instead is something like:
> 
> --8<---------------cut here---------------start------------->8---
> ((specification
>    (name "kernel-updates")
>    (build '(manifests "etc/kernels-manifest.scm"))
>    (period 7200)
>    (priority 2)
>    (systems '#$systems))
>  (specification
>    (name "kernel-updates-images")
>    (build 'images)
>    (period 7200)
>    (priority 2)
>    (systems '#$systems))
>  (specification
>    (name "kernel-updates-tests")
>    (build 'system-tests)
>    (period 7200)
>    (priority 2)
>    (systems '("x86_64-linux"))))
> --8<---------------cut here---------------end--------------->8---

That's perfect! I've attached an updated patch.

> Then we could think about supporting multiple build subsets.

Well, your suggested workaround is great, so it's not necessary to add
the functionality.

What's the procedure for making these changes? I can push to guix.git
and maintenance.git, but I'll need assistance putting the changes into
practice.

[-- Attachment #1.2: v2-0001-hydra-cuirass-Add-jobsets-for-testing-kernel-upda.patch --]
[-- Type: text/plain, Size: 2020 bytes --]

From 299039cf429210392b979a57e898c4a00ca4a6c3 Mon Sep 17 00:00:00 2001
From: Leo Famulari <leo@famulari.name>
Date: Tue, 27 Dec 2022 21:26:16 -0500
Subject: [PATCH v2] hydra: cuirass: Add jobsets for testing kernel updates.

* hydra/modules/sysadmin/services.scm (cuirass-specs): Add 'kernel-updates',
'kernel-updates-images', and 'kernel-updates-system-tests' jobsets.
---
 hydra/modules/sysadmin/services.scm | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/hydra/modules/sysadmin/services.scm b/hydra/modules/sysadmin/services.scm
index d0c5b24..66bf231 100644
--- a/hydra/modules/sysadmin/services.scm
+++ b/hydra/modules/sysadmin/services.scm
@@ -2,6 +2,7 @@
 ;;;
 ;;; Copyright © 2016-2022 Ludovic Courtès <ludo@gnu.org>
 ;;; Copyright © 2017, 2018, 2020, 2022 Ricardo Wurmus <rekado@elephly.net>
+;;; Copyright © 2022 Leo Famulari <leo@famulari.name>
 ;;;
 ;;; This program is free software: you can redistribute it and/or modify
 ;;; it under the terms of the GNU General Public License as published by
@@ -347,6 +348,26 @@
                         (branch "core-updates"))))
                 (priority 4)
                 (systems '#$systems)))
+            #~())
+     #$@(if (member "kernel-updates" branches)
+            #~((specification
+                (name "kernel-updates")
+                (build '(manifests "etc/kernels-manifests.scm"))
+                (period 7200)
+                (priority 2)
+                (systems '#$systems))
+               (specification
+                (name "kernel-updates-images")
+                (build 'images)
+                (period 7200)
+                (priority 2)
+                (systems '#$systems))
+               (specification
+                (name "kernel-updates-system-tests")
+                (build 'system-tests)
+                (period 7200)
+                (priority 2)
+                (systems '("x86_64-linux"))))
             #~())))
 
 \f
-- 
2.38.1


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

  reply	other threads:[~2022-12-28 22:51 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-12-28  2:22 [bug#60373] Kernel job for Cuirass Leo Famulari
2022-12-28  2:25 ` [bug#60373] [PATCH] maint: Add a manifest for everything related to linux-libre Leo Famulari
2022-12-28  2:29 ` [bug#60373] [PATCH] hydra: cuirass: Add 'kernel-updates' jobset Leo Famulari
2022-12-28 14:35   ` [bug#60373] Kernel job for Cuirass Mathieu Othacehe
2022-12-28 22:50     ` Leo Famulari [this message]
2022-12-29 10:33       ` Mathieu Othacehe
2022-12-29 18:54         ` Leo Famulari
2023-01-05  3:48         ` Leo Famulari
2023-01-05 17:37           ` Mathieu Othacehe
2023-01-06  1:19             ` Leo Famulari
2023-01-06  3:00               ` Leo Famulari
2023-01-06  5:33                 ` Leo Famulari
2023-01-06  6:27                   ` Leo Famulari
2023-01-07 14:50                   ` Mathieu Othacehe
2023-01-07 14:47                 ` Mathieu Othacehe
2023-01-07 18:38                   ` Leo Famulari
2022-12-28 22:54 ` [bug#60373] [PATCH v2] maint: Add a manifest for everything related to linux-libre Leo Famulari

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=Y6zIJb4Pk2mbDQ/6@jasmine.lan \
    --to=leo@famulari.name \
    --cc=60373@debbugs.gnu.org \
    --cc=othacehe@gnu.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.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

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.