GNU bug report logs - #77019
[PATCH 0/1] machine: hetzner: Allow attaching existing public IPs.

Previous Next

Full log

Message #26 received at 77019 <at> debbugs.gnu.org (full text, mbox):

From: Sergey Trofimov <sarg <at> sarg.org.ru>
To: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
Cc: Ludovic Courtès <ludo <at> gnu.org>,
 Roman Scherer <roman <at> burningswell.com>,
 Roman Scherer <roman.scherer <at> burningswell.com>, 77019 <at> debbugs.gnu.org
Subject: Re: bug#77019: [PATCH 0/1] machine: hetzner: Allow attaching
 existing public IPs.
Date: Sun, 20 Apr 2025 16:30:02 +0200

Hi Maxim,

Maxim Cournoyer <maxim.cournoyer <at> gmail.com> writes:

> Hi,
>
> Roman Scherer <roman.scherer <at> burningswell.com> writes:
>
>> Hi Ludovic and Sergey,
>>
>> the patch looks good to me. Thanks for adding the tests.
>>
>> I would have expected #t, #f or a string as the value of
>> hetzner-configuration-ipv4 and hetzner-configuration-ipv6.
>>
>> It's a pitty the null issue in guile-json has no comments yet.
>>
>> I would say, let's merge it. The default behaviour right now is to
>> enable ipv4 and ipv6 and this patch does the same. Once the issue in
>> guile-json has been fixed we gain the ability to disable via #f, right?
>
> nitpick: If these fields are booleans, they should be suffixed by '?',
> e.g. 'ipv6?' or 'ipv4?', for clarity.

They're "boolean | string", with the values being: #t - create new, #f -
don't assign an IP, a string - assign the IP under that name.

Previous Next

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