Re: [Int-dir] Int-Dir - Review of draft-ietf-softwire-dslite-mib-11

"Yu Fu" <fuyu@cnnic.cn> Thu, 26 November 2015 09:10 UTC

Return-Path: <fuyu@cnnic.cn>
X-Original-To: int-dir@ietfa.amsl.com
Delivered-To: int-dir@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 4A1991A904B; Thu, 26 Nov 2015 01:10:49 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -3.186
X-Spam-Level:
X-Spam-Status: No, score=-3.186 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_LOW=-0.7, RP_MATCHES_RCVD=-0.585, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001] autolearn=ham
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 5Lm7FqckEQTb; Thu, 26 Nov 2015 01:10:43 -0800 (PST)
Received: from cnnic.cn (smtp13.cnnic.cn [218.241.118.13]) by ietfa.amsl.com (Postfix) with ESMTP id 79A121B378C; Thu, 26 Nov 2015 01:10:42 -0800 (PST)
Received: from LIUXD (unknown [218.241.103.218]) by ocmail02.zx.nicx.cn (Coremail) with SMTP id AQAAf0ApMEyDzFZWKkDNCA--.65493S3; Thu, 26 Nov 2015 17:10:27 +0800 (CST)
From: Yu Fu <fuyu@cnnic.cn>
To: "'Carlos Pignataro (cpignata)'" <cpignata@cisco.com>, int-dir@ietf.org, int-ads@ietf.org
References: <A3D39C46-3E64-40EA-9D7C-E5818C1CFFAF@cisco.com>
In-Reply-To: <A3D39C46-3E64-40EA-9D7C-E5818C1CFFAF@cisco.com>
Date: Thu, 26 Nov 2015 17:10:33 +0800
Message-ID: <000c01d1282a$4ebb24c0$ec316e40$@cn>
MIME-Version: 1.0
Content-Type: multipart/alternative; boundary="----=_NextPart_000_000D_01D1286D.5CDE64C0"
X-Mailer: Microsoft Office Outlook 12.0
Thread-Index: AQHRDPv8uQQq0547Q0u+jVdDMLjGGp6uIPpQ
Content-Language: zh-cn
X-CM-TRANSID: AQAAf0ApMEyDzFZWKkDNCA--.65493S3
X-Coremail-Antispam: 1UD129KBjvJXoWxXr4fuF1fWFWxCFWkKFyDJrb_yoW5tr45pF s7W3y5Kryktr1xJrn7uF48Wa4Yka9agFWagFyDCr1UAws8G3Z2yryF9ay5AayDGryfWF48 Zw12vr93Wan5ZFJanT9S1TB71UUUUU7qnTZGkaVYY2UrUUUUjbIjqfuFe4nvWSU5nxnvy2 9KBjDU0xBIdaVrnRJUUUBab7Iv0xC_Kw4lb4IE77IF4wAFF20E14v26r1j6r4UM7CY07I2 0VC2zVCF04k26cxKx2IYs7xG6rWj6s0DM7CIcVAFz4kK6r1j6r18M28lY4IEw2IIxxk0rw A2F7IY1VAKz4vEj48ve4kI8wA2z4x0Y4vE2Ix0cI8IcVAFwI0_Xr0_Ar1l84ACjcxK6xII jxv20xvEc7CjxVAFwI0_Cr0_Gr1UM28EF7xvwVC2z280aVAFwI0_GcCE3s1l84ACjcxK6I 8E87Iv6xkF7I0E14v26rxl6s0DM2AIxVAIcxkEcVAq07x20xvEncxIr21l5I8CrVAYj202 j2C_Xr0_Wr1l5I8CrVAqjxCE14ACF2xKxwAqx4xG64kEw2xG04xIwI0_Jr0_Gr1l5I8CrV CF0I0E4I0vr24lYx0E2Ix0cI8IcVAFwI0_Jr0_Jr4lYx0Ex4A2jsIE14v26r1j6r4UMcvj eVCFs4IE7xkEbVWUJVW8JwACjcxG0xvY0x0EwIxGrwCY02Avz4vE14v_GFWl42xK82IYc2 Ij64vIr41l4I8I3I0E4IkC6x0Yz7v_Jr0_Gr1lx2IqxVAqx4xG67AKxVWUGVWUWwC20s02 6x8GjcxK67AKxVWUGVWUWwC2zVAF1VAY17CE14v26r126r1DMIIYrxkI7VAKI48JMIIF0x vE2Ix0cI8IcVAFwI0_Jr0_JF4lIxAIcVC0I7IYx2IY6xkF7I0E14v26r1j6r4UMIIF0xvE 42xK8VAvwI8IcIk0rVWrJr0_WFyUJwCI42IY6I8E87Iv67AKxVWUJVW8JwCI42IY6I8E87 Iv6xkF7I0E14v26r1j6r4UYxBIdaVFxhVjvjDU0xZFpf9x07boUDXUUUUU=
X-CM-SenderInfo: pix13q5fqqxugofq/
Archived-At: <http://mailarchive.ietf.org/arch/msg/int-dir/tOgXHZjAjFp1DYcvnVRbi7cNj2E>
X-Mailman-Approved-At: Thu, 26 Nov 2015 06:22:31 -0800
Cc: draft-ietf-softwire-dslite-mib.all@ietf.org
Subject: Re: [Int-dir] Int-Dir - Review of draft-ietf-softwire-dslite-mib-11
X-BeenThere: int-dir@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: "This list is for discussion between the members of the Internet Area directorate." <int-dir.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/int-dir>, <mailto:int-dir-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/int-dir/>
List-Post: <mailto:int-dir@ietf.org>
List-Help: <mailto:int-dir-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/int-dir>, <mailto:int-dir-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 26 Nov 2015 09:10:49 -0000

Hi Carlos,

 

Thanks for your review. We have updated our draft based on you comments. Please see my reply inline.

 

>>CMP: Should the above say that this only supports IPv4-in-IPv6?

 

[Yu]:Yes, it only supports IPv4 in IPv6 tunnel.

 

>>CMP: Might be useful to add that this is because the tunnel is not point-to-point.

 

[Yu]: We have added it in the updated version.

 

>>CMP:Has this been checked? https://www.ietf.org/iesg/statement/writable-mib-module.html

 

[Yu]:It has been reviewed by mib doctor during the WGLC.  I am not sure is that could be considered as it has been checked?

 

>>CMP: I only saw one read-write and no read-create. Are there “a number of …”?

 

[Yu]:We have updated this sentence to “There is only one objects defined in this MIB module…..”

 

>>CMP: Why is RFC 2119 Informative?

 

[Yu]: We have changed it into Normative References in the new version.

 

Thanks again

 

BR

Yu

 

From: Carlos Pignataro (cpignata) [mailto:cpignata@cisco.com] 
Sent: Friday, October 23, 2015 3:01 AM
To: int-dir@ietf.org; int-ads@ietf.org
Cc: draft-ietf-softwire-dslite-mib.all@ietf.org
Subject: Int-Dir - Review of draft-ietf-softwire-dslite-mib-11 

 

Hi,

 

I am an assigned INT directorate reviewer for draft-ietf-softwire-dslite-mib-11. These comments were written primarily for the benefit of the Internet Area Directors. Document editors and shepherd(s) should treat these comments just like they would treat comments from any other IETF contributors and resolve them along with any other Last Call comments that have been received. For more details on the INT Directorate, see http://www.ietf.org/iesg/directorate/intarea.html.

 

This document defines MIB objects to manage DS-Lite solutions, and targets the Standards Track.

 

Please find some minor review comments:

 

5.  Difference from the IP tunnel MIB and NATV2-MIB

 

   Notes: According to section 5.2 of [RFC6333], DS-Lite only defines

   IPv4 in IPv6 tunnels at this moment, but other types of encapsulation

   could be defined in the future.  So this DS-Lite MIB only supports IP

   in IP encapsulation, if another RFC defined other tunnel types in the

   future, this DS-Lite MIB will be updated then.

 

CMP: Should the above say that this only supports IPv4-in-IPv6?

 

   The implementation of the IP Tunnel MIB is required for DS-Lite.  The

   tunnelIfEncapsMethod in the tunnelIfEntry should be set to

   dsLite("xx"), and a corresponding entry in the DS-Lite module will

   exist for every tunnelIfEntry with this tunnelIfEncapsMethod.  The

   tunnelIfRemoteInetAddress must be set to "::”.

 

CMP: Might be useful to add that this is because the tunnel is not point-to-point.

 

      dsliteAFTRAlarmConnectNumber OBJECT-TYPE

         SYNTAX Integer32 (60..90)

         MAX-ACCESS read-write

 

CMP: Has this been checked? https://www.ietf.org/iesg/statement/writable-mib-module.html

 

9.  Security Considerations

   There are a number of management objects defined in this MIB module

   with a MAX-ACCESS clause of read-write and/or read-create. 

 

CMP: I only saw one read-write and no read-create. Are there “a number of …”?

 

12.2.  Informative References

 

   [RFC2119]  Bradner, S., "Key words for use in RFCs to Indicate

              Requirement Levels", BCP 14, RFC 2119,

              DOI 10.17487/RFC2119, March 1997,

              <http://www.rfc-editor.org/info/rfc2119>.

 

CMP: Why is RFC 2119 Informative?

 

I hope these are useful!

 

Thanks,

 

— Carlos.