Re: Last Call: <draft-ietf-ippm-twamp-yang-07.txt> (Two-Way Active Measurement Protocol (TWAMP) Data Model) to Proposed Standard

[Mahesh Jethanandani <mjethanandani@xxxxxxxxx>]

Tom,

On this particular comment ….

On Apr 12, 2018, at 5:13 PM, Mahesh Jethanandani <mjethanandani@xxxxxxxxx> wrote:

"container session-sender {
       if-feature session-sender;
       presence  "Enables TWAMP Session-Sender functionality.";
       description
         "Configuration of the TWAMP Session-Sender logical entity";
       leaf admin-state {
         type boolean;
         mandatory true;
         description
           "Indicates whether the device is allowed to operate
            as a TWAMP Session-Sender.";
"

A presence container is a boolean so I am unclear what it is that the
admin-state boolean adds here since the presence container "Enables
TWAMP Session-Sender functionality.”;

[mj] Good catch. Will remove the presence statement.

[[mj]] In addition to removing the presence statement, we decided to change the ‘mandatory true' statement to a ‘default true’ statement. Just removing the presence statement results in the following error:

Validating yang/ietf-twamp@xxxxxxxxxxxxxxx

yang/ietf-twamp@xxxxxxxxxxxxxxx:445: error: RFC 6087: 4.9: top-level node twamp must not be mandatory

Cheers.

Mahesh Jethanandani

mjethanandani@xxxxxxxxx