[Teas] Genart last call review of draft-ietf-teas-sr-rsvp-coexistence-rec-02
Joel Halpern <jmh@joelhalpern.com> Thu, 19 April 2018 10:15 UTC
Return-Path: <jmh@joelhalpern.com>
X-Original-To: teas@ietf.org
Delivered-To: teas@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id BC09A12D942; Thu, 19 Apr 2018 03:15:45 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Joel Halpern <jmh@joelhalpern.com>
To: gen-art@ietf.org
Cc: ietf@ietf.org, teas@ietf.org, draft-ietf-teas-sr-rsvp-coexistence-rec.all@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.78.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <152413294572.28724.1279477772622511232@ietfa.amsl.com>
Date: Thu, 19 Apr 2018 03:15:45 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/teas/8Xt3F9hEbSpa_njuq8xuO-6FAuI>
Subject: [Teas] Genart last call review of draft-ietf-teas-sr-rsvp-coexistence-rec-02
X-BeenThere: teas@ietf.org
X-Mailman-Version: 2.1.22
List-Id: Traffic Engineering Architecture and Signaling working group discussion list <teas.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/teas>, <mailto:teas-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/teas/>
List-Post: <mailto:teas@ietf.org>
List-Help: <mailto:teas-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/teas>, <mailto:teas-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 19 Apr 2018 10:15:46 -0000
Reviewer: Joel Halpern
Review result: Almost Ready
I am the assigned Gen-ART reviewer for this draft. The General Area
Review Team (Gen-ART) reviews all IETF documents being processed
by the IESG for the IETF Chair. Please treat these comments just
like any other last call comments.
For more information, please see the FAQ at
<https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.
Document: draft-ietf-teas-sr-rsvp-coexistence-rec-02
Reviewer: Joel Halpern
Review Date: 2018-04-19
IETF LC End Date: 2018-04-20
IESG Telechat date: 2018-05-10
Summary:
Major issues:
The focus of the draft seems to be the recommendation in section 3.5 that
the maximum reservable bandwidth on a link be adjusted to reflect the SR
traffic consumption. There appear to be two issues that need to be
discussed, both related to the difference between what the SR controller
wants to reserve and what the router observes. First, an SR controller may
be performing calculations without requiring that bandwidth be committed to
the traffic. The recommendation here assumes that all traffic using SR is
high priority. It may suffice to note that QoS markings in the labels
(corresponding to diffserv markings in the underlying packet may hel with
this. Given the range of allowed behaviors in when RSVP-TE and SR are
separate, it may also be necessary to restrict what the SR controllers do
in these interworking cases. Second, and more importantly, this solution
assumes that short term traffic measurements are a good proxy for intended
reservation. Even assuming edge policing so that usage is less than or
equal to the reservation, this will frequently underestimate the traffic
reservation. Such underestimates would seem to be able to cause
significant problems.
Minor issues:
Section 3.5 assumes that the router can measure the traffic using SR. This
seems to rely on the unstated premise that the measurement is conditioned
by the recognition of which labels are being used for SR. This is
reasonable. It should be stated.
Nits/editorial comments:
The second paragraph of the introduction seems to have the opening text
repeated twice.
The third paragraph of section 3.1 seems to be a repetition of the end of
the second paragraph using slightly different words.
- [Teas] Genart last call review of draft-ietf-teas… Joel Halpern
- Re: [Teas] Genart last call review of draft-ietf-… Harish Sitaraman
- Re: [Teas] Genart last call review of draft-ietf-… Joel M. Halpern
- Re: [Teas] Genart last call review of draft-ietf-… Joel M. Halpern
- Re: [Teas] Genart last call review of draft-ietf-… Harish Sitaraman