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 _______________________________________________ GROW mailing list GROW@ietf.org https://www.ietf.org/mailman/listinfo/grow
