Robert Sparks | 28 Jun 21:33 2016

Genart LC (and likely telechat) review : draft-ietf-trill-tree-selection-04

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>.

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. 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. 
(Continue reading)

Paul Kyzivat | 28 Jun 00:46 2016
Picon

Gen-ART Last Call review of draft-ietf-trill-ia-appsubtlv

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>.

Document: draft-ietf-trill-ia-appsubtlv
Reviewer: Paul Kyzivat
Review Date: 2016-06-27
IETF LC End Date: 2016-06-28
IESG Telechat date: 2016-07-07

Summary:

This draft is on the right track but has open issues, described in the 
review.

This is a well written document. I was generally able to follow it even 
though I know nothing about the subject.

Issues:

Major: 0
Minor: 7
Nits:  2

(1) MINOR: (Section 2)

"Addr Sets End" is described as follows:

(Continue reading)

Russ Housley | 24 Jun 23:41 2016

Gen-ART Review of draft-ietf-grow-blackholing-00

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
<http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.

Document: draft-ietf-grow-blackholing-00
Reviewer: Russ Housley
Review Date: 2016-06-24
IETF LC End Date: 2016-07-04
IESG Telechat date: unknown

Summary:  Ready

Major Concerns: None

Minor Concerns: None

Nits:

In Section 1: s/next- hop IP addresses/next-hop IP addresses/

The requested value from IANA is quite appropriate.  ;-)
Russ Housley | 24 Jun 22:10 2016

Gen-ART Review of draft-ietf-forces-interfelfb-05

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
<http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.

Document: draft-ietf-forces-interfelfb-05
Reviewer: Russ Housley
Review Date: 2016-06-24
IETF LC End Date: 2016-06-22
IESG Telechat date: 2016-06-30

Summary:  Ready

Thanks for addressing the comments that I raised on
draft-ietf-forces-interfelfb-04.
Brian E Carpenter | 24 Jun 04:48 2016
Picon

Gen-ART telechat review of draft-ietf-alto-deployments-15

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
<http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.

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.
(Continue reading)

A. Jean Mahoney | 24 Jun 00:33 2016

Assignments for the 2016-06-30 Telechat

Hi all,

The following reviewers have assignments:

Reviewer          LC end   Draft
-------------------------------------------------------------------------
Brian Carpenter   16-06-21 draft-ietf-alto-deployments-15 **

Christer Holmberg 16-06-21 draft-ietf-mile-implementreport-09 **

Dan Romascanu     16-06-22 draft-ietf-calext-extensions-03 **

Francis Dupont    16-06-24 draft-ietf-trill-irb-13 **

Fernando Gont     16-06-23 draft-ietf-ospf-transition-to-ospfv3-09

Russ Housley      16-06-22 draft-ietf-forces-interfelfb-05 *

* Earlier draft reviewed
** Already reviewed

I have made the assignments in the review tool:
http://art.tools.ietf.org/tools/art/genart/

And the assignments are captured in the spreadsheets:
http://wiki.tools.ietf.org/dav/genart/gen-art.html
http://wiki.tools.ietf.org/dav/genart/gen-art-by-reviewer.html

For your convenience, the review boilerplate template is included below.

(Continue reading)

A. Jean Mahoney | 24 Jun 00:31 2016

A *new* batch of IETF LC reviews - 2016-06-23

Hi all,

The following reviewers have assignments:

Reviewer          LC end      Draft
---------------------------------------------------------------------
Brian Carpenter   2016-07-06  draft-ietf-nfsv4-multi-domain-fs-reqs-8

Christer Holmberg 2016-07-07  draft-ietf-calext-availability-03 *

Dan Romascanu     2016-07-21  draft-hardy-pdf-mime-02

Russ Housley      2016-07-04  draft-ietf-grow-blackholing-00

On the 7/7 Telechat

I have made the assignments in the review tool:
http://art.tools.ietf.org/tools/art/genart/

And the assignments are captured in the spreadsheets:
http://wiki.tools.ietf.org/dav/genart/gen-art.html
http://wiki.tools.ietf.org/dav/genart/gen-art-by-reviewer.html

The updated template is included below.

Thanks,

Jean

-------------------------------------------------------------------
(Continue reading)

Francis Dupont | 22 Jun 16:22 2016
Picon

review of draft-ietf-trill-irb-13.txt

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>.

Document: draft-ietf-trill-irb-13.txt 
Reviewer: Francis Dupont
Review Date: 20160618
IETF LC End Date: 20160624
IESG Telechat date: 20150630

Summary: Ready

Major issues: None

Minor issues: None

Nits/editorial comments:
 - you have 6 (with a soft limit of 5) authors

 - 1 page 3: please expand the first occurrence of the FGL abbrev

  (in theory you should expand the TRILL abbrev and in the Abstract too
   but IMHO it is unlikely to get a reader of this document who does
   not know the TRILL abbrev meaning and raises a concern. BTW VLAN
   is well known (*) so doesn't need to be expanded)
(Continue reading)

Romascanu, Dan (Dan | 21 Jun 13:48 2016

Gen-ART LC review for draft-ietf-calext-extensions-03

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.tools.ietf.org/area/gen/trac/wiki/GenArtfaq >.

 

Document: draft-ietf-calext-extensions-03

Reviewer: Dan Romascanu

Review Date: 6/21/16

IETF LC End Date: 6/22/16

IESG Telechat date: 7/7/16

 

Summary: A clear document, almost ready from the Gen-ART point of view. I recommend to clarify the two minor issues below before approval.

 

Major issues:

 

Minor issues:

 

1.       Should not this document be marked as ‘Updates RFC 5545’? The document recommends that all properties not defined in 5545 always include a “VALUE” parameter if the type is other than “TEXT” It also modifies some existing properties and defines new properties and elements that bring into the standard extension elements added by vendors into their specification.

2.       I had a hard time understanding some of the details in 5.7 (REFRESH-INTERVAL Property). These may be however due to my lack of familiarity with the style of iCal definitions and notations – so only explanations may be sufficient. First I do not understand what is covered under Property Parameters: what means ‘IANA and non-standard property parameters’ that can be defined here – and why here? Maybe an example would help. Second in the format definition: dur-value … consisting of a positive duration of time – should not units be specified here as well? Last, I did not understand VALUE=DURATION:P1W – maybe it’s just me missing the notation

 

Nits/editorial comments:

 

 

_______________________________________________
Gen-art mailing list
Gen-art <at> ietf.org
https://www.ietf.org/mailman/listinfo/gen-art
A. Jean Mahoney | 21 Jun 00:00 2016

A *new* batch of IETF LC reviews - 2016-06-20

Hi all,

The following reviewers have assignments:

Reviewer        LC end      Draft
---------------------------------------------------------------------
Paul Kyzivat   16-06-28  draft-ietf-trill-ia-appsubtlv-08 *

Pete Resnick   16-07-01  draft-ietf-trill-arp-optimization-06 *

Peter Yee      16-07-01  draft-ietf-trill-channel-tunnel-09 *

Ralph Droms    16-07-18  draft-holmberg-dispatch-rfc7315-updates-06

Robert Sparks  16-07-01  draft-ietf-trill-tree-selection-04 *

Ron Bonica     16-07-04  draft-ietf-sipcore-dns-dual-stack-06

Roni Even    16-07-04  draft-ietf-pals-mpls-tp-pw-over-bidir-lsp-08 *

* on the 07/07 Telechat

I have made the assignments in the review tool:
http://art.tools.ietf.org/tools/art/genart/

And the assignments are captured in the spreadsheets:
http://wiki.tools.ietf.org/dav/genart/gen-art.html
http://wiki.tools.ietf.org/dav/genart/gen-art-by-reviewer.html

The updated template is included below.

Thanks,

Jean

-------------------------------------------------------------------

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>.

Document:
Reviewer:
Review Date:
IETF LC End Date:
IESG Telechat date: (if known)

Summary:

Major issues:

Minor issues:

Nits/editorial comments:
A. Jean Mahoney | 16 Jun 22:25 2016

A *new* batch of IETF LC reviews - 2016-06-16

Hi all,

The following reviewers have assignments:

Reviewer          LC end      Draft
---------------------------------------------------------------------
Francis Dupont    2016-06-24  draft-ietf-trill-irb-13 *

Joel Halpern      2016-07-08  draft-pd-dispatch-msrp-websocket-12

Jouni Korhonen    2016-06-28  draft-ietf-trill-ia-appsubtlv-08

Matt Miller       2016-07-11  draft-sweet-rfc2910bis-08

Meral Shirazipour 2016-06-27  draft-ietf-cdni-metadata-18

* on the 6/30 Telechat

I have made the assignments in the review tool:
http://art.tools.ietf.org/tools/art/genart/

And the assignments are captured in the spreadsheets:
http://wiki.tools.ietf.org/dav/genart/gen-art.html
http://wiki.tools.ietf.org/dav/genart/gen-art-by-reviewer.html

The updated template is included below.

Thanks,

Jean

-------------------------------------------------------------------

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>.

Document:
Reviewer:
Review Date:
IETF LC End Date:
IESG Telechat date: (if known)

Summary:

Major issues:

Minor issues:

Nits/editorial comments:

Gmane