Re: [Dime] [dime] #54: OC-Report-Type as mandatory AVP

["TROTTIN, JEAN-JACQUES (JEAN-JACQUES)" <[e-mail]>]

Hi Mcruz

The current description indicates that when not present the OLR is of type Host, which was fine for me and keeps my preference. 
We may have  deployments where Realm OLR is not used, or where statistically the HOST type is the most frequent, so to have the grouped OLR-AVP containing a minimum of AVPs minimizes parsing. I agree it is a small optimization.

Best regards

JJacques 




-----Message d'origine-----
De : DiME [mailto:[e-mail]] De la part de [e-mail]
Envoyé : mercredi 12 février 2014 15:46
À : [e-mail]; [e-mail]
Objet : Re: [Dime] [dime] #54: OC-Report-Type as mandatory AVP

Hi Maria Cruz,

I'm assuming that you mean "required" instead of "mandatory", right?

So instead of:

   OC-OLR ::= < AVP Header: TBD2 >
              < OC-Sequence-Number >
              [ OC-Report-Type ]
              [ OC-Reduction-Percentage ]
              [ OC-Validity-Duration ]
            * [ AVP ]

You would prefer:

   OC-OLR ::= < AVP Header: TBD2 >
              < OC-Sequence-Number >
              { OC-Report-Type }
              [ OC-Reduction-Percentage ]
              [ OC-Validity-Duration ]
            * [ AVP ]

And I'm fine with this proposal.

Cheers,

Lionel

-----Message d'origine-----
De : DiME [mailto:[e-mail]] De la part de dime issue tracker Envoyé : mercredi 12 février 2014 15:26 À : [e-mail] Cc : [e-mail] Objet : [Dime] [dime] #54: OC-Report-Type as mandatory AVP

#54: OC-Report-Type as mandatory AVP

 Now in chapter 4.6:

    The default value of the OC-Report-Type AVP is 0 (i.e. the host
    report).

 This AVP is always required, right? Then, I think it is more precise that  we define this AVP as mandatory.

-- 
-----------------------------------------------+------------------------
-----------------------------------------------+---
 Reporter:  [e-mail]  |      Owner:  MCruz
     Type:  defect                             |  Bartolomé
 Priority:  major                              |     Status:  new
Component:  draft-docdt-dime-ovli              |  Milestone:
 Severity:  Active WG Document                 |    Version:  1.0
                                               |   Keywords:
-----------------------------------------------+------------------------
-----------------------------------------------+---

Ticket URL: <http://trac.tools.ietf.org/wg/dime/trac/ticket/54>
dime <http://tools.ietf.org/wg/dime/>

_______________________________________________
DiME mailing list
[e-mail]
https://www.ietf.org/mailman/listinfo/dime

_________________________________________________________________________________________________________________________

Ce message et ses pieces jointes peuvent contenir des informations confidentielles ou privilegiees et ne doivent donc pas etre diffuses, exploites ou copies sans autorisation. Si vous avez recu ce message par erreur, veuillez le signaler a l'expediteur et le detruire ainsi que les pieces jointes. Les messages electroniques etant susceptibles d'alteration, Orange decline toute responsabilite si ce message a ete altere, deforme ou falsifie. Merci.

This message and its attachments may contain confidential or privileged information that may be protected by law; they should not be distributed, used or copied without authorisation.
If you have received this email in error, please notify the sender and delete this message and its attachments.
As emails may be altered, Orange is not liable for messages that have been modified, changed or falsified.
Thank you.

_______________________________________________
DiME mailing list
[e-mail]
https://www.ietf.org/mailman/listinfo/dime