Re: [mpls] Extended Working Gorup Last Call and IPR Poll on draft-ietf-mpls-spring-inter-domain-oam-06

[Shraddha Hegde <[e-mail]>]

Dhruv,

Thanks for the follow-up.
Pls see inline for replies.
Posting -09 with changes

Rgds
Shraddha



Juniper Business Use Only
From: Dhruv Dhody <[e-mail]>
Sent: Friday, January 26, 2024 12:56 PM
To: Shraddha Hegde <[e-mail]>
Cc: [e-mail]; [e-mail]
Subject: Re: [mpls] Extended Working Gorup Last Call and IPR Poll on draft-ietf-mpls-spring-inter-domain-oam-06

[External Email. Be cautious of content]

Hi Shraddha,

Snipping to open points...

* You should add a reference to how EPE SIDs are handled, as your examples use them. Any issue adding text and reference to https://datatracker.ietf.org/doc/draft-ietf-mpls-sr-epe-oam/<https://urldefense.com/v3/__https:/datatracker.ietf.org/doc/draft-ietf-mpls-sr-epe-oam/__;!!NEt6yMaO-gk!DwN3BBepkygqgUdOp3H4S-t2LoMtwUuEzFn_V_TL4Yk7UG4FwVzOPTiUEuGNFjHzcVpjgrmec-QpkHhVnw0y$>?
<SH> https://datatracker.ietf.org/doc/draft-ietf-mpls-sr-epe-oam<https://urldefense.com/v3/__https:/datatracker.ietf.org/doc/draft-ietf-mpls-sr-epe-oam__;!!NEt6yMaO-gk!GeD7LXSXenrWU5MEGHg6TfUmDY0eDxy5l_eLVSKx1-M4yWo0sqZMbqaJlvNv0W3nv1cFuux7BrAR2kPNJ1nH$> is for defining FEC for verifying the EPE-SIDs themselves. In this draft EPE-SIDs are being used to construct reverse paths. The two topics are quite independent and I don’t see the need to reference the epe-oam draft and create unnecessary reference to another
WG document.


Dhruv: You are correct. But then what would be the best reference for EPE SIDs? Note that RFC 9087 uses different terms. In the example what you call EPE SID should it be called PeerAdj SID? Or am I missing something....

 <SH> I see your point. I added below statement to define EPE SID in this document
PeerNode SID/PeerAdj SID/PeerSet SID are
referred to as Egress Peer Engineering (EPE) SIDs in this document.

* Suggest to follow the RFC7942 template for section 12.1
<SH> Its not clear what template to be followed. An example would help.

Dhruv: Template is defined at https://datatracker.ietf.org/doc/html/rfc7942#section-2<https://urldefense.com/v3/__https:/datatracker.ietf.org/doc/html/rfc7942*section-2__;Iw!!NEt6yMaO-gk!GeD7LXSXenrWU5MEGHg6TfUmDY0eDxy5l_eLVSKx1-M4yWo0sqZMbqaJlvNv0W3nv1cFuux7BrAR2lc7vKZn$>
Example would be: https://www.ietf.org/archive/id/draft-ietf-pce-local-protection-enforcement-11.html#section-6<https://urldefense.com/v3/__https:/www.ietf.org/archive/id/draft-ietf-pce-local-protection-enforcement-11.html*section-6__;Iw!!NEt6yMaO-gk!GeD7LXSXenrWU5MEGHg6TfUmDY0eDxy5l_eLVSKx1-M4yWo0sqZMbqaJlvNv0W3nv1cFuux7BrAR2nbHZfAH$>

--
<SH> Updated

Looking at the IANA section, just one more suggestion -

- Section 10.1, Reference should be "this document" of "Section X of this document", just "Section X" would be incorrect! Also better to say in text that the below allocation should be via standard action range of 0-16383 instead of putting it in the table, where it can cause confusion.
- Section 10.3, Same comment as above.
<SH> Done
Thanks!
Dhruv