Re: Review of draft-ietf-dime-load-07

Steve Donovan <srdonovan@usdonovans.com> Tue, 07 March 2017 15:45 UTC

Return-Path: <srdonovan@usdonovans.com>
X-Original-To: ietf@ietfa.amsl.com
Delivered-To: ietf@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 8B4A012959D for <ietf@ietfa.amsl.com>; Tue, 7 Mar 2017 07:45:25 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.12
X-Spam-Level:
X-Spam-Status: No, score=-1.12 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, SPF_NEUTRAL=0.779, URIBL_BLOCKED=0.001] autolearn=no 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 UDP_CyhFfx4A for <ietf@ietfa.amsl.com>; Tue, 7 Mar 2017 07:45:24 -0800 (PST)
Received: from biz131.inmotionhosting.com (biz131.inmotionhosting.com [173.247.247.114]) (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 77C041295B9 for <ietf@ietf.org>; Tue, 7 Mar 2017 07:45:04 -0800 (PST)
Received: from cpe-97-99-50-102.tx.res.rr.com ([97.99.50.102]:49238 helo=Steves-MacBook-Air.local) by biz131.inmotionhosting.com with esmtpsa (TLSv1.2:ECDHE-RSA-AES128-GCM-SHA256:128) (Exim 4.87) (envelope-from <srdonovan@usdonovans.com>) id 1clHII-000EXm-77 for ietf@ietf.org; Tue, 07 Mar 2017 07:45:04 -0800
Subject: Re: Review of draft-ietf-dime-load-07
To: ietf@ietf.org
References: <148784049413.20240.15876379956092850264.idtracker@ietfa.amsl.com>
From: Steve Donovan <srdonovan@usdonovans.com>
Message-ID: <0390f072-bf67-d77b-a47f-b76140be81ae@usdonovans.com>
Date: Tue, 07 Mar 2017 09:44:57 -0600
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:45.0) Gecko/20100101 Thunderbird/45.7.1
MIME-Version: 1.0
In-Reply-To: <148784049413.20240.15876379956092850264.idtracker@ietfa.amsl.com>
Content-Type: text/plain; charset="utf-8"; format="flowed"
Content-Transfer-Encoding: 7bit
X-OutGoing-Spam-Status: No, score=-1.0
X-AntiAbuse: This header was added to track abuse, please include it with any abuse report
X-AntiAbuse: Primary Hostname - biz131.inmotionhosting.com
X-AntiAbuse: Original Domain - ietf.org
X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12]
X-AntiAbuse: Sender Address Domain - usdonovans.com
X-Get-Message-Sender-Via: biz131.inmotionhosting.com: authenticated_id: srdonovan@usdonovans.com
X-Authenticated-Sender: biz131.inmotionhosting.com: srdonovan@usdonovans.com
X-Source:
X-Source-Args:
X-Source-Dir:
Archived-At: <https://mailarchive.ietf.org/arch/msg/ietf/Rcnb9dXn319XEOJuC6X6weutiqo>
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ietf/>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 07 Mar 2017 15:45:25 -0000

Roni,

Thank you for your review.  Please see my comments inline.

Steve

On 2/23/17 3:01 AM, Roni Even wrote:
> Reviewer: Roni Even
> Review result: Ready with Nits
>
> 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
>
> <https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.
>
> Document: draft-ietf-dime-load-07
> Reviewer: Roni Even
> Review Date: 2017-02-23
> IETF LC End Date: 2017-02-27
> IESG Telechat date: 2017-03-16
>
> Summary: This draft is almost ready for publication as a standard
> track RFC
>
> Major issues:
>
> Minor issues:
>
> I understand that each node can calculate the load differently , the
> example in figure 8 demonstrate that the agent selection may be
> different if the agent aggregates load from the servers to calculate
> its load or just conveys his load, possibly even that each one of the
> agents will use different method. Why not mandate load calculation
> using aggregated weighted loads?
SRD> The working group thought it was best to leave this as an 
implementation decision.
>
> Nits/editorial comments:
> 1. In section 5 paragraph 9 "The load report includes a value
> indicating the load of the sending
>     node relative load of the sending node, " should be just "The load
> report includes a value indicating the relative load of the sending
> node,"
SRD> Yes, change made.
> 2. In section 6.2 "weigth "
SRD> Change made.
> 3. in the security consideration what about an endpoint in the middle
> changing the host load value causing a change in the routing
> decisions.
SRD> I'm assuming that you mean an agent in the middle changing the host 
value.  I've added the following to the security considerations section 
-- "Given that routing decisions are impacted by load information, there 
is potential for negative impacts on a Diameter network caused by 
erroneous or malicious load reports. This includes the malicious 
changing of load values by Diameter Agents."
>
>
>
>