Subject: [Netconf] Comments on draft-ietf-netconf-with-defaults-04

From: Martin Bjorklund <[email-masked]>
Date: Mon, 09 Nov 2009 10:31:03 +0100 (CET)

Hi,

Here are some comments on this document.

1.1
---

      External management
      application/user can reach the device e.g. via the NETCONF, CLI,
      SNMP, GUI.

Can you rephrase the definition where this sentence occurs?  It seems
out-of-place.

1.1
---

   In addition the following terms are defined in RFC 4741 and are not
   redefined here:
   o  application

I don't think it is defined in 4741, and you are using the term
"management application".

1.2
---

OLD:

   o  trim: Values are not reported if they match the default.

NEW: (Is this a correct interpretation?)

   o  trim: Values are not reported if they match the default value as
   specified in the data model schema.

2.5
---

OLD:

   A new <with-defaults> XML child element is added to the method-name
   element.  This is the element that indicates the type of the
   operation e.g. <get>, <get-config> or <copy-config>.

NEW:

   A new <with-defaults> XML child element is added to the <get>,
   <get-config>, and <copy-config> rpc operations.

4.
--

s/Data Model XSD/XML Schema for the with-defaults capability/

8.
--

    // for the uri data type
    import ietf-netconf { prefix nc; }

remove the comment; it is wrong.

8.
--

Since you want other operations to also add the with-defaults
parameter, would it make sense to place it in a reusable grouping?

  grouping with-defaults-parameters {
      leaf with-defaults {
          type with-defaults-mode;
      }
  }

  augment /nc:get-config/nc:input {
      uses with-defaults-parameters;
  }

General
-------

Can we change the parameter 'basic' to 'basic-mode'?

/martin