Re: [MIB-DOCTORS] Update to draft-ietf-bess-mvpn-mib-07

Glenn Mansfield Keeni <glenn@cysols.com> Sat, 28 July 2018 12:36 UTC

Return-Path: <glenn@cysols.com>
X-Original-To: mib-doctors@ietfa.amsl.com
Delivered-To: mib-doctors@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id ED707130EDB; Sat, 28 Jul 2018 05:36:31 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -0.5
X-Spam-Level:
X-Spam-Status: No, score=-0.5 tagged_above=-999 required=5 tests=[BAYES_05=-0.5, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id AWfT93R2ctb9; Sat, 28 Jul 2018 05:36:30 -0700 (PDT)
Received: from niseko.cysol.co.jp (niseko.cysol.co.jp [210.233.3.236]) (using TLSv1 with cipher DHE-RSA-CAMELLIA256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id C80AE130E83; Sat, 28 Jul 2018 05:36:28 -0700 (PDT)
Received: from [192.168.0.94] (cysvpn07.priv.cysol.co.jp [192.168.0.94]) (authenticated bits=0) by aso.priv.cysol.co.jp (8.14.9/8.14.9) with ESMTP id w6SCa8Bp082858 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES128-SHA bits=128 verify=NO); Sat, 28 Jul 2018 21:36:09 +0900 (JST) (envelope-from glenn@cysols.com)
From: Glenn Mansfield Keeni <glenn@cysols.com>
To: Hiroshi Tsunoda <tsuno@m.ieice.org>
Cc: mib-doctors@ietf.org, Mach Chen <mach.chen@huawei.com>, bess@ietf.org
References: <CAPbjwkxTvjiV-wyB-pY+1_51PRkNyGg5M23H-uTdOZHetsubyA@mail.gmail.com> <1912cdf9-ef2f-4631-a9dc-6d9bcd64b00f@cysols.com>
Message-ID: <a8c13d7a-883b-a0f6-cc3b-fe9dde295724@cysols.com>
Date: Sat, 28 Jul 2018 21:36:03 +0900
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1
MIME-Version: 1.0
In-Reply-To: <1912cdf9-ef2f-4631-a9dc-6d9bcd64b00f@cysols.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 8bit
Archived-At: <https://mailarchive.ietf.org/arch/msg/mib-doctors/wEqlG5gUs4xqfG_3MsE4OXQikSQ>
Subject: Re: [MIB-DOCTORS] Update to draft-ietf-bess-mvpn-mib-07
X-BeenThere: mib-doctors@ietf.org
X-Mailman-Version: 2.1.27
Precedence: list
List-Id: MIB Doctors list <mib-doctors.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/mib-doctors>, <mailto:mib-doctors-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/mib-doctors/>
List-Post: <mailto:mib-doctors@ietf.org>
List-Help: <mailto:mib-doctors-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/mib-doctors>, <mailto:mib-doctors-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sat, 28 Jul 2018 12:36:32 -0000

Dear Tsunoda,
Thanks for addressing all the comments.
There are a few additional comments. Could you please take
care of these?
1. Delete the last paragraph of IANA considerations.
2. Move
       mvpnBgpCmcastRouteWithdrawalTimer,
       mvpnBgpSrcSharedTreeJoinTimer,
    from mvpnBgpGroup to mvpnScalarGroup
    unless there is a good reason to keep those in mvpnBgpGroup

Once the above is done, I will be done with my review!
Thank you for your cooperation.

Glenn


On 2018/07/24 10:28, Glenn Mansfield Keeni wrote:
> Hi Tsuno,
>     Thanks for the quick response.
> Will confirm the changes and get back to you
> before the end of the week.
> 
> Glenn
> On 2018/07/23 20:44, Hiroshi Tsunoda wrote:
>> Hi Glenn,
>>
>> I have submitted the updated version of draft-ietf-bess-mvpn-mib.
>> Links to the draft and diff are as follows.
>>
>> URL:
>> https://www.ietf.org/internet-drafts/draft-ietf-bess-mvpn-mib-07.txt
>> Htmlized:       https://tools.ietf.org/html/draft-ietf-bess-mvpn-mib-07
>> Htmlized:       
>> https://datatracker.ietf.org/doc/html/draft-ietf-bess-mvpn-mib
>> Diff:           
>> https://www.ietf.org/rfcdiff?url2=draft-ietf-bess-mvpn-mib-07
>>
>> I believe that all comments that you gave as of now are
>> addressed in this version.
>> I hope this revision fulfills your requirements.
>>
>> ----- Major changes are as follows -------------------------------
>>
>>    - Simplification of branch structure
>>         mvpn{Generic, Config, States} branches are removed.
>>         All tables are directly under mvpnObjects branch now.
>>
>>    - Re-organization of table structures
>>         mvpnAdvtStatsTable was added and
>>         mvpn{Ipms, InterAsIpmsi, Spmsi}AdvtTable were removed.
>>         All objects representing advertisement statistics
>>         are defined in mvpnAdvtStatsTable now.
>>
>>    - Relocation and removal of some managed objects
>>
>>       - mvpnSPTunnelLimit, mvpnBgpCmcastRouteWithdrawalTimer, and
>>         mvpnBgpSrcSharedTreeJoinTimer are defined per PE instead of
>>         per MVPN.
>>       - mvpnSpmsiThreshold, mvpnSpmsiAdvtRefCnt, and
>>         some objects representing "address family" in a table
>>         are removed.
>>
>>    - Update of security considerations
>>
>> -------------------------------------------------------------------------------- 
>>
>>
>> -- tsuno
>>
>> _______________________________________________
>> BESS mailing list
>> BESS@ietf.org
>> https://www.ietf.org/mailman/listinfo/bess
>>
>