Re: [Gen-art] Genart last call review of draft-ietf-dmm-ondemand-mobility-15

"Moses, Danny" <danny.moses@intel.com> Mon, 07 January 2019 09:52 UTC

Return-Path: <danny.moses@intel.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 35F9812D4F1; Mon, 7 Jan 2019 01:52:01 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.901
X-Spam-Level:
X-Spam-Status: No, score=-6.901 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_HI=-5, SPF_PASS=-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 9PXsCT9DYA16; Mon, 7 Jan 2019 01:51:59 -0800 (PST)
Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id A71E3126BED; Mon, 7 Jan 2019 01:51:55 -0800 (PST)
X-Amp-Result: SKIPPED(no attachment in message)
X-Amp-File-Uploaded: False
Received: from orsmga008.jf.intel.com ([10.7.209.65]) by orsmga103.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 07 Jan 2019 01:51:54 -0800
X-ExtLoop1: 1
X-IronPort-AV: E=Sophos;i="5.56,450,1539673200"; d="scan'208";a="107834573"
Received: from fmsmsx104.amr.corp.intel.com ([10.18.124.202]) by orsmga008.jf.intel.com with ESMTP; 07 Jan 2019 01:51:53 -0800
Received: from fmsmsx120.amr.corp.intel.com (10.18.124.208) by fmsmsx104.amr.corp.intel.com (10.18.124.202) with Microsoft SMTP Server (TLS) id 14.3.408.0; Mon, 7 Jan 2019 01:51:53 -0800
Received: from HASMSX109.ger.corp.intel.com (10.184.198.21) by fmsmsx120.amr.corp.intel.com (10.18.124.208) with Microsoft SMTP Server (TLS) id 14.3.408.0; Mon, 7 Jan 2019 01:51:53 -0800
Received: from hasmsx106.ger.corp.intel.com ([169.254.10.8]) by hasmsx109.ger.corp.intel.com ([169.254.3.40]) with mapi id 14.03.0415.000; Mon, 7 Jan 2019 11:51:50 +0200
From: "Moses, Danny" <danny.moses@intel.com>
To: Russ Housley <housley@vigilsec.com>
CC: "draft-ietf-dmm-ondemand-mobility.all@ietf.org" <draft-ietf-dmm-ondemand-mobility.all@ietf.org>, "ietf@ietf.org" <ietf@ietf.org>, "dmm@ietf.org" <dmm@ietf.org>, "gen-art@ietf.org" <gen-art@ietf.org>
Thread-Topic: Genart last call review of draft-ietf-dmm-ondemand-mobility-15
Thread-Index: AQHUo7fHoXJLHmSsSEuV/AzLEJhaF6WjlKfQ
Date: Mon, 07 Jan 2019 09:51:50 +0000
Message-ID: <F0CF5715D3D1884BAC731EA1103AC281441B5FA1@HASMSX106.ger.corp.intel.com>
References: <154655627155.29540.12228087320701274157@ietfa.amsl.com>
In-Reply-To: <154655627155.29540.12228087320701274157@ietfa.amsl.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-ctpclassification: CTP_NT
x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiY2E2NDYzMDMtOTc0MC00YTExLWFiZGItZDZmNzFhOGRmNjEzIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiK3pMekZ5NTRXTDk0OHNLbnRCTFlGaXY1aHlZXC9lVlRtXC84YXp4NjZqeTU3K09pWGZNN2M1RUZSWGVSSVc3WGxuIn0=
dlp-product: dlpe-windows
dlp-version: 11.0.400.15
dlp-reaction: no-action
x-originating-ip: [10.124.184.102]
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: base64
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/UbjWEk07HeXutlV3XXM0xjowhpM>
Subject: Re: [Gen-art] Genart last call review of draft-ietf-dmm-ondemand-mobility-15
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.29
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: Mon, 07 Jan 2019 09:52:01 -0000

Hi Russ,

Thank you for your comments and questions. Please see the responses to each issue.
I copied the comments and followed each one with a response for ease of review.


Minor Concerns:

Section 2: Please update the first paragraph to reference RFC 8174 in addition to RFC 2119, as follows: 

   The key words "MUST", "MUST NOT", "REQUIRED", "SHALL", "SHALL NOT",
   "SHOULD", "SHOULD NOT", "RECOMMENDED", "NOT RECOMMENDED", "MAY", and
   "OPTIONAL" in this document are to be interpreted as described in
   BCP 14 [RFC2119] [RFC8174] when, and only when, they appear in all
   capitals, as shown here.
>>Response: Yes, I will apply the newer text and add a reference to RFC8174.


Nits:

Section 1: s/It should be noted that in/In/
>>Response: I will apply the change.


Section 1 uses one style for listing two properties, and then Section 3 uses another style for listing four types of IP address.  Please pick one style and use it in both places.
>>Response: I am changing the listing style in section 1 to the one used in section 3.


Section 4.1: s/secsc(/setsc(/  -- in a comment
>>Response: I will correct the typo in the comment


Questions:

Should getsc() also be described in Section 6?
>>Response: Actually, ‘getsc()’ is a typo. The draft proposes only one new API: ‘setsc()’. I will correct all occurrences of ‘getsc()’ to ‘setsc()’.

Should anything be added to the Security Considerations about CGA?
>>Response: I do not think so. This draft does not deal with how source IP addresses are generated. Specifically, it does not change the way a source IPv6 addresses are constructed from a given source IPv6 prefix. It only specifies a way for applications to express their desire regarding the type of service provided by networks for mobility management.  







Thanks and regards,
Danny



-----Original Message-----
From: Russ Housley [mailto:housley@vigilsec.com] 
Sent: Friday, January 04, 2019 00:58
To: gen-art@ietf.org
Cc: draft-ietf-dmm-ondemand-mobility.all@ietf.org; ietf@ietf.org; dmm@ietf.org
Subject: Genart last call review of draft-ietf-dmm-ondemand-mobility-15

Reviewer: Russ Housley
Review result: Almost Ready

I am the assigned Gen-ART reviewer for this draft. The General Area Review Team (Gen-ART) reviews all IETF documents being processed by the IESG for the IETF Chair.  Please treat these comments just like any other last call comments.

For more information, please see the FAQ at <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.

Document: draft-ietf-dmm-ondemand-mobility-15
Reviewer: Russ Housley
Review Date: 2019-01-03
IETF LC End Date: 2019-01-16
IESG Telechat date: unknown

Summary: Almost Ready


Major Concerns:

None.


Minor Concerns:

Section 2: Please update the first paragraph to reference RFC 8174 in addition to RFC 2119, as follows: 

   The key words "MUST", "MUST NOT", "REQUIRED", "SHALL", "SHALL NOT",
   "SHOULD", "SHOULD NOT", "RECOMMENDED", "NOT RECOMMENDED", "MAY", and
   "OPTIONAL" in this document are to be interpreted as described in
   BCP 14 [RFC2119] [RFC8174] when, and only when, they appear in all
   capitals, as shown here.


Nits:

Section 1: s/It should be noted that in/In/

Section 1 uses one style for listing two properties, and then Section 3 uses another style for listing four types of IP address.  Please pick one style and use it in both places.

Section 4.1: s/secsc(/setsc(/  -- in a comment


Questions:

Should getsc() also be described in Section 6?

Should anything be added to the Security Considerations about CGA?


---------------------------------------------------------------------
A member of the Intel Corporation group of companies

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.