[sfc] Genart last call review of draft-ietf-sfc-multi-layer-oam-23

Behcet Sarikaya via Datatracker <noreply@ietf.org> Mon, 15 May 2023 15:41 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: sfc@ietf.org
Delivered-To: sfc@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id AB5D3C17B35A; Mon, 15 May 2023 08:41:18 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Behcet Sarikaya via Datatracker <noreply@ietf.org>
To: gen-art@ietf.org
Cc: draft-ietf-sfc-multi-layer-oam.all@ietf.org, last-call@ietf.org, sfc@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 10.3.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <168416527869.53196.15333590507548997462@ietfa.amsl.com>
Reply-To: Behcet Sarikaya <sarikaya@ieee.org>
Date: Mon, 15 May 2023 08:41:18 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/sfc/1YGsolhncOFx5MGTULFbGwAPSpU>
Subject: [sfc] Genart last call review of draft-ietf-sfc-multi-layer-oam-23
X-BeenThere: sfc@ietf.org
X-Mailman-Version: 2.1.39
List-Id: Network Service Chaining <sfc.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/sfc>, <mailto:sfc-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/sfc/>
List-Post: <mailto:sfc@ietf.org>
List-Help: <mailto:sfc-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/sfc>, <mailto:sfc-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 15 May 2023 15:41:18 -0000

Reviewer: Behcet Sarikaya
Review result: Ready with Nits

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://wiki.ietf.org/en/group/gen/GenArtFAQ>.

Document: draft-ietf-sfc-multi-layer-oam-??
Reviewer: Behcet Sarikaya
Review Date: 2023-05-15
IETF LC End Date: 2023-05-17
IESG Telechat date: Not scheduled for a telechat

Summary:
This document defines  Service Function Chaining (SFC) Operation, Maintenance
and Management (OAM) protocol. It defines in Section 6 ICMP like Echo
Request/Reply for SFC to do Operation, Maintenance and Management mainly for
Fault Management as SFC messages which constitutes the bulk of SFC OAM
protocol. As shepherd write-up explains it grew out of 3 documents all lead by
Greg Mirsky when he was with ZTE. Major issues:

Minor issues:
Author list has 6 people as also indicated in shepherd write-up. The document
already contains two names in the contributor list. Strongly recommend having
one more in the contributor list.

Nits/editorial comments: please add the and as shown below:
This mechanism enables on-demand Continuity Check and
   Connectivity Verification among other operations over SFC in networks
  and addresses functionalities discussed in Sections 4.1, 4.2, and 4.3 of
   [RFC8924].

The draft name contains multi-layer which is not mentioned elsewhere in the
draft.

draft-ietf-sfc-nsh-tlv -> RFC 9263

draft-ietf-sfc-oam-packet normative reference is in RFC Editor Queue

Behcet