[sidr] Review of draft-ietf-sidr-publication-09

2017-01-06 Thread Peter Yee
Reviewer: Peter Yee Review result: Ready with Nits 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

Re: [sidr] draft-ietf-sidr-bgpsec-protocol

2017-01-06 Thread Randy Bush
re keyur's point 4. the issue is that 4271 has semantics of AS_PATH, bgpsec replaces it with BGPSEC_PATH, but bgpsec does not explicitly repeat things such as loop detection using BGPSEC_PATH. instead of this becoming another text explosion, a simple statement that, in bgpsec, the following

Re: [sidr] draft-ietf-sidr-bgpsec-protocol

2017-01-06 Thread Alvaro Retana (aretana)
On 1/6/17, 12:40 AM, "Sriram, Kotikalapudi (Fed)" wrote: [Cut the distribution list a little.] Sriram: Hi! Happy New Year! I have some comments on this, please see below. Thanks! Alvaro. … | | 1) Section 4.1 “The BGPsec Path attribute and

Re: [sidr] AD Review of draft-ietf-sidr-delta-protocol-04

2017-01-06 Thread Alvaro Retana (aretana)
On 12/28/16, 11:43 AM, "Tim Bruijnzeels" wrote: Tim: Happy New Year!! | Thank you for your in-depth review :) It really helps to clarify the document. Replies in-line. | | I attached an updated document, but.. I did not discuss this with co-authors yet. So, I invite

Re: [sidr] draft-ietf-sidr-bgpsec-protocol

2017-01-06 Thread Sriram, Kotikalapudi (Fed)
My previous post of this message seems to have line wrap issue. Resending ... hopefully this avoids that issue. - Sriram --- Keyur, Thank you for taking the time to read and offer comments. I find the comments very insightful and helpful. My comments are marked with