[GROW] AD Review of draft-ietf-grow-bgp-session-culling

[Warren Kumari <[e-mail]>]

Hi there,

I've just completed my AD review of draft-ietf-grow-bgp-session-culling.

I only had a few small nits:

Section 3:
"Involuntary BGP Session Teardown: The Caretaker of the lower layer
  network disrupts BGP control-plane traffic in the upper layer,
  causing the BGP Hold Timers of the affected BGP session to expire,"
-- it took me a few readings to parse this sentence -- I think that
the "in the upper layer" is redundant and confuses the sentence.
I think just removing it and

"Involuntary BGP Session Teardown:  The Caretaker of the lower layer
  network disrupts (higher layer) BGP control-plane traffic,
  causing the BGP Hold Timers of the affected BGP session to expire,..." ?


2: Section 3.2. Involuntary BGP Session Teardown Recommendations
"Such culling of control-plane traffic will pre-empt the" - s/pre-empt/preempt/


3: I really like the fact that this has actual exmaple config. I think
it would be nice it if also included some more vendors.

Anyway, I'm fine to start IETF LC like this, but it you are able to
post a new version I think things might go smoother.

Please let me know either way.
W



-- 
I don't think the execution is relevant when it was obviously a bad
idea in the first place.
This is like putting rabid weasels in your pants, and later expressing
regret at having chosen those particular rabid weasels and that pair
of pants.
   ---maf