[Pce] 答复: Secdir early review of draft-ietf-pce-pcep-extension-native-ip-29

Aijun Wang <wangaijun@tsinghua.org.cn> Thu, 01 February 2024 03:41 UTC

Return-Path: <wangaijun@tsinghua.org.cn>
X-Original-To: pce@ietfa.amsl.com
Delivered-To: pce@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id BCC07C14F71E; Wed, 31 Jan 2024 19:41:18 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.908
X-Spam-Level:
X-Spam-Status: No, score=-1.908 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_MSPIKE_H2=-0.001, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01, URIBL_DBL_BLOCKED_OPENDNS=0.001, URIBL_ZEN_BLOCKED_OPENDNS=0.001] autolearn=ham autolearn_force=no
Received: from mail.ietf.org ([50.223.129.194]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id L0EthK-nud0P; Wed, 31 Jan 2024 19:41:17 -0800 (PST)
Received: from mail-m49198.qiye.163.com (mail-m49198.qiye.163.com [45.254.49.198]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 7C57AC14E515; Wed, 31 Jan 2024 19:41:09 -0800 (PST)
Received: from LAPTOP09T7970K (unknown [219.142.69.75]) by smtp.qiye.163.com (Hmail) with ESMTPA id D4B917E0136; Thu, 1 Feb 2024 11:40:53 +0800 (CST)
From: Aijun Wang <wangaijun@tsinghua.org.cn>
To: 'Ned Smith' <ned.smith@intel.com>, secdir@ietf.org
Cc: draft-ietf-pce-pcep-extension-native-ip.all@ietf.org, pce@ietf.org
References: <170673086511.57536.5395552393929333066@ietfa.amsl.com>
In-Reply-To: <170673086511.57536.5395552393929333066@ietfa.amsl.com>
Date: Thu, 01 Feb 2024 11:40:52 +0800
Message-ID: <000e01da54c0$75120460$5f360d20$@tsinghua.org.cn>
MIME-Version: 1.0
Content-Type: text/plain; charset="gb2312"
Content-Transfer-Encoding: quoted-printable
X-Mailer: Microsoft Outlook 16.0
Thread-Index: AQFKn0D6BkMydBm+Li1/j2uGSWXDj7IUKJUA
Content-Language: zh-cn
X-HM-Spam-Status: e1kfGhgUHx5ZQUpXWQgPGg8OCBgUHx5ZQUlOS1dZFg8aDwILHllBWSg2Ly tZV1koWUFKTEtLSjdXWS1ZQUlXWQ8JGhUIEh9ZQVlDGklKVkwYGUpCGBkaGkNDH1UTARMWGhIXJB QOD1lXWRgSC1lBWUlKQlVKT0lVTUJVTE5ZV1kWGg8SFR0UWUFZT0tIVUpNT0lMTlVKS0tVSkJLS1 kG
X-HM-Tid: 0a8d62c1558903a2kunmd4b917e0136
X-HM-MType: 10
X-HM-Sender-Digest: e1kMHhlZQR0aFwgeV1kSHx4VD1lBWUc6NAw6USo5HDMXMk0DLBpLLU0v NC8wFE5VSlVKTEtNTE5DQ05PT0pMVTMWGhIXVQwaFRwaEhEOFTsPCBIVHBMOGlUUCRxVGBVFWVdZ EgtZQVlJSkJVSk9JVU1CVUxOWVdZCAFZQU9IQ0s3Bg++
Archived-At: <https://mailarchive.ietf.org/arch/msg/pce/9YAnYH1yMHm6J1aXlYb8SZLPZ5E>
Subject: [Pce] 答复: Secdir early review of draft-ietf-pce-pcep-extension-native-ip-29
X-BeenThere: pce@ietf.org
X-Mailman-Version: 2.1.39
Precedence: list
List-Id: Path Computation Element <pce.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/pce>, <mailto:pce-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/pce/>
List-Post: <mailto:pce@ietf.org>
List-Help: <mailto:pce-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/pce>, <mailto:pce-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 01 Feb 2024 03:41:18 -0000

Hi, Ned:

Thanks for your review.
Some detail replies are inline below.

If you have no further comments, I will upload it to address your
concerns/comments then.

Aijun Wang

-----邮件原件-----
发件人: forwardingalgorithm@ietf.org [mailto:forwardingalgorithm@ietf.org]
代表 Ned Smith via Datatracker
发送时间: 2024年2月1日 3:54
收件人: secdir@ietf.org
抄送: draft-ietf-pce-pcep-extension-native-ip.all@ietf.org; pce@ietf.org
主题: [Pce] Secdir early review of
draft-ietf-pce-pcep-extension-native-ip-29

Reviewer: Ned Smith
Review result: Has Nits

1) Section 5 header should capitalize "messages"
【WAJ】:Done, together with the sub section 5.1 and 5.2

2) The introduction says, "It is necessary to use the central control mode
described in [RFC8283]". This reads like a mandatory constraint on
implementers but is listed as an informative reference. If the I-D doesn't
actually depend on RFC8283, but the authors are assuming this context, then
the wording could be adjusted to describe the author's assumptions rather
than describe it in
(almost) normative style. Alternatively, it is a normative requirement that
shouldn't exist in the introduction and the reference should move to the
normative list. 
【WAJ】We would like to keep it as an informative reference.  How about the
following description:(change "necessary" to "feasible")
"It is feasible to use the central control mode described in RFC 8283... ...
" or your suggestions?


3) Figure 16 - It isn't clear if TBD1 and TBD2 are gaps in the I-D that the
WG has yet to complete vs. extension / extensibility properties that a
subsequent I-D might define. 
【WAJ】TBD1 and TBD2 will be assigned the values by the IANA later. They are
added after the other early IANA temporary allocations.

4) Section 10 "The communication of PCE and PCC described in this document
should also follow the same procedures, treat the three newly defined
objects (BPI, EPR, PPA) associated with the same symbolic path name as the
attribute of the same path in the LSP-DB (LSP State Database)." The "should"
in this sentence reads as though it is normative.
Consider making it upper case. The sentence is potentially easily misread as
it isn't clear which procedures is meant by "...the same procedures,...".
【WAJ】How about the following descriptions: 
The communication of PCE and PCC described in this document SHOULD follow
the state synchronization procedures described in RFC8232, treat the three
newly defined objects (BPI, EPR, PPA) associated with the same symbolic path
name as the attribute of the same path in the LSP-DB (LSP State Database).


_______________________________________________
Pce mailing list
Pce@ietf.org
https://www.ietf.org/mailman/listinfo/pce