Early Review of draft-ietf-idr-sla-exchange-10
review-ietf-idr-sla-exchange-10-rtgdir-early-bonica-2017-02-16-00

Hello,

I have been selected as the Routing Directorate reviewer for this draft. The
Routing Directorate seeks to review all routing or routing-related drafts as
they pass through IETF last call and IESG review. The purpose of the review is
to provide assistance to the Routing ADs. For more information about the
Routing Directorate, please see
​http://trac.tools.ietf.org/area/rtg/trac/wiki/RtgDir

Although these comments are primarily for the use of the Routing ADs, it would
be helpful if you could consider them along with any other IETF Last Call
comments that you receive, and strive to resolve them through discussion or by
updating the draft.

Document:  draft-ietf-idr-sla-exchange-10
Reviewer: Ron Bonica
Review Date:  2/16/2017
IETF LC End Date: TBD
Intended Status: Standards Track

Summary:

I have some minor concerns about this document that I think should be resolved
before publication.

Comments:

Major Issues:

This document might benefit from discussion of operational issues. I assume
that when a BGP listener learns a route with the SLA Exchange Attribute, it
provisions class of service forwarding classes on interfaces. I also assume
that a) it takes time to provision class of service forwarding classes and b)
the number of forwarding classes that can be provisioned are finite. What does
the BGP listener do when the number of forwarding classes requested exceeds its
capacity to deliver? When a route flaps? How does the router protect itself

In the Security Considerations section, I am concerned about the possibility of
intermediate AS's modifying the SLA Exchange Attribute. It seems that you need
to have some degree of trust in every AS on the path (not only those included
in the attribute)

Minor Issues:

In Section 3.2, is the flag really needed? Doesn't an AS list containing only
the receivers AS have exactly the same meaning?

Nits:

Miscellaneous warnings:
  ----------------------------------------------------------------------------

  == The document seems to use 'NOT RECOMMENDED' as an RFC 2119 keyword, but
     does not include the phrase in its RFC 2119 key words list.

  Checking references for intended status: Proposed Standard
  ----------------------------------------------------------------------------

     (See RFCs 3967 and 4897 for information about using normative references
     to lower-maturity documents in RFCs)

  == Unused Reference: 'RFC2434' is defined on line 1279, but no explicit
     reference was found in the text

  == Unused Reference: 'RFC6793' is defined on line 1301, but no explicit
     reference was found in the text

  ** Obsolete normative reference: RFC 2434 (Obsoleted by RFC 5226)

  ** Downref: Normative reference to an Informational RFC: RFC 4272

  ** Downref: Normative reference to an Informational RFC: RFC 7132

  == Outdated reference: draft-ietf-netconf-restconf has been published as
     RFC 8040