Re: [Dime] OVLI: comments to 4.6

Maria Cruz Bartolome <maria.cruz.bartolome@ericsson.com> Fri, 13 December 2013 09:28 UTC

Return-Path: <maria.cruz.bartolome@ericsson.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 8DE1A1AE01F for <dime@ietfa.amsl.com>; Fri, 13 Dec 2013 01:28:23 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.24
X-Spam-Level:
X-Spam-Status: No, score=-1.24 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HELO_EQ_SE=0.35, HOST_MISMATCH_NET=0.311, SPF_PASS=-0.001] autolearn=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 psCFElcmfLM3 for <dime@ietfa.amsl.com>; Fri, 13 Dec 2013 01:28:20 -0800 (PST)
Received: from sesbmg21.mgmt.ericsson.se (sesbmg21.ericsson.net [193.180.251.49]) by ietfa.amsl.com (Postfix) with ESMTP id BB1521A802A for <dime@ietf.org>; Fri, 13 Dec 2013 01:28:19 -0800 (PST)
X-AuditID: c1b4fb31-b7fa78e0000005dd-bf-52aad32c1f12
Received: from ESESSHC023.ericsson.se (Unknown_Domain [153.88.253.125]) by sesbmg21.mgmt.ericsson.se (Symantec Mail Security) with SMTP id 57.7E.01501.C23DAA25; Fri, 13 Dec 2013 10:28:12 +0100 (CET)
Received: from ESESSMB101.ericsson.se ([169.254.1.197]) by ESESSHC023.ericsson.se ([153.88.183.87]) with mapi id 14.02.0347.000; Fri, 13 Dec 2013 10:28:09 +0100
From: Maria Cruz Bartolome <maria.cruz.bartolome@ericsson.com>
To: "dime@ietf.org" <dime@ietf.org>
Thread-Topic: [Dime] OVLI: comments to 4.6
Thread-Index: Ac7yfC5RbFsL+gV9TT6AJTwE2XmtwwANV4kAAAOr9wAAG9aeQAB3FIIAAAup3DD//0H8gP/58ajA
Date: Fri, 13 Dec 2013 09:28:08 +0000
Message-ID: <087A34937E64E74E848732CFF8354B920973B124@ESESSMB101.ericsson.se>
References: <5BCBA1FC2B7F0B4C9D935572D90006681519DCBC@DEMUMBX014.nsn-intra.net> <6950EE6B-689E-47A1-88AA-1A67C47BC82E@gmail.com> <5BCBA1FC2B7F0B4C9D935572D90006681519DD4D@DEMUMBX014.nsn-intra.net> <A9CA33BB78081F478946E4F34BF9AAA014D2977C@xmb-rcd-x10.cisco.com> <5BCBA1FC2B7F0B4C9D935572D90006681519E00D@DEMUMBX014.nsn-intra.net> <A9CA33BB78081F478946E4F34BF9AAA014D2D5A9@xmb-rcd-x10.cisco.com> <5BCBA1FC2B7F0B4C9D935572D90006681519E086@DEMUMBX014.nsn-intra.net>
In-Reply-To: <5BCBA1FC2B7F0B4C9D935572D90006681519E086@DEMUMBX014.nsn-intra.net>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [153.88.183.148]
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFjrOLMWRmVeSWpSXmKPExsUyM+Jvra7O5VVBBpOXaFnM7V3B5sDosWTJ T6YAxigum5TUnMyy1CJ9uwSujL37lrIXrImv2HCyk62BcY1vFyMnh4SAicSJ+ecZIWwxiQv3 1rOB2EICJxglPh6z6GLkArKXMEr8/naFHSTBJmAncen0C6YuRg4OEQFlidO/HEDCwgKaEi8f HWIFsUUEtCRuzv7DClESJXHjjiJImEVAVaL74k0mEJtXwFfi+qvjbBDjbzFL7Ju+C6yXUyBA 4vL17WBFjED3fD+1BsxmFhCXuPVkPhPEnQISS/acZ4awRSVePv7HCmErSTQuecIKUa8jsWD3 JzYIW1ti2cLXzBCLBSVOznzCMoFRdBaSsbOQtMxC0jILScsCRpZVjJLFqcVJuelGhnq56bkl eqlFmcnFxfl5esWpmxiBkXFwy2/DHYwTr9kfYpTmYFES52WY3hkkJJCeWJKanZpakFoUX1Sa k1p8iJGJg1OqgXFKibXZHn/b6GkuUZft9qtGLf7IrKJr9+7r77PBqanV3+bFH75YPuX3q45/ CybfKjuXx9hXoyci/MSNbYmj/7yjfxtr3AWWf/9g9LLlcwfLjaTniW3vCl9tc+P5Kuxz++LN r3/2mPTf3Lz0jkLbfXeHJWscwlWPhH+yf//XJuzHVD3/lEUvuQKUWIozEg21mIuKEwHOVRWF WgIAAA==
Subject: Re: [Dime] OVLI: comments to 4.6
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: Fri, 13 Dec 2013 09:28:23 -0000

Hello Ulrich, all,

I agree with Nirav, I think in the base IETF draft we need to keep what is required for the use cases identified so far, (as long as ReportType is extensible).

Therefore I think that DESTINATION_HOST_MATCH and DESTINATION_REALM_MATCH are enough in the draft.

Going a bit more in detail into your mail, I will comment on why rest of values are not required: 

>   APPLICATION_ID_MATCH (0x0000000000000001)
Agreement for the baseline is to infer this always from the Diameter encapsulating message. Therefore, this is always done, no reason to add a ReportType for this.
> 
>   DESTINATION_HOST_PRESENT (0x0000000000000002)
What we need to know if whether overload-report shall apply for either HOST_MATCH or REALM_MATCH. 
This value is included into HOST_MATCH, then I do not see a reason to have it as an independent value.

>   DESTINATION_HOST_MATCH (0x0000000000000004)
Fine, I think this is required

>   DESTINATION_HOST_ABSENT (0x0000000000000008)
This value is included into REALM_MATCH. Any reason to have it as an independent value? 

>   DESTINATION_REALM_MATCH (0x0000000000000010)
Fine, I think this is required

About decision on whether it should be Unsigned or Enumerated, not sure which one could be best, I then to agree with Jouni that both could be valid as long as we need to define new values on application documentation, explaining expected behavior.

Best regards
/MCruz

-----Original Message-----
From: DiME [mailto:dime-bounces@ietf.org] On Behalf Of Wiehe, Ulrich (NSN - DE/Munich)
Sent: lunes, 09 de diciembre de 2013 14:38
To: ext Nirav Salot (nsalot); ext Jouni
Cc: dime@ietf.org
Subject: Re: [Dime] OVLI: comments to 4.6

Hi Nirav,

in the end its a matter of taste, and I definitly prefer the Unsigned64 approach.

Please note: The Enumerated approach also has a value (0 Reserved) without use case. And there is confusing text e.g. "...should apply to requests that the reacting node knows will reach the overloaded node." 
How would the reacting node know that a request will actually reach the overloaded node? And even if it knows this, wouldn't the overload treatment apply only to those requests that match the application-id from the answer?

Best regards
Ulrich


-----Original Message-----
From: ext Nirav Salot (nsalot) [mailto:nsalot@cisco.com] 
Sent: Monday, December 09, 2013 1:56 PM
To: Wiehe, Ulrich (NSN - DE/Munich); ext Jouni
Cc: dime@ietf.org
Subject: RE: [Dime] OVLI: comments to 4.6

Ulrich,

Defining a bit "APPLICATION_ID_MATCH" and not having a use case when this bit is set to "0" (at least in the context of the current draft) is confusing to me. Same is the case with some other bits you have proposed.

In summary, I do not see any issue with the current ReportType. 
For the extensibility, we have to deal with it anyway when we have corresponding use case in 3GPP or in IETF. In my view, defining the ReportType with some assumption of its extensibility is confusing since those use cases are not discussed here yet.

Regards,
Nirav.

-----Original Message-----
From: Wiehe, Ulrich (NSN - DE/Munich) [mailto:ulrich.wiehe@nsn.com] 
Sent: Monday, December 09, 2013 5:55 PM
To: Nirav Salot (nsalot); ext Jouni
Cc: dime@ietf.org
Subject: RE: [Dime] OVLI: comments to 4.6

Hi Nirav,
can you please point out what actually is confusing. I would have thougth that my proposal is more clear and precise as it exactly identifies when a given request matches the OLR (i.e. must undergo the throttling).

With regard to complexity there is no difference between checking the two allowed enumeration values 1 and 2 (current version) or the two allowed Unsigned64 values (0x0000000000000007) and (0x0000000000000019) (my proposal).

The main benefit I see with my proposal is the extensibility e.g.
One new value of   ORIGIN_HOST_MATCH (0x0000000000000020) for Overload Mitigation Differentiation per client rather than
Two new values of "host report with origin-host match" and realm-report with origin-host match".

Best regards
Ulrich



-----Original Message-----
From: ext Nirav Salot (nsalot) [mailto:nsalot@cisco.com] 
Sent: Saturday, December 07, 2013 10:44 AM
To: Wiehe, Ulrich (NSN - DE/Munich); ext Jouni
Cc: dime@ietf.org
Subject: RE: [Dime] OVLI: comments to 4.6

Hi Ulrich,

Instead of re-packing, I see your proposal as confusing or adding more implementation complexity to the exiting simple proposal of having just two Report-Type.
e.g. with your proposal, now the nodes have to validate if the combination of Report-Type flag is correct/allowed or not and perform the error handling if there is any discrepancy. 

Besides, I do not see the need for some obvious Report-Type e.g. "APPLICATION_ID_MATCH" since it is always present and/or implicit.
Same is the case with "DESTINATION_HOST_PRESENT" and "DESTINATION_HOST_MATCH". What is the use case for destination-host present but not match? In other words, the reacting node should use this type of report towards which node/realm?

I guess this time you need to convince me (at least) as why you think so many different Report-Types are needed and what is the issue with existing definition of Report-Type.

Regards,
Nirav.

-----Original Message-----
From: DiME [mailto:dime-bounces@ietf.org] On Behalf Of Wiehe, Ulrich (NSN - DE/Munich)
Sent: Friday, December 06, 2013 7:48 PM
To: ext Jouni
Cc: dime@ietf.org
Subject: Re: [Dime] OVLI: comments to 4.6

Hi Jouni,

maybe that my proposal is just kind of repackaging. But still I think it is much clearer than the existing text, and you seem not to object.
Please consider accepting the proposed modification.

Best regards
Ulrich

-----Original Message-----
From: ext Jouni [mailto:jouni.nospam@gmail.com] 
Sent: Friday, December 06, 2013 1:33 PM
To: Wiehe, Ulrich (NSN - DE/Munich)
Cc: dime@ietf.org
Subject: Re: [Dime] OVLI: comments to 4.6


On Dec 6, 2013, at 2:10 PM, Wiehe, Ulrich (NSN - DE/Munich) wrote:

> Dear all,
>  
> here are comments to clause 4.6:
>  
> It has already been proposed to change the type of the OC-Report-Type AVP from Enumerated to Unsinged64 in order to avoid potential extensibility problems.  

I do not see how changing the type to unsigned would help with the extensibility on
an assumption we create an IANA maintained registry for it. Unsigned64 will have
exactly the same issues as enumerated unless we define report type semantics to
something like what we have on OC-Feature AVP. How the receiver of the report type
reacts when it sees a flag bit that is does not understand? If the content and
handling  of the OC-OLR somehow depends on the unknown flag bit, the receiver has
no other choice than drop the OLR, since it cannot be sure how to handle the report
as a whole.

The only ways around I see here are:
a) you can extend report types when defining new applications
b) or when both ends have a mutually supported & advertised feature that maps
   to a report type (that has been defined after the publication of this 
   spec)

Other than those, IMHO, are just repackaging the issue into different form.


- Jouni

> In addition to that, I think that the purpose of the Report-Type is to let the reacting node know to which (future) request commands the overload treatment should apply:
> 
> For a host report-type the overload treatment should apply to all request commands for which
> a) The request command's Application-ID matches the Application-Id of the OC-Report-Type AVP's encapsulating answer command and
> b) The request command's Destination-Host is present and 
> c) The request command's Destination-Host matches the Origin-Host of the OC-Report-Type AVP's encapsulating answer command
> 
> For a realm report-type the overload treatment should apply to all request commands for which
> a) <see a) above> and
> d) The request command's Destination-Host is absent and 
> e) The request command's Destination Realm matches the Origin-Realm of the OC-Report-Type AVP's encapsulating answer command.
> 
> The proposal now is to assign individual bits of the Unsinged64 type to a), b), c), d), and e):
> 
> Proposed text:
>   4.6 OC-Report-Type AVP
> 
>   The OC-Report-Type AVP (AVP code TBD5) is type of Unsigned64 and contains a 64 bit flags field of a request
>   command's characteristics.
> 
>   The following characteristics are defined in this document:
> 
>   APPLICATION_ID_MATCH (0x0000000000000001)
> 
>   When this flag is set by the overload reporting endpoint it means that the overload treatment should apply only
>   to those request commands with an Application-ID that matches the Application-Id of the OC-Report-Type AVP's
>   encapsulating answer command.
> 
>   DESTINATION_HOST_PRESENT (0x0000000000000002)
> 
>   When this flag is set by the overload reporting endpoint it means that the overload treatment should apply only
>   to those request commands containing a Destination-Host AVP.
> 
>   DESTINATION_HOST_MATCH (0x0000000000000004)
> 
>   When this flag is set by the overload reporting endpoint it means that the overload treatment should apply only
>   To those request commands with an Destination-Host AVP that matches the Origin-Host of the OC-Report-Type AVP's
>   encapsulating answer command.
> 
>   DESTINATION_HOST_ABSENT (0x0000000000000008)
> 
>   When this flag is set by the overload reporting endpoint it means that the overload treatment should apply only
>   to those request commands not containing a Destination-Host AVP.
> 
>   DESTINATION_REALM_MATCH (0x0000000000000010)
> 
>   When this flag is set by the overload reporting endpoint it means that the overload treatment should apply only
>   to those request commands with an Destination-Host AVP that matches the Origin-Host of the OC-Report-Type AVP's
>   encapsulating answer command.
> 
>   Combinations other than 
>   1) APPLICATION_ID_MATCH and DESTINATION_HOST_PRESENT and DESTINATION_HOST_MATCH
>   and
>   2) APPLICATION_ID_MATCH and DESTINATION_HOST_ABSENT and DESTINATION_REALM_MATCH
>   require a mutually agreed extension.
> 
> 
> 
> 
> One extension required by 3GPP applications is the Overload Mitigation Differentiation per client (see 3GPP TR 29.809 clause 6.4.7. To address this, a new value would be needed e.g.
> 
>   ORIGIN_HOST_MATCH (0x0000000000000020)
> 
>   When this flag is set by the overload reporting endpoint it means that the overload treatment should apply only
>   to those request commands with an Origin-Host AVP that matches the Destination-Host of the OC-Report-Type AVP's
>   encapsulating answer command.
> 
> With this extension the following additional combinations would be allowed:
> 3) APPLICATION_ID_MATCH and DESTINATION_HOST_PRESENT and DESTINATION_HOST_MATCH and ORIGIN_HOST_MATCH
> and
> 4) APPLICATION_ID_MATCH and DESTINATION_HOST_ABSENT and DESTINATION_REALM_MATCH and ORIGIN_HOST_MATCH
> 
> Another potential extension is Diameter Agent Overload. To address this, a new value would be needed e.g.
> 
>   NEXT_HOP_MATCH (0x0000000000000040)
> 
>   When this flag is set by the overload reporting endpoint it means that the overload treatment should apply only
>   to those request commands sent to the same next hop the OC-Report-Type AVP's encapsulating answer command was received from.
> 
> 
> Best regards
> Ulrich
> 
> 
> _______________________________________________
> DiME mailing list
> DiME@ietf.org
> https://www.ietf.org/mailman/listinfo/dime

_______________________________________________
DiME mailing list
DiME@ietf.org
https://www.ietf.org/mailman/listinfo/dime
_______________________________________________
DiME mailing list
DiME@ietf.org
https://www.ietf.org/mailman/listinfo/dime