Last Call Review of draft-ietf-grow-bgp-gshut-13
review-ietf-grow-bgp-gshut-13-opsdir-lc-hares-2018-01-18-00

Request Review of draft-ietf-grow-bgp-gshut
Requested rev. no specific revision (document currently at 13)
Type Last Call Review
Team Ops Directorate (opsdir)
Deadline 2018-01-11
Requested 2017-12-14
Authors Pierre Francois, Bruno Decraene, Cristel Pelsser, Keyur Patel, Clarence Filsfils
Draft last updated 2018-01-18
Completed reviews Rtgdir Last Call review of -11 by Matthew Bocci (diff)
Genart Last Call review of -11 by Matthew Miller (diff)
Opsdir Last Call review of -13 by Susan Hares
Assignment Reviewer Susan Hares
State Completed
Review review-ietf-grow-bgp-gshut-13-opsdir-lc-hares-2018-01-18
Reviewed rev. 13
Review result Has Nits
Review completed: 2018-01-18

Review
review-ietf-grow-bgp-gshut-13-opsdir-lc-hares-2018-01-18

Status: Nits 

The operational procedures described in this process for the gshut comment are accurately covered, and SHOULD work well.  The Appendices A-C add to an operations document and should be retained for publication. 

Technical nit: 
location of technical nit: (section 4.3) 
The document indicats that the "BGP implementers SHOULD provide configuration knobs that utilize teh GRACEFUL_SHUTDOWN community."   

What the problem is: 
The document does not say is that their should be error reporting knobs to track the use of GRACEFUL_SHUTDOWN community.  This can go in section 4.3 in one or two sentences.

Editorial nit: 
section 3. paragraph 2, p. 3

/This is because alternate paths can be hidden by knodes of an AS./
commment:  The implied "this" is too vague for a specification. 

Fix:/This lack of path occurs because alternate paths can be hidden by nodes of an AS."/