IETF Logo Mail Archive

Re: [Dime] [dime] #77 (draft-ietf-dime-ovli): Section 4.2.3 - Reporting node behavior (overload report handling)

Ben Campbell <ben@nostrum.com> Mon, 13 October 2014 23:15 UTC

Return-Path: <ben@nostrum.com>
X-Original-To: dime@ietfa.amsl.com
Delivered-To: dime@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id B3DEE1A1A59 for <dime@ietfa.amsl.com>; Mon, 13 Oct 2014 16:15:17 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.354
X-Spam-Level:
X-Spam-Status: No, score=-2.354 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, IP_NOT_FRIENDLY=0.334, RP_MATCHES_RCVD=-0.786, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001] autolearn=ham
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 6vcK1pHOysVr for <dime@ietfa.amsl.com>; Mon, 13 Oct 2014 16:15:15 -0700 (PDT)
Received: from nostrum.com (raven.nostrum.com [69.55.229.100]) (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 2CC661A1A45 for <dime@ietf.org>; Mon, 13 Oct 2014 16:15:14 -0700 (PDT)
Received: from [10.0.1.23] (cpe-173-172-146-58.tx.res.rr.com [173.172.146.58]) (authenticated bits=0) by nostrum.com (8.14.9/8.14.7) with ESMTP id s9DNFC1p045604 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=NO); Mon, 13 Oct 2014 18:15:13 -0500 (CDT) (envelope-from ben@nostrum.com)
X-Authentication-Warning: raven.nostrum.com: Host cpe-173-172-146-58.tx.res.rr.com [173.172.146.58] claimed to be [10.0.1.23]
Content-Type: text/plain; charset="us-ascii"
Mime-Version: 1.0 (Mac OS X Mail 7.3 \(1878.6\))
From: Ben Campbell <ben@nostrum.com>
In-Reply-To: <066.d4c405dbb2245ca94afc738db13a2a20@trac.tools.ietf.org>
Date: Mon, 13 Oct 2014 18:15:11 -0500
X-Mao-Original-Outgoing-Id: 434934911.633419-b811d5ecd14b9525719224cbc4f02f5c
Content-Transfer-Encoding: quoted-printable
Message-Id: <E76694D4-6CEC-4508-A2BB-DD754F432C22@nostrum.com>
References: <066.d4c405dbb2245ca94afc738db13a2a20@trac.tools.ietf.org>
To: dime@ietf.org
X-Mailer: Apple Mail (2.1878.6)
Archived-At: http://mailarchive.ietf.org/arch/msg/dime/SRlLhFJDaixIOzqyVJmD-0DSpLM
Subject: Re: [Dime] [dime] #77 (draft-ietf-dime-ovli): Section 4.2.3 - Reporting node behavior (overload report handling)
X-BeenThere: dime@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: Diameter Maintanence and Extentions Working Group <dime.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/dime>, <mailto:dime-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/dime/>
List-Post: <mailto:dime@ietf.org>
List-Help: <mailto:dime-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/dime>, <mailto:dime-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 13 Oct 2014 23:15:18 -0000

Are these not mandatory in the in the AVP definition? If they aren't they should be, and if they are, the text is redundant.

On Oct 13, 2014, at 3:54 PM, dime issue tracker <trac+dime@zinfandel.tools.ietf.org> wrote:

> #77: Section 4.2.3 - Reporting node behavior (overload report handling)
> 
> Propose adding the following two paragraphs after the third paragraph
> (location isn't critically important):
> 
>    The reporting node MUST include the OC-Validity-Duration AVP in the
> OLR.
> 
>    The reporting node MUST include a report-type in the OLR.
> 
> -- 
> -------------------------------------+-------------------------------------
> Reporter:                           |      Owner:  draft-ietf-dime-
>  srdonovan@usdonovans.com           |  ovli@tools.ietf.org
>     Type:  defect                   |     Status:  new
> Priority:  major                    |  Milestone:
> Component:  draft-ietf-dime-ovli     |    Version:
> Severity:  Active WG Document       |   Keywords:
> -------------------------------------+-------------------------------------
> 
> Ticket URL: <http://trac.tools.ietf.org/wg/dime/trac/ticket/77>
> dime <http://tools.ietf.org/wg/dime/>
>

v2.23.0 | Report a Bug | By Email