Re: [mpls] MPLS-RT review of draft-smack-mpls-rfc4379bis

[Kireeti Kompella <[e-mail]>]

Hi Yimin,

Thanks for your review!

On Fri, Dec 4, 2015 at 10:17 PM, Yimin Shen <[e-mail]> wrote:

> Hi authors,
>
> I was selected as a reviewer for the draft-smack-mpls-rfc4379bis, and I
> have completed my review.
>
> I think this draft is useful, providing an overdue update for RFC 4379,
> which is the base RFC of a widely deployed protocol in MPLS networks.
>
> One issue I have is that the draft is still specifying the packet format
> and mechanisms based on the Downstream Mapping TLV, which has been
> deprecated by the Downstream Detailed Mapping TLV defined by RFC 6424. I
> notice the section 1.5 where it briefly mentions the evaluation of
> incorporating some new RFCs to this draft. But it is unclear to me what the
> decision will be, whether the draft would stop here to provide just
> backward compatibility or it will take further steps to incorporate RFC
> 6424. So, before the draft is adopted, I'd like to see draft progress a
> little further on the to-do list and give and clearer strategy on this.
>

I don't really care whether this is done before or after WG adoption.
However, one question for the WG is, should deprecated TLVs be moved into
an appendix?  Sort of as follows:

3.2.8 <http://tools.ietf.org/html/rfc4379#section-3.2.8>.  FEC 128
Pseudowire (Deprecated)


(See Appendix A for details.)


3.2.9 <http://tools.ietf.org/html/rfc4379#section-3.2.9>.  FEC 128
Pseudowire (Current)


*...*


[and of course put the text currently in section 3.2.8 in the
Appendix.]  This preserves the historical stuff while moving them out
of the way of implementors.


Cheers,

Kireeti.



> Thanks,
>
> /Yimin Shen
>
>


-- 
Kireeti