[Resending]
I am the assigned Gen-ART reviewer for this draft. For background on
Gen-ART, please see the FAQ at
< http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.
Please wait for direction from your document shepherd
or AD before posting a new version of the draft.
Document: draft-ietf-idr-ls-distribution-10.txt
Reviewer: Alexey Melnikov
Review Date: 2015-05-10
IETF LC End Date: 2015-04-08
IESG Telechat date: N/A
My apologies for the late review of this document.
Summary: Ready with nits
Minor (but some of these might be more serious):
In 6.2.2:
If an implementation of BGP-LS detects a malformed attribute, then it
SHOULD use the Â’Attribute DiscardÂ’ action as per
[I-D.ietf-idr-error-handling] Section 2.
This needs to be a Normative reference. Or you can keep it as
Informative, if you change the sentence not to use RFC 2119 language.
In 3.3.1.1 - does this need a new IANA registry? (I am fine if you think
you don't).
In 3.3.1.3/3.3.2.7 - what is "subset of the FQDN"?
In 3.3.2.3:
The TE Default Metric TLV carries the TE-metric for this link.
The length of this TLV is fixed at 4 octets.
I am probably showing my ignorance, but is the term "TE-metric" defined
somewhere? The description below suggests it has substructure, which I
don't know anything about.
If a source protocol (e.g.
IS-IS) does not support a Metric width of 32 bits then the high
order octet MUST be set to zero.
Best Regards,
Alexey
_______________________________________________
Gen-art mailing list
Gen-art@ietf.org
https://www.ietf.org/mailman/listinfo/gen-art
