Re: [mpls] MPLS-RT review of draft-malis-mpls-sfc-encapsulation
"Andrew G. Malis" <agmalis@gmail.com> Thu, 11 October 2018 13:56 UTC
Return-Path: <agmalis@gmail.com>
X-Original-To: mpls@ietfa.amsl.com
Delivered-To: mpls@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 8B0C7130E77; Thu, 11 Oct 2018 06:56:33 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.988
X-Spam-Level:
X-Spam-Status: No, score=-1.988 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_PASS=-0.001, T_KAM_HTML_FONT_INVALID=0.01, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.com
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 Ua77UmQIuXib; Thu, 11 Oct 2018 06:56:31 -0700 (PDT)
Received: from mail-qt1-x829.google.com (mail-qt1-x829.google.com [IPv6:2607:f8b0:4864:20::829]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id E093A130E74; Thu, 11 Oct 2018 06:56:30 -0700 (PDT)
Received: by mail-qt1-x829.google.com with SMTP id j46-v6so9874709qtc.9; Thu, 11 Oct 2018 06:56:30 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=P/ztHdNpaP/Y3OQFqHT/zYE3n0/XTC/ZOTsJ1uXXyM4=; b=VrmofTcrf/+IHoQq/eijd7EuMigjjkHKqLfbRei3MpnvvVcH9f6AE62GvgZlp4E1OE ch1Yu5nyWxN4R22f8KmGKgiMEd0z/5jNWAYdR4QTd3edPRvZzBWcAW4d5m+P1MwGt3RY sJqfdHx9T6ccOo4WSBXaQNEYGisi2m9/zSWSpXgR8msRSvbBclmyLOlib6ITaS8njZYi 3opj26dvKah7bsEDoAMp6fkERlGcYfbxUD3w1Q/Yrcx6JiylUVylFW/X2n4Il3d7ACqQ 1NjE0KDtW5n8rzLtaJ9/ljX6Q37EJ2fzo6V1hZ+o/TBwLdZm3ZmuNynzfYMj+KjgBWRk u23A==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=P/ztHdNpaP/Y3OQFqHT/zYE3n0/XTC/ZOTsJ1uXXyM4=; b=jCTBH4vfPVWJQ9CaZvIBx6/EtcxkzuqaZUOZMk2uzwTiTejnXXexheAeY6Cab04BUC AHTGy1B54CURuMBCuuziPlrSYX1bzo1oIM5c+qh+JR9+Ynrsfrv8k5iCTW6F+H81nbte wUA8povNZtDLB1P95Hs8ztd4z2XNc9zhRZwDdt8LcaGuO5eUrnkqOpfpKfX50YeX28/H yt692uA93qDx1+dAFV1+ZcZLucO51L2BeMop/ReiZ3+FzIaCGKwuxt6rVtCS1uoRFkhU QUnSoB+p+68fL6r8Xx4SSE9zUfrrQux/OaxihASaJLamVOXbmV3IEf6H5/1hkNLgVjj5 tC6w==
X-Gm-Message-State: ABuFfojIoTeGTfwG9L9C241JilbC64ChlgCyswc0SqwJ/yEsWgz4u2jg 9DuLhiScWNctMY/iOcElvcQ7P3gtsYjkUrd2I9o=
X-Google-Smtp-Source: ACcGV635CqZmAZHSMfZxb0UJz6qwF8t8Qns7LfZ75XQrOEkzXxAdh6vLKJaolZ3Hh40sTSs5qR4yhICIYuM2XR9/BGc=
X-Received: by 2002:a05:6214:114e:: with SMTP id b14mr1650617qvt.234.1539266190018; Thu, 11 Oct 2018 06:56:30 -0700 (PDT)
MIME-Version: 1.0
References: <735916399E11684EAF4EB4FB376B719553B6714F@dggema523-mbs.china.huawei.com>
In-Reply-To: <735916399E11684EAF4EB4FB376B719553B6714F@dggema523-mbs.china.huawei.com>
From: "Andrew G. Malis" <agmalis@gmail.com>
Date: Thu, 11 Oct 2018 09:56:19 -0400
Message-ID: <CAA=duU2xH2ke3UyHzwd=r7fgY=Av=ShnSzMk2fkHCce-mDzFdQ@mail.gmail.com>
To: Jia HE <hejia@huawei.com>
Cc: Loa Andersson <loa@pi.nu>, Sam Aldrin <aldrin.ietf@gmail.com>, BOCCI Matthew <matthew.bocci@nokia.com>, Yimin Shen <yshen@juniper.net>, draft-malis-mpls-sfc-encapsulation@ietf.org, mpls-chairs <mpls-chairs@ietf.org>, mpls@ietf.org
Content-Type: multipart/alternative; boundary="000000000000788efb0577f456cb"
Archived-At: <https://mailarchive.ietf.org/arch/msg/mpls/Y_wSKc3w6gEZv9thlK0ecnlytIA>
Subject: Re: [mpls] MPLS-RT review of draft-malis-mpls-sfc-encapsulation
X-BeenThere: mpls@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Multi-Protocol Label Switching WG <mpls.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/mpls>, <mailto:mpls-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/mpls/>
List-Post: <mailto:mpls@ietf.org>
List-Help: <mailto:mpls-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/mpls>, <mailto:mpls-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 11 Oct 2018 13:56:34 -0000
Jia, Thanks for your review. I personally prefer keeping section 2 as it is, so that a reader doesn't have to looking through the document for all of the cases, as someone not particularly interested (for example) in OAM may not read the section on OAM considerations. Of course, once this is a WG document, if the WG prefers a restructuring, we can do that. Your suggestion to illustrate the label stack is a good one, and something we can pursue once it's a WG draft. Thanks again, Andy On Thu, Oct 11, 2018 at 5:31 AM Hejia (Jia) <hejia@huawei.com> wrote: > Hi, > > > > I have reviewed draft-malis-mpls-sfc-encapsulation-02. The draft > introduces a SFF label in the MPLS label stack, which is used to transport > SFC packets with the NSH over an MPLS infrastructure. In summary, I think > the draft is useful and ready to be considered for WG adoption. > > I have found one comment and some nits, which can be resolved either > before or after WG adoption. > > > > Comments: > > As indicated by the first sentence of Section 2.1, it is to describe the > steps of constructing label stack to "send an SFC packet with the NSF > (typo? NSH?)". However, the first step that follows describes the special > cases and uses OAM as an example using GAL/G-ACh where no NSH is applied if > I understand correctly. It looks inconsistent somehow. Since there are > already separate sections for OAM and ECMP considerations. Can I suggest > leaving Section 2 to describe only the general SFF label processing and > leave the special cases to OAM and ECMP sections respectively? And if > possible, a figure to show the structure of label stack (e.g. the general > case, the OAM case, the ECMP case) will be more direct and absolutely clear > :-) > > > > Nits: > > Section 2, s/by having the receiving advertise .../by having the receiving > node advertise ... > > Section 2.1, s/When one SFF wishes to send an SFC packet with the NSF to > another SFF.../When one SFF wishes to send an SFC packet with the NSH to > another SFF... > > > > > > B.R. > > Jia > > > > -----邮件原件----- > 发件人: Loa Andersson [mailto:loa@pi.nu] > 发送时间: 2018年9月26日 20:34 > 收件人: Sam Aldrin <aldrin.ietf@gmail.com>; Hejia (Jia) <hejia@huawei.com>; > Bocci, Matthew (Nokia - GB) <matthew.bocci@nokia.com>; Yimin Shen < > yshen@juniper.net> > 抄送: draft-malis-mpls-sfc-encapsulation@ietf.org; mpls-chairs@ietf.org > 主题: MPLS-RT review of draft-malis-mpls-sfc-encapsulation > > > > Sam, Jia, Matthew and Yimin, > > > > > > You have been selected as MPLS-RT reviewers for draft-malis-mpls-sfc- > encapsulation. > > > > Note to authors: You have been CC'd on this email so that you can know > that this review is going on. However, please do not review your own > document. > > > > Reviews should comment on whether the document is coherent, is it useful > (ie, is it likely to be actually useful in operational networks), and is > the document technically sound? > > > > We are interested in knowing whether the document is ready to be > considered for WG adoption (ie, it doesn't have to be perfect at this > point, but should be a good start). Please remember that it often is easier > to progress the document when it has become a working group document. All > comments in the MPLS-RT review needs to be addressed, but please think > carefully about whether a comment is gating the adoption or could just as > easily be addressed after the adoption. > > > > Reviews should be sent to the document authors, WG co-chairs and WG > secretary, and CC'd to the MPLS WG email list. If necessary, comments may > be sent privately to only the WG chairs. > > > > If you have technical comments you should try to be explicit about what > needs to be resolved before adopting it as a working group document, and > what can wait until the document is a working group document and the > working group has the revision control. > > > > Are you able to review this draft by October 11, 2018? Please respond > whether you are available to do the review in a timely fashion. > > > > > > Thanks, Loa > > (as MPLS WG co-chair) > > -- > > > > > > Loa Andersson email: loa@pi.nu > > Senior MPLS Expert > > Bronze Dragon Consulting phone: +46 739 81 21 64 >
- Re: [mpls] MPLS-RT review of draft-malis-mpls-sfc… Hejia (Jia)
- Re: [mpls] MPLS-RT review of draft-malis-mpls-sfc… Andrew G. Malis