Skip to main content

Signaling Maximum SID Depth (MSD) Using OSPF
draft-ietf-ospf-segment-routing-msd-25

Yes

(Alvaro Retana)

No Objection

(Adam Roach)
(Alexey Melnikov)
(Alissa Cooper)
(Ben Campbell)
(Deborah Brungard)
(Ignas Bagdonas)
(Martin Vigoureux)
(Mirja Kühlewind)
(Suresh Krishnan)
(Terry Manderson)

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

Alvaro Retana Former IESG member
Yes
Yes (for -20) Unknown

                            
Adam Roach Former IESG member
No Objection
No Objection (for -20) Unknown

                            
Alexey Melnikov Former IESG member
No Objection
No Objection (for -21) Unknown

                            
Alissa Cooper Former IESG member
No Objection
No Objection (for -21) Unknown

                            
Ben Campbell Former IESG member
No Objection
No Objection (for -21) Unknown

                            
Benjamin Kaduk Former IESG member
(was Discuss) No Objection
No Objection (2018-10-15 for -23) Sent
Thanks for addressing my Discuss point; original ballot comment preserved below.

Can SID be expanded on first usage --
https://www.rfc-editor.org/materials/abbrev.expansion.txt does not list it
as "well known".  (It also doesn't appear to list "Segment Identifier" as
one of the expansions.)

This is basically the same thing I said for the IS-IS document that creates
the MSD types registry, but I'm not sure I followed correctly the meaning
of MSD type 1 for SR-enabled vs.  non-SR-enabled networks.  In particular,
I still don't really understand why it's okay to use the same codepoint for
the max SID depth in SR-enabled networks and for the max label depth in
non-SR MPLS networks.  Why couldn't they just be separate MSD Type
codepoints?

Section-by-section comments follow.

Section 2

                  If the Node MSD TLV appears in multiple Router
   Information LSAs that have the same flooding scope, the Node MSD TLV
   in the Router Information (RI) LSA with the numerically smallest
   Instance ID MUST be used and subsequent instances of the Node MSD TLV
   MUST be ignored. [...]

Unless there is a sorting requirement I've forgotten about, shouldn't this
be "other" rather than "subsequent"?

Section 6

Thanks for the updates in response to the secdir review; they help a lot.

   If the value is larger than supported - instantiation of a path that
   can't be supported by the head-end (the node performing the SID
   imposition).

This is supposed to mean "(instantiation by the head-end) of a (path that
can't be supported)", not "instantiation of a path (that can't be supported
by the head-end)", right?
Deborah Brungard Former IESG member
No Objection
No Objection (for -20) Unknown

                            
Ignas Bagdonas Former IESG member
No Objection
No Objection (for -21) Unknown

                            
Martin Vigoureux Former IESG member
No Objection
No Objection (for -21) Unknown

                            
Mirja Kühlewind Former IESG member
No Objection
No Objection (for -20) Unknown

                            
Suresh Krishnan Former IESG member
No Objection
No Objection (for -21) Unknown

                            
Terry Manderson Former IESG member
No Objection
No Objection (for -21) Unknown