bug#26829: Add knot service

[julien lepiller]

Le 2017-05-28 19:34, address@hidden a écrit :
> Hi Julien,
>
> Julien Lepiller <address@hidden> skribis:
>
> > So, pushed as ba69e8f7ce21a81bdd5b99fdb1cc64492443e15c.
>
> Very nice and very useful notably for self-hosting use cases!
That's exactly why I did that :). I'm currently self-hosting my services 
on debian, and I would like to move to guixSD at some point, so I'm 
writing services I need.

> I have two minor comments:
>
>   1. What it would it take to write tests for this?  I assume we could
>      simply make DNS requests locally?
I have never written a system test, so I don't know how it works. If we 
check only that requests are answered correctly locally, I think that 
would be great but limited to the "master" scenario. Then there is the 
"slave" scenario, where we would need a zone transfer between two hosts. 
And we could also check that transfer occurs only to authorized slaves.

>   2. For things like:
>
>      address@hidden @code{nsec3-salt-lifetime} (default: @code{"30d"})
>
>      I prefer using an integer denoting the number of seconds, and then
>      letting users do the multiplication or use (string->duration 
> "30d").
>      I did that for ‘guix-publish-service’.
I didn't know it existed, but it looks nicer indeed. I'll see what I can 
do.

>      Thoughts?
I'll be away for a few days, but I'll send patches as soon as I can.

> Thank you!
>
> Ludo’.