From: "Ludovic Courtès" <ludo@gnu.org>
To: Christopher Baines <mail@cbaines.net>
Cc: 43494@debbugs.gnu.org
Subject: [bug#43494] [PATCH 4/4] services: guix: Add guix-build-coordinator-queue-builds-service-type.
Date: Fri, 18 Sep 2020 22:31:14 +0200 [thread overview]
Message-ID: <87pn6iu8rx.fsf@gnu.org> (raw)
In-Reply-To: <20200918184042.22660-4-mail@cbaines.net> (Christopher Baines's message of "Fri, 18 Sep 2020 19:40:42 +0100")
Christopher Baines <mail@cbaines.net> skribis:
> * gnu/services/guix.scm (<guix-build-coordinator-queue-builds-configuration>):
> New record type.
> (guix-build-coordinator-queue-builds-configuration,
> guix-build-coordinator-queue-builds-configuration?,
> guix-build-coordinator-queue-builds-configuration-package,
> guix-build-coordinator-queue-builds-configuration-user,
> guix-build-coordinator-queue-builds-coordinator,
> guix-build-coordinator-queue-builds-configuration-systems,
> guix-build-coordinator-queue-builds-configuration-system-and-targets,
> guix-build-coordinator-queue-builds-configuration-guix-data-service,
> guix-build-coordinator-queue-builds-configuration-processed-commits-file,
> guix-build-coordinator-queue-builds-shepherd-services,
> guix-build-coordinator-queue-builds-activation,
> guix-build-coordinator-queue-builds-account): New procedures.
> (guix-build-coordinator-queue-builds-service-type): New variable.
[...]
> +@defvar {Scheme Variable} guix-build-coordinator-queue-builds-service-type
> +Service type for the
> +guix-build-coordinator-queue-builds-from-guix-data-service script. Its
Oh! :-)
‘guix-build-queue’ maybe? Because after all, it can queue builds coming
from anywhere, not just from the Data Service, right?
It’d be nice to add a sentence or two explaining how the “queue” fits in
the big picture (I’m not quite sure :-)).
> +@item @code{systems-and-targets} (default: @code{#f})
> +An association list of system and target pairs for which to fetch
> +derivations to build.
Cross-compilation target triplets are a notion that’s not available at
the derivation level; it only exists for packages.
Are we mixing things here?
> +@item @code{guix-data-service} (default: @code{"https://data.guix.gnu.org"})
> +The Guix Data Service instance from which to query to find out about
> +derivations to build.
Ah so the queue pulls from the Data Service. Got it!
> +@item @code{processed-commits-file} (default: @code{"/var/lib/guix-build-coordinator-queue-builds/processed-commits"})
> +A file to record which commits have been processed, to avoid needlessly
> +processing them again if the service is restarted.
Maybe in /var/cache by default, no?
> + (processed-commits-file
> + guix-build-coordinator-queue-builds-configuration-processed-commits-file
> + (default "/var/lib/guix-build-coordinator-queue-builds/processed-commits")))
Maybe “state-file”? In addition to saving space :-), it’d leave room
for additional bits of state.
That’s all, thank you!
Ludo’.
next prev parent reply other threads:[~2020-09-18 20:33 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-09-18 18:34 [bug#43494] [PATCH 0/4] Add package and services for the Guix Build Coordinator Christopher Baines
2020-09-18 18:40 ` [bug#43494] [PATCH 1/4] gnu: Add guix-build-coordinator Christopher Baines
2020-09-18 18:40 ` [bug#43494] [PATCH 2/4] services: guix: Add guix-build-coordinator-service-type Christopher Baines
2020-09-18 20:20 ` Ludovic Courtès
2020-09-19 9:13 ` Christopher Baines
2020-09-25 9:42 ` Ludovic Courtès
2020-09-26 8:43 ` Christopher Baines
2020-09-18 18:40 ` [bug#43494] [PATCH 3/4] services: guix: Add guix-build-coordinator-agent-service-type Christopher Baines
2020-09-18 20:25 ` Ludovic Courtès
2020-09-19 9:49 ` Christopher Baines
2020-09-18 18:40 ` [bug#43494] [PATCH 4/4] services: guix: Add guix-build-coordinator-queue-builds-service-type Christopher Baines
2020-09-18 20:31 ` Ludovic Courtès [this message]
2020-09-19 10:05 ` Christopher Baines
2020-09-18 20:08 ` [bug#43494] [PATCH 1/4] gnu: Add guix-build-coordinator Ludovic Courtès
2020-09-18 23:34 ` Jonathan Brielmaier
2020-09-19 9:00 ` Christopher Baines
2020-09-19 9:05 ` Christopher Baines
2020-09-19 10:10 ` [bug#43494] [PATCH v2 " Christopher Baines
2020-09-19 10:10 ` [bug#43494] [PATCH v2 2/4] services: guix: Add guix-build-coordinator-service-type Christopher Baines
2020-09-19 10:10 ` [bug#43494] [PATCH v2 3/4] services: guix: Add guix-build-coordinator-agent-service-type Christopher Baines
2020-09-19 10:10 ` [bug#43494] [PATCH v2 4/4] services: guix: Add guix-build-coordinator-queue-builds-service-type Christopher Baines
2020-10-05 8:00 ` Ludovic Courtès
2020-10-05 17:16 ` bug#43494: " Christopher Baines
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=87pn6iu8rx.fsf@gnu.org \
--to=ludo@gnu.org \
--cc=43494@debbugs.gnu.org \
--cc=mail@cbaines.net \
/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.