Hi Murray,

Thank you for your comments! Please find the diff and the responses in line 
below. Thank you!

Diff: 
https://tools.ietf.org/tools/rfcdiff/rfcdiff.pyht?url1=draft-ietf-pce-pcep-extension-for-pce-controller-12&url2=https://raw.githubusercontent.com/dhruvdhody/ietf/master/draft-ietf-pce-pcep-extension-for-pce-controller-13.txt
 


-----Original Message-----
From: Murray Kucherawy via Datatracker [mailto:nore...@ietf.org] 
Sent: Thursday, February 25, 2021 3:42 PM
To: The IESG <i...@ietf.org>
Cc: draft-ietf-pce-pcep-extension-for-pce-control...@ietf.org; 
pce-cha...@ietf.org; pce@ietf.org; Julien Meuric <julien.meu...@orange.com>
Subject: Murray Kucherawy's No Objection on 
draft-ietf-pce-pcep-extension-for-pce-controller-12: (with COMMENT)

Murray Kucherawy has entered the following ballot position for
draft-ietf-pce-pcep-extension-for-pce-controller-12: No Objection

When responding, please keep the subject line intact and reply to all email 
addresses included in the To and CC lines. (Feel free to cut this introductory 
paragraph, however.)


Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html
for more information about IESG DISCUSS and COMMENT positions.


The document, along with other ballot positions, can be found here:
https://datatracker.ietf.org/doc/draft-ietf-pce-pcep-extension-for-pce-controller/



----------------------------------------------------------------------
COMMENT:
----------------------------------------------------------------------

In Section 2, RFC 8283 isn't a "draft".

Shuping> Removed!

In Section 5.5.1:

   Once the label operations are completed, the PCE SHOULD send a PCUpd
   message to the ingress PCC.

Why "SHOULD"?  Is there another option?   Why might an implementer do something 
else?

Shuping> Changed to MUST. Thanks for spotting this!

The SHOULDs elsewhere in Section 5 are probably worth a second look too.

Shuping> Ack

Best Regards, 
Shuping 

_______________________________________________
Pce mailing list
Pce@ietf.org
https://www.ietf.org/mailman/listinfo/pce

Reply via email to