[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#29732] [PATCH 1/1] services: Add dhcpd-service-type and <dhcpd-conf
[bug#29732] [PATCH 1/1] services: Add dhcpd-service-type and <dhcpd-configuration>.
Wed, 18 Apr 2018 22:28:33 +0200
Gnus/5.13 (Gnus v5.13) Emacs/25.3 (gnu/linux)
Chris Marusich <address@hidden> skribis:
> Here's a new patch. It mainly tidies up some formatting, makes it
> possible to run more than one version of the DHCP daemon at the same
> time (e.g., for IPv4, IPv6, and IPv4 over IPv6), and adds a system test
> to verify that the DHCPv4 daemon can start up without error.
> I tried adding a test case for DHCPv6, but I ran into complications.
> Specifically, I'm not sure how to give an IPv6 address and subnet to the
> eth0 interface within the test VM using the static-networking-service.
> The DHCPv6 daemon requires the interface to have an IPv6 subnet
> configured, so it refuses to run on that interface.
> For now, it's nice enough, I think, that we have a DHCPv4 system test!
> Please let me know what you think.
I agree, nice job!
> From 5dd2e6853f1a332e55f3a4ba69b9baf199458fcb Mon Sep 17 00:00:00 2001
> From: Chris Marusich <address@hidden>
> Date: Sat, 16 Dec 2017 00:52:42 -0800
> Subject: [PATCH] services: Add dhcpd-service-type and <dhcpd-configuration>.
> * doc/guix.texi (Networking Services): Document it.
> * gnu/services/networking.scm (dhcpd-service-type): Add it.
> (dhcpd-configuration, dhcpd-configuration?): Add it.
> (dhcpd-configuration-package): Add it.
> (dhcpd-configuration-config-file): Add it.
> (dhcpd-configuration-version): Add it.
> (dhcpd-configuration-run-directory): Add it.
> (dhcpd-configuration-lease-file): Add it.
> (dhcpd-configuration-pid-file): Add it.
> (dhcpd-configuration-interfaces): Add it.
> doc/guix.texi | 17 +++++++
> gnu/services/networking.scm | 80 ++++++++++++++++++++++++++++++
> gnu/tests/networking.scm | 97 ++++++++++++++++++++++++++++++++++++-
Please mention the gnu/tests/networking.scm changes in the log.
Otherwise LGTM, thank you!