I reviewed the document draft-ietf-l3vpn-mvpn-considerations in general
and for its operational impact.
Operations directorate reviews are solicited primarily to help the area
directors improve their efficiency, particularly when preparing for IESG
telechats, and allowing them to focus on documents requiring their attention
and spend less time on the trouble-free ones.
Improving the documents is important, but clearly a secondary purpose.
A third purpose is to broaden the OpsDir reviewers' exposure to work going
on in other parts of the IETF.
Reviews from OpsDir members do not in and of themselves cause the IESG to
raise issue with a document. The reviews may, however, convince individual
IESG members to raise concern over a particular document requiring further
discussion. The reviews, particularly those conducted in last call and
earlier, may also help the document editors improve their documents.
--
Review Summary:
Intended status: Doesn't say
More that one set of mechanisms to support multicast in a layer 3
BGP/MPLS VPN has been defined. These are presented in the documents
that define them as optional building blocks.
To enable interoperability between implementations, this document
defines a subset of features that is considered mandatory for a
multicast BGP/MPLS VPN implementation. This will help implementers
and deployers understand which L3VPN multicast requirements are best
satisfied by each option.
Is the document readable?
Yes.
Does it contain nits?
While there were no errors, idnits did spit out quite a few warnings:
idnits 2.12.01
tmp/draft-ietf-l3vpn-mvpn-considerations-06.txt:
tmp/draft-ietf-l3vpn-mvpn-considerations-06.txt(366): Found possible IPv4
address '3.4.1.1' in position 8; this doesn't match the suggested
documentation address ranges specified in RFC 3330 (or successor): blocks
192.0.2.0/24 (TEST-NET-1), 198.51.100.0/24 (TEST-NET-2), and 203.0.113.0/24
(TEST-NET-3); or the suggested 233.252.0.0/24 example multicast address
range.
tmp/draft-ietf-l3vpn-mvpn-considerations-06.txt(369): Found possible IPv4
address '3.4.1.2' in position 8; this doesn't match the suggested
documentation address ranges specified in RFC 3330 (or successor): blocks
192.0.2.0/24 (TEST-NET-1), 198.51.100.0/24 (TEST-NET-2), and 203.0.113.0/24
(TEST-NET-3); or the suggested 233.252.0.0/24 example multicast address
range.
tmp/draft-ietf-l3vpn-mvpn-considerations-06.txt(372): Found possible IPv4
address '3.4.1.3' in position 8; this doesn't match the suggested
documentation address ranges specified in RFC 3330 (or successor): blocks
192.0.2.0/24 (TEST-NET-1), 198.51.100.0/24 (TEST-NET-2), and 203.0.113.0/24
(TEST-NET-3); or the suggested 233.252.0.0/24 example multicast address
range.
tmp/draft-ietf-l3vpn-mvpn-considerations-06.txt(531): Line has weird
spacing: '... or the us...'
Checking boilerplate required by RFC 5378 and the IETF Trust (see
http://trustee.ietf.org/license-info):
== You're using the IETF Trust Provisions' Section 6.b License Notice from
12 Sep 2009 rather than the newer Notice from 28 Dec 2009. (See
http://trustee.ietf.org/license-info/)
Checking nits according to http://www.ietf.org/ietf/1id-guidelines.txt:
== No 'Intended status' indicated for this document; assuming Proposed
Standard
Checking nits according to http://www.ietf.org/id-info/checklist :
== There are 3 instances of lines with non-RFC3330-compliant IPv4
addresses
in the document. If these are example addresses, they should be
changed.
Miscellaneous warnings:
No issues found here.
Checking references for intended status: Proposed Standard
(See RFCs 3967 and 4897 for information about using normative
references
to lower-maturity documents in RFCs)
== Missing Reference: 'RFC4364' is mentioned on line 747, but not
defined
'Options A, B and C (as described in section 10 of [RFC4364]) are...'
== Outdated reference: A later version (-10) exists of
draft-ietf-l3vpn-2547bis-mcast-09
== Outdated reference: A later version (-13) exists of
draft-rosen-vpn-mcast-12
== Outdated reference: A later version (-10) exists of
draft-ietf-pim-sm-linklocal-08
Summary: 0 errors (**), 7 warnings (==), 0 comments (--).
Is the document class appropriate?
No class is stated, so I can't tell.
Is the problem well stated?
Yes.
Is the problem really