Re: [pim] Fw: I-D Action: draft-ietf-pim-msdp-yang-16.txt

Hi Bob,
I'd like to ask a question in-line below.

On Mon, Mar 9, 2020 at 7:54 AM Rob Wilton (rwilton) <rwilton=
40cisco.com@dmarc.ietf.org> wrote:

> Hi Sandy,
>
>
>
> Thanks for making these markups.
>
>
>
> I wasn’t sure that your Actions example in A.3 was complete, it doesn’t
> look to be structurally correct, e.g. it doesn’t seem to identify the
> action that is being invoked.
>

Is there any complete JSON encoded action example? Based on RFC8040, the
given JSON data seems to be complete. The object that the action is applied
is identified by the resource url (Sec 3.6. of RFC8040):

    POST {+restconf}/data/<data-resource-identifier>/<action>

Page 35 gives an example:

      { "example-actions:input" : {
          "delay" : 600
        }
      }

Based on this example, A.3 seems correct.

It would also be useful to have a sentence before the action to describe
> what it is doing.
>

This could be the solution.
Thanks,
- Xufeng

>
>
> As a minor nit on 3.2, I would change reword the last paragraph:
>
>
>
> OLD:
>
>
>
>    The actions can be used to clear the connection of one specific MSDP
>
>    peer, or clear the connections of all the MSDP peers, or clear some
>
>    or all the SA caches.
>
>
>
>
>
> PROPOSED:
>
>
>
>    YANG actions are defined to clear the connection of one specific MSDP
>
>    peer, clear the connections of all MSDP peers, or clear some
>
>    or all the SA caches.
>
>
>
> Regards,
> Rob
>
>
>
>
>
> *From:* zhang.zheng@zte.com.cn <zhang.zheng@zte.com.cn>
> *Sent:* 09 March 2020 03:46
> *To:* kaduk@mit.edu; rdd@cert.org; barryleiba@computer.org; Rob Wilton
> (rwilton) <rwilton@cisco.com>; aretana.ietf@gmail.com; Reshad Rahman
> (rrahman) <rrahman@cisco.com>
> *Cc:* pim-chairs@ietf.org; pim@ietf.org
> *Subject:* Fw:[pim] I-D Action: draft-ietf-pim-msdp-yang-16.txt
>
>
>
> Hi Ben, Roman, Barry, Rob, Alvaro, Reshad,
>
>
>
> Thank you very much for your comments and modification suggestions.
>
> And many thanks to everyone who give us suggestions.
>
> We updated the draft to 16 version.
>
> Please review it. Hope this verson can be well.
>
>
>
> Best regards,
>
> Sandy
>
>
>
> 原始邮件
>
> *发件人：*internet-drafts@ietf.org <internet-drafts@ietf.org>
>
> *收件人：*i-d-announce@ietf.org <i-d-announce@ietf.org>;
>
> *抄送人：*pim@ietf.org <pim@ietf.org>;
>
> *日* *期* *：*2020年03月09日 11:26
>
> *主* *题* *：**[pim] I-D Action: draft-ietf-pim-msdp-yang-16.txt*
>
>
>
> A New Internet-Draft is available from the on-line Internet-Drafts directories.
> This draft is a work item of the Protocols for IP Multicast WG of the IETF.
>
>
>         Title           : A YANG Data Model for Multicast Source Discovery Protocol (MSDP)
>         Authors         : Xufeng Liu
>                           Zheng Zhang
>                           Anish Peter
>                           Mahesh Sivakumar
>                           Feng Guo
>                           Pete McAllister
>     Filename        : draft-ietf-pim-msdp-yang-16.txt
>     Pages           : 34
>     Date            : 2020-03-08
>
> Abstract:
>    This document defines a YANG data model for the configuration and
>    management of Multicast Source Discovery Protocol (MSDP) Protocol.
>
>
> The IETF datatracker status page for this draft is:
> https://datatracker.ietf.org/doc/draft-ietf-pim-msdp-yang/
>
> There are also htmlized versions available at:
> https://tools.ietf.org/html/draft-ietf-pim-msdp-yang-16
> https://datatracker.ietf.org/doc/html/draft-ietf-pim-msdp-yang-16
>
> A diff from the previous version is available at:
> https://www.ietf.org/rfcdiff?url2=draft-ietf-pim-msdp-yang-16
>
>
>
> Please note that it may take a couple of minutes from the time of submission
> until the htmlized version and diff are available at tools.ietf.org.
>
> Internet-Drafts are also available by anonymous FTP at:
> ftp://ftp.ietf.org/internet-drafts/
>
>
> _______________________________________________
> pim mailing list
> pim@ietf.org
> https://www.ietf.org/mailman/listinfo/pim
>
>
> _______________________________________________
> pim mailing list
> pim@ietf.org
> https://www.ietf.org/mailman/listinfo/pim
>