[bug#33515] [PATCH 0/5] Cuirass/Hydra: evaluate jobs in an inferior

[Mark H Weaver]

Hi Ludovic,

Ludovic Courtès <address@hidden> writes:

> Ludovic Courtès <address@hidden> skribis:
>
>> This patch set changes the way we compute continuous integration jobs:
>> instead of letting Guile auto-compile all of Guix from its checkout,
>> we first build Guix in the same way as ‘guix pull’, open an inferior
>> to that Guix, and run the job evaluation code in that inferior.
>>
>> I think it’s cleaner and it should be faster and less resource-hungry
>> than the current approach.
>>
>> The build-aux/hydra/gnu-system.scm file will now rely on the
>> (guix channels) and (guix inferior) with the new ‘checkout->channel-instance’
>> and ‘inferior-eval-with-store’ procedures, which means that Cuirass
>> (and Hydra) will need to be using a recent Guix to be able to perform
>> the evaluation.  Apart from that ‘gnu-system.scm’ is rather decoupled
>> from the Guix APIs.
>
> I’ve pushed the patches to the ‘wip-ci-inferior’ branch and created
> these two jobsets:
>
>   https://berlin.guixsd.org/jobset/wip-ci-inferior
>   https://hydra.gnu.org/jobset/gnu/wip-ci-inferior
>
> Evaluation with Cuirass on berlin went fine.
>
> I suspect evaluation on hydra will fail though, because it’s probably
> running an older Guix version; we’ll have to upgrade there.  Mark, would
> you like to take a look?

Indeed, the evaluation eventually failed on hydra.gnu.org.  I then
upgraded 'guix' in both root's and hydra's profiles to the latest
version and tried again.  This time it failed almost immediately, with
the following error:

--8<---------------cut here---------------start------------->8---
evaluator hydra-eval-guile-jobs
hydra-eval-guile-jobs returned exit code 1:
adding `/gnu/store/y4m61z51s28kmiff2hzbr7xm6f4lsk80-git-export' to the load path
Backtrace:
In ice-9/eval.scm:
   293:34 19 (_ #<module (#{ g257}#) 2949a00>)
In ice-9/boot-9.scm:
   2862:4 18 (define-module* _ #:filename _ #:pure _ #:version _ # _ ?)
  2875:24 17 (_)
   222:17 16 (map1 (((guix git)) ((guix records)) ((guix gexp)) (#) ?))
  2788:17 15 (resolve-interface (guix git) #:select _ #:hide _ # _ # ?)
  2714:10 14 (_ (guix git) _ _ #:ensure _)
  2982:16 13 (try-module-autoload _ _)
   2312:4 12 (save-module-excursion #<procedure 472d510 at ice-9/boo?>)
  3002:22 11 (_)
In unknown file:
          10 (primitive-load-path "guix/git" #<procedure 4749f20 at ?>)
In ice-9/eval.scm:
   721:20  9 (primitive-eval (define-module (guix git) #:use-module ?))
In ice-9/psyntax.scm:
  1235:36  8 (expand-top-sequence ((define-module (guix git) # # ?)) ?)
  1182:24  7 (parse _ (("placeholder" placeholder)) ((top) #(# # ?)) ?)
   285:10  6 (parse _ (("placeholder" placeholder)) (()) _ c&e (eval) ?)
In ice-9/eval.scm:
   293:34  5 (_ #<module (#{ g258}#) 2949780>)
In ice-9/boot-9.scm:
   2862:4  4 (define-module* _ #:filename _ #:pure _ #:version _ # _ ?)
  2875:24  3 (_)
   222:17  2 (map1 (((git)) ((git object)) ((guix i18n)) ((guix ?)) ?))
   2791:6  1 (resolve-interface _ #:select _ #:hide _ #:prefix _ # _ ?)
In unknown file:
           0 (scm-error misc-error #f "~A ~S" ("no code for modu?" ?) ?)

ERROR: In procedure scm-error:
no code for module (git)

Some deprecated features have been used.  Set the environment
variable GUILE_WARN_DEPRECATED to "detailed" and rerun the
program to get more information.  Set it to "no" to suppress
this message.
--8<---------------cut here---------------end--------------->8---

For now, I rolled back both of the profiles that I updated.

This looks similar to the errors I encountered when trying to update
'guix' on the armhf build slaves, although the module it failed to load
in that case was different.  In both cases, the wrapper for 'guix' is
failing to populate the GUILE_LOAD_PATH with all of the needed modules.

       Mark