[bess] AD Review of draft-ietf-bess-fat-pw-bgp-03

[Alvaro Retana <[e-mail]>]

Dear authors:

I just finished reading this document.  Thank you for a well written and
straight forward document!!

I have some comments (see below) that I think are easy to address.  I am
then starting the IETF Last Call.

Thanks!

Alvaro.


Major:

M1. All the rfc2119 keywords in this text should not be capitalized because
they are part of an example:

   For example, a PE part of a VPLS and with a local T = 1,
   MUST only transmit traffic with a flow label to those peers that
   signaled R = 1.  And if the same PE has local R = 1, it MUST only
   expect to receive traffic with a flow label from peers with T = 1.
   Any other traffic MUST NOT have a flow label.


M2. Security Considerations: I agree that there are no new issues.
However, please also point to rfc4761 and any other document that defines
the base functionality being modified here.


Minor:

P1. "This draft introduces an OPTIONAL mode of operation..."  There's no
need for "OPTIONAL" to be Normative in this sentence since it is just
describing what it is, not specifying behavior.  s/OPTIONAL/optional


P2. The new registry has a policy of "IETF Review", which basically means
that any RFC (not just Standards Track RFCs) can use the bits in the
registry.  I ask because there are only 4 bits left.  Note that I'm not
asking you to necessarily change the policy...just pointing it out.


P3. "T   When the bit value is 1, the PE is requesting the ability..."  Did
you mean "announce the ability" instead?


P4. s/NUST NOT/MUST NOT


P5. References:  I think these can be Informative: rfc4385, rfc8077, rfc4928