Skip to main content

OSPF Graceful Link Shutdown
draft-ietf-ospf-link-overload-16

Yes

(Alia Atlas)

No Objection

Warren Kumari
(Adam Roach)
(Alissa Cooper)
(Kathleen Moriarty)
(Mirja Kühlewind)
(Spencer Dawkins)
(Suresh Krishnan)
(Terry Manderson)

Note: This ballot was opened for revision 12 and is now closed.

Yes (for -12) Unknown

                            
No Objection (for -13) Unknown

                            
No Objection (for -13) Unknown

                            
No Objection (2018-01-24 for -13) Unknown
I debated about filing my first comment as a DISCUSS [1], but decided against it because it should be very easy to solve.  The rest are non-blocking comments.

(1) The following should be Normative references: rfc2119 and rfc6987 -- this last one because MaxLinkMetric (which is defined there) is extensively used (as a MUST) throughout the document.

(2) Section 3. (Flooding Scope) provides information about the flooding scope, but only references for OSPFv2.  It would be nice if the references for OSPFv3 were included there as well.

(3) Section 4.5. mentions that a "new TLV called Graceful-Link-Shutdown is defined" for BGP-LS, but there are no details on the format, etc.  The IANA Considerations section suggests a value, not for a TLV but for an NLRI Type!  

(4) Section 5: "The node that has the link to be taken out of service SHOULD advertise the Graceful-Link-Shutdown sub-TLV..."  When would the node not advertise the sub-TLV?  IOW, why is "MUST" not used?

(5) In 5.1: "MAX-TE-METRIC is a constant defined by this draft and set to 0xfffffffe."  Assuming that the intent is to define a new architectural constant... I would rather see this constant defined separately (in it's own section/sub-section with a formal definition) instead of "in passing" while describing how to use it (a la MaxLinkMetric in rfc6987).

(6) 5.1 says that the metrics "MUST be set to MaxLinkMetric...and SHOULD be set to MAX-TE-METRIC".  Why is there a difference?

(7) s/MAX_METRIC/MaxLinkMetric

[1] https://www.ietf.org/iesg/statement/discuss-criteria.html
No Objection (2018-01-23 for -13) Unknown
-8: It would be helpful to see a few sentences about how the security considerations in 2328 and 5340 apply to the mechanisms in this draft, rather than just a "no new considerations" assertion.
No Objection (2018-01-25 for -14) Unknown
As mentioned by Tim, part of the OPS DIR review. It's the authors and responsible AD to decide whether to act on those comments.

I believe the document is Ready for publication.  I have only three minor
comments below, which the authors may choose to act on.

Overall the document reads reasonably well. Not being overly familiar with the
material, I needed to read it through end-to-end more than once to better
understand its scope and intent. My first comment would be that perhaps the
introduction section could be better written; the abstract seemed clear on the
purpose of the draft, while the introduction felt a little muddled.  Sections
2, 3 and 4, which detail the motivations and extensions, were much clearer.

Secondly, there are some minor typographic errors throughout the document,
generally missing (in)definite articles.  While the RFC Editor would pick these
up, it would be nice for the authors to have a final pass and fix those before
submission.

Thirdly, the document does not give any advice on the timing of using the
extensions - how far in advance is it recommended to use the extensions? - or
on the return to 'normal' state once the maintenance is completed.  So perhaps
consider adding a short section on this, maybe in Section 5.
No Objection (2018-02-05) Unknown
Thanks for addressing my Discuss and comment.
No Objection (2018-01-18 for -12) Unknown
I think this document would be clearer if the example in S 7.1 were
in the intro. I was scratching my head a bit at the beginning and
then got to 7.1 and it made more sense.
No Objection (for -13) Unknown

                            
No Objection (for -13) Unknown

                            
No Objection (for -13) Unknown

                            
No Objection (for -14) Unknown

                            
No Objection (for -13) Unknown