Skip to main content

Definitions of Managed Objects for IP Flow Information Export
draft-ietf-ipfix-mib-10

Yes

(Dan Romascanu)

No Objection

(Cullen Jennings)
(Jari Arkko)
(Lisa Dusseault)
(Magnus Westerlund)
(Pasi Eronen)
(Ralph Droms)
(Robert Sparks)
(Ron Bonica)
(Ross Callon)
(Russ Housley)
(Tim Polk)

Note: This ballot was opened for revision 10 and is now closed.

Dan Romascanu Former IESG member
Yes
Yes () Unknown

                            
Adrian Farrel Former IESG member
No Objection
No Objection (2009-11-28) Unknown
Just a couple of minor issues for you to think about if the I-D is revised or during AUTH-48.

---

Use of citations within REFERENCE clauses.
I think you should not use square brackets for referenced documents 
inside the MIB module itself. Just remove the [], but you may need to
fix up some text elsewhere to ensure that the referenced documents
are cited from somewhere int he document.

---

Section 5.3 
Right at the end of the example in the section you have:
                  +- ipfixTemplateDefinitionEnterprise (5) = 0
                  +- ipfixTemplateDefinitionFlags (4) = 0
The OIDs are reversed.

---
ipfixTemplateDefinitionFlags

You have...

   Thus we get the following values for an Information Element:

   '0'H
       The Information Element is neither used for scoping nor
       as Flow Key.
   '1'H (scope)
       The Information Element is used for scoping.
   '2'H (flowKey)
       The Information Element is used as Flow Key.
   '3'H (scope | flowKey)
       This combination is not allowed."

I know what you are trying to say, but ipfixTemplateDefinitionFlags has
SYNTAX BITS and you can't convert that into hex (if you had wanted to
you could have used SYNTAX INTEGER).

I think you just have to rephrase this in terms of bits.
Alexey Melnikov Former IESG member
No Objection
No Objection (2009-12-13) Unknown
5.3.  The Template Definition Table

    ipfixTemplateDefinitionTable (4)
    |
    +- ipfixTemplateDefinitionEntry (1)
       |
       +- index (5) (ipfixTransportSessionIndex)
          +- index (3) (ipfixTemplateObservationDomainId)
             + index (257) (ipfixTemplateId)
               +- index (1) (ipfixTemplateDefinitionIndex)
               |  +- ipfixTemplateDefinitionIndex (1) = 1
               |  +- ipfixTemplateDefinitionIeId (2) = 158
               |  |                      (flowStartDeltaMicroseconds)
               |  +- ipfixTemplateDefinitionIeLength (3) = 4
               |  +- ipfixTemplateDefinitionEnterprise (4) = 0
               |  +- ipfixTemplateDefinitionFlags (5) = 0
               |
               +- index (2) (ipfixTemplateDefinitionIndex)
               |  +- ipfixTemplateDefinitionIndex (1) = 2
               |  +- ipfixTemplateDefinitionIeId (2) = 159
               |  |                      (flowStartDeltaMicroseconds)

flowStartDeltaMicroseconds is listed twice (for 158 and for 159). This looks wrong.
Cullen Jennings Former IESG member
No Objection
No Objection () Unknown

                            
Jari Arkko Former IESG member
No Objection
No Objection () Unknown

                            
Lisa Dusseault Former IESG member
No Objection
No Objection () Unknown

                            
Magnus Westerlund Former IESG member
No Objection
No Objection () Unknown

                            
Pasi Eronen Former IESG member
No Objection
No Objection () Unknown

                            
Ralph Droms Former IESG member
No Objection
No Objection () Unknown

                            
Robert Sparks Former IESG member
No Objection
No Objection () Unknown

                            
Ron Bonica Former IESG member
No Objection
No Objection () Unknown

                            
Ross Callon Former IESG member
No Objection
No Objection () Unknown

                            
Russ Housley Former IESG member
No Objection
No Objection () Unknown

                            
Tim Polk Former IESG member
No Objection
No Objection () Unknown