[v6ops] Genart last call review of draft-ietf-v6ops-cpe-slaac-renum-04

[Pete Resnick via Datatracker <[e-mail]>]

Reviewer: Pete Resnick
Review result: Ready with Issues

I am the assigned Gen-ART reviewer for this draft. The General Area
Review Team (Gen-ART) reviews all IETF documents being processed
by the IESG for the IETF Chair.  Please treat these comments just
like any other last call comments.

For more information, please see the FAQ at

<https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.

Document: draft-ietf-v6ops-cpe-slaac-renum-04
Reviewer: Pete Resnick
Review Date: 2020-09-09
IETF LC End Date: 2020-09-09
IESG Telechat date: Not scheduled for a telechat

Summary: Ready with one process issue, and some assorted nits.

Major issues: None

Minor issues:

The shepherd writeup says:

   The document so far has been approved by the V6OPS working group
   (successful working group last call). The document does not specify
   new protocol, but rather changes to the default parameters in
   existing protocols.

However, the document is Informational, as confirmed by the shepherd writeup.
If this is actually updating default parameters in protocols, that sounds like
it should either be a Standards Track document or more likely a BCP. As 2026
says:

   The BCP subseries of the RFC series is designed to be a way to
   standardize practices and the results of community deliberations. [...]

   ...[G]ood user
   service requires that the operators and administrators of the
   Internet follow some common guidelines for policies and operations.
   While these guidelines are generally different in scope and style
   from protocol standards, their establishment needs a similar process
   for consensus building.

That sounds like what this is doing, especially with all of the 2119 language
in here. Maybe this is Informational because 7084 (and 6204 before it) were
Informational, but perhaps 7084 (and other such document) should be BCP as
well. Indeed, it sounds like all of these SLAAC operational documents could be
in one BCP together. Either way, Informational seems wrong.

Nits/editorial comments:

Throughout the document, it says, "This document RECOMMENDS..." or "This
document also RECOMMENDS" or "Additionally, this document RECOMMENDS". RFC 2119
does not use "RECOMMENDS". You can say "CE Routers SHOULD..." or "A Router
Lifetime of ND_PREFERRED_LIMIT is RECOMMENDED" or if you must "It is
RECOMMENDED that..." (blech, I hate the passive form), since SHOULD and
RECOMMENDED are equivalent in 2119, but using the "This document RECOMMENDS..."
form is weird and isn't in 2119.

In 3.3, it says:

   o  Upon changes to the advertised prefixes, and after bootstrapping,
      the CE router advertising prefix information via SLAAC SHOULD
      proceed as follows:

But then each of the things under there has a SHOULD or a MUST. The SHOULD here
is confusing. Instead, the sentence could simply be:

   o  Upon changes to the advertised prefixes, and after bootstrapping,
   the CE router advertising prefix information via SLAAC proceeds as
   follows:

Similarly:

   This document RECOMMENDS that if a CE Router provides LAN-side DHCPv6
   (address assignment or prefix delegation), the following behavior be
   implemented:

Just make the sentence:

   If a CE Router that provides LAN-side DHCPv6 (address assignment or
   prefix delegation), then: