[mpls] Re: [Last-Call] [RTG-DIR]Re: Rtgdir last call review of draft-ietf-mpls-msd-yang-07
Acee Lindem <acee.ietf@gmail.com> Thu, 06 June 2024 17:27 UTC
Return-Path: <acee.ietf@gmail.com>
X-Original-To: mpls@ietfa.amsl.com
Delivered-To: mpls@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id BA57CC151087; Thu, 6 Jun 2024 10:27:25 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.096
X-Spam-Level:
X-Spam-Status: No, score=-2.096 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_DBL_BLOCKED_OPENDNS=0.001, URIBL_ZEN_BLOCKED_OPENDNS=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.com
Received: from mail.ietf.org ([50.223.129.194]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id izi8oGiCYUGm; Thu, 6 Jun 2024 10:27:21 -0700 (PDT)
Received: from mail-qt1-x82c.google.com (mail-qt1-x82c.google.com [IPv6:2607:f8b0:4864:20::82c]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 08E73C14F748; Thu, 6 Jun 2024 10:27:20 -0700 (PDT)
Received: by mail-qt1-x82c.google.com with SMTP id d75a77b69052e-43fdd85a284so6602351cf.0; Thu, 06 Jun 2024 10:27:20 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1717694840; x=1718299640; darn=ietf.org; h=references:to:cc:in-reply-to:date:subject:mime-version:message-id :from:from:to:cc:subject:date:message-id:reply-to; bh=Vkga1eUg84b/PqKImPf2Yn3ER+OYY/mAYBDoHqC5gT8=; b=cSWkjYrslLuyNeWf18BnqQpxMJ9nVHmNGGuAdDbS25koGDR0m9N173mk0XgIL6pnPK Tfsor3mIVBSvEOJf81goKjY3dNAu5SF2afd19hlkdgCqBoTNsugipJE6CQDlx84J6l1N cC3aNp36+4DLWHvFhA84ZkBk0m70cXCa+0jipwwY78UU9SEWCMZ5bsBR7E2PBieOcSyD 6uWNueTf9wprvYEFqZF/2rUy2YuhSHz2UnBtWya5l5vyu/4rp0KjDR5KMlXsSj39HRbs gA5/NLMQJQZKeSsEK14V0bE2zh90ENWSFs6xA0OI1zZryeOjPhimPhJZOLtEgGUXlf8g +a4w==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1717694840; x=1718299640; h=references:to:cc:in-reply-to:date:subject:mime-version:message-id :from:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=Vkga1eUg84b/PqKImPf2Yn3ER+OYY/mAYBDoHqC5gT8=; b=qJyHx6g6u7dVXVIIR4rj//UhboyStta5l4dIDMAtCHVWOCQF9q5vK091Fs1uUGKOjL 33iXpqkDJ5H6Ufh5cvb9QpGUZtIP0UbtC83wgARDMqV7wtoh9DxYuZclZehV2U+TbyFu o7Hga9UOOVzEq48Qb1J13+kQYMgPT+eyRMCZEGJ2ZpFdgeOd6mOCbUGgyONNAZVyWueu qeGfx8hFr3Dmm6yXZqzJzbEjn1C/uLqXQO+2+tdROcljtgl/IxZ7hQ5XIDTuM88/WCoE SFlJ0mPLWlJX63qeV64QUgVAX8uLbnhGxON8oC+k4QvP45+wvRiDsXjJIQxXrf+P0Cc7 ZLXw==
X-Forwarded-Encrypted: i=1; AJvYcCVAJF6vEo33czTywDxfKmbt58nOsgxNp+c/rIL5lmCZB9eRNcaQ5DicJG5TaX0HY4BzCUCvOAth90W2kR8rnp0Dkfub1cmB9Tv+2aq+LH/nkIUnk+5MD+f51RQYpun5smfl2zvBDG0lHAAqCeDSNSEH5YAJKGG52apbKzhFKUO3bm6lvzv/MKjJ08E=
X-Gm-Message-State: AOJu0YzVubXvdZqugjEgK4vOE9V+1YMCHgy9OFaqMIezcMbQhRkW+spI cP1wzeYTMivKM86E2H52HBCq5LtI798cHMF+W1gJq2Qwwo96I2+Suc3fnw==
X-Google-Smtp-Source: AGHT+IGEud0/yd9/Pgox+3alU5RzHVxuhzERRKsleZyNN4Y/wKcLyV5mYe91qikbk7xAfLT01Hmf7A==
X-Received: by 2002:a05:622a:1108:b0:43a:d063:c8b1 with SMTP id d75a77b69052e-44041bf4d2amr2869701cf.33.1717694839345; Thu, 06 Jun 2024 10:27:19 -0700 (PDT)
Received: from smtpclient.apple ([2605:a601:918a:7600:56d:ff2e:8bf5:9b1a]) by smtp.gmail.com with ESMTPSA id d75a77b69052e-44038a684c4sm6006011cf.21.2024.06.06.10.27.18 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Thu, 06 Jun 2024 10:27:18 -0700 (PDT)
From: Acee Lindem <acee.ietf@gmail.com>
Message-Id: <C8364DF4-202D-4247-A6A1-3C6200D65EC3@gmail.com>
Content-Type: multipart/alternative; boundary="Apple-Mail=_99BC6387-153B-4538-B88B-7C1FC7273D98"
Mime-Version: 1.0 (Mac OS X Mail 16.0 \(3774.600.62\))
Date: Thu, 06 Jun 2024 13:27:08 -0400
In-Reply-To: <DU2PR02MB1016069DAE5EBDBD02DAD3C0688FA2@DU2PR02MB10160.eurprd02.prod.outlook.com>
To: Med Boucadair <mohamed.boucadair@orange.com>
References: <171743797081.42914.4518891340142384843@ietfa.amsl.com> <DU2PR02MB1016077CA409CD39DB09F04E988F82@DU2PR02MB10160.eurprd02.prod.outlook.com> <CABY-gOOhPR=3nixD2qwW99i9DArYNiY3Xk-w258B2Pa4vqz2bg@mail.gmail.com> <DU2PR02MB101603134893C129D606A271288F92@DU2PR02MB10160.eurprd02.prod.outlook.com> <92571ED1-FD5F-47C6-A158-2E3BE2B2B0CB@gmail.com> <DU2PR02MB101608D94311BD0B28776344788F92@DU2PR02MB10160.eurprd02.prod.outlook.com> <CABY-gOPB=HKpTVX-nzy9HNabv0bXNU6Fz5AK4KA-8V9kfJjVvg@mail.gmail.com> <DU2PR02MB101605909EAA0561B18194E1A88F92@DU2PR02MB10160.eurprd02.prod.outlook.com> <CABY-gOOxwwAm+SmxqLZKkyDOre1rW4UT=vTO_YWFkshs3DyAyQ@mail.gmail.com> <DU2PR02MB101607B0BC82F9128AD02C35688FA2@DU2PR02MB10160.eurprd02.prod.outlook.com> <CAP7zK5aKn4N+hUYNk45He+MPROThcBSv=8ypLYKtRW3hc500XA@mail.gmail.com> <DU2PR02MB10160C927F47A9063CA4BBC0C88FA2@DU2PR02MB10160.eurprd02.prod.outlook.com> <CAP7zK5acAj4hcLkKv+TN661bVvW64Hfh8R-9uxbGMoqjzG-ozQ@mail.gmail.com> <DU2PR02MB1016069DAE5EBDBD02DAD3C0688FA2@DU2PR02MB10160.eurprd02.prod.outlook.com>
X-Mailer: Apple Mail (2.3774.600.62)
Message-ID-Hash: 7JNWYBUUSNODAURKL3RFVWKHYI7ZAP4J
X-Message-ID-Hash: 7JNWYBUUSNODAURKL3RFVWKHYI7ZAP4J
X-MailFrom: acee.ietf@gmail.com
X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; emergency; loop; banned-address; member-moderation; header-match-mpls.ietf.org-0; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header
CC: Dhruv Dhody <dd@dhruvdhody.com>, "rtg-dir@ietf.org" <rtg-dir@ietf.org>, "draft-ietf-mpls-msd-yang.all@ietf.org" <draft-ietf-mpls-msd-yang.all@ietf.org>, Last Call <last-call@ietf.org>, mpls <mpls@ietf.org>
X-Mailman-Version: 3.3.9rc4
Precedence: list
Subject: [mpls] Re: [Last-Call] [RTG-DIR]Re: Rtgdir last call review of draft-ietf-mpls-msd-yang-07
List-Id: Multi-Protocol Label Switching WG <mpls.ietf.org>
Archived-At: <https://mailarchive.ietf.org/arch/msg/mpls/Ju6ZLOCMNzETPw_pGthl-fWRkS8>
List-Archive: <https://mailarchive.ietf.org/arch/browse/mpls>
List-Help: <mailto:mpls-request@ietf.org?subject=help>
List-Owner: <mailto:mpls-owner@ietf.org>
List-Post: <mailto:mpls@ietf.org>
List-Subscribe: <mailto:mpls-join@ietf.org>
List-Unsubscribe: <mailto:mpls-leave@ietf.org>
Hi Med, > On Jun 6, 2024, at 05:49, mohamed.boucadair@orange.com wrote: > > Re-, > > Please see inline. > > Cheers, > Med > > De : Dhruv Dhody <dd@dhruvdhody.com <mailto:dd@dhruvdhody.com>> > Envoyé : jeudi 6 juin 2024 11:22 > À : BOUCADAIR Mohamed INNOV/NET <mohamed.boucadair@orange.com <mailto:mohamed.boucadair@orange.com>> > Cc : Yingzhen Qu <yingzhen.ietf@gmail.com <mailto:yingzhen.ietf@gmail.com>>; Acee Lindem <acee.ietf@gmail.com <mailto:acee.ietf@gmail.com>>; rtg-dir@ietf.org <mailto:rtg-dir@ietf.org>; draft-ietf-mpls-msd-yang.all@ietf.org <mailto:draft-ietf-mpls-msd-yang.all@ietf.org>; Last Call <last-call@ietf.org <mailto:last-call@ietf.org>>; mpls <mpls@ietf.org <mailto:mpls@ietf.org>> > Objet : [Last-Call] Re: [RTG-DIR]Re: Rtgdir last call review of draft-ietf-mpls-msd-yang-07 > > Hi Med, > > On Thu, Jun 6, 2024 at 8:24 AM <mohamed.boucadair@orange.com <mailto:mohamed.boucadair@orange.com>> wrote: > Hi Dhruv, > > Yes, that would work assuming that we will have a new column to record explicitly the data plane name for each type. > > Dhruv: What you are asking is not wrong "technically" but at the same time I can't envision someone defining a MSD without being explicit on the data plane it applies to. > [Med] These two are not conflicting :-) If the information is explicit, then record it in the registry, not infer it from the narrative text or the MSD name. > > Can this be a guideline for designated experts to verify without adding a new column? > > [Med] The concern is beyond the DE review. Once a change is made to the registry: how to mirror that automatically in the IANA-maintained module using the current instructions in the spec? Which information in the registry will be used to trigger whether the base identity is to be used, existing child ones, or creating new child ones, etc. We need to make the maintenance task implementable, but easy. Are you envisioning this being automated with a very naive script? We are in the age of AI and the discerning the data-plane seems trivial. Would adding the data-plane column to the IANA registry satisfy your concern? Thanks, Acee > > Some checks are worth though: Do we foresee cases where the same type can be used by multiple “data planes”? > > > Dhruv: Unlikely IMHO. > [Med] ACK. Thanks. > > Thanks, > Dhruv > > > Cheers, > Med > > De : Dhruv Dhody <dd@dhruvdhody.com <mailto:dd@dhruvdhody.com>> > Envoyé : jeudi 6 juin 2024 08:59 > À : BOUCADAIR Mohamed INNOV/NET <mohamed.boucadair@orange.com <mailto:mohamed.boucadair@orange.com>> > Cc : Yingzhen Qu <yingzhen.ietf@gmail.com <mailto:yingzhen.ietf@gmail.com>>; Acee Lindem <acee.ietf@gmail.com <mailto:acee.ietf@gmail.com>>; rtg-dir@ietf.org <mailto:rtg-dir@ietf.org>; draft-ietf-mpls-msd-yang.all@ietf.org <mailto:draft-ietf-mpls-msd-yang.all@ietf.org>; Last Call <last-call@ietf.org <mailto:last-call@ietf.org>>; mpls <mpls@ietf.org <mailto:mpls@ietf.org>> > Objet : [RTG-DIR]Re: Rtgdir last call review of draft-ietf-mpls-msd-yang-07 > > Hi Med, > > How about this I-D adds some suitable guidance for designated experts (as the IANA registry is under 'Expert Review')? We can also make this I-D update RFC8491. Could that be a way forward? > > Thanks! > Dhruv > > On Thu, Jun 6, 2024 at 6:05 AM <mohamed.boucadair@orange.com <mailto:mohamed.boucadair@orange.com>> wrote: > Hi Yingzhen, > > Thank you for addressing the sec comment. I trust that you have also fixed the content mismatch between the module and the parent registry points. > > Thank you also for confirming that there are no such IANA considerations. The narrative text in 9352 does not change the rules set in 8491. > > I consider that the last comment is still valid and not addressed. Thank you. > > Cheers, > Med > > De : Yingzhen Qu <yingzhen.ietf@gmail.com <mailto:yingzhen.ietf@gmail.com>> > Envoyé : jeudi 6 juin 2024 01:05 > À : BOUCADAIR Mohamed INNOV/NET <mohamed.boucadair@orange.com <mailto:mohamed.boucadair@orange.com>> > Cc : Acee Lindem <acee.ietf@gmail.com <mailto:acee.ietf@gmail.com>>; Dhruv Dhody <dd@dhruvdhody.com <mailto:dd@dhruvdhody.com>>; rtg-dir@ietf.org <mailto:rtg-dir@ietf.org>; draft-ietf-mpls-msd-yang.all@ietf.org <mailto:draft-ietf-mpls-msd-yang.all@ietf.org>; Last Call <last-call@ietf.org <mailto:last-call@ietf.org>>; mpls <mpls@ietf.org <mailto:mpls@ietf.org>> > Objet : Re: [RTG-DIR]Re: Rtgdir last call review of draft-ietf-mpls-msd-yang-07 > > Hi Med, > > You're right, it is not explicitly stated in the IANA consideration section. However you can find "This document only defines one type of MSD: Base MPLS Imposition. " in the abstract of RFC 8491. > > And the following text is at the beginning of section 4 in RFC 9352 that defines the SRH MSDs: > " > [RFC8491 <https://www.rfc-editor.org/rfc/rfc9352.html#RFC8491>] defines the means to advertise node-/link-specific values for MSDs of various types. Node MSDs are advertised in a sub-TLV of the Router Capability TLV [RFC7981 <https://www.rfc-editor.org/rfc/rfc9352.html#RFC7981>]. Link MSDs are advertised in a sub-TLV of TLVs 22, 23, 25, 141, 222, and 223.¶ <https://www.rfc-editor.org/rfc/rfc9352.html#section-4-1> > This document defines the relevant SRv6 MSDs and requests MSD type assignments in the "IGP MSD-Types" registry created by [RFC8491 <https://www.rfc-editor.org/rfc/rfc9352.html#RFC8491>]. > > " > I've updated the security consideration in version -09. Please let us know whether this addressed your comments. > > Thanks, > Yingzhen > > > > On Wed, Jun 5, 2024 at 9:27 AM <mohamed.boucadair@orange.com <mailto:mohamed.boucadair@orange.com>> wrote: > Re-, > > I actually checked that before making the comment. There is no such requirement for new requests in https://www.rfc-editor.org/rfc/rfc8491.html#section-6. > > May be I’m not looking in the right place?! > > I would appreciate if you can point me to where I can find this requirement: “the document that defines the new entry should state what the MSD is for” > > Cheers, > Med > > De : Yingzhen Qu <yingzhen.ietf@gmail.com <mailto:yingzhen.ietf@gmail.com>> > Envoyé : mercredi 5 juin 2024 18:02 > À : BOUCADAIR Mohamed INNOV/NET <mohamed.boucadair@orange.com <mailto:mohamed.boucadair@orange.com>> > Cc : Acee Lindem <acee.ietf@gmail.com <mailto:acee.ietf@gmail.com>>; Dhruv Dhody <dd@dhruvdhody.com <mailto:dd@dhruvdhody.com>>;rtg-dir@ietf.org <mailto:rtg-dir@ietf.org>; draft-ietf-mpls-msd-yang.all@ietf.org <mailto:draft-ietf-mpls-msd-yang.all@ietf.org>; Last Call <last-call@ietf.org <mailto:last-call@ietf.org>>; mpls <mpls@ietf.org <mailto:mpls@ietf.org>> > Objet : [RTG-DIR]Re: Rtgdir last call review of draft-ietf-mpls-msd-yang-07 > > > > Hi Med, > > When a new entry is added to the "IGP MSD-Type" registry, the document that defines the new entry should state what the MSD is for. > > I'll update the security consideration. > > Thanks, > Yingzhen > > On Wed, Jun 5, 2024 at 7:06 AM <mohamed.boucadair@orange.com <mailto:mohamed.boucadair@orange.com>> wrote: > Hi Acee, > > Please see inline. > > Cheers, > Med > > > Orange Restricted > De : Acee Lindem <acee.ietf@gmail.com <mailto:acee.ietf@gmail.com>> > Envoyé : mercredi 5 juin 2024 15:35 > À : BOUCADAIR Mohamed INNOV/NET <mohamed.boucadair@orange.com <mailto:mohamed.boucadair@orange.com>> > Cc : Yingzhen Qu <yingzhen.ietf@gmail.com <mailto:yingzhen.ietf@gmail.com>>; Dhruv Dhody <dd@dhruvdhody.com <mailto:dd@dhruvdhody.com>>; rtg-dir@ietf.org <mailto:rtg-dir@ietf.org>; draft-ietf-mpls-msd-yang.all@ietf.org <mailto:draft-ietf-mpls-msd-yang.all@ietf.org>; Last Call <last-call@ietf.org <mailto:last-call@ietf.org>>; mpls <mpls@ietf.org <mailto:mpls@ietf.org>> > Objet : Re: [RTG-DIR]Rtgdir last call review of draft-ietf-mpls-msd-yang-07 > > > Hi Med, > > See inline. > > > On Jun 5, 2024, at 04:16, mohamed.boucadair@orange.com <mailto:mohamed.boucadair@orange.com> wrote: > > Hi Yingzhen, > > Thanks for taking care of this. The new version looks better. > > Still, I don’t see how the hierarchical identity structure can be automatically inferred from the current flat registry structure. I’m afraid that the instructions in the 3rd para of 6.2 are not sufficient as I don’t think that we can trust the presence of SRH or some magic words in the description to decide which identity to use, and more generally if “a new data plane” is used. > > Both the description and references listed in the IANA module diverge from the actual content of the registry. I would avoid that. Please refer to this clarification in the 8407bis (see the last sentence, in particular): > > The content of these registries are usually available using various > formats (e.g., plain text, XML). However, there were some confusion > in the past about whether the content of some registries is dependent > on a specific representation format. For example, Section 5 of > [RFC8892] was published to clarify that MIB and YANG modules are > merely additional formats in which the "Interface Types (ifType)" and > "Tunnel Types (tunnelType)" registries are available. The MIB > [RFC2863] and YANG modules [RFC7224][RFC8675] are not separate > registries, and the same values are always present in all formats of > the same registry. > > I disagree. The fact that we have a hierarchy of identities wouldn’t confuse anyone. They are all part of the > iana-msd-types.yang model and all have “msd-base” as the root identity. We are rejecting this rather > subjective comment. > > [Med] Hmm…this is not subjective :-) Let’s consider that I registered a new entry in Interior Gateway Protocol (IGP) Parameters (iana.org) <https://www.iana.org/assignments/igp-parameters/igp-parameters.xhtml> > > 115 > > My own type > > [MyREF <https://www.iana.org/go/rfc8491>] > > > How IANA will follow the following guidance? How IANA will know this is about an existing “data plane” or a new one? How it will decide to mirror it using existing one msd-base or msd-base-srh? How to infer a data plane from a registration? Etc. > > The identities defined in the iana-msd-types YANG module are > organized hierarchically based on the data plane. In this initial > version, only MPLS and SRv6 data planes are supported, hence "msd- > base-mpls" and "msd-base-srh" are defined. When a new data plane is > added to the "IGP MSD-Types" registry, a new "identity" statement > should be added to the "iana-msd-types" YANG module. The name of the > "identity" is the prefix "msd-base-" plus a lower-case version of the > data plane name . The identity statement should have the following > sub-statements defined: > > > If you want to suggest further text to explain this, we’ll consider inclusion. > > Some other misc. comments: > > (1) “lower-case version of the data plane name”: you may also indicate that the space is replaced with “-“, not trimmed. > > (2) "description": Replicates the description from the registry. > > I guess you meant replicate the “name” from the registry. There is no description in the IGP MSD Type reg. > > (3) > > OLD: > name: iana-msd-types > namespace: urn:ietf:params:xml:ns:yang:iana-msd-types > prefix: iana-msd-types > reference: RFC XXXX > > name: ietf-mpls-msd > namespace: urn:ietf:params:xml:ns:yang:ietf-mpls-msd > prefix: mpls-msd > reference: RFC XXXX > > NEW: > name: iana-msd-types > namespace: urn:ietf:params:xml:ns:yang:iana-msd-types > prefix: iana-msd-types > maintained by IANA? Y > reference: RFC XXXX > > name: ietf-mpls-msd > namespace: urn:ietf:params:xml:ns:yang:ietf-mpls-msd > prefix: mpls-msd > maintained by IANA? N > reference: RFC XXXX > > (4) the security section does not follow the template + does not cover the IANA module. Please refer tohttps://datatracker.ietf.org/doc/html/draft-ietf-netmod-rfc8407bis-11#name-security-considerations-sect. > > The iana-msd-types.yang modules doesn’t include any data leafs so there are no associated security considerations. We could state this. > We’ll check the latest template in the draft. > > [Med] There is a para in the template for that as well. > > Thanks, > Acee > > Cheers, > Med > > De : Yingzhen Qu <yingzhen.ietf@gmail.com <mailto:yingzhen.ietf@gmail.com>> > Envoyé : mercredi 5 juin 2024 07:55 > À : BOUCADAIR Mohamed INNOV/NET <mohamed.boucadair@orange.com <mailto:mohamed.boucadair@orange.com>> > Cc : Dhruv Dhody <dd@dhruvdhody.com <mailto:dd@dhruvdhody.com>>; rtg-dir@ietf.org <mailto:rtg-dir@ietf.org>; draft-ietf-mpls-msd-yang.all@ietf.org <mailto:draft-ietf-mpls-msd-yang.all@ietf.org>; last-call@ietf.org <mailto:last-call@ietf.org>; mpls@ietf.org <mailto:mpls@ietf.org> > Objet : Re: [RTG-DIR]Rtgdir last call review of draft-ietf-mpls-msd-yang-07 > > Hi Mohamed, > > Thanks for the review and pointer. I've uploaded version -08 to address your comments, please review and let me know your comments, especially about the hierarchical identities. > > Thanks, > Yingzhen > > On Tue, Jun 4, 2024 at 12:18 AM <mohamed.boucadair@orange.com <mailto:mohamed.boucadair@orange.com>> wrote: > Hi all, > > In addition to the comments raised by Dhruv, the authors may look at the guidance athttps://datatracker.ietf.org/doc/html/draft-ietf-netmod-rfc8407bis-11#name-iana-maintained-modules for the required details for IANA-maintained modules. > > ## Lack of the details to maintain the module > > There is currently no guidance in draft-ietf-mpls-msd-yang about how the module will be maintained. For example, given that there is no label but only a description field in the authoritative IANA registry, the doc should explain how names will be echoed in the module. > > ## Mirror the content of the authoritative registry > > The content of the IANA module does not mirror the details in the registry. For example, there are many refs that are listed in draft-ietf-mpls-msd-yang, but those are not present in the parent registry. > > ## Hierarchy > > The IANA module defines this hierarchy, while there is no such hierarchy in the IANA registry. I understand that the authors want to structure the types, but is this really required here? Absent guidance about how new entries will be echoed from the registry, I don't think this structure is easily maintainable. Please keep in mind that registrants of new types are not even aware that an IANA-maintained module exists. So, they cannot be involved in the process of maintaining the module. > > == > identity msd-base-srh { > base msd-base; > description > "Identity for MSD types for Segment Routing Header (SRH)."; > } > > identity msd-srh-max-sl { > base msd-base-srh; > description > "The Maximum Segment Left MSD type."; > reference > "RFC 9352: IS-IS Extensions to Support Segment Routing > over the IPv6 Data Plane"; > } > > identity msd-srh-max-end-pop { > base msd-base-srh; > description > "The Maximum End Pop MSD Type."; > reference > "RFC 9352: IS-IS Extensions to Support Segment Routing > over the IPv6 Data Plane"; > } > == > > Hope this helps. > > Cheers, > Med > > > -----Message d'origine----- > > De : Dhruv Dhody via Datatracker <noreply@ietf.org <mailto:noreply@ietf.org>> > > Envoyé : lundi 3 juin 2024 20:06 > > À : rtg-dir@ietf.org <mailto:rtg-dir@ietf.org> > > Cc : draft-ietf-mpls-msd-yang.all@ietf.org <mailto:draft-ietf-mpls-msd-yang.all@ietf.org>; last-call@ietf.org <mailto:last-call@ietf.org>; > > mpls@ietf.org <mailto:mpls@ietf.org> > > Objet : [RTG-DIR]Rtgdir last call review of draft-ietf-mpls-msd- > > yang-07 > > > > > > Reviewer: Dhruv Dhody > > Review result: Has Issues > > > > Hello, > > > > I have been selected as the Routing Directorate reviewer for this > > draft. The Routing Directorate seeks to review all routing or > > routing-related drafts as they pass through the IETF last call > > and IESG review, and sometimes on special request. The purpose of > > the review is to assist the Routing ADs. For more information > > about the Routing Directorate, please see > > https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2 <https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%252> > > Fwiki.ietf.org <http://fwiki.ietf.org/>%2Fen%2Fgroup%2Frtg%2FRtgDir&data=05%7C02%7Cmohamed > > .boucadair%40orange.com <http://40orange.com/>%7C6ecf264db0bb43052c3b08dc83f8121b%7C90c7 > > a20af34b40bfbc48b9253b6f5d20%7C0%7C0%7C638530348673738183%7CUnkno > > wn%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1h > > aWwiLCJXVCI6Mn0%3D%7C0%7C%7C%7C&sdata=vfo%2F%2BxP9zc3YIrI1b9RjmRl > > XL3MicMrirSECkDHfM3c%3D&reserved=0 > > > > Although these comments are primarily for the use of the Routing > > ADs, it would be helpful if you could consider them along with > > any other IETF Last Call comments that you receive, and strive to > > resolve them through discussion or by updating the draft. > > > > Document: draft-ietf-mpls-msd-yang-07 > > Reviewer: Dhruv Dhody > > Review Date: 2024-06-03 > > IETF LC End Date: 2024-06-04 > > Intended Status: Proposed Standard > > > > ## Summary: > > > > * I have some minor concerns about this document that I think > > should be resolved before publication. > > > > ## Comment: > > > > * This draft defines 2 YANG models one is IANA-maintained to > > mirror the msd-type registry and the other is augmenting base > > MPLS to include MSD values. > > > > ### Major Issues: > > > > - Please remove the BCP14 boilerplate (Section 1.1) as you are > > not using any of those keywords. Also, remove from the ietf-mpls- > > msd YANG model. > > > > - You should explicitly state that this is an initial version of > > "iana-msd-types" YANG model - "This document defines the initial > > version of the IANA-maintained 'iana-msd-types' YANG module." > > > > ### Minor Issues: > > > > - Title: Please change to "A YANG Data Model for MPLS Maximum > > Segment Identifier (SID) Depth (MSD)". Also, update the reference > > in the YANG model around RFC XXXX. > > > > - The abstract suggests that only one YANG model is defined in > > this I-D. > > Consider rephrasing or adding some hints about the IANA model as > > well. > > > > - Section 1, "YANG [RFC7950] is a data definition language.."; I > > suggest changing it to data modeling as that is the term used in > > the referenced RFC. > > > > - Section 1, I am unsure about the text "The augmentation defined > > in this document requires support..."; isn't it obvious that one > > needs to support the model one is augmenting... > > > > - Section 4, please add this text in the description inside the > > YANG module - "This YANG module is maintained by IANA and > > reflects the 'IGP MSD-Types' > > registry." > > > > - identity msd-erld, should also have a reference to RFC9088. > > > > - In "ietf-mpls-msd", please remove the reference "RFC XXXX: A > > YANG Data Model for MPLS MSD." immediately after the module > > description. The revision statement is the correct place to have > > this reference. > > > > - leaf msd-value should also include text for "0 represents the > > lack of ability to support a SID stack of any depth". > > > > - I can not parse "A type of Node MSD is the smallest same type > > link MSD supported by the node.";" > > > > - RFC8340 should be normatively referenced. > > > > ### Nits: > > > > - s/(MSD) Types as the IANA the IGP MSD-Types registry/(MSD) > > Types as per the IANA IGP MSD-Types registry/ > > > > - s/which itself augments [RFC8349]/which itself augments routing > > RIB data model [RFC8349]/ > > > > - s/IANA maintained module/IANA-maintained module/ > > > > - s/This module will be maintained by IANA if more MSD types are > > added to the registry./This module will be maintained by IANA and > > updated if and when there is any change in the registry./ > > > > - s/and it is to provide support of different types of MSDs in > > MPLS data plane./and it provides support for different types of > > MSDs in the MPLS data plane./ > > > > - s/read-only data decided by/read-only data as per/ > > > > - Section 4, expand SID on first use in the YANG model. > > > > Thanks, > > Dhruv > > > > > > ____________________________________________________________________________________________________________ > 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. > > ____________________________________________________________________________________________________________ > 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. > > ____________________________________________________________________________________________________________ > 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. > ____________________________________________________________________________________________________________ > 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. > ____________________________________________________________________________________________________________ > 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. > ____________________________________________________________________________________________________________ > 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. > ____________________________________________________________________________________________________________ > 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.
- [mpls] Rtgdir last call review of draft-ietf-mpls… Dhruv Dhody via Datatracker
- [mpls] Re: [RTG-DIR]Rtgdir last call review of dr… mohamed.boucadair
- [mpls] Re: [RTG-DIR]Rtgdir last call review of dr… Yingzhen Qu
- [mpls] Re: [RTG-DIR]Rtgdir last call review of dr… mohamed.boucadair
- [mpls] Re: [RTG-DIR]Rtgdir last call review of dr… Acee Lindem
- [mpls] Re: [RTG-DIR]Rtgdir last call review of dr… mohamed.boucadair
- [mpls] Re: [RTG-DIR]Rtgdir last call review of dr… Yingzhen Qu
- [mpls] Re: [RTG-DIR]Re: Rtgdir last call review o… mohamed.boucadair
- [mpls] Re: [RTG-DIR]Re: Rtgdir last call review o… Yingzhen Qu
- [mpls] Re: [RTG-DIR]Re: Rtgdir last call review o… mohamed.boucadair
- [mpls] Re: [RTG-DIR]Re: Rtgdir last call review o… Dhruv Dhody
- [mpls] Re: [RTG-DIR]Re: Rtgdir last call review o… mohamed.boucadair
- [mpls] Re: [RTG-DIR]Re: Rtgdir last call review o… Dhruv Dhody
- [mpls] Re: [Last-Call] Re: [RTG-DIR]Re: Rtgdir la… mohamed.boucadair
- [mpls] Re: [Last-Call] [RTG-DIR]Re: Rtgdir last c… Acee Lindem
- [mpls] Re: [Last-Call] [RTG-DIR]Re: Rtgdir last c… mohamed.boucadair
- [mpls] Re: [Ext] [Last-Call] Re: [RTG-DIR]Re: Rtg… Amanda Baber
- [mpls] Re: [Last-Call] [RTG-DIR]Re: Rtgdir last c… Yingzhen Qu
- [mpls] Re: [Last-Call] [RTG-DIR]Re: Rtgdir last c… mohamed.boucadair
- [mpls] Re: [RTG-DIR]Re: Rtgdir last call review o… Mahesh Jethanandani
- [mpls] Re: Rtgdir last call review of draft-ietf-… Yingzhen Qu
- [mpls] Re: [Last-Call] Re: Rtgdir last call revie… Dhruv Dhody