Re: [Pals] [Gen-art] Genart last call review of draft-ietf-pals-ethernet-cw-06

Alissa Cooper <[email protected]>
Newsgroups gmane.ietf.pwe3,gmane.ietf.gen-art
Message-ID <[email protected]>
Thanks!

> On Jun 19, 2018, at 5:35 AM, Stewart Bryant <[email protected]> wrote:
> 
> Hi Alissa
> 
> Yes, we will address all review comments that we receive.
> 
> - Stewart
> 
> 
> On 18/06/2018 18:44, Alissa Cooper wrote:
>> Brian, thanks for your review. Andy, thanks for your response; hopefully it means Brian’s comments will be addressed? I have entered a No Objection ballot.
>> 
>> Alissa
>> 
>>> On Jun 12, 2018, at 1:50 PM, Andrew G. Malis <[email protected] <mailto:[email protected]>> wrote:
>>> 
>>> Brian,
>>> 
>>> Thanks, your comments are much appreciated.
>>> 
>>> Cheers,
>>> Andy
>>> 
>>> 
>>> On Mon, Jun 11, 2018 at 11:30 PM, Brian Carpenter <[email protected] <mailto:[email protected]>> wrote:
>>> Reviewer: Brian Carpenter
>>> Review result: Ready with Nits
>>> 
>>> Gen-ART Last Call review of draft-ietf-pals-ethernet-cw-06
>>> 
>>> 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
>>> <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>>.
>>> 
>>> Document: draft-ietf-pals-ethernet-cw-06.txt
>>> Reviewer: Brian Carpenter
>>> Review Date: 2018-06-12
>>> IETF LC End Date: 2018-06-15
>>> IESG Telechat date: 2018-06-21
>>> 
>>> Summary: Ready with nits
>>> --------
>>> 
>>> Comments: 
>>> ---------
>>> 
>>> This (with RFC4928) is a wonderful example of why layer violations are a Bad Thing.
>>> 
>>> Nits:
>>> -----
>>> 
>>> > 1.  Introduction
>>> ....
>>> >   This document recommends the use of the Ethernet pseudowire control
>>> >   word in all but exceptional circumstances.
>>> 
>>> That's wrong, it *mandates* this usage with a MUST (first paragraph of section 4).
>>> 
>>> > 3.  Background
>>> ....
>>> >   A recent posting on the Nanog email list has highlighted this
>>> >   problem:
>>> >
>>> >   https://mailman.nanog.org/pipermail/nanog/2016-December/089395.html <https://mailman.nanog.org/pipermail/nanog/2016-December/089395.html>
>>> 
>>> No, it's no longer recent. How about:
>>> 
>>>    For example, a posting on the Nanog email list highlighted this
>>>    problem:
>>> 
>>> > 7.  Operational Considerations
>>> >
>>> >   CW presence on the PW is controlled by the configuration and may be
>>> >   subject to default operational mode of not being enabled. 
>>> 
>>> That sentence is hard to parse. Try this:
>>> 
>>>    A configuration switch might determine whether the CW is used on the PW. 
>>>    The default configuration might be to disable use of the CW.
>>> 
>>> 
>>> _______________________________________________
>>> Gen-art mailing list
>>> [email protected] <mailto:[email protected]>
>>> https://www.ietf.org/mailman/listinfo/gen-art <https://www.ietf.org/mailman/listinfo/gen-art>
>> 
>

_______________________________________________
Pals mailing list
[email protected]
https://www.ietf.org/mailman/listinfo/pals
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.