Re: [mpls] AD review of draft-ietf-mpls-entropy-label

"Adrian Farrel" <adrian@olddog.co.uk> Mon, 20 August 2012 18:42 UTC

Return-Path: <adrian@olddog.co.uk>
X-Original-To: mpls@ietfa.amsl.com
Delivered-To: mpls@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id C30A821F8606 for <mpls@ietfa.amsl.com>; Mon, 20 Aug 2012 11:42:01 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.519
X-Spam-Level:
X-Spam-Status: No, score=-2.519 tagged_above=-999 required=5 tests=[AWL=0.080, BAYES_00=-2.599]
Received: from mail.ietf.org ([64.170.98.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id qywrn+zLA6Nl for <mpls@ietfa.amsl.com>; Mon, 20 Aug 2012 11:42:01 -0700 (PDT)
Received: from asmtp3.iomartmail.com (asmtp3.iomartmail.com [62.128.201.159]) by ietfa.amsl.com (Postfix) with ESMTP id 05A7621F84F5 for <mpls@ietf.org>; Mon, 20 Aug 2012 11:42:00 -0700 (PDT)
Received: from asmtp3.iomartmail.com (localhost.localdomain [127.0.0.1]) by asmtp3.iomartmail.com (8.13.8/8.13.8) with ESMTP id q7KIfxV0001564; Mon, 20 Aug 2012 19:41:59 +0100
Received: from 950129200 (dsl-sp-81-140-15-32.in-addr.broadbandscope.com [81.140.15.32]) (authenticated bits=0) by asmtp3.iomartmail.com (8.13.8/8.13.8) with ESMTP id q7KIfwnl001544 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=NO); Mon, 20 Aug 2012 19:41:58 +0100
From: Adrian Farrel <adrian@olddog.co.uk>
To: 'Kireeti Kompella' <kireeti@juniper.net>
References: <0dc501cd7bb1$d318ad10$794a0730$@olddog.co.uk> <C1AFAC36-95CE-47AE-8FA3-633288144AB4@juniper.net> <0edf01cd7c43$94feec00$befcc400$@olddog.co.uk> <3E2B22DB-9FF6-41D0-92C3-85E25FF57ECF@juniper.net> <10c101cd7d65$7bd332e0$737998a0$@olddog.co.uk> <A151C7AC-E538-4800-99C0-85516FBF1E66@juniper.net>
In-Reply-To: <A151C7AC-E538-4800-99C0-85516FBF1E66@juniper.net>
Date: Mon, 20 Aug 2012 19:41:57 +0100
Message-ID: <129401cd7f03$7aa67090$6ff351b0$@olddog.co.uk>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
X-Mailer: Microsoft Outlook 14.0
Thread-Index: AQHPHuSqRl1cgpHy+Pw5z8k+/Sb7UAKzUXN9AjoNQFkBrUf7lwK6ATZgATWL1IeXC3dO4A==
Content-Language: en-gb
Cc: mpls@ietf.org, draft-ietf-mpls-entropy-label@tools.ietf.org
Subject: Re: [mpls] AD review of draft-ietf-mpls-entropy-label
X-BeenThere: mpls@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
Reply-To: adrian@olddog.co.uk
List-Id: Multi-Protocol Label Switching WG <mpls.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/mpls>, <mailto:mpls-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/mpls>
List-Post: <mailto:mpls@ietf.org>
List-Help: <mailto:mpls-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/mpls>, <mailto:mpls-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 20 Aug 2012 18:42:01 -0000

Yes, I like that.

Store it up as another last call comment.

A
(PS, thanks for keeping these comments public to the MPLS WG. That really helps)

> -----Original Message-----
> From: Kireeti Kompella [mailto:kireeti@juniper.net]
> Sent: 20 August 2012 19:38
> To: adrian@olddog.co.uk
> Cc: Kireeti Kompella; draft-ietf-mpls-entropy-label@tools.ietf.org;
mpls@ietf.org
> Subject: Re: AD review of draft-ietf-mpls-entropy-label
> 
> While we're on the subject, I'd like to make another change -- a
clarification.
> 
> section 4.3:
> 
> OLD:
> 
>    If a transit LSR recognizes the ELI, it MAY choose to load balance
>    solely on the following label (the EL); otherwise, it SHOULD use as
>    much of the whole label stack as feasible as keys for the load
>    balancing function, with the exception that reserved labels MUST NOT
>    be used.
> 
> NEW:
> 
>    If a transit LSR recognizes the ELI, it MAY choose to load balance
>    solely on the following label (the EL); otherwise, it SHOULD use as
>    much of the whole label stack as feasible as keys for the load
>    balancing function.  In any case, reserved labels MUST NOT be used
>    as keys for the load balancing function.
> 
> Kireeti.
> 
> PS: there, you see, I do (sometimes) use MUST/MUST NOT.