Re: [netext] Gen-ART LC review: draft-ietf-netext-update-notifications-07

Suresh Krishnan <suresh.krishnan@ericsson.com> Thu, 26 September 2013 06:43 UTC

Return-Path: <suresh.krishnan@ericsson.com>
X-Original-To: netext@ietfa.amsl.com
Delivered-To: netext@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 3A5CD21F9FD5; Wed, 25 Sep 2013 23:43:02 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -102.599
X-Spam-Level:
X-Spam-Status: No, score=-102.599 tagged_above=-999 required=5 tests=[AWL=0.000, BAYES_00=-2.599, USER_IN_WHITELIST=-100]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id wrtZ9pTJqqZY; Wed, 25 Sep 2013 23:42:56 -0700 (PDT)
Received: from usevmg20.ericsson.net (usevmg20.ericsson.net [198.24.6.45]) by ietfa.amsl.com (Postfix) with ESMTP id 9A95021F9A96; Wed, 25 Sep 2013 23:42:56 -0700 (PDT)
X-AuditID: c618062d-b7fda8e0000024c6-a0-5243d76fe87c
Received: from EUSAAHC008.ericsson.se (Unknown_Domain [147.117.188.96]) by usevmg20.ericsson.net (Symantec Mail Security) with SMTP id 0D.CD.09414.F67D3425; Thu, 26 Sep 2013 08:42:55 +0200 (CEST)
Received: from EUSAAMB107.ericsson.se ([147.117.188.124]) by EUSAAHC008.ericsson.se ([147.117.188.96]) with mapi id 14.02.0328.009; Thu, 26 Sep 2013 02:42:54 -0400
From: Suresh Krishnan <suresh.krishnan@ericsson.com>
To: Robert Sparks <rjsparks@nostrum.com>
Thread-Topic: [netext] Gen-ART LC review: draft-ietf-netext-update-notifications-07
Thread-Index: AQHOn3viWuJBnfE5x0mfHumRiJHjvJmiB4GAgDObmQCAAcit4A==
Date: Thu, 26 Sep 2013 06:42:54 +0000
Message-ID: <E87B771635882B4BA20096B589152EF6808395@eusaamb107.ericsson.se>
References: <521652E0.8030300@nostrum.com> <52167D67.2010103@ericsson.com> <521687CD.40502@nostrum.com> <5241D27E.4040201@nostrum.com>
In-Reply-To: <5241D27E.4040201@nostrum.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [147.117.188.135]
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFrrLLMWRmVeSWpSXmKPExsUyuXRPgm7+decggy0NRhYPNixhtLj66jOL xbWfT9ktrs1pZHNg8Viy5CeTx6ydT1g8vlz+zBbAHMVlk5Kak1mWWqRvl8CVsXP2BraCTbIV J8+cZm5gbBTvYuTkkBAwkbg4YT0zhC0mceHeerYuRi4OIYGjjBJ3J79iBEkICSxnlDh+pxzE ZgNq2LDzMxOILSKgIXFtyRJ2EJtZ4CSjxPW2XBBbWCBE4v2vB6wQNaESH3/2s0PYThITby0E 62URUJX4erUJrIZXwFtixtpXzBCLOxglzvVuBruIU0BbYt6zdrBmRqDrvp9awwSxTFzi1pP5 TBBXC0gs2XMe6gNRiZeP/7FC2MoS3+c8YoGo15FYsPsTG4StLbFs4WtmiMWCEidnPmGZwCg2 C8nYWUhaZiFpmYWkZQEjyypGjtLi1LLcdCODTYzAKDomwaa7g3HPS8tDjNIcLErivKv0zgQK CaQnlqRmp6YWpBbFF5XmpBYfYmTi4JRqYKx4pJ2ZI+Ky9ekTzZZvoZaP0yq2W4UIdX+WzdjL ZfB4y6JSbiNJjU3v6to9D3mnfM1X5FerPlnHH6qdnpUom/8u2HNixNuPRW2GeglcjMIcv30L 1FVnOB62Zy4Mlbunv0Zjva9Nb07broqp7aJWlY7nfZV6ufvWK8j9q1zFud5LlE/uZIcSS3FG oqEWc1FxIgAlVOG6cAIAAA==
Cc: "netext@ietf.org" <netext@ietf.org>, General Area Review Team <gen-art@ietf.org>, "draft-ietf-netext-update-notifications@tools.ietf.org" <draft-ietf-netext-update-notifications@tools.ietf.org>
Subject: Re: [netext] Gen-ART LC review: draft-ietf-netext-update-notifications-07
X-BeenThere: netext@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: "Mailing list for discusion of extensions to network mobility protocol, i.e PMIP6. " <netext.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/netext>, <mailto:netext-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/netext>
List-Post: <mailto:netext@ietf.org>
List-Help: <mailto:netext-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/netext>, <mailto:netext-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 26 Sep 2013 06:43:02 -0000

Hi Robert,
  Due to an editorial mixup the changes did not make it to -08.  We apologize for that. We have made the changes to the draft now and will ensure that they are included in the -09 rev or RFC editor notes depending on whether any other changes are needed.

Thanks
Suresh

> -----Original Message-----
> From: Robert Sparks [mailto:rjsparks@nostrum.com]
> Sent: September-24-13 1:57 PM
> To: Suresh Krishnan
> Cc: General Area Review Team; netext@ietf.org; draft-ietf-netext-update-
> notifications@tools.ietf.org
> Subject: Re: [netext] Gen-ART LC review: draft-ietf-netext-update-
> notifications-07
> 
> Hi Suresh -
> 
> It doesn't look like any of these changes made it into -08?
> 
> RjS
> 
> On 8/22/13 4:51 PM, Robert Sparks wrote:
> > On 8/22/13 4:06 PM, Suresh Krishnan wrote:
> >> Hi Robert,
> >>    Thanks a lot for the review. We will include the changes in the
> >> next revision we submit. Please see proposed changes inline.
> >>
> >> On 08/22/2013 02:05 PM, Robert Sparks wrote:
> >>> 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>.
> >>>
> >>> Please resolve these comments along with any other Last Call
> >>> comments you may receive.
> >>>
> >>> Document: draft-ietf-netext-update-notifications-07
> >>> Reviewer: Robert Sparks
> >>> Review Date: 2013-08-22
> >>> IETF LC End Date: 2013-08-29
> >>> IESG Telechat date: not scheduled
> >>>
> >>> Summary: This draft is ready for publication as Proposed Standard
> >>>
> >>> I had to read through this text several times to convince myself
> >>> implementers could figure out what order they were required to take
> >>> steps in vs where they had flexibility:
> >>>
> >>>     o  Upon accepting the Update Notification message, the mobile
> >>> access
> >>>        gateway MUST process the message and perform the actions
> >>> based on
> >>>        the Notification Reason.
> >>>        *  If the (A) flag in the message is set to a value of (1), the
> >>>           mobile access gateway MUST first send an Update Notification
> >>>           Acknowledgement message and set the status code field
> >>> according
> >>>           to the result of processing the Update Notification message.
> >>>
> >>> In particular, it's not immediately obvious if there is tension
> >>> between that "MUST first" and having "the result of processing"
> available.
> >>> Please consider rewording to make it clearer that this "result of
> >>> processing" is not intended to include waiting for the result of
> >>> some action processing this notification message might trigger.
> >> I think we can lose the word first without losing anything. Does the
> >> following rewording work for you?
> > Yes, thanks!
> >>
> >> OLD:
> >> If the (A) flag in the message is set to a value of (1), the mobile
> >> access gateway MUST first send an Update Notification
> Acknowledgement
> >> message and set the status code field according to the result of
> >> processing the Update Notification message.
> >>
> >> NEW:
> >> If the (A) flag in the message is set to a value of (1), the mobile
> >> access gateway MUST send an Update Notification Acknowledgement
> >> message with the status code field set based on the result of
> >> processing the Update Notification message.
> >>
> >>> It might help readers understand the intended usual case
> >>> retransmission mechanics if the expected default values listed in
> >>> section 7 were called out earlier in the document.
> >> Sure. Will call out the defaults at first use.
> >>
> >> Thanks
> >> Suresh
> >>
> >