Re: [Gen-art] Gen-ART review of draft-ietf-dhc-dhcpv6-failover-protocol-04

Christer Holmberg <christer.holmberg@ericsson.com> Wed, 01 February 2017 22:58 UTC

Return-Path: <christer.holmberg@ericsson.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 7BD9C1295EB for <gen-art@ietfa.amsl.com>; Wed, 1 Feb 2017 14:58:15 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.221
X-Spam-Level:
X-Spam-Status: No, score=-4.221 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id cB26_S1zSeyK for <gen-art@ietfa.amsl.com>; Wed, 1 Feb 2017 14:58:14 -0800 (PST)
Received: from sessmg22.ericsson.net (sessmg22.ericsson.net [193.180.251.58]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 5920D129577 for <gen-art@ietf.org>; Wed, 1 Feb 2017 14:58:14 -0800 (PST)
X-AuditID: c1b4fb3a-12eaf98000004068-1e-58926804c0f9
Received: from ESESSHC024.ericsson.se (Unknown_Domain [153.88.183.90]) by (Symantec Mail Security) with SMTP id 1E.9B.16488.40862985; Wed, 1 Feb 2017 23:58:12 +0100 (CET)
Received: from ESESSMB209.ericsson.se ([169.254.9.76]) by ESESSHC024.ericsson.se ([153.88.183.90]) with mapi id 14.03.0319.002; Wed, 1 Feb 2017 23:57:39 +0100
From: Christer Holmberg <christer.holmberg@ericsson.com>
To: kkinnear <kkinnear@cisco.com>
Thread-Topic: Gen-ART review of draft-ietf-dhc-dhcpv6-failover-protocol-04
Thread-Index: AdJ8ysFOq+nkh2z8R/yqSz6QRbIwJQACS1KAAAJnoyA=
Date: Wed, 01 Feb 2017 22:57:39 +0000
Message-ID: <7594FB04B1934943A5C02806D1A2204B4BFD921A@ESESSMB209.ericsson.se>
References: <7594FB04B1934943A5C02806D1A2204B4BFD8F92@ESESSMB209.ericsson.se> <F5C2B9B7-F35D-4E92-829E-05100DDFBDA5@cisco.com>
In-Reply-To: <F5C2B9B7-F35D-4E92-829E-05100DDFBDA5@cisco.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [153.88.183.154]
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFmplkeLIzCtJLcpLzFFi42KZGbE9SpclY1KEwfwXqhYnN8xjs7j66jOL xbaGz0wOzB5Tfm9k9Viy5CeTx5fLn9kCmKO4bFJSczLLUov07RK4Mtqm3WQpaNCo+P58IlsD 4x+1LkZODgkBE4mVTX3MILaQwDpGiRut/l2MXED2IkaJU7N7mLoYOTjYBCwkuv9pg9SICChK /P19nx2khllgBaPEsa0P2UESwgKeEus63zNBFHlJrHq9ihnCtpKYumg+mM0ioCJx8f0kRhCb V8BX4nLXNGaIZQ2MEmtadoM1cwrYSmze3QBWxCggJvH91BqwOLOAuMStJ/OZIK4WkFiy5zwz hC0q8fLxP1YIW0li0e3PYEczC2hKrN+lD9GqKDGlG+JOXgFBiZMzn7BMYBSdhWTqLISOWUg6 ZiHpWMDIsopRtDi1uDg33chIL7UoM7m4OD9PLy+1ZBMjMHIObvlttYPx4HPHQ4wCHIxKPLwG BpMihFgTy4orcw8xSnAwK4nwFiQDhXhTEiurUovy44tKc1KLDzFKc7AoifOarbwfLiSQnliS mp2aWpBaBJNl4uCUamBsbUmYdODEDI7vUVxWV1aHfpQ0YtpVXPFzAotWWYKwm+fr8pzMS9sk mtJrltw6cXyiUgGbvbn9frGl07K3uJ2bENsg+W2m+7+SNJlYPeblhnPY04tyP6ndOLPk++ng 870pDjM/7juxWju8YN7aJBULZ2bLI/PXb5xhvFy+RXWeweY3nBUsM34osRRnJBpqMRcVJwIA CgrzC5gCAAA=
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/Lz1pZfn0Rji297hcwdJL01Z3IGk>
Cc: "gen-art@ietf.org" <gen-art@ietf.org>, "draft-ietf-dhc-dhcpv6-failover-protocol.all@tools.ietf.org" <draft-ietf-dhc-dhcpv6-failover-protocol.all@tools.ietf.org>
Subject: Re: [Gen-art] Gen-ART review of draft-ietf-dhc-dhcpv6-failover-protocol-04
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 01 Feb 2017 22:58:15 -0000

Hi,

>I will make all of the changes that you suggest, and have offered details (indented) below.  Note that while I'm willing to make the change you have discussed in Q3, I don't >actually understand what you would like us to do.
>
>More below...

INTRODUCTION:
  
>> Q1:        In the first sentence of the Introduction, I suggest to say:
>>  
>> “The failover protocol defined in this document provides…”
>>  
>> Otherwise it’s a little unclear what failover protocol you are talking about.
>
>	Sure, good idea.  See Q2, immediately below.
>  
>> Q2:        In the Introduction, before the first sentence, shouldn’t there be some background text, including some information 
>>about the problem that the document solves. I know there is something in the Abstract, but I think there should also be something 
>>in the Introduction, before jumping into the solution.
>
>	I've been chastised for repeating the abstract in the
>	introduction, so I was trying to not do that.  How about we
>	start the introduction with:
>
>	"The DHCPv6 protocol [RFC3315] does not provide for server
>	redundancy.  The failover protocol defined in this document
>	provides a means for cooperating DHCP servers to work together
>	to provide a DHCP service with availability that is increased
>	beyond that which could be provided by a single DHCP server
>	operating alone. ..."

Looks very good.

I know some people don't want to repeat Abstract text in the Introduction, but I still think the Introduction should provide an introduction to the problem :)

>	Note that the last sentence of the Introduction already points
>	to the DHCPv6 failover requirements RFC, which has a lot more
>	to say about this.
>  
>> Q3:        In the Introduction, I suggest adding a reference to the first occurrences of “DHCP service” and “DHCP server”.
>
>	While I'm more than happy to do this, I don't actually know
>	what you would like me to reference?  We haven't defined
>	either "DHCP service" or "DHCP server" in the glossary since
>	we felt they were reasonably apparent from the context of this
>	document.  Are you thinking that we should reference RFC3315
>	on the first occurrence of "DHCP service" and "DHCP server"?

Yes.


>> Q4:        In the Introduction, you switch between “This protocol” and “The failover protocol”. Please use consistent terminology. This applies to the document in general.
>
>	Ok, we'll go with "The failover protocol" since it appears 
>	more times so far.  I will make this global change when I
>	next update the document.  

Sounds good.

  
>> SECTION 4:
>>  
>> Q5:        In the Abstract and Introduction it is said that DHCPv6 does not provide server redundancy. Then section 4 talks about failover concepts and mechanism.
>>  
>> Are those concepts something used for DHCPv6 today, but for some reason do not fulfil the failover protocol requirements?
>>  
>> OR, are these general concepts that will be supported by implementing the failover protocol?
>>  
>> I think it would be good to have an introduction statement clarifying that.
>
>	The concepts and mechanisms discussed in Section 4 relate to
>	the failover protocol, they aren't present in the regular
>	RFC3315(et. al.) DHCPv6 protocol.  I will add the following to
>	Section 4 to clarify that:
>
>	"4.  Failover Concepts and Mechanisms
>
>	The following concepts and mechanisms are necessary to the operation
>	of the failover protocol, and they are not currently employed by
>	the DHCPv6 protocol [RFC3315].

That's better. But, will the failover protocol provide support for those concepts? If so, please indicate it.

Thanks!
 
Regards,

Christer