[mpls] MPLS-RT comments on draft-ali-mpls-inter-domain-p2mp-rsvp-te-lsp-08

["Eric Osborne (eosborne)" <[e-mail]>]

These are my review team comments on draft-ali-mpls-inter-domain-p2mp-rsvp-te-lsp-08.  

* Is the document coherent?
 
It needs work.  The formatting is off (try to print the pdf version of the txt doc and you'll see what I mean).  It needs a serious editorial cleanup - for example, section 4.1/4.2 refer to a flag as both Recording and Recording Request, and another as Re-merge Presence and Re-merge Present.  It also has a structure that makes it hard to follow, and a large number of SHOULDs where I think there ought to be MUSTs.


* Is it useful (ie, is it likely to be actually useful in operational networks), and is the document technically sound?  

I believe that this document solves the problem it purports to solve.  It does so largely by adding three bits and an error code to extend existing p2mp single-area procedures across domain boundaries.  I think at its heart it is technically sound, although its current state makes it hard to follow.


* Is the document ready to be considered for WG adoption?

There are three parts to this question: technical suitability, document quality, and desirability.

The document seems technically suitable; the procedures it defines are a good start to solving the problem, and that's what a document needs to be before WG status.

As I said above, the quality is not quite there.

As far as desirabilty...I have reviewed v08 of the document.  v00 was posted in July of 2008, so the doc is quite old.  Although I buy into the general idea that things which work in a single domain should work across domains, I'd like to see whether this document has interest from others, primarily operators.  



eric