Re: [Pce] Review of draft-ietf-pce-stateful-pce-18

<lionel.morand@orange.com> Wed, 12 April 2017 16:29 UTC

Return-Path: <lionel.morand@orange.com>
X-Original-To: pce@ietfa.amsl.com
Delivered-To: pce@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 21789131786; Wed, 12 Apr 2017 09:29:01 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.62
X-Spam-Level:
X-Spam-Status: No, score=-2.62 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, RP_MATCHES_RCVD=-0.001, SPF_PASS=-0.001, UNPARSEABLE_RELAY=0.001] autolearn=ham autolearn_force=no
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id dD_RJ7Ol2KMy; Wed, 12 Apr 2017 09:28:59 -0700 (PDT)
Received: from relais-inet.orange.com (mta239.mail.business.static.orange.com [80.12.66.39]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 2E1C713178B; Wed, 12 Apr 2017 09:28:57 -0700 (PDT)
Received: from opfedar03.francetelecom.fr (unknown [xx.xx.xx.5]) by opfedar27.francetelecom.fr (ESMTP service) with ESMTP id 5AC706045B; Wed, 12 Apr 2017 18:28:55 +0200 (CEST)
Received: from Exchangemail-eme2.itn.ftgroup (unknown [xx.xx.31.2]) by opfedar03.francetelecom.fr (ESMTP service) with ESMTP id 25351180066; Wed, 12 Apr 2017 18:28:55 +0200 (CEST)
Received: from OPEXCLILM43.corporate.adroot.infra.ftgroup ([fe80::ec23:902:c31f:731c]) by OPEXCLILM21.corporate.adroot.infra.ftgroup ([fe80::e92a:c932:907e:8f06%19]) with mapi id 14.03.0319.002; Wed, 12 Apr 2017 18:28:52 +0200
From: lionel.morand@orange.com
To: Jonathan Hardwick <Jonathan.Hardwick@metaswitch.com>, MEURIC Julien IMT/OLN <julien.meuric@orange.com>, "ops-dir@ietf.org" <ops-dir@ietf.org>
CC: "draft-ietf-pce-stateful-pce.all@ietf.org" <draft-ietf-pce-stateful-pce.all@ietf.org>, "pce@ietf.org" <pce@ietf.org>, "ietf@ietf.org" <ietf@ietf.org>
Thread-Topic: [Pce] Review of draft-ietf-pce-stateful-pce-18
Thread-Index: AQHSssnNDQvmKPrPVk6JDOdpTPcg8qHAIZWAgAAHxYCAAbQL8A==
Date: Wed, 12 Apr 2017 16:28:51 +0000
Message-ID: <26612_1492014535_58EE55C7_26612_8402_1_d18aea96-35a8-419a-b7fe-9f6231f2aaaf@OPEXCLILM21.corporate.adroot.infra.ftgroup>
References: <148965756308.14230.13426886469262710918@ietfa.amsl.com> <BY2PR0201MB1910B9060DF50A938DC05D6984000@BY2PR0201MB1910.namprd02.prod.outlook.com> <2ff156bc-c198-80fe-eccf-b45b6db978df@orange.com> <BY2PR0201MB19101ABC5D53474ED6DB96B484000@BY2PR0201MB1910.namprd02.prod.outlook.com>
In-Reply-To: <BY2PR0201MB19101ABC5D53474ED6DB96B484000@BY2PR0201MB1910.namprd02.prod.outlook.com>
Accept-Language: fr-FR, en-US
Content-Language: fr-FR
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.168.234.3]
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Archived-At: <https://mailarchive.ietf.org/arch/msg/pce/03aw9aI_X2gS3sEMt33MPr-NI6I>
Subject: Re: [Pce] Review of draft-ietf-pce-stateful-pce-18
X-BeenThere: pce@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: Path Computation Element <pce.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/pce>, <mailto:pce-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/pce/>
List-Post: <mailto:pce@ietf.org>
List-Help: <mailto:pce-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/pce>, <mailto:pce-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 12 Apr 2017 16:29:01 -0000

Hi Julien, Jon,

Thank you for your feedback.
Please see below my answer/feedback (indicated with [LM2])

Regards,

Lionel
 
> Hi Julien
> 
> You are right, but it is *really easy* for the reader to confuse "stateful
> capability" with "update capability" and "active stateful capability".  Case in
> point: I just confused them in my reply to Lionel.
> 
> We should fix each of the three points below to make this clearer.

[LM2] I agree :)

> Jon, Lionel,
> 
> I believe Lionel got confused by the wording introduced in RFC 8051:
> - no report, no update means stateless PCE;
> - report, no update means passive stateful PCE;
> - report and update means active (stateful) PCE.

[LM2] understood!
> 
> > =====
> >
> > [LM] active/passive mode are not  advertized in PCEP. s/if active
> > stateful PCE capability was not advertised/if stateful PCE capability
> > was not advertised
> >
> > Jon> ACK
> >
> > =====
> [JM] NACK! ;-)
> Actually, the passive mode is advertised using the Stateful-capability-object TLV
> with the U bit unset, the active mode by setting the U bit.
> 
[LM2] "il faut être sorti de Saint-Cyr pour comprendre" as we say in french :)
Could be good to add something like "(as indicated by the U-bit clear in Stateful-capability-object)"

> > =====
> >
> > Note that even if the update capability has not been advertised, a PCE
> > can still accept LSP Status Reports from a PCC and build and maintain
> > an up to date view of the state of the PCC's LSPs.
> >
> > [LM] I don't undersand. Is it not in contradiction with
> >
> > "If the PCEP Speaker on the PCE supports the extensions of this draft
> > but did not advertise this capability, then upon receipt of a PCRpt
> > message from the PCC, it MUST generate a PCErr with error- type
> > 19 (Invalid Operation), error-value 5 (Attempted LSP State Report if
> > active stateful PCE capability was not advertised) (see Section
> > 8.5) and it SHOULD terminate the PCEP session."
> >
> > Or does it mean that there is another way than PCRpt message for the
> > PCC to send LSP status reports to the PCE?
> >
> > Jon> ACK.  I think that the statement in the draft is bogus and I
> > propose to delete this sentence from it.
> >
> > =====
> [JM] I do not think that the text is bogus:
> - case 1: no advertised capability on update but advertised on report (i.e. passive
> stateful) => no error message;
> - case 2: no advertised capability on update nor report (i.e. stateless) => error.

[LM2] After multiple readings and thanks to your explanation, I think I have understood. Am I correct saying that the PCE will accept LSP Status Reports from a PCC ONLY if the stateful PCE capability has been advertised (i.e. Stateful Capability TLV with the 'LSP Update' Flag cleared)? If it is the case, is it really required to keep this text, as in the previous paragraph we find the conditions to accept/reject reports from the PCC?
> 
> > =====
> >
> > [LM] Would it be useful to discover (using another TLV) whether the
> > PCE is an active/passive stateful PCE, as in IGP-based capabilities
> > discovery mechanism?
> >
> > Jon> This can be inferred immediately from the U flag in the
> > STATEFUL-PCE-CAPABILITY TLV.  Passive mode is synonymous with not
> > sending / handling PCUpd messages.
> >
> > =====
> [JM] The mechanism is there, but section 7.1.1 may deserve an explicit use of
> the "passive/active" terms, to make sure the capability terminology is aligned
> with the vocabulary in the IGP section.

[LM2] I can only be agree with you :)


_________________________________________________________________________________________________________________________

Ce message et ses pieces jointes peuvent contenir des informations confidentielles ou privilegiees et ne doivent donc
pas etre diffuses, exploites ou copies sans autorisation. Si vous avez recu ce message par erreur, veuillez le signaler
a l'expediteur et le detruire ainsi que les pieces jointes. Les messages electroniques etant susceptibles d'alteration,
Orange decline toute responsabilite si ce message a ete altere, deforme ou falsifie. Merci.

This message and its attachments may contain confidential or privileged information that may be protected by law;
they should not be distributed, used or copied without authorisation.
If you have received this email in error, please notify the sender and delete this message and its attachments.
As emails may be altered, Orange is not liable for messages that have been modified, changed or falsified.
Thank you.