Re: [OPSAWG] please see draft-lear-opsawg-ol on licensing

[Eliot Lear <[e-mail]>]

Hi Med,

Thanks for your comments.  Please see below.

On 28.05.21 11:18, [e-mail] 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