[Gen-art] 答复: Review of draft-ietf-ccamp-flexible-grid-ospf-ext-08

[Zhenghaomian <[e-mail]>]

Hi, Pete, 

Thanks for your review. Please find my considerations inline...

Best wishes,
Haomian

-----邮件原件-----
发件人: Pete Resnick [mailto:[e-mail]] 
发送时间: 2017年2月11日 7:24
收件人: [e-mail]
抄送: [e-mail]; [e-mail]
主题: 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: