Hi Weiqiang,
Thanks for your support and comments!
Please see inline with [Quan].
Here I have some minor comments for the authors' consideration:
1. Section 2.1: Terminology
Instead of solely referencing external documents for terminology, it would be
beneficial to compile a dedicated list of terms used within this draft.
Furthermore, it's advisable to ensure that all terms, such as "LSR," are
provided with their full names upon first mention.
[Quan]:Thanks for your remind. I will check the draft and provide the full
names. But this draft did not propose new terms other than existing RFCs.
2. Entropy Label Position (ELP)
To maintain consistency and clarity, I suggest introducing the term "Entropy
Label Position (ELP)" upon its initial mention in the document, followed by the
abbreviated form "ELP" for subsequent references.
[Quan]:Agree with you. ELP has been described and referenced to
[I-D-ietf-idr-bgp-srmpls-elp] based on Xuesong's comment.
3. PCRep, PCUpd, or PCInitiate Messages
In Section 4.2, it would be beneficial to include a brief description of the
original documents that define terms such as PCRep, PCUpd, or PCInitiate
messages.
[Quan]: Thanks, I will add the according documents in next version.
4. Section 5: Operations
I agree with Xuesong's comments to rename the section title to "Operational
Example" for improved clarity. Additionally, I recommend expanding the
description to encompass the behaviors associated with different roles,
providing a more comprehensive understanding.
[Quan]: Thanks, I will add behaviors description from PCC and PCE in next
version.
Best Regards,
Quan
Original
From: WeiqiangCheng <chengweiqi...@chinamobile.com>
To: dd <d...@dhruvdhody.com>;pce <pce@ietf.org>;
Cc: pce-chairs <pce-cha...@ietf.org>;draft-peng-pce-entropy-label-position
<draft-peng-pce-entropy-label-posit...@ietf.org>;
Date: 2024年02月06日 17:26
Subject: Re: [Pce] WG Adoption of draft-peng-pce-entropy-label-position-10
Hi,
I've reviewed the draft and I support the adoption. This draft proposes a set
of extensions for PCEP to configure the ELP information for SR-MPLS networks.
It can effectively promote the application of entropy label in centralized
control scenarios.
Here I have some minor comments for the authors' consideration:
1. Section 2.1: Terminology
Instead of solely referencing external documents for terminology, it would be
beneficial to compile a dedicated list of terms used within this draft.
Furthermore, it's advisable to ensure that all terms, such as "LSR," are
provided with their full names upon first mention.
2. Entropy Label Position (ELP)
To maintain consistency and clarity, I suggest introducing the term "Entropy
Label Position (ELP)" upon its initial mention in the document, followed by the
abbreviated form "ELP" for subsequent references.
3. PCRep, PCUpd, or PCInitiate Messages
In Section 4.2, it would be beneficial to include a brief description of the
original documents that define terms such as PCRep, PCUpd, or PCInitiate
messages.
4. Section 5: Operations
I agree with Xuesong's comments to rename the section title to "Operational
Example" for improved clarity. Additionally, I recommend expanding the
description to encompass the behaviors associated with different roles,
providing a more comprehensive understanding.
Best regards,
Weiqiang Cheng
From: Dhruv Dhody
Date: 2024-01-27 01:19
To: pce
CC: pce-chairs; draft-peng-pce-entropy-label-position
Subject: [Pce] WG Adoption of draft-peng-pce-entropy-label-position-10
Hi WG,
This email begins the WG adoption poll for
draft-peng-pce-entropy-label-position-10
https://datatracker.ietf.org/doc/draft-peng-pce-entropy-label-position/
Should this draft be adopted by the PCE WG? Please state your reasons - Why /
Why not? What needs to be fixed before or after adoption? Are you willing to
work on this draft? Review comments should be posted to the list.
Please respond by Monday 12th Feb 2024.
Please be more vocal during WG polls!
Thanks!
Dhruv & Julien
_______________________________________________
Pce mailing list
Pce@ietf.org
https://www.ietf.org/mailman/listinfo/pce
