Hi Giuseppe, Thanks for letting the WG know about your I-D.
<wg-contributor> I have some suggestions to improve this I-D - (0) Can these TLVs be carried inside the LSPA object, and allow the IFIT to be used for all paths (including SR, SRv6, RSVP-TE, PCECC, etc). I understand that your key case is with SR-Policy but it is better to put the TLVs in a generic object rather than in an SR specific one. I am guessing you might be influenced by BGP SR Policy (where it makes sense to limit it to SR-Policy as BGP is used only for SR-Policy) but in the case of PCEP, making it generic is a better idea IMHO. (1) Add capability exchange between PCEP speakers by adding a flag in SR-PCE-CAPABILITY Sub-TLV. PCE WG has a long history of explicit capability exchange in open before using new features. (2) You need to re-write section 5.1, it does not come across as an example - it describes some new procedure to handle SR Policy that is not aligned with the rest of PCE documents. I suggest adding an example figure with a PCInitiate message from PCE to PCC carrying the SR Policy/Candidate Path/IFIT attributes to explain the flow with an example. (3) The document is light on motivation, you say "So that IFIT behavior can be enabled automatically when the SR policy is applied". I wish you can expand on this a little more. Something on lines of - When a PCE is used to initiate paths using PCEP, it is important that the head end of the path also understands the IFIT behavior that is intended for the path. When PCEP is in use for path initiation it makes sense for that same protocol to be used to also carry the IFIT attributes that describe the IOAM header/procedure that needs to be applied to the data that flow those paths. (4) Only one of these TLVs can be enabled right? This needs to be described. Also, add text to describe disabling of IFIT or making changes. Some tightening of the procedure is needed here. (5) PCE would need to maintain the uniqueness of Flow ID, FlowMonID etc. This can be highlighted with a reference to other I-Ds that describe the procedure. (6) It would be good to also point to the registries created by other documents that we would be re-using. I hope you find this useful. I would encourage the WG to provide their thoughts as well. </wg-contributor> Thanks! Dhruv On Fri, Jul 10, 2020 at 8:21 PM Giuseppe Fioccola <[email protected]> wrote: > > Dear All, > We have recently updated draft-chen-pce-sr-policy-ifit. It aims to define the > Extensions to PCEP to enable automatically IOAM and Alternate Marking when > the SR policy is applied. In particular we have clarified better the PCEP > usage in this scenario. > > Feedbacks and suggestions are always welcome. > > Best Regards, > > Giuseppe > > -----Original Message----- > From: [email protected] [mailto:[email protected]] > Sent: Friday, July 10, 2020 2:28 PM > To: Giuseppe Fioccola <[email protected]>; Hang Yuan > <[email protected]>; Huanan Chen <[email protected]>; wangyali > <[email protected]>; Liweidong (Poly) <[email protected]>; Tianran Zhou > <[email protected]> > Subject: New Version Notification for draft-chen-pce-sr-policy-ifit-02.txt > > > A new version of I-D, draft-chen-pce-sr-policy-ifit-02.txt > has been successfully submitted by Giuseppe Fioccola and posted to the IETF > repository. > > Name: draft-chen-pce-sr-policy-ifit > Revision: 02 > Title: PCEP SR Policy Extensions to Enable IFIT > Document date: 2020-07-10 > Group: Individual Submission > Pages: 12 > URL: > https://www.ietf.org/internet-drafts/draft-chen-pce-sr-policy-ifit-02.txt > Status: > https://datatracker.ietf.org/doc/draft-chen-pce-sr-policy-ifit/ > Htmlized: https://tools.ietf.org/html/draft-chen-pce-sr-policy-ifit-02 > Htmlized: > https://datatracker.ietf.org/doc/html/draft-chen-pce-sr-policy-ifit > Diff: > https://www.ietf.org/rfcdiff?url2=draft-chen-pce-sr-policy-ifit-02 > > Abstract: > Segment Routing (SR) policy is a set of candidate SR paths consisting > of one or more segment lists and necessary path attributes. It > enables instantiation of an ordered list of segments with a specific > intent for traffic steering. In-situ Flow Information Telemetry > (IFIT) refers to network OAM applications that apply dataplane on- > path telemetry techniques. This document defines extensions to PCEP > to distribute SR policies carrying IFIT information. So that IFIT > behavior can be enabled automatically when the SR policy is applied. > > > > > > Please note that it may take a couple of minutes from the time of submission > until the htmlized version and diff are available at tools.ietf.org. > > The IETF Secretariat > > > _______________________________________________ > Pce mailing list > [email protected] > https://www.ietf.org/mailman/listinfo/pce _______________________________________________ Pce mailing list [email protected] https://www.ietf.org/mailman/listinfo/pce
