Re: [Lsr] draft-ietf-ospf-yang

"Acee Lindem (acee)" <acee@cisco.com> Fri, 07 December 2018 00:26 UTC

Return-Path: <acee@cisco.com>
X-Original-To: lsr@ietfa.amsl.com
Delivered-To: lsr@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id A9615131238; Thu, 6 Dec 2018 16:26:58 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -15.959
X-Spam-Level:
X-Spam-Status: No, score=-15.959 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-1.46, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_HI=-5, SPF_PASS=-0.001, URIBL_BLOCKED=0.001, USER_IN_DEF_DKIM_WL=-7.5] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=cisco.com
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 a8PdC1cvYLxo; Thu, 6 Dec 2018 16:26:53 -0800 (PST)
Received: from alln-iport-1.cisco.com (alln-iport-1.cisco.com [173.37.142.88]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id D235B13124C; Thu, 6 Dec 2018 16:26:52 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=41936; q=dns/txt; s=iport; t=1544142412; x=1545352012; h=from:to:cc:subject:date:message-id:references: in-reply-to:mime-version; bh=LybatpOEZctFgtCAkNmE2KNOwerOBWip57BBov4IY2s=; b=fExOaO7uu0yl7PEGVjkUUxCGq/RfPHzUacGkMxlVrp1tjY/Dmx39+isC 99Ajkhy9DlwqDGWzpu+lq0sAWDHO2IUfp8J+RGYpRnrNtNcwJwbSZjDxK wAklUhu8bEKLU24+Dns4AVGQDoBsGNUx+h3tFKsgOziZ8uynzHstdT4Wk w=;
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: =?us-ascii?q?A0ADAADEvQlc/5xdJa1jGQEBAQEBAQE?= =?us-ascii?q?BAQEBAQcBAQEBAQGBUQQBAQEBAQsBgQ1NKWaBAicKg3CIGYwQgg2JEo48FIF?= =?us-ascii?q?mCwEBhGwCF4J+IjQJDQEDAQECAQECbSiFPAEBAQECASNEEgUHBAIBCBEDAQE?= =?us-ascii?q?BASAHAwICAh8RFAkIAgQOBYMhAYEdTAMNCKYhgS+IBg2CHIwfF4F/gRABJx+?= =?us-ascii?q?CTIJXgXcBEgE2CQYQgk4xgiYCiREIMoFZg3uGTYpFLgkCjhaDLhiBXIUVikW?= =?us-ascii?q?PCIldAhEUgScfOGRxcBVlAYJBgicXfwEIjRVBMQGJBIEfgR8BAQ?=
X-IronPort-AV: E=Sophos;i="5.56,324,1539648000"; d="scan'208,217";a="209490382"
Received: from rcdn-core-5.cisco.com ([173.37.93.156]) by alln-iport-1.cisco.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 07 Dec 2018 00:26:51 +0000
Received: from XCH-RTP-014.cisco.com (xch-rtp-014.cisco.com [64.101.220.154]) by rcdn-core-5.cisco.com (8.15.2/8.15.2) with ESMTPS id wB70QpMV014589 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=FAIL); Fri, 7 Dec 2018 00:26:51 GMT
Received: from xch-rtp-015.cisco.com (64.101.220.155) by XCH-RTP-014.cisco.com (64.101.220.154) with Microsoft SMTP Server (TLS) id 15.0.1395.4; Thu, 6 Dec 2018 19:26:50 -0500
Received: from xch-rtp-015.cisco.com ([64.101.220.155]) by XCH-RTP-015.cisco.com ([64.101.220.155]) with mapi id 15.00.1395.000; Thu, 6 Dec 2018 19:26:50 -0500
From: "Acee Lindem (acee)" <acee@cisco.com>
To: Xufeng Liu <xufeng.liu.ietf@gmail.com>
CC: "t.petch" <ietfc@btconnect.com>, Stephane Litkowski <stephane.litkowski@orange.com>, "lsr@ietf.org" <lsr@ietf.org>, "draft-ietf-ospf-yang@ietf.org" <draft-ietf-ospf-yang@ietf.org>, "draft-ietf-teas-yang-te-types@ietf.org" <draft-ietf-teas-yang-te-types@ietf.org>
Thread-Topic: draft-ietf-ospf-yang
Thread-Index: AQHUi/kT6LQa9fvqSUmy655qm6w/ZqVx/OCAgACdzAD//9R3gA==
Date: Fri, 7 Dec 2018 00:26:50 +0000
Message-ID: <AF107FDD-E15E-4FE0-826B-38D5CC467842@cisco.com>
References: <576_1542796445_5BF5349D_576_261_1_9E32478DFA9976438E7A22F69B08FF924B7731BE@OPEXCLILMA4.corporate.adroot.infra.ftgroup> <58C71B78-1C6A-4FB5-B64A-7A38628028C1@cisco.com> <19021_1543406661_5BFE8445_19021_254_3_9E32478DFA9976438E7A22F69B08FF924B776CB0@OPEXCLILMA4.corporate.adroot.infra.ftgroup> <31F7DFA5-7BB5-4E79-AFD9-829AE34BC485@cisco.com> <26904_1543488239_5BFFC2EF_26904_436_1_9E32478DFA9976438E7A22F69B08FF924B777AA8@OPEXCLILMA4.corporate.adroot.infra.ftgroup> <00ce01d48bf8$be184980$4001a8c0@gateway.2wire.net> <41B51A9E-9831-4669-AA87-AFA289303B71@cisco.com> <02b901d48c8b$48d5c920$4001a8c0@gateway.2wire.net> <31017_1544014638_5C07CB2E_31017_130_1_9E32478DFA9976438E7A22F69B08FF924B77DE59@OPEXCLILMA4.corporate.adroot.infra.ftgroup> <002201d48cb4$eb6d5580$4001a8c0@gateway.2wire.net> <C13962BC-98F2-4775-8A7C-0DF186B26F4D@cisco.com> <CAEz6PPRMCCtj2yo0RPKnGR-3FadTzt9iuM7Eav_A5fn59enKNw@mail.gmail.com>
In-Reply-To: <CAEz6PPRMCCtj2yo0RPKnGR-3FadTzt9iuM7Eav_A5fn59enKNw@mail.gmail.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-ms-exchange-messagesentrepresentingtype: 1
x-ms-exchange-transport-fromentityheader: Hosted
x-originating-ip: [10.116.152.196]
Content-Type: multipart/alternative; boundary="_000_AF107FDDE15E4FE0826B38D5CC467842ciscocom_"
MIME-Version: 1.0
X-Outbound-SMTP-Client: 64.101.220.154, xch-rtp-014.cisco.com
X-Outbound-Node: rcdn-core-5.cisco.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/lsr/EkiNdbG3xRK9jiRZuBoM42sA4Ws>
Subject: Re: [Lsr] draft-ietf-ospf-yang
X-BeenThere: lsr@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Link State Routing Working Group <lsr.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/lsr>, <mailto:lsr-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/lsr/>
List-Post: <mailto:lsr@ietf.org>
List-Help: <mailto:lsr-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/lsr>, <mailto:lsr-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 07 Dec 2018 00:26:59 -0000

Hi Xufeng,
Sounds good.
Acee

From: Xufeng Liu <xufeng.liu.ietf@gmail.com>
Date: Thursday, December 6, 2018 at 5:03 PM
To: Acee Lindem <acee@cisco.com>
Cc: Tom Petch <ietfc@btconnect.com>om>, Stephane Litkowski <stephane.litkowski@orange.com>om>, "lsr@ietf.org" <lsr@ietf.org>rg>, "draft-ietf-ospf-yang@ietf.org" <draft-ietf-ospf-yang@ietf.org>rg>, "draft-ietf-teas-yang-te-types@ietf.org" <draft-ietf-teas-yang-te-types@ietf.org>
Subject: Re: draft-ietf-ospf-yang

Hi Acee, Tom, and All,

Several authors of draft-ietf-teas-yang-te-types had a brief discussion on this topic. Our take on the te-node-id and te-router-id is:

- In TEAS, the te-node-id specified in draft-ietf-teas-yang-te-types has a wider use scope than IP MPLS TE. The system may or may not run OSPF TE, and may not use IPv4. The 32-bit ID number is used only for uniquely identifying the TE node, and it may or may not be a routable address.
- When RFC3630 is implemented, it is ok to map a routable IPv4 address (such as the address of loopbak0) to the te-node-id, but it is not required.
- We intentionally use the term "te-node-id" instead of "te-router-id" to convey the concept that this ID is on a TE node, which may or may or be a router.
- We will clarify the description to say that "This attribute is MAY be mapped to TE Router ID in [RFC3630], [RFC5329], [RFC5305], and [RFC6119]."

Thanks,
- Xufeng

On Thu, Dec 6, 2018 at 12:38 PM Acee Lindem (acee) <acee@cisco.com<mailto:acee@cisco.com>> wrote:
Hi Tom,
I think the only action here is for the authors of draft-ietf-teas-yang-te-types to fix their te-node-id definition. As for the OSPF Router ID and OSPF/ISIS TE Router IDs we can't change the decades old definitions to achieve uniformity.
Thanks,
Acee

On 12/5/18, 11:12 AM, "tom petch" <ietfc@btconnect.com<mailto:ietfc@btconnect.com>> wrote:

    ----- Original Message -----
    From: <stephane.litkowski@orange.com<mailto:stephane.litkowski@orange.com>>
    Sent: Wednesday, December 05, 2018 12:57 PM

    > Hi Tom,
    >
    > I think that having a different router-id configured per protocol is a
    matter of deployment. I don't think that we can impose anything in this
    area. There are use cases where it is good to have separate router-ids
    per protocol or instances of a protocol. For instance, when a router is
    part of multiple "administrative domains", it is worth having separate
    router-ids per admin domain.
    >
    > However I have a concern about the router-id or te-node-id  bound to a
    32 bits number only. How do we do in a pure IPv6 network ?

    Stephane

    I am used to configuring a router-id as a 32-bit number with no
    requirement for that to be an address that can be accessed over the
    internet (so I have always found the idea of 'loopback0' unfortunate).
    Yes, the router needs to be addressable, but merging that concept with a
    router id has always seemed to me unfortunate because they are two
    separate concepts.  (In fact, I would regard good practice as giving a
    router multiple addresses for different functions, so that e.g. syslog
    can be separated from SNMP or FTP).

    Thus I have no problem with a 32-bit router-id in an IPv6 network.
    Indeed, RFC5329 defines a 32-bit router-id in an OSPFv3
    Intra-Area-TE-LSA.  It is the Router IPv6 Address TLV that carries the
    128-bit address.

    When ospf-yang says
             container te-rid {
               if-feature te-rid;
               description  "Stable OSPF Router IP Address used for Traffic
                  Engineering (TE)";
               leaf ipv4-router-id { type inet:ipv4-address; description
                   "Explicitly configure the TE IPv4 Router ID.";
               }
               leaf ipv6-router-id {
                 type inet:ipv6-address;
                 description "Explicitly configure the TE IPv6 Router ID.";

    then that is when I wonder what is going on.  That looks to me like
    configuring
    Router IPv6 Address TLV
    not the router id.

    Meanwhile, te-yang-te-types has

       te-node-id:
          A type representing the identifier for a node in a topology.  The
          identifier is represented as 32-bit unsigned integer in the
          dotted-quad notation.  This attribute is mapped to Router ID in
          [RFC3630], [RFC5329], [RFC5305], and [RFC6119].

    Well, I disagree with their choice of YANG type but agree that it is
    32-bit and not 128.

    Tom Petch.

    > Brgds,
    >
    >
    > -----Original Message-----
    > From: tom petch [mailto:ietfc@btconnect.com<mailto:ietfc@btconnect.com>]
    > Sent: Wednesday, December 05, 2018 12:14
    > To: Acee Lindem (acee); LITKOWSKI Stephane OBS/OINIS; lsr@ietf.org<mailto:lsr@ietf.org>;
    draft-ietf-ospf-yang@ietf.org<mailto:draft-ietf-ospf-yang@ietf.org>; draft-ietf-teas-yang-te-types@ietf.org<mailto:draft-ietf-teas-yang-te-types@ietf.org>
    > Subject: Re: draft-ietf-ospf-yang
    >
    > Acee
    >
    > (Top-posting because the indentation usually fails)
    >
    > On the TEAS te-types, I had a quick look at where
    > typedef te-node-id
    > is used and the answer is lots of places, because it is part of
    >   grouping explicit-route-hop {
    >     description    "The explicit route subobject grouping";
    >     choice type {
    >       description   "The explicit route subobject type";
    >       case num-unnum-hop {
    >         container num-unnum-hop {
    >           leaf node-id {
    >             type te-types:te-node-id;
    >             description   "The identifier of a node in the TE
    > topology.";
    > and YANG uses of that grouping are many, in several WGs; however,
    > because it is a grouping, then the impact of changing the type should
    be
    > minimal at least in terms of the I-Ds.
    >
    > On the multiple router definitions, my research of the IETF memo only
    > came up with the two cited RFC which, to me, say that you should use
    an
    > existing router-id if there is one.
    >
    > I did look at the documentation of A Major Router Manufacturer and
    while
    > they did not give any advice, the default for a te router-id was
    > loopback0
    > while the default for a more general router-id, one without te, was
    > loopback0
    > which gives me the message, you can make them different but SHOULD NOT
    > (in IETF terminology).
    >
    > So while I agree that the two lsr modules should allow per-protocol
    > configuration, I think that it should carry a health warning in the
    body
    > of the I-D that this is not a good idea (I struggle to think of when
    it
    > would be a good idea, to use three separate identifiers for, say, BGP
    > and the two lsr protocols).
    >
    > Tom Petch
    >
    > ----- Original Message -----
    > From: "Acee Lindem (acee)" <acee@cisco.com<mailto:acee@cisco.com>>
    > To: "tom petch" <ietfc@btconnect.com<mailto:ietfc@btconnect.com>>;
    <stephane.litkowski@orange.com<mailto:stephane.litkowski@orange.com>>;
    > <lsr@ietf.org<mailto:lsr@ietf.org>>; <draft-ietf-ospf-yang@ietf.org<mailto:draft-ietf-ospf-yang@ietf.org>>;
    > <draft-ietf-teas-yang-te-types@ietf.org<mailto:draft-ietf-teas-yang-te-types@ietf.org>>
    > Sent: Tuesday, December 04, 2018 7:46 PM
    >
    > > Hi Tom,
    > >
    > > Let me try to explain.
    > >
    > > On 12/4/18, 12:44 PM, "tom petch" <ietfc@btconnect.com<mailto:ietfc@btconnect.com>> wrote:
    > >
    > >     The router id in this I-D confuse me.
    > >
    > >     RFC8294 defines
    > >          typedef router-id { type yang:dotted-quad;
    > >
    > > Some implementations configure a global router-id while others only
    > allow it at the control-plane-protocol level. This is why we have it
    in
    > both places.
    > >
    > >     ospf-yang defines
    > >      leaf ipv4-router-id { type inet:ipv4-address;
    > >
    > > For better or worse, OSPF has a separate TE address that is routable
    > and referred to as the TE router-id. You'll note that this is part of
    > the te-rid container in both the OSPF and IS-IS YANG models. We could
    > add "-te-" to the leaves to avoid confusion.
    > >
    > >     draft-ietf-teas-yang-te-types defines
    > >       typedef te-node-id {     type yang:dotted-quad;
    > >      ...       This attribute is mapped to Router ID ....
    > >
    > > This is just wrong. It is a routable address in the IGP TE
    extensions.
    > I've copied the draft authors.
    > >
    > > Thanks,
    > > Acee Lindem
    > >
    > >
    > >     Three different YANG types for a router id.
    > >
    > >     Why?
    > >
    > >     Behind this, ospf-yang gives as references for a router te id
    > >     RFC3630(V2) and RFC5329(V3).  Reading these, my take is that a
    > router id
    > >     is needed for te but that the existing id should be used where
    > possible
    > >     i.e. creating an additional identifier for the same instance of
    > the same
    > >     entity is A Bad Thing (which sounds like a good general
    > principle).
    > >     With two objects in the lsr protocols, that would appear to make
    > at
    > >     least three identifiers for the same instance of the same
    entity.
    > >
    > >     Why?
    > >
    > >     I copy Stephane on this since the same issues apply to the other
    > lsr
    > >     protocol, mutatis mutandi.
    > >
    > >     Tom Petch
    > >
    > >
    > >
    > >
    >
    >
    >
    ________________________________________________________________________
    _________________________________________________
    >
    > 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.
    >
    >