Re: [Gen-art] Gen-ART telechat review of draft-ietf-alto-deployments-15

[Mirja Kuehlewind (IETF)]

Sorry it’s late here…

> Am 30.06.2016 um 00:23 schrieb Mirja Kuehlewind (IETF) :
> 
> Hi Brian, hi Jari,
> 
> I’ve put updating Martin’s affliction incl. email address as well as removing 
> the conclusion already in the RFC editor (in case we don’t do another update 
> before publication).
> 
> I note your concerns about the references to individual drafst, however, not 
> sure how this fix this.

s/darfst/drafts/ and s/how this fix this/how to fix this/

> 
> Thanks!
> Mirja
> 
> 
>> Am 30.06.2016 um 00:19 schrieb Jari Arkko :
>> 
>> Thanks much, Brian!
>> 
>> Authors, do note Brian’s comments.
>> 
>> Jari
>> 
>> On 24 Jun 2016, at 05:48, Brian E Carpenter  
>> wrote:
>> 
>>> 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 wait for direction from your
>>> document shepherd or AD before posting a new version of the draft.
>>> 
>>> For more information, please see the FAQ at
>>> .
>>> 
>>> Document: draft-ietf-alto-deployments-15.txt
>>> Reviewer: Brian Carpenter
>>> Review Date: 2016-06-13
>>> IETF LC End Date: 2016-06-21
>>> IESG Telechat date: 2016-06-30
>>> 
>>> Summary: Ready
>>> 
>>> 
>>> Comment:
>>> 
>>> 
>>> This is a long document but well written, so there is little to say about 
>>> it.
>>> 
>>> Minor issues:
>>> -
>>> 
>>> The Conclusion section could be deleted. It doesn't add anything.
>>> 
>>> There are a lot of Informational references that are still personal I-Ds. 
>>> Many of
>>> them are not even listed as "Related Internet-Drafts" for the Alto WG. I 
>>> think readers
>>> of the future RFC will find this a bit annoying.
>>> 
>>> Nits:
>>> -
>>> 
>>>  and  bounced
>>> 
>>> ___
>>> Gen-art mailing list
>>> Gen-art@ietf.org
>>> https://www.ietf.org/mailman/listinfo/gen-art
>> 
> 

___
Gen-art mailing list
Gen-art@ietf.org
https://www.ietf.org/mailman/listinfo/gen-art

Re: [Gen-art] Gen-ART telechat review of draft-ietf-alto-deployments-15

[Mirja Kuehlewind (IETF)]

Hi Brian, hi Jari,

I’ve put updating Martin’s affliction incl. email address as well as removing 
the conclusion already in the RFC editor (in case we don’t do another update 
before publication).

I note your concerns about the references to individual drafst, however, not 
sure how this fix this.

Thanks!
Mirja


> Am 30.06.2016 um 00:19 schrieb Jari Arkko :
> 
> Thanks much, Brian!
> 
> Authors, do note Brian’s comments.
> 
> Jari
> 
> On 24 Jun 2016, at 05:48, Brian E Carpenter  
> wrote:
> 
>> 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 wait for direction from your
>> document shepherd or AD before posting a new version of the draft.
>> 
>> For more information, please see the FAQ at
>> .
>> 
>> Document: draft-ietf-alto-deployments-15.txt
>> Reviewer: Brian Carpenter
>> Review Date: 2016-06-13
>> IETF LC End Date: 2016-06-21
>> IESG Telechat date: 2016-06-30
>> 
>> Summary: Ready
>> 
>> 
>> Comment:
>> 
>> 
>> This is a long document but well written, so there is little to say about it.
>> 
>> Minor issues:
>> -
>> 
>> The Conclusion section could be deleted. It doesn't add anything.
>> 
>> There are a lot of Informational references that are still personal I-Ds. 
>> Many of
>> them are not even listed as "Related Internet-Drafts" for the Alto WG. I 
>> think readers
>> of the future RFC will find this a bit annoying.
>> 
>> Nits:
>> -
>> 
>>  and  bounced
>> 
>> ___
>> Gen-art mailing list
>> Gen-art@ietf.org
>> https://www.ietf.org/mailman/listinfo/gen-art
> 

___
Gen-art mailing list
Gen-art@ietf.org
https://www.ietf.org/mailman/listinfo/gen-art

Re: [Gen-art] Gen-ART telechat review of draft-ietf-alto-deployments-15

[Jari Arkko]

Thanks much, Brian!

Authors, do note Brian’s comments.

Jari

On 24 Jun 2016, at 05:48, Brian E Carpenter  wrote:

> 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 wait for direction from your
> document shepherd or AD before posting a new version of the draft.
> 
> For more information, please see the FAQ at
> .
> 
> Document: draft-ietf-alto-deployments-15.txt
> Reviewer: Brian Carpenter
> Review Date: 2016-06-13
> IETF LC End Date: 2016-06-21
> IESG Telechat date: 2016-06-30
> 
> Summary: Ready
> 
> 
> Comment:
> 
> 
> This is a long document but well written, so there is little to say about it.
> 
> Minor issues:
> -
> 
> The Conclusion section could be deleted. It doesn't add anything.
> 
> There are a lot of Informational references that are still personal I-Ds. 
> Many of
> them are not even listed as "Related Internet-Drafts" for the Alto WG. I 
> think readers
> of the future RFC will find this a bit annoying.
> 
> Nits:
> -
> 
>  and  bounced
> 
> ___
> Gen-art mailing list
> Gen-art@ietf.org
> https://www.ietf.org/mailman/listinfo/gen-art



signature.asc
Description: Message signed with OpenPGP using GPGMail
___
Gen-art mailing list
Gen-art@ietf.org
https://www.ietf.org/mailman/listinfo/gen-art

Re: [Gen-art] Gen-ART review of draft-ietf-mile-implementreport-09

[Jari Arkko]

Thanks!

On 10 Jun 2016, at 15:08, Christer Holmberg  
wrote:

> 
> I am the assigned Gen-ART reviewer for this draft. For background on Gen-ART, 
> please see the FAQ at 
> 
> 
> Document:
> draft-ietf-mile-implementreport-09
> Reviewer:
> Christer Holmberg
> Review Date:
> 10 June 2016
> IETF LC End Date:
> 21 June 2016
> IETF Telechat Date:
> N/A
> Summary:
> The document is well written, and ready for publication is informational RFC.
> Major Issues:None
> Minor Issues:None
> Editorial Issues:
> None
> 
> ___
> Gen-art mailing list
> Gen-art@ietf.org
> https://www.ietf.org/mailman/listinfo/gen-art



signature.asc
Description: Message signed with OpenPGP using GPGMail
___
Gen-art mailing list
Gen-art@ietf.org
https://www.ietf.org/mailman/listinfo/gen-art

Re: [Gen-art] Gen-ART Review of draft-ietf-forces-interfelfb-05

[Jari Arkko]

> Summary:  Ready


Thanks, all!

Jari



signature.asc
Description: Message signed with OpenPGP using GPGMail
___
Gen-art mailing list
Gen-art@ietf.org
https://www.ietf.org/mailman/listinfo/gen-art

Re: [Gen-art] draft-ietf-calext-extensions-03 (was: RE: Assignments for the 2016-06-30 Telechat)

[Jari Arkko]

Thanks, all!

Jari

On 29 Jun 2016, at 17:07, Romascanu, Dan (Dan)  wrote:

> Perfect timing. My only pending issue is addressed. From the Gen-ART point pf 
> view this document is Ready.
> 
> Thanks and Regards,
> 
> Dan
> 
> 
>> -Original Message-
>> From: Cyrus Daboo [mailto:cy...@daboo.name]
>> Sent: Wednesday, June 29, 2016 5:04 PM
>> To: Romascanu, Dan (Dan); A. Jean Mahoney; General Area Review Team
>> Cc: draft-ietf-calext-extensions@tools.ietf.org
>> Subject: Re: draft-ietf-calext-extensions-03 (was: RE: [Gen-art] Assignments
>> for the 2016-06-30 Telechat)
>> 
>> Hi Dan,
>> 
>> --On June 29, 2016 at 1:44:03 PM + "Romascanu, Dan (Dan)"
>>  wrote:
>> 
>>> Hi,
>>> 
>>> There was no update to this document since my previous review. My
>>> original review raised two issues - one was clarified by explanations
>>> from Alexey (thanks!); the other is about the document status (should
>>> it be marked as 'Updates RFC 5545'?). I did not see that addressed. It
>>> would be nice to get an answer, but I do not believe this is a show-stopper.
>> 
>> A new -04 version of the draft has just been published with your review
>> issues addressed.
>> 
>> 
>> --
>> Cyrus Daboo
> 
> ___
> Gen-art mailing list
> Gen-art@ietf.org
> https://www.ietf.org/mailman/listinfo/gen-art



signature.asc
Description: Message signed with OpenPGP using GPGMail
___
Gen-art mailing list
Gen-art@ietf.org
https://www.ietf.org/mailman/listinfo/gen-art

Re: [Gen-art] Genart LC (and likely telechat) review : draft-ietf-trill-tree-selection-04

[Donald Eastlake]

Hi Robert,

Thanks for your thorough comments. See below.

On Tue, Jun 28, 2016 at 3:33 PM, Robert Sparks 
wrote:
> 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
>
> .
>
> Document: draft-ietf-trill-tree-selection-04
> Reviewer: Robert Sparks
> Review Date: 28 Jun 2016
> IETF LC End Date: 1 Jul 2016
> IESG Telechat date: 7 Jul 2016
>
> Summary: Ready (with nits) for publication as Proposed Standard
>
> This document is easy to read, even for someone not deeply steeped
> in trill.
>
> I have a few questions and suggestions to consider
>
> 1) The essence of the idea this document provides support for is that an
> operator will create and install a configuration that meets the one tree per
> identifiable thing (such as VLAN) constraint.

Well, it helps as long as multi-destination traffic identified by VLAN
or whatever is carried by fewer than all trees. It need not be one.

> The protocol proposed here
> does not try to enforce that the operator supplies a configuration meeting
> that constraint. Should the things that generate messages with the TLVs
> defined in this document be restricted from sending messages that would map
> the same VLAN to two trees? I understand things will still work
> (suboptimally, as pointed out in the backwards-compatibility section), but
> it seems this configuration error should be mitigated. Section 3.3 also
> pulls the punch a little with it's discussion at the end of the second
> paragraph. If you're going to leave it up to the unspecified way the
> operator installs this configuration, you might at least point out that this
> is something to look for and complain about. If you think the optimal
> configuration isn't a likely thing to reach, then consider a pass through
> the document that sets that expectation consistently.

While restricting, for example, VLAN-x to one tree is optimal from the
point of view of using up the least amount of fast path FIB
(Forwarding Information Base) resources in some hardware
implementations, it is not optimal from the point of view of load
spreading. To get optimal load spreading, you would want to spread
different multi-destination flows onto different distribution trees.

> 2) There are a couple of places where you use 2119 where you appear to be
> restating requirements from other documents. That's dangerous, from a
> document set maintenance point of view. Please consider changing these to
> simple prose, leaving the 2119 requirements to the protocol you're defining
> in this document. Please look at the SHOULD in the Background Description,
> and the SHOULD NOT in the first paragraph of the Overview. (2119 in sections
> like backgrounds and overviews is usually a sign that somethings in the
> wrong place.)

The SHOULD in the Background Description is indeed just echoing the
same provision from [RFC6325] and can be changed to not use a 2119 keyword.

The SHOULD NOT in the first paragraph of the Overview (Section 3.1) is
entirely due to theis draft and not inhereted from any other document.

> 3) In the 3rd paragraph of 3.3, why is the requirement SHOULD strength? What
> else would the RBridge do, and when would it be reasonable for it to do that
> something else?

The "SHOULD" requirement is to use a tree that the choosing RBridge
has advertised it will use; however, it is not actually required to
advertise which tree(s) it will use. Furthermore, even if it has, that
tree(s) might just have become unavailable due to one or more
failures.  We can probably add some words to clarify that.

> Nits/editorial comments:
>
> * You use a lot of domain-specific acronyms in section 1 before saying what
> they mean in section 2.

Looks to me like the terminology section could be moved up.

> * The first sentence in the 8th paragraph of 1.2 is very
> complex. (It's the one that starts "In cases where blocks
> of"). Please consider simplifying it.

I think it can be re-worded.

> * Section 2: (I'm no fun) Do you want this alternate expansion of
> FGL to stand?

Nope... Looks like a global replace run amok or something, that should
be fixed :-)

> * Figure 2: the left table has a VLAN of 4095, which is inconsistent
> with the prose.

Shold be fixed. 4095 (0xFFF) is not a valid VLAN ID.

> * In section 3.4 you use 2119 RECOMMENDED (which is equivalent to
> SHOULD) when describing how the operator configures things. This
> isn't a constraint on the protocol defined in this document. Please
> consider rewriting the sentence without the 2119 keyword.

Humm. I think those are good operational recommendations. We can try
changing "RECOMMEND" to "suggest" and see if we get push back to
change it back :-)

> * Micronits: there's spurious space at the beginning of t

Re: [Gen-art] review of draft-ietf-trill-irb-13.txt

[Jari Arkko]

Francis, Donald, thank you for the review & updates.

Jari



signature.asc
Description: Message signed with OpenPGP using GPGMail
___
Gen-art mailing list
Gen-art@ietf.org
https://www.ietf.org/mailman/listinfo/gen-art

Re: [Gen-art] draft-ietf-calext-extensions-03 (was: RE: Assignments for the 2016-06-30 Telechat)

[Romascanu, Dan (Dan)]

Perfect timing. My only pending issue is addressed. From the Gen-ART point pf 
view this document is Ready. 

Thanks and Regards,

Dan


> -Original Message-
> From: Cyrus Daboo [mailto:cy...@daboo.name]
> Sent: Wednesday, June 29, 2016 5:04 PM
> To: Romascanu, Dan (Dan); A. Jean Mahoney; General Area Review Team
> Cc: draft-ietf-calext-extensions@tools.ietf.org
> Subject: Re: draft-ietf-calext-extensions-03 (was: RE: [Gen-art] Assignments
> for the 2016-06-30 Telechat)
> 
> Hi Dan,
> 
> --On June 29, 2016 at 1:44:03 PM + "Romascanu, Dan (Dan)"
>  wrote:
> 
> > Hi,
> >
> > There was no update to this document since my previous review. My
> > original review raised two issues - one was clarified by explanations
> > from Alexey (thanks!); the other is about the document status (should
> > it be marked as 'Updates RFC 5545'?). I did not see that addressed. It
> > would be nice to get an answer, but I do not believe this is a show-stopper.
> 
> A new -04 version of the draft has just been published with your review
> issues addressed.
> 
> 
> --
> Cyrus Daboo

___
Gen-art mailing list
Gen-art@ietf.org
https://www.ietf.org/mailman/listinfo/gen-art

Re: [Gen-art] draft-ietf-calext-extensions-03 (was: RE: Assignments for the 2016-06-30 Telechat)

[Cyrus Daboo]

Hi Dan,

--On June 29, 2016 at 1:44:03 PM + "Romascanu, Dan (Dan)" 
 wrote:



Hi,

There was no update to this document since my previous review. My
original review raised two issues - one was clarified by explanations
from Alexey (thanks!); the other is about the document status (should it
be marked as 'Updates RFC 5545'?). I did not see that addressed. It would
be nice to get an answer, but I do not believe this is a show-stopper.


A new -04 version of the draft has just been published with your review 
issues addressed.



--
Cyrus Daboo

___
Gen-art mailing list
Gen-art@ietf.org
https://www.ietf.org/mailman/listinfo/gen-art

Re: [Gen-art] draft-ietf-calext-extensions-03 (was: RE: Assignments for the 2016-06-30 Telechat)

[Alexey Melnikov]

On 29/06/2016 14:44, Romascanu, Dan (Dan) wrote:


Hi,

There was no update to this document since my previous review. My original 
review raised two issues - one was clarified by explanations from Alexey 
(thanks!); the other is about the document status (should it be marked as 
'Updates RFC 5545'?). I did not see that addressed. It would be nice to get an 
answer, but I do not believe this is a show-stopper.

Hi Dan,
Ben incorporated your question into his DISCUSS, so this will either be 
addressed or you will get an answer from author arguing otherwise.


Best Regards,
Alexey

___
Gen-art mailing list
Gen-art@ietf.org
https://www.ietf.org/mailman/listinfo/gen-art

[Gen-art] draft-ietf-calext-extensions-03 (was: RE: Assignments for the 2016-06-30 Telechat)

[Romascanu, Dan (Dan)]

Hi,

There was no update to this document since my previous review. My original 
review raised two issues - one was clarified by explanations from Alexey 
(thanks!); the other is about the document status (should it be marked as 
'Updates RFC 5545'?). I did not see that addressed. It would be nice to get an 
answer, but I do not believe this is a show-stopper. 

Regards,

Dan


> -Original Message-
> From: Gen-art [mailto:gen-art-boun...@ietf.org] On Behalf Of A. Jean
> Mahoney
> Sent: Friday, June 24, 2016 1:34 AM
> To: General Area Review Team
> Subject: [Gen-art] Assignments for the 2016-06-30 Telechat
> 
> 
> Dan Romascanu 16-06-22 draft-ietf-calext-extensions-03 **
> 

___
Gen-art mailing list
Gen-art@ietf.org
https://www.ietf.org/mailman/listinfo/gen-art