Re: [CCAMP] Mirja Kühlewind's No Objection on draft-ietf-ccamp-ospf-availability-extension-10: (with COMMENT)

"Mirja Kuehlewind (IETF)" <ietf@kuehlewind.net> Mon, 09 October 2017 08:21 UTC

Return-Path: <ietf@kuehlewind.net>
X-Original-To: ccamp@ietfa.amsl.com
Delivered-To: ccamp@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 7ACBB134CEA for <ccamp@ietfa.amsl.com>; Mon, 9 Oct 2017 01:21:19 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.001
X-Spam-Level:
X-Spam-Status: No, score=-2.001 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, SPF_PASS=-0.001] autolearn=unavailable autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); domainkeys=pass (1024-bit key) header.from=ietf@kuehlewind.net header.d=kuehlewind.net
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 V0BUKl3A7hiJ for <ccamp@ietfa.amsl.com>; Mon, 9 Oct 2017 01:21:16 -0700 (PDT)
Received: from kuehlewind.net (kuehlewind.net [83.169.45.111]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 89253134CEB for <ccamp@ietf.org>; Mon, 9 Oct 2017 01:15:19 -0700 (PDT)
DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=default; d=kuehlewind.net; b=qw5zYWlBOQdwA5+g3mTuXXEMhwXG+GbPcsyNMsiz6NwzdBjt4tGYiykW3Mgm9JzarZgg1vYbsDktwiPairyvt2UBGw9DORFmv78aSC7sbTics+FU9FIU7C4sV9I3T1mfhdbTSp6Jp+5tm/KN0sYcoIyha/KaLwdB7f6vkzK7QQs=; h=Received:Received:Content-Type:Mime-Version:Subject:From:In-Reply-To:Date:Cc:Content-Transfer-Encoding:Message-Id:References:To:X-Mailer:X-PPP-Message-ID:X-PPP-Vhost;
Received: (qmail 21596 invoked from network); 9 Oct 2017 10:15:17 +0200
Received: from pd9e11169.dip0.t-ipconnect.de (HELO ?192.168.178.33?) (217.225.17.105) by kuehlewind.net with ESMTPSA (DHE-RSA-AES256-SHA encrypted, authenticated); 9 Oct 2017 10:15:17 +0200
Content-Type: text/plain; charset="utf-8"
Mime-Version: 1.0 (Mac OS X Mail 10.3 \(3273\))
From: "Mirja Kuehlewind (IETF)" <ietf@kuehlewind.net>
In-Reply-To: <9C5FD3EFA72E1740A3D41BADDE0B461FC7590FDB@DGGEMA501-MBX.china.huawei.com>
Date: Mon, 09 Oct 2017 10:15:16 +0200
Cc: Greg Mirsky <gregimirsky@gmail.com>, "ccamp-chairs@ietf.org" <ccamp-chairs@ietf.org>, "ccamp@ietf.org" <ccamp@ietf.org>, Fatai Zhang <zhangfatai@huawei.com>, The IESG <iesg@ietf.org>, "draft-ietf-ccamp-ospf-availability-extension@ietf.org" <draft-ietf-ccamp-ospf-availability-extension@ietf.org>
Content-Transfer-Encoding: quoted-printable
Message-Id: <F8A97491-74B2-41AB-9FAC-F60BCCE1FB61@kuehlewind.net>
References: <150722574941.1338.17678983518705938171.idtracker@ietfa.amsl.com> <CA+RyBmW1T0zY_nDt15xgxOsdBtmuMSRa=2+bE8HF8eQGN8uswQ@mail.gmail.com> <9C5FD3EFA72E1740A3D41BADDE0B461FC7590FDB@DGGEMA501-MBX.china.huawei.com>
To: "Yemin (Amy)" <amy.yemin@huawei.com>
X-Mailer: Apple Mail (2.3273)
X-PPP-Message-ID: <20171009081517.21586.55111@lvps83-169-45-111.dedicated.hosteurope.de>
X-PPP-Vhost: kuehlewind.net
Archived-At: <https://mailarchive.ietf.org/arch/msg/ccamp/xFtEQkqTXceVwR2CIlmDr-eLHrM>
Subject: Re: [CCAMP] Mirja Kühlewind's No Objection on draft-ietf-ccamp-ospf-availability-extension-10: (with COMMENT)
X-BeenThere: ccamp@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: Discussion list for the CCAMP working group <ccamp.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ccamp>, <mailto:ccamp-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ccamp/>
List-Post: <mailto:ccamp@ietf.org>
List-Help: <mailto:ccamp-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ccamp>, <mailto:ccamp-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 09 Oct 2017 08:21:19 -0000

Great. Thanks!

> Am 09.10.2017 um 04:21 schrieb Yemin (Amy) <amy.yemin@huawei.com>:
> 
> Hi Mirja,
>  
> Thanks for the comments.
> We will change according to the first comment.
> For the second comment, I think Greg’s text is good. What do you think?
>  
> BR,
> Amy
> From: Greg Mirsky [mailto:gregimirsky@gmail.com] 
> Sent: Saturday, October 07, 2017 1:23 AM
> To: Mirja Kühlewind <ietf@kuehlewind.net>
> Cc: The IESG <iesg@ietf.org>; draft-ietf-ccamp-ospf-availability-extension@ietf.org; Fatai Zhang <zhangfatai@huawei.com>; ccamp-chairs@ietf.org; ccamp@ietf.org
> Subject: Re: Mirja Kühlewind's No Objection on draft-ietf-ccamp-ospf-availability-extension-10: (with COMMENT)
>  
> Hi Mirja,
> would the following be acceptable to address your latter comment:
> OLD
> "Length: A 16 bits field that expresses the length of the TLV in
>    bytes. "
> NEW
> "Length: 2 octets, 16 bits."
>  
> Regards,
> Greg
>  
>  
> On Thu, Oct 5, 2017 at 10:49 AM, Mirja Kühlewind <ietf@kuehlewind.net> wrote:
> Mirja Kühlewind has entered the following ballot position for
> draft-ietf-ccamp-ospf-availability-extension-10: No Objection
> 
> When responding, please keep the subject line intact and reply to all
> email addresses included in the To and CC lines. (Feel free to cut this
> introductory paragraph, however.)
> 
> 
> Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html
> for more information about IESG DISCUSS and COMMENT positions.
> 
> 
> The document, along with other ballot positions, can be found here:
> https://datatracker.ietf.org/doc/draft-ietf-ccamp-ospf-availability-extension/
> 
> 
> 
> ----------------------------------------------------------------------
> COMMENT:
> ----------------------------------------------------------------------
> 
> This could lead to random outcomes
> "If multiple are present, only the first Availability
>    SCSI-TLV for an availability level carried in the same ISCD SHALL be
>    processed. "
> I would suggest the following instead:
> "If multiple are present, the Availability
>    SCSI-TLV with the lowest bandwidth value SHALL be
>    processed. "
> 
> Nit:
> In section 3.1 you may actually specify the actual length value as done for the type:
> OLD
> "Length: A 16 bits field that expresses the length of the TLV in
>    bytes. "
> NEW
> "Length: 4 (bytes), 16 bits."
>