Skip to main content

Last Call Review of draft-ietf-bmwg-sdn-controller-benchmark-meth-07
review-ietf-bmwg-sdn-controller-benchmark-meth-07-rtgdir-lc-rogge-2018-01-29-00

Request Review of draft-ietf-bmwg-sdn-controller-benchmark-meth
Requested revision No specific revision (document currently at 09)
Type Last Call Review
Team Routing Area Directorate (rtgdir)
Deadline 2018-02-02
Requested 2018-01-19
Requested by Alvaro Retana
Authors Bhuvaneswaran Vengainathan , Anton Basil , Mark Tassinari , Vishwas Manral , Sarah Banks
I-D last updated 2018-01-29
Completed reviews Rtgdir Last Call review of -07 by Henning Rogge (diff)
Opsdir Last Call review of -07 by Scott O. Bradner (diff)
Genart Last Call review of -07 by Stewart Bryant (diff)
Secdir Last Call review of -07 by Russ Housley (diff)
Genart Telechat review of -08 by Stewart Bryant (diff)
Assignment Reviewer Henning Rogge
State Completed
Request Last Call review on draft-ietf-bmwg-sdn-controller-benchmark-meth by Routing Area Directorate Assigned
Reviewed revision 07 (document currently at 09)
Result Has nits
Completed 2018-01-29
review-ietf-bmwg-sdn-controller-benchmark-meth-07-rtgdir-lc-rogge-2018-01-29-00
Hi,

Min Ye asked me to do a review of
draft-ietf-bmwg-sdn-controller-benchmark-meth-07.


I first have to say I like the fact that the draft has a "sequence
diagram" for each of the test cases in the appendix. This will make it
easier to design test software without forgetting small things or
misinterpreting the text version of the test.

Second, in chapter 5.1.1 you might want to consider that topology
discovery can have transient results that might be (accidentally) the
correct one. Just because the you get the right topology once it
doesn't mean that the algorithm is already in a stable state. I would
suggest keeping the "test three times for same result" also for the
correct topology.

Similar in chapter 5.2.2, it might be good to compare the results of
procedure step 3 and 4... see if they have a different of one of more.
I am not sure if a test result should be just invalid if the
difference is larger or if the difference should recorded in the
results.

Last, the Table of Content link to chapter 3 and the "section 10" link
in chapter 4.1 seem to be broken.

Henning Rogge