unofficial mirror of bug-guix@gnu.org 
 help / color / mirror / code / Atom feed
* bug#73916: [Cuirass] Queued derivations have their GC roots removed
@ 2024-10-20 20:06 Ludovic Courtès
  2024-11-10 11:49 ` Ludovic Courtès
  0 siblings, 1 reply; 2+ messages in thread
From: Ludovic Courtès @ 2024-10-20 20:06 UTC (permalink / raw)
  To: 73916

Cuirass stores GC roots for all the derivations of its jobsets under
/var/guix/profiles/per-user/cuirass/cuirass (or similar).

The ‘delete-old-gc-roots’ periodically deletes these GC roots (which are
not just derivations), but it’s supposed to keep GC roots to derivations
that are queued: that was done in
55af0f70c0d4938b8eda777382bbc4d8f5698a37.

Unfortunately, there’s ample evidence on berlin that GC roots of .drv
currently queued get removed anyway.  This causes workers to try in vain
to substitute them, to eventually reject the build, which gets canceled
by ‘remote-server’.

Ludo’.




^ permalink raw reply	[flat|nested] 2+ messages in thread

* bug#73916: [Cuirass] Queued derivations have their GC roots removed
  2024-10-20 20:06 bug#73916: [Cuirass] Queued derivations have their GC roots removed Ludovic Courtès
@ 2024-11-10 11:49 ` Ludovic Courtès
  0 siblings, 0 replies; 2+ messages in thread
From: Ludovic Courtès @ 2024-11-10 11:49 UTC (permalink / raw)
  To: 73916

Ludovic Courtès <ludovic.courtes@inria.fr> skribis:

> Cuirass stores GC roots for all the derivations of its jobsets under
> /var/guix/profiles/per-user/cuirass/cuirass (or similar).
>
> The ‘delete-old-gc-roots’ periodically deletes these GC roots (which are
> not just derivations), but it’s supposed to keep GC roots to derivations
> that are queued: that was done in
> 55af0f70c0d4938b8eda777382bbc4d8f5698a37.
>
> Unfortunately, there’s ample evidence on berlin that GC roots of .drv
> currently queued get removed anyway.  This causes workers to try in vain
> to substitute them, to eventually reject the build, which gets canceled
> by ‘remote-server’.

This was fixed or at least mitigated by these Cuirass commits:

  e96f088 * register: Add ‘--derivation-ttl’ option, default to twice ‘--ttl’.
  304c09b * base: Be more conservative in deciding whether to remove a .drv GC root.

In particular, .drv for builds currently in ‘failed-derivation’ status
are no longer GC’d when other builds in that evaluation might eventually
trigger a failed-derivation -> scheduled transition.

Ludo’.




^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2024-11-10 13:53 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-10-20 20:06 bug#73916: [Cuirass] Queued derivations have their GC roots removed Ludovic Courtès
2024-11-10 11:49 ` Ludovic Courtès

Code repositories for project(s) associated with this public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).