[Gen-art] Gen-ART review of draft-ietf-v6ops-siit-dc-2xlat-01

Christer Holmberg <christer.holmberg@ericsson.com> Thu, 17 September 2015 10:13 UTC

Return-Path: <christer.holmberg@ericsson.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id BBB351B2C9D for <gen-art@ietfa.amsl.com>; Thu, 17 Sep 2015 03:13:23 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.2
X-Spam-Level:
X-Spam-Status: No, score=-4.2 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_MED=-2.3, 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 Mq6Y1QcqXCMh for <gen-art@ietfa.amsl.com>; Thu, 17 Sep 2015 03:13:20 -0700 (PDT)
Received: from sessmg23.ericsson.net (sessmg23.ericsson.net [193.180.251.45]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 139DA1B2C8F for <gen-art@ietf.org>; Thu, 17 Sep 2015 03:13:19 -0700 (PDT)
X-AuditID: c1b4fb2d-f79626d000004282-94-55fa923d2628
Received: from ESESSHC009.ericsson.se (Unknown_Domain [153.88.253.124]) by sessmg23.ericsson.net (Symantec Mail Security) with SMTP id E0.63.17026.D329AF55; Thu, 17 Sep 2015 12:13:18 +0200 (CEST)
Received: from ESESSMB209.ericsson.se ([169.254.9.81]) by ESESSHC009.ericsson.se ([153.88.183.45]) with mapi id 14.03.0248.002; Thu, 17 Sep 2015 12:13:17 +0200
From: Christer Holmberg <christer.holmberg@ericsson.com>
To: "gen-art@ietf.org" <gen-art@ietf.org>
Thread-Topic: Gen-ART review of draft-ietf-v6ops-siit-dc-2xlat-01
Thread-Index: AdDxKwm6Kg1XXwLDQu6Os4qiD+X9Dw==
Date: Thu, 17 Sep 2015 10:13:17 +0000
Message-ID: <7594FB04B1934943A5C02806D1A2204B37A84174@ESESSMB209.ericsson.se>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [153.88.183.147]
Content-Type: multipart/alternative; boundary="_000_7594FB04B1934943A5C02806D1A2204B37A84174ESESSMB209erics_"
MIME-Version: 1.0
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrFLMWRmVeSWpSXmKPExsUyM+Jvja7dpF+hBrc2c1icPuRtcfXVZxYH Jo8lS34yeXy5/JktgCmKyyYlNSezLLVI3y6BK+PBnE62gneVFa8OXGdsYDyS08XIySEhYCLx 7sp9RghbTOLCvfVsXYxcHEICRxklOnfcgnIWM0os2TuHqYuRg4NNwEKi+582SIOIgKbE3BVv mUBsZoFcidbGu2CDhAVsJKZMnMEGUeMo8fp7AwtIq4iAnsTcf0YgYRYBVYnm5bNZQWxeAV+J K0tbwFoZgW74fmoN1EhxiVtP5jNB3CYgsWTPeWYIW1Ti5eN/rCAjJQSUJKZtTYMoz5dobf/J CDFSUOLkzCcsExiFZyGZNAtJ2SwkZRBxHYkFuz+xQdjaEssWvmaGsc8ceMyELL6AkX0Vo2hx anFxbrqRsV5qUWZycXF+nl5easkmRmDkHNzyW3cH4+rXjocYBTgYlXh4H4T9ChViTSwrrsw9 xCjNwaIkztvC9CBUSCA9sSQ1OzW1ILUovqg0J7X4ECMTB6dUA6PfxOerT3fcbnpg/t/N/OPZ MoaCCaujby5o3N4gmpD27h2n1YZJ5x+kslmXavIqVfUxXZu4LbDRLvxXzsqnpn8NOrKj9k/8 Hda6THB+2qbUA+nbg1efXFi/2fBk6xae1g3N+X8MObnsa+br+6TNkT12/4HypPreCZ+m3k9h PDjr3vrwI8aL/tcpsRRnJBpqMRcVJwIA8sbdEH0CAAA=
Archived-At: <http://mailarchive.ietf.org/arch/msg/gen-art/SmLIODZHzfYX4oe9Qlz6sHiCkq8>
Cc: "draft-ietf-v6ops-siit-dc-2xlat.all@tools.ietf.org" <draft-ietf-v6ops-siit-dc-2xlat.all@tools.ietf.org>
Subject: [Gen-art] Gen-ART review of draft-ietf-v6ops-siit-dc-2xlat-01
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 17 Sep 2015 10:13:23 -0000

I am the assigned Gen-ART reviewer for this draft. For background on Gen-ART, please see the FAQ at <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>
Document:                                   draft-ietf-v6ops-siit-dc-2xlat-01.txt
Reviewer:                                     Christer Holmberg
Review Date:                               17 September 2015
IETF LC End Date:                       22 September 2015
IETF Telechat Date:                   N/A
Summary:                                     The document is well written, and almost ready for publication. However, there are a few editorial nits that I ask the author to address.
Major Issues: None
Minor Issues: None
Editorial Issues:

Section 2 (Terminology):
------------------------------

Q2_1: Many of the definitions have been defined in draft-ietf-v6ops-siit-dc. Now they are re-defined, and sometimes with a little different wording.

For those definitions, my suggestion would be to say:

"As defined in [draft-ietf-v6ops-siit-dc], a XXX is a blah blah blah" - copy/pasting the text from draft-ietf-v6ops-siit-dc.


Q2_2: In the Edge Relay, I think it would be good to mention the two types (node-based and network-based).


Section 4 (Deployment Considerations):
---------------------------------------------------

Q4_1:

The text in section 4.1. says:

                             "The IPv6 Path MTU between the ER and the BR will typically be larger
   than the default value defined in Section 4 of [RFC6145] (1280),"

What is (1280)?


Section 5 (Intra-IDC IPv4 Communication):
---------------------------------------------------

Q5_1:

The text in section 5.1 says:

"If the BR supports hairpinning as described in Section 4.2 of I-D
   .ietf-v6ops-siit-eam [I-D.ietf-v6ops-siit-eam],"

I suggest to remove I-D.ietf-v6ops-siit-eam. The reference is enough.


Section 7 (IANA Considerations):
----------------------------------------

Q7_1: Do we normally remove the section if there are no requests from IANA? Personally I prefer to keep the explicit "This draft makes no request of the IANA." sentence.

(I had the same comment on draft-ietf-v6ops-siit-dc, so whatever the outcome is it can be applied to both documents).


Section 8 (Security Considerations):
----------------------------------------

Q8_1:

The text says:

"See the Security Considerations section in
   [I-D.ietf-v6ops-siit-dc] for additional security considerations
   applicable to the SIIT-DC architecture in general."

I suggest to remove "additional".


Q8_2:

Is there a need to have section 8.1, or can all text be put in section 8?