GNU bug report logs - #76081
OCI provisioning service

Previous Next

Package: guix-patches;

Reported by: paul <goodoldpaul <at> autistici.org>

Date: Wed, 5 Feb 2025 22:01:03 UTC

Severity: normal

Tags: moreinfo

Full log


View this message in rfc822 format

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Giacomo Leidi <goodoldpaul <at> autistici.org>
Cc: 76081 <at> debbugs.gnu.org
Subject: [bug#76081] [PATCH v10 6/7] services: oci: Migrate oci-configuration to (guix records).
Date: Fri, 16 May 2025 11:02:34 +0900
Hi,

Giacomo Leidi <goodoldpaul <at> autistici.org> writes:

> This commit migrates oci-configuration to (guix records) singe it
> appears (for-home (oci-configuration ...)) does not work as expected
> with (gnu services configuration).  This is supposed to be completely
> transparent for users and can be reverted in the
> future once this has been implemented.
>
> * gnu/service/containers.scm: Migrate oci-configuration to (guix records).

I'd like to understand and resolve this issue and avoid pushing this
commit.  What is the errors you are seeing?  It at least byte-compiles
fine with this commit reverted and the following diff applied:

--8<---------------cut here---------------start------------->8---
modified   gnu/home/services/containers.scm
@@ -32,16 +32,14 @@ (define home-oci-service-type
                 (extensions
                  (list
                   (service-extension home-profile-service-type
-                                     (oci-service-extension-wrap-validate
-                                      (lambda (config)
-                                        (let ((runtime-cli
-                                               (oci-configuration-runtime-cli config))
-                                              (runtime
-                                               (oci-configuration-runtime config)))
-                                          (oci-service-profile runtime runtime-cli)))))
+                                     (lambda (config)
+                                       (let ((runtime-cli
+                                              (oci-configuration-runtime-cli config))
+                                             (runtime
+                                              (oci-configuration-runtime config)))
+                                         (oci-service-profile runtime runtime-cli))))
                   (service-extension home-shepherd-service-type
-                                     (oci-service-extension-wrap-validate
-                                      oci-configuration->shepherd-services))))
+                                     oci-configuration->shepherd-services)))
                 (extend
                  (lambda (config extension)
                    (for-home
--8<---------------cut here---------------end--------------->8---

Also, there are other home services using a define-configuration'd
record with for-home, such as the home-snuik-service-type, so it should
be possible to make it work.

-- 
Thanks,
Maxim




This bug report was last modified 12 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.