Hi Med,

Thanks for your comments.  Please see below.

On 28.05.21 11:18, [email protected] wrote:
Hi Eliot, all,

This extension makes sense to me. I support it to be adopted in opsawg (where 
8520 was developed).

That's said, this draft does not update 8520. I suggest to remove "Updates: 8520 (if 
approved)"

Please find below some other comments, fwiw.

(1)

OLD:

      import ietf-inet-types {
        prefix inet;
      }

NEW
      import ietf-inet-types {
        prefix inet;
        reference
          "RFC 6991: Common YANG Data Types, Section 4";
      }


Sure, thanks.


(2) Use the prefix as defined in 8520:

OLD:

      import ietf-mud {
        prefix mud;
      }

NEW:

      import ietf-mud {
        prefix ietf-mud;
        reference
          "RFC 8520: Manufacturer Usage Description Specification";
      }


Right.


(3)

You may remove the note about normative language in the description of the 
module given that you don't use it.

I think we need to discuss this more, because we SHOULD be applying a SHOULD on this document to 8520 so that there is no IPR confusion.  And that implies an update.


(4)

OLD:
        reference
          "RFC XXXX: Extension for ownership and licensing";
NEW:
        reference
          "RFC XXXX: Ownership and licensing statements in YANG";


Corrected, thanks.


(5)

Rather than using "leaf-list owners", I would define this as a list keyed by owners and 
which may include a leaf-list of licenses. Rather than defining license as a choice, I would use a 
"union".
We're still discussing the substance of this.
(6)

OLD:

        "ol": {
          "owners": [
            "Copyright (c) FrobMaster 2021. All Rights Reserved"
          ],
          "spdx-tag": "0BSD"
        },

NEW:
        "ietf-ol:ol": {
          "owners": [
            "Copyright (c) FrobMaster 2021. All Rights Reserved"
          ],
          "spdx-tag": "0BSD"
        },
Still working on this.
(7) Please update the IANA section as follows:

OLD:
    The IANA is requested to add "ol" to the MUD extensions registry as
    follows:

      Extension Name: ol
      Standard reference: This document

NEW:
    The IANA is requested the following entry to the MUD extensions registry
    available at: https://www.iana.org/assignments/mud/mud.xhtml#mud-extensions

      Extension Name: ol
      Reference: This document
Done.  Thanks.
(8) Please update the IANA section with the following:

NEW

    This document requests IANA to register the following URI in the "ns"
    subregistry within the "IETF XML Registry" [RFC3688]:

          URI: urn:ietf:params:xml:ns:yang:ietf-ol
          Registrant Contact: The IESG.
          XML: N/A; the requested URI is an XML namespace.

    This document requests IANA to register the following YANG module in
    the "YANG Module Names" subregistry [RFC6020] within the "YANG
    Parameters" registry.

          name: ietf-ol
          namespace: urn:ietf:params:xml:ns:yang:ietf-ol
          maintained by IANA: N
          prefix: ol
          reference: RFC XXXX
What does "maintained by IANA" mean in this context?

Thanks again for the update.

Eliot


Attachment: OpenPGP_signature
Description: OpenPGP digital signature

_______________________________________________
OPSAWG mailing list
[email protected]
https://www.ietf.org/mailman/listinfo/opsawg

Reply via email to