Hi, Pete, Thanks for your review. Please find my considerations inline...
Best wishes, Haomian -----邮件原件----- 发件人: Pete Resnick [mailto:[email protected]] 发送时间: 2017年2月11日 7:24 收件人: [email protected] 抄送: [email protected]; [email protected] 主题: Review of draft-ietf-ccamp-flexible-grid-ospf-ext-08 Reviewer: Pete Resnick 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 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 <https://trac.ietf.org/trac/gen/wiki/GenArtfaq>. Document: draft-ietf-ccamp-flexible-grid-ospf-ext-08 Reviewer: Pete Resnick Review Date: 2017-02-10 IETF LC End Date: 2017-01-31 IESG Telechat date: 2017-02-16 Summary: Ready with Nits A couple of nits that I mentioned in my earlier review that you might want to address, but none of them are essential. (You may have decided that I was wrong; that's OK too.) I didn't bother Cc'ing the IETF list on this, since they're both very minor. Major issues: None Minor issues: None Nits/editorial comments: 3.1: A set of non-overlapping available frequency ranges MUST be disseminated in order to allow efficient resource management of flexi-grid DWDM links and RSA procedures which are described in Section 4.8 of [RFC7698]. Those MUSTs look weird to me. I think instead of "MUST be" you mean "are", since it doesn't look like an implementation really has a choice here. [Haomian] Noted and will change in next version. 3.2: Hence, in order to support all possible applications and implementations the following information should be advertised for a flexi-grid DWDM link: Is that "should" in there meant to be normative? That is, do bad things happen if I don't advertise one of those items? Or do you just mean "the following information is advertised..."? [Haomian] I feel weird if replace 'should be' with 'is', as you cannot support some application/implementation (rather than do bad things) if you don't advertise... How about following change? OLD Hence, in order to support all possible applications and implementations the following information should be advertised for a flexi-grid DWDM link: NEW Hence, in order to support all possible applications and implementations the following information is required to be advertised for a flexi-grid DWDM link: _______________________________________________ Gen-art mailing list [email protected] https://www.ietf.org/mailman/listinfo/gen-art
