Re: [Pce] FW: New Version Notification for draft-chen-pce-sr-policy-ifit-02.txt

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
<giuseppe.fioccola@huawei.com> 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: internet-drafts@ietf.org [mailto:internet-drafts@ietf.org]
> Sent: Friday, July 10, 2020 2:28 PM
> To: Giuseppe Fioccola <giuseppe.fioccola@huawei.com>; Hang Yuan <yuanhang@unionpay.com>; Huanan Chen <chenhuan6@chinatelecom.cn>; wangyali <wangyali11@huawei.com>; Liweidong (Poly) <poly.li@huawei.com>; Tianran Zhou <zhoutianran@huawei.com>
> 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
> Pce@ietf.org
> https://www.ietf.org/mailman/listinfo/pce