Re: [sfc] John Scudder's Discuss on draft-ietf-sfc-nsh-tlv-09: (with DISCUSS and COMMENT)

wei.yuehua@zte.com.cn Fri, 03 December 2021 08:53 UTC

Return-Path: <wei.yuehua@zte.com.cn>
X-Original-To: sfc@ietfa.amsl.com
Delivered-To: sfc@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 5FAAF3A0CDF; Fri, 3 Dec 2021 00:53:19 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.897
X-Spam-Level:
X-Spam-Status: No, score=-1.897 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, CTE_8BIT_MISMATCH=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, UNPARSEABLE_RELAY=0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id YmO-weJywb8T; Fri, 3 Dec 2021 00:53:15 -0800 (PST)
Received: from mxde.zte.com.cn (mxde.zte.com.cn [209.9.37.27]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 46A213A0CDE; Fri, 3 Dec 2021 00:53:12 -0800 (PST)
Received: from mse-eu.zte.com.cn (unknown [10.35.13.51]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mxde.zte.com.cn (FangMail) with ESMTPS id 4J566M0s7qzB5l0N; Fri, 3 Dec 2021 16:52:59 +0800 (CST)
Received: from dgapp01.zte.com.cn ([10.35.13.16]) by mse-eu.zte.com.cn with SMTP id 1B38vpmo090857; Fri, 3 Dec 2021 16:57:51 +0800 (GMT-8) (envelope-from wei.yuehua@zte.com.cn)
Received: from mapi (dgapp01[null]) by mapi (Zmail) with MAPI id mid1; Fri, 3 Dec 2021 16:52:53 +0800 (CST)
Date: Fri, 3 Dec 2021 16:52:53 +0800 (CST)
X-Zmail-TransId: 2af961a9dae581440ae0
X-Mailer: Zmail v1.0
Message-ID: <202112031652538441517@zte.com.cn>
Mime-Version: 1.0
From: <wei.yuehua@zte.com.cn>
To: <noreply@ietf.org>
Cc: <iesg@ietf.org>, <draft-ietf-sfc-nsh-tlv@ietf.org>, <sfc-chairs@ietf.org>, <sfc@ietf.org>, <gregimirsky@gmail.com>
Content-Type: text/plain; charset="UTF-8"
X-MAIL: mse-eu.zte.com.cn 1B38vpmo090857
X-Fangmail-Gw-Spam-Type: 0
X-FangMail-Miltered: at 10-35-8-64 with ID 61A9DAEB.000 by FangMail milter!
X-FangMail-Envelope: 1638521579/4J566M0s7qzB5l0N/61A9DAEB.000/10.35.13.51/[10.35.13.51]/mse-eu.zte.com.cn/<wei.yuehua@zte.com.cn>
X-Fangmail-Anti-Spam-Filtered: true
X-Fangmail-MID-QID: 61A9DAEB.000/4J566M0s7qzB5l0N
Archived-At: <https://mailarchive.ietf.org/arch/msg/sfc/p3rlLxmMkPd16Bq0rqAZLzpSLpk>
Subject: Re: [sfc] =?utf-8?q?John_Scudder=27s_Discuss_on_draft-ietf-sfc-nsh-t?= =?utf-8?q?lv-09=3A_=28with_DISCUSS_and_COMMENT=29?=
X-BeenThere: sfc@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
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: Fri, 03 Dec 2021 08:53:20 -0000

John,
I appreciate your comments.
See comments resolution inline begin with Yuehua>>

Best Regards,
Yuehua Wei
------------------原始邮件------------------
发件人:JohnScudderviaDatatracker
收件人:The IESG;
抄送人:draft-ietf-sfc-nsh-tlv@ietf.org;sfc-chairs@ietf.org;sfc@ietf.org;gregimirsky@gmail.com;gregimirsky@gmail.com;
日 期 :2021年12月02日 03:29
主 题 :John Scudder's Discuss on draft-ietf-sfc-nsh-tlv-09: (with DISCUSS and COMMENT)
John Scudder has entered the following ballot position for
draft-ietf-sfc-nsh-tlv-09: Discuss
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/blog/handling-iesg-ballot-positions/
for more information about how to handle DISCUSS and COMMENT positions.
The document, along with other ballot positions, can be found here:
https://datatracker.ietf.org/doc/draft-ietf-sfc-nsh-tlv/
----------------------------------------------------------------------
DISCUSS:
----------------------------------------------------------------------
1. I notice that in his RTGDIR review of version 08 [*], Stig Venaas suggested
some improvements to the security considerations section. This was subsequently
discussed and Yuehua Wei proposed some new text [**] for version 09. That text
isn’t present, and I don’t see any further resolution on the mailing list
either. I’d appreciate it if the topic were closed by either adding the
proposed text, or some other text to resolve Stig’s concern, or explanation of
why no change was made.
[*]
https://datatracker.ietf.org/doc/review-ietf-sfc-nsh-tlv-08-rtgdir-lc-venaas-2021-09-29/
[**] https://mailarchive.ietf.org/arch/msg/sfc/Q2Snf_ZLTkJ1augbaWpmNYlwFBU/

Yuehua>> Thank you ,you are right , I  will merge the proposed text to ver10 for further review and discussion.

2. In §8.2, the two first references, [GROUPBASEDPOLICY] and [GROUPPOLICY] are
deficient. At a minimum, a reference should provide enough information to allow
a reader to straightforwardly determine how to retrieve it. This is true even
if it’s not an openly-available online source. These two references have less
than the bare bones, I don’t know how to find them or refer to them.

Yuehua>> Thank you, you are right, The weblinks in the current XML file don't show well in the txt or html file , will fix them to ver10.

----------------------------------------------------------------------
COMMENT:
----------------------------------------------------------------------
1. I support all of Ben’s discuss points. I also want to reiterate his comment
about the desirability of having useful captions on the figures.

Yuehua>>will state the resolutions to the thread of Ben's comments

2. In §4.2, you write,
This context header carries both the format
and value of the Tenant identifier.
However, I don’t see anywhere that the header “carries… the format”. Indeed,
you write that the Tenant ID is an opaque value. As far as I can tell, there’s
no way to infer anything about its structure without a priori knowledge.
If that is correct, you can simplify the sentence to “This context header
carries the Tenant Identifier.” If it’s not correct, please explain?

Yuehua>> Accepted

3. Nit, in §4.7 the words “quite efficiently” don’t seem to serve any useful
purpose; the document would be better off without them IMO.

Yuehua>> Accepted. will delete “quite efficiently”