[Lsr] Opsdir last call review of draft-ietf-ospf-te-link-attr-reuse-12

Scott Bradner via Datatracker <noreply@ietf.org> Wed, 27 May 2020 15:17 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: lsr@ietf.org
Delivered-To: lsr@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 742043A0E90; Wed, 27 May 2020 08:17:05 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
From: Scott Bradner via Datatracker <noreply@ietf.org>
To: <ops-dir@ietf.org>
Cc: draft-ietf-ospf-te-link-attr-reuse.all@ietf.org, lsr@ietf.org, last-call@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 7.1.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <159059262542.19823.6779966735787003447@ietfa.amsl.com>
Reply-To: Scott Bradner <sob@sobco.com>
Date: Wed, 27 May 2020 08:17:05 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/lsr/svTw0Mitm2L_Rl5qHnYY-tQGZ3E>
Subject: [Lsr] Opsdir last call review of draft-ietf-ospf-te-link-attr-reuse-12
X-BeenThere: lsr@ietf.org
X-Mailman-Version: 2.1.29
List-Id: Link State Routing Working Group <lsr.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/lsr>, <mailto:lsr-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/lsr/>
List-Post: <mailto:lsr@ietf.org>
List-Help: <mailto:lsr-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/lsr>, <mailto:lsr-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 27 May 2020 15:17:06 -0000

Reviewer: Scott Bradner
Review result: Not Ready

This is an OPS-DIR review of OSPF Link Traffic Engineering Attribute Reuse
(draft-ietf-ospf-te-link-attr-reuse)

This ID describes application-specific attribute advertisements for use in OSPF.

I found this ID hard to read and recommend that it be reviewed for readability.

I have a basic question about this proposal – the ID describes specific
advertisements to be used when particular applications want to make use of
specific link attributes and says that other applications should not make of
the information in the advertisement without saying why such use would be a
problem.  I can imagine some reasons but it seems to me that it would be good
if this document just explained the problem it is trying to solve.

Some specific issues in the document

Page 6 – the text says “Standard Application Identifier Bit Mask: Optional set
of bits, where each bit represents a single standard application.  Bits are
defined in [I-D.ietf-isis-te-app].”  - it seems to me that this should be in an
IANA registry for extensibility but it does not seem to be in the referenced ID
but I could not actually tell

Page 6 – text says “The bits are repeated here for informational purpose” 
maybe point to a IANA registry or say “current assignments”

Page 6 – text says “If the link attribute advertisement is limited to be used
by a specific set of applications”  - maybe say “intended” rather than
“limited” since I do not see a way to actually limit a future application from
eavesdropping on the advertisement

Page 7 – the text says “If the SABM or UDABM length is other than 0, 4, or 8,
the ASLA sub-TLV MUST be ignored by the receiver.”  - it would seem to be
useful operations-wise to say that an indication of an error should be recorded
somewhere

Page 7 – a “User Defined Application Identifier” is introduced but never
described – what uses it and what is it used for

Section 11 – I found this discussion of the relationship between the existence
of a particular advertisement and the possible existence of an application to
use that advertisement to be quite confusing – if the existence of a particular
advertisement does not indicate that any application is listening why not just
say that?

Section 12.1 – it would help to say what problem is trying to be solved – why
is the use of RSVP-TE LSA advertisements a problem?

Section 12.3.3 – I could not tell if this section is saying that the
application specific attribute advertisements could not be used if there is
even a single legacy router present of if the presence of such a router means
that the application specific attribute advertisements can be used but the old
advertisements must also be used Section 14 – it might help to say how new
Sub-TLV types can be added to the registry