Hi Ron,

Thanks for your review.

Unfortunately, you seem to have not get the latest version from the Github 
(which was later posted as v03).
Good news is that your comments were addressed in the latest version.

Please see specifics in-line.

Happy Holidays!

Thanks

Regards … Zafar

From: spring <spring-boun...@ietf.org> on behalf of Ron Bonica 
<rbonica=40juniper....@dmarc.ietf.org>
Date: Wednesday, December 18, 2019 at 4:07 PM
To: Ole Troan <otr...@employees.org>, 6man WG <i...@ietf.org>, SPRING WG 
<spring@ietf.org>
Cc: 6man Chairs <6man-cha...@ietf.org>
Subject: Re: [spring] 6man w.g. last call for <draft-ietf-6man-spring-srv6-oam>

Ole,

I have reviewed version-02 of this document and believe that it still has the 
following major issues:

- Section 3.1.1 assumes implementation details (e.g., that the implementation 
has an OAM process) but fails to describe any externally observable behavior. 
The externally observable behaviors are described, incompletely, in Sections 
4.1.2 and 4.2.2.

[ZA] Like you mentioned, processing details were embedded in the Section 4. We 
brought them up in the Section 3 and also added additional normative language 
in Section 4. Specifically, In the new revision:

  *   We have added normative text at the beginning of 3.1.1 where O-bit is 
defined.
  *   Sections 3.3 and 3.4 adds normative texts for OAM SIDs.
  *   4.1.2 and 4.2.2 further adds additional normative text for Ping and 
traceroute use-cases, respectively.

- Section 3.5 seems to be incomplete. What TLV are we talking about and what 
role does it play?

[ZA] This has been already addressed. Specifically, indeed, the draft does not 
define any TLV for OAM purposes. However, section was added as future drafts 
may define OAM TLVs. However, the section has been removed in the new revision.

- Section 4 talks about future revisions of this document. Aren't we in WGLC?

[ZA] This has been already addressed in the latest version.

- Section 4.1.2.1 - When the ping succeeds, what does the success message look 
like? An ICMP Echo Response? What is the source address? A SID?

[ZA] Yes, it is an ICMP Echo Response ICMP Echo Response. The v03 specifically 
mentions the same. As mentioned in the draft, the upper layer header processing 
follows RFC4443.

- Section 4.1.2.2 - Does this work with PSP?

[ZA] This does not apply. There is no PSP flavor defined for END.OP SID.

- Section 4.2.2.1 - Do we need to examine flags even when SL == 0?

[ZA] The O-bit is for telemetry purpose and packets with SL=0 are also 
telemetered.


                                                                                
                    Ron



Juniper Business Use Only

-----Original Message-----
From: ipv6 <ipv6-boun...@ietf.org<mailto:ipv6-boun...@ietf.org>> On Behalf Of 
Ole Troan
Sent: Wednesday, December 4, 2019 3:53 PM
To: 6man WG <i...@ietf.org<mailto:i...@ietf.org>>; SPRING WG 
<spring@ietf.org<mailto:spring@ietf.org>>
Cc: 6man Chairs <6man-cha...@ietf.org<mailto:6man-cha...@ietf.org>>
Subject: 6man w.g. last call for <draft-ietf-6man-spring-srv6-oam>

Hello,

  As agreed in the working group session in Singapore, this message starts a 
new two week 6MAN Working Group Last Call on advancing:

  Title    : Operations, Administration, and Maintenance (OAM) in Segment 
Routing Networks with IPv6 Data plane (SRv6)
  Author   : Z. Ali, C. Filsfils, S. Matsushima, D. Voyer, M. Chen
  Filename : draft-ietf-6man-spring-srv6-oam-02
  Pages    : 23
  Date     : 2019-11-20

    
https://urldefense.com/v3/__https://datatracker.ietf.org/doc/draft-ietf-6man-spring-srv6-oam/__;!8WoA6RjC81c!XSqnzjZKANP41cvNa8ybSjYCuTDO6XCHjQyZ4ARNtx4MhYWG7vdEvXi-22No-zDm$<https://urldefense.com/v3/__https:/datatracker.ietf.org/doc/draft-ietf-6man-spring-srv6-oam/__;!8WoA6RjC81c!XSqnzjZKANP41cvNa8ybSjYCuTDO6XCHjQyZ4ARNtx4MhYWG7vdEvXi-22No-zDm$>

as a Proposed Standard.

Substantive comments and statements of support for publishing this document 
should be directed to the mailing list.
Editorial suggestions can be sent to the author. This last call will end on the 
18th of December 2019.

To improve document quality and ensure that bugs are caught as early as 
possible, we would require at least two reviewers to do a complete review of 
the document.  Please let the chairs know if you are willing to be a reviewer.

The last call will be forwarded to the spring working group, with discussion 
directed to the ipv6 list.

Thanks,
Bob & Ole, 6man co-chairs


--------------------------------------------------------------------
IETF IPv6 working group mailing list
i...@ietf.org<mailto:i...@ietf.org>
Administrative Requests: 
https://urldefense.com/v3/__https://www.ietf.org/mailman/listinfo/ipv6__;!8WoA6RjC81c!XSqnzjZKANP41cvNa8ybSjYCuTDO6XCHjQyZ4ARNtx4MhYWG7vdEvXi-2-1yR5ZL$<https://urldefense.com/v3/__https:/www.ietf.org/mailman/listinfo/ipv6__;!8WoA6RjC81c!XSqnzjZKANP41cvNa8ybSjYCuTDO6XCHjQyZ4ARNtx4MhYWG7vdEvXi-2-1yR5ZL$>
--------------------------------------------------------------------

_______________________________________________
spring mailing list
spring@ietf.org<mailto:spring@ietf.org>
https://www.ietf.org/mailman/listinfo/spring

_______________________________________________
spring mailing list
spring@ietf.org
https://www.ietf.org/mailman/listinfo/spring

Reply via email to