unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: ludo@gnu.org (Ludovic Courtès)
To: Carlo Zancanaro <carlo@zancanaro.id.au>
Cc: guix-devel <guix-devel@gnu.org>
Subject: Re: Services can now have a default value
Date: Sat, 22 Apr 2017 00:04:09 +0200	[thread overview]
Message-ID: <87bmrp8lk6.fsf@gnu.org> (raw)
In-Reply-To: <874lxjnzyx.fsf@zancanaro.id.au> (Carlo Zancanaro's message of "Thu, 20 Apr 2017 20:19:18 +1000")

Hello,

Carlo Zancanaro <carlo@zancanaro.id.au> skribis:

> On Thu, Apr 20 2017, Ludovic Courtès wrote:
>> There must be some sort of a mapping between service types and
>> configuration types, indeed, but I’m not sure how to achieve it.
>>
>> One solution would be to have all the <foo-configuration> records
>> inherit (in the OO sense) from <service>, or something along these
>> lines.
>
> This was my first thought. I couldn't see how to do OO-style inheritance
> with the SRFI-9 API, though. I'm not very experienced with Guile (or
> scheme generally), so I might do some more reading about that.

SRFI-99 supports inheritance, though there’s currently no SRFI-99 module
in Guile proper:

  https://srfi.schemers.org/srfi-99/srfi-99.html

Oh and there’s also R6RS records, SRFI-35… no shortage of record APIs!
:-)

>> Or we could have a ‘define-service’ macro that defines both the
>> <service-type> and the <foo-configuration>, and defines a ‘foo-service’
>> macro equivalent to (service foo-service-type (foo-configuration …)).
>>
>>   (define-service-type openssh-service-type
>>     openssh-service
>>     (extensions …)
>>     (configuration
>>       (port openssh-service-port (default 22))
>>       (use-pam? openssh-service-use-pam? (default #t))))
>>
>> and then:
>>
>>    (operating-system
>>      ;; …
>>      (services (cons (openssh-service (port 2222)) %base-services)))
>
> I also thought about this, but I was concerned about things like
> dovecot-service, where there are two configuration objects. I wouldn't
> want to force us to duplicate code, and create two different service
> types, if we wanted services like that in future.
>
> Although, maybe we would actually rather enforce a "one configuration
> type per service type" rule, for the sake of modifying services? It's
> hard to modify a service if you can't be sure of what the type of the
> configuration will be.

Right, I would prefer one type per service.  I didn’t know dovecot was
different.

> Do you have a preference for what approach to use? If we use a macro to
> generate things then we retain the same flexibility as the current
> approach which removing a bunch of boilerplate, but I'm not sure I have
> the best view of the trade-offs involved.

A ‘define-service-type’ macro or similar could generate either code the
current framework (with <service-type> and <service> and
<foo-configuration>) or for SRFI-99-style records if we later to go that
route.

So I think we should start by designing this macro.

How does that sound?

>> I’m not sure what you mean.  Is it something like what ‘simple-service’
>> does?
>
> I meant something more like what I did with exim-service-type, where I
> extend a service just to ensure its presence, then I had to document
> you have to have a mail-aliases-service-type in order to use exim. With
> a default configuration the mail-aliases-service-type could be
> automatically instantiated if it doesn't exist.

Oh right.

Well I don’t know, perhaps in some cases it might make sense to
automatically instantiate things depended on.  The advantage is that as
a user of the service (exim for instance) you don’t have to be aware of
the services it expects (improves separation of concern).

So you could blissfully write just:

  (cons (service mediagoblin-service-type)
        %base-services)

and behind the scenes it would add an nginx instance, an mcron instance
with a couple of jobs, a rottlog instance, and so on.

WDYT?

Thanks,
Ludo’.

  reply	other threads:[~2017-04-21 22:04 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-04-15 22:51 Services can now have a default value Ludovic Courtès
2017-04-15 23:11 ` ng0
2017-04-17 11:56   ` Ricardo Wurmus
2017-04-19 23:22     ` ng0
2017-04-20  8:53       ` Ricardo Wurmus
2017-04-20  9:09         ` ng0
2017-04-23 10:23           ` Ricardo Wurmus
2017-04-17 11:58 ` Ricardo Wurmus
2017-04-19 14:42 ` Carlo Zancanaro
2017-04-19 15:18   ` ng0
2017-04-19 21:20   ` Ludovic Courtès
     [not found]     ` <8737d32abz.fsf@zancanaro.id.au>
2017-04-20  8:42       ` Ludovic Courtès
2017-04-20 10:19         ` Carlo Zancanaro
2017-04-21 22:04           ` Ludovic Courtès [this message]
2017-04-21 23:41             ` Carlo Zancanaro
2017-04-22  0:46               ` We need an RFC procedure [Re: Services can now have a default value] ng0
2017-04-22  7:12                 ` Ricardo Wurmus
2017-04-22 10:08                   ` ng0
2017-04-22 22:55                     ` Ludovic Courtès
2017-04-23 10:13                       ` Ricardo Wurmus
2017-04-23 12:02                         ` ng0
2017-04-27 13:29                         ` Ludovic Courtès
2017-04-27 16:37                           ` Petter
2017-05-02 12:42                             ` Ludovic Courtès
2017-05-22 21:23                               ` Ricardo Wurmus
2017-05-22 22:45                                 ` Leo Famulari
2017-04-23 11:52                       ` ng0
2017-05-13 10:39               ` Services can now have a default value Carlo Zancanaro
2017-05-13 22:53                 ` Carlo Zancanaro
2017-05-15 12:48                   ` Ludovic Courtès
2017-04-22 14:46             ` Christopher Allan Webber
2017-04-22 14:59               ` Jan Nieuwenhuizen
2017-04-22 22:57               ` Ludovic Courtès

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

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87bmrp8lk6.fsf@gnu.org \
    --to=ludo@gnu.org \
    --cc=carlo@zancanaro.id.au \
    --cc=guix-devel@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 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).