Re: [bmwg] I-D Action: draft-ietf-bmwg-network-tester-cfg-00.txt

[Vladimir Vassilev <[e-mail]>]

Hi Sarah,

On 7/26/22 00:31, Sarah Banks wrote:
> Hi,
>     Some feedback from me as a participant, for the authors, is below. :)
>
> Thank you,
> Sarah
>
> Section 1.3 Objectives
> - "this section describes some of the design objectives ..." - as a reader, I expect this section to authoritatively tell me what the objectives are. I recommend either a. filling in the entire list or b. removing the words "some of" to make it clearer.
OK. I will remove "some of" in -01.
> - there's a mention of counters and 1 single stat (the latency stat) - but no other stats here - is that really the design intent?

Yes. This is specified in the last 3 objectives in the list:

...

* provide counters for received test traffic frames and octets.

* provide latency statistic in the case of test traffic with dynamic 
frame data that includes timestamp.

* provide sequence number errors in the case of test traffic with 
dynamic frame data that includes sequence number.

...


The tree diagram shows the relevant data nodes under 
/if:interfaces/if:interface/ntta:traffic-analyzer :

...
     |  +--ro state
     |     +--ro pkts?              yang:counter64
     |     +--ro octets?            yang:counter64
     |     +--ro idle-octets?       yang:counter64 {idle-octets-counter}?
     |     +--ro errors?            yang:counter64
     |     +--ro testframe-stats
     |     |  +--ro testframe-pkts?    yang:counter64
     |     |  +--ro sequence-errors?   yang:counter64
     |     |  +--ro payload-errors?    yang:counter64
     |     |  +--ro latency
     |     |     +--ro samples?   uint64
     |     |     +--ro min?       uint64
     |     |     +--ro max?       uint64
     |     |     +--ro average?   uint64
     |     |     +--ro latest?    uint64
     |     +--ro capture {capture}?
     |        +--ro frame* [sequence-number]
     |           +--ro sequence-number             uint64
     |           +--ro timestamp?                  yang:date-and-time
     |           +--ro length?                     uint32
     |           +--ro preceding-interframe-gap?   uint32
     |           +--ro data?                       string
...

The capture mechanism does not have corresponding entry in the objectives list. I will add one in -01.

>
> Has this draft seen a -YANGdoctors review yet?

No. I think requesting a review is a good idea.


/Vladimir


>
>
>
>> On Jun 17, 2022, at 3:19 PM, [e-mail] wrote:
>>
>>
>> A New Internet-Draft is available from the on-line Internet-Drafts directories.
>> This draft is a work item of the Benchmarking Methodology WG of the IETF.
>>
>>         Title           : A YANG Data Model for Network Tester Management
>>         Author          : Vladimir Vassilev
>> 	Filename        : draft-ietf-bmwg-network-tester-cfg-00.txt
>> 	Pages           : 28
>> 	Date            : 2022-06-17
>>
>> Abstract:
>>    This document introduces new YANG model for use in network
>>    interconnect testing containing modules of traffic generator and
>>    traffic analyzer.
>>
>>
>> The IETF datatracker status page for this draft is:
>> https://datatracker.ietf.org/doc/draft-ietf-bmwg-network-tester-cfg/
>>
>> There is also an htmlized version available at:
>> https://datatracker.ietf.org/doc/html/draft-ietf-bmwg-network-tester-cfg-00
>>
>>
>> Internet-Drafts are also available by rsync at rsync.ietf.org::internet-drafts
>>
>>
>> _______________________________________________
>> bmwg mailing list
>> [e-mail]
>> https://www.ietf.org/mailman/listinfo/bmwg
> _______________________________________________
> bmwg mailing list
> [e-mail]
> https://www.ietf.org/mailman/listinfo/bmwg