Ron's Comments WGLC draft-ietf-6man-segment-routing-header-11

["Darren Dukes (ddukes)" <[e-mail]>]

Ron, thanks for working with me through the issues you raised, it's been valuable and helped improve the specification.  We still have more work to do but I wanted to update the group on the items we’ve been able to close.  We’ll close off

Reviewed and Closed
#28<https://trac.ietf.org/trac/6man/ticket/28> 2.2 SRv6 Segment (Comment 1)
Rev 12 to describe how interface addresses are handled in end and end.x pseudocode.

#29<https://trac.ietf.org/trac/6man/ticket/29> Section 2.2 SRv6 Segment (Comment 2)

Rev 12 to add this text to 2.1

"A SID is routed through an IPv6 network as any other IPv6 destination address.  IPv6 SIDs, or the prefix(es) covering all SIDs in the "My local SID" table, and their reachability may be distributed by means outside the scope of this document.  For example, [RFC5308] or [RFC5340] may be used to advertise a prefix covering the My Local SID Table."

#31<https://trac.ietf.org/trac/6man/ticket/31> Section 2.2 SRv6 Segment (Comment 4)
Rev 12 to add two descriptions:
1 - Describe default SID handling for addresses within the prefix covering my Local SID Table that have not END or END.X function define.
2 - Describe error handling when receiving packets with SRH and SL!=0 where DA is in interface address that is not in my Local SID Table

#33<https://trac.ietf.org/trac/6man/ticket/33> Section 2.3 Segment Routing (SR) Domain (Comment 1)
Rev 12 to change text to
"The SRH is added to the packet by its source, consistent with the source routing model defined in [RFC8200].  For example:”


#34<https://trac.ietf.org/trac/6man/ticket/34> Section 3 Segment Routing Header (Comment 1)
Rev 12 removes normative references to draft-filsfils-spring-srv6-network-programming, agreement from London was to deal with SRH separately.

#35<https://trac.ietf.org/trac/6man/ticket/35> Section 3 Segment Routing Header (Comment 2)
Rev 12 Removes P Flag for definition in a draft with the usecase - i.e. draft-bashandy-rtgwg-segment-routing-ti-lfa

#36<https://trac.ietf.org/trac/6man/ticket/36> Section 3 Segment Routing Header (Comment 3)
Rev 12 Removes O Flag for definition in a draft with the usecase - i.e. draft-ali-spring-srv6-oam

#39<https://trac.ietf.org/trac/6man/ticket/39> Section 3.1.1 Ingress Node TLV (Comment 1)
Rev 12 Removes the TLV

#40<https://trac.ietf.org/trac/6man/ticket/40> Section 3.1.1 Ingress Node TLV (Comment 2)
Rev 12 Removes the TLV

#41<https://trac.ietf.org/trac/6man/ticket/41> Section 3.1.2 Egress Node TLV (Comment 1)
Rev 12 Removes the TLV

#42<https://trac.ietf.org/trac/6man/ticket/42> Section 3.1.3 Opaque Container TLV Comment 1)
Rev 12 Removes the TLV

#43<https://trac.ietf.org/trac/6man/ticket/43> Section 3.1.6 NSH Carrier TLV (Comment 1)
Rev 12 Removes the NSH TLV for definition in a draft with the usecase - i.e. draft-xuclad-spring-sr-service-chaining

#44<https://trac.ietf.org/trac/6man/ticket/44> Section 3.1.6 NSH Carrier TLV (Comment 2)
Same as #43

#45<https://trac.ietf.org/trac/6man/ticket/45> Section 3.2. SRH and RFC8200 behavior (Comment 1)
Rev 12 removes the text in question, defaulting to definition in RFC8200

#46<https://trac.ietf.org/trac/6man/ticket/46> Section 4 SRH Functions (Comment 1)
Rev 12 removes the reference

#47<https://trac.ietf.org/trac/6man/ticket/47> Section 4.1 Endpoint Function (End) (Comment 1)
Rev 12 removes the multicast comment, any discussion of multicast can be held to a document that concentrates on the usecase.


Responses not reviewed with Ron but likely to close
#52<https://trac.ietf.org/trac/6man/ticket/52> MINOR ISSUES:
U bits are not contiguous given released version of Linux Kernel with H flag support.
A-flag is removed from Rev 12
Ingress/Egress/Opaque TLVs are removed
The SRH draft is the normative document for END and END.X, draft-filsfils-spring-srv6-network-programming will be changed to reference it.



Issues from Ron that are in progress:
#27,28,32 - all share the same fate, rough agreement on what’s needed, I’m working on text.
#37 - adding text
#38 - re-review with Ron
#48 - adding text
#51 - review with Ron


Thanks
  Darren