[mpls] Roman Danyliw's No Objection on draft-ietf-mpls-sfl-framework-10: (with COMMENT)

Roman Danyliw via Datatracker <noreply@ietf.org> Wed, 23 September 2020 16:02 UTC

MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
From: Roman Danyliw via Datatracker <noreply@ietf.org>
To: The IESG <iesg@ietf.org>
Cc: draft-ietf-mpls-sfl-framework@ietf.org, mpls-chairs@ietf.org, mpls@ietf.org, Tarek Saad <tsaad.net@gmail.com>, tsaad.net@gmail.com
Auto-Submitted: auto-generated
Precedence: bulk
Reply-To: Roman Danyliw <rdd@cert.org>
Message-ID: <160087694154.15979.1302812867971446914@ietfa.amsl.com>
Date: Wed, 23 Sep 2020 09:02:21 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/mpls/JJJa8qtNIp7RhXq2S_iufAzxboQ>
Subject: [mpls] Roman Danyliw's No Objection on draft-ietf-mpls-sfl-framework-10: (with COMMENT)

Roman Danyliw has entered the following ballot position for
draft-ietf-mpls-sfl-framework-10: No Objection

When responding, please keep the subject line intact and reply to all
email addresses included in the To and CC lines. (Feel free to cut this
introductory paragraph, however.)


Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html
for more information about IESG DISCUSS and COMMENT positions.


The document, along with other ballot positions, can be found here:
https://datatracker.ietf.org/doc/draft-ietf-mpls-sfl-framework/



----------------------------------------------------------------------
COMMENT:
----------------------------------------------------------------------

Thank you for responding to the SECDIR review (and thank you to Tero Kivinen
for doing it)

I had the same reaction as Martin Duke – I found one RFC2119/normative phrase
in the document (Section 5).  I’m not sure if this means it should be
information or normative requirements should be articulated more clearly.

The reviews TSVART and Ben Kaduk already capture my feedback on Sections 6 and
7.

** Section 3.  What is “triggering IPFIX inspection”?  I wasn’t familiar with
this phrasing.  Is the intent to trigger more in-depth analysis of header
information, as opposed to “triggering … DPI”, which will do content inspection?

** Editorial Nits:
-- Section 3.  Typo. s/co-ordinating/coordinating/

-- Section 4.1.  Typo. s/are present/is present/

-- Section 5.  Typo. s/invalidate the certain uses/invalidate certain uses/

[mpls] Roman Danyliw's No Objection on draft-ietf… Roman Danyliw via Datatracker