Re: [mpls] MPLS-RT review on draft-cdh-mpls-tp-psc-non-revertive-00//RE: MPLS-RT review of mpls psc documents

정태식 <cts@etri.re.kr> Mon, 26 August 2013 02:47 UTC

Return-Path: <cts@etri.re.kr>
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 808EC21E804C for <mpls@ietfa.amsl.com>; Sun, 25 Aug 2013 19:47:06 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -97.346
X-Spam-Level:
X-Spam-Status: No, score=-97.346 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, CHARSET_FARAWAY_HEADER=3.2, MIME_8BIT_HEADER=0.3, MIME_BASE64_TEXT=1.753, USER_IN_WHITELIST=-100]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id LBixYX6i3itT for <mpls@ietfa.amsl.com>; Sun, 25 Aug 2013 19:47:01 -0700 (PDT)
Received: from smtpeg.etri.re.kr (smtpeg1.etri.re.kr [129.254.27.141]) by ietfa.amsl.com (Postfix) with ESMTP id B50ED11E8138 for <mpls@ietf.org>; Sun, 25 Aug 2013 19:46:59 -0700 (PDT)
Received: from SMTP2.etri.info (129.254.28.72) by SMTPEG1.etri.info (129.254.27.141) with Microsoft SMTP Server (TLS) id 14.1.355.2; Mon, 26 Aug 2013 11:46:48 +0900
Received: from SMTP4.etri.info ([169.254.3.68]) by SMTP2.etri.info ([129.254.28.72]) with mapi id 14.01.0355.002; Mon, 26 Aug 2013 11:46:49 +0900
From: 정태식 <cts@etri.re.kr>
To: Mach Chen <mach.chen@huawei.com>, "draft-cdh-mpls-tp-psc-non-revertive@tools.ietf.org" <draft-cdh-mpls-tp-psc-non-revertive@tools.ietf.org>
Thread-Topic: [mpls] MPLS-RT review on draft-cdh-mpls-tp-psc-non-revertive-00//RE: MPLS-RT review of mpls psc documents
Thread-Index: AQHOlPghtHYllm9ZlUWJChJo0dWyb5mgbNEAgAZzA2A=
Date: Mon, 26 Aug 2013 02:46:49 +0000
Message-ID: <AD98114A73E97041A2EDDCC3F3D10B031103AEAB@SMTP4.etri.info>
References: <5204D9DE.6060405@pi.nu> <F73A3CB31E8BE34FA1BBE3C8F0CB2AE255C02212@szxeml558-mbs.china.huawei.com>
In-Reply-To: <F73A3CB31E8BE34FA1BBE3C8F0CB2AE255C02212@szxeml558-mbs.china.huawei.com>
Accept-Language: ko-KR, en-US
Content-Language: ko-KR
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [129.254.73.69]
Content-Type: text/plain; charset="ks_c_5601-1987"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
Cc: "mpls@ietf.org" <mpls@ietf.org>, "mpls-chairs@tools.ietf.org" <mpls-chairs@tools.ietf.org>
Subject: Re: [mpls] MPLS-RT review on draft-cdh-mpls-tp-psc-non-revertive-00//RE: MPLS-RT review of mpls psc documents
X-BeenThere: mpls@ietf.org
X-Mailman-Version: 2.1.12
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: <http://www.ietf.org/mail-archive/web/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: Mon, 26 Aug 2013 02:47:07 -0000

Hi Mach,

Thank you for reviewing draft-cdh-mpls-psc-non-revertive-00.

As one of authors of the draft, I answered your comments.
Please see inline.

Best regards,
Taesik


-----Original Message-----
From: mpls-bounces@ietf.org [mailto:mpls-bounces@ietf.org] On Behalf Of Mach Chen
Sent: Thursday, August 22, 2013 6:06 PM
To: draft-cdh-mpls-tp-psc-non-revertive@tools.ietf.org
Cc: mpls@ietf.org; mpls-chairs@tools.ietf.org
Subject: [mpls] MPLS-RT review on draft-cdh-mpls-tp-psc-non-revertive-00//RE: MPLS-RT review of mpls psc documents


Hi,

I have done my MPLS-RT review on draft-cdh-mpls-tp-psc-non-revertive-00, here are my comments:

I think that modification to Non-revertive mode, adding MS-W and renaming MS to MS-F are valid points. But I not sure whether there is a need to replace " Protecting administrative state" to "Switching administrative state".
[Taesik] I guess "MS-W" as "MS-P" because the draft does not use the term "MS-F". 
[Taesik] According to Sec 3.6 of RFC6378, “Protecting administrative state” means that the operator has issued a command switching the user traffic to the “protection” path. So, the current definition cannot cover the case where the operator has issued a MS-W command requesting to switch the user traffic to the “working” path. By replacing “Protecting administrative state” by “Switching administrative state”, it can cover both cases.

Read through the draft, it gives me the feeling that the draft just lists a set of erratas, I am not sure that this is the right way to progress the draft as it be. If the WG have the consensus on the content of the draft, IMHO, it's better to do a bis to RFC6378.  
[Taesik] I think that this comment is not for the authors of the draft.

Minor comment:
It's better to expand the acronym when first use, for example the MS-F and MS-W, I have to guess the meaning of until I see the Acronyms section. 
[Taesik] MS-W is already expanded in both Abstract and Section 1 of the draft. For MS-P, it can be expanded in both Abstract and Section 1 in the next revision of the draft. But, before doing this, we need to agree on the renaming because there is another review's comment not supporting this renaming.


Best regards,
Mach

> -----Original Message-----
> From: Loa Andersson [mailto:loa@pi.nu]
> Sent: Friday, August 09, 2013 8:01 PM
> To: Eric Osborne (eosborne); Eric Gray; kenji.fujihira.dj@hitachi.com; 
> Yaacov Weingarten; Sam Aldrin; Mach Chen; Kamran Raza (skraza); 
> Henderickx, Wim (Wim); thomas.morin@orange.com; mjork@juniper.net
> Cc: draft-osborne-mpls-psc-updates@tools.ietf.org;
> draft-dj-mpls-tp-exer-psc@tools.ietf.org;
> draft-rhd-mpls-tp-psc-sd@tools.ietf.org;
> draft-cdh-mpls-tp-psc-non-revertive@tools.ietf.org;
> draft-rhd-mpls-tp-psc-priority@tools.ietf.org; 
> mpls-chairs@tools.ietf.org; VIGOUREUX, MARTIN (MARTIN)
> Subject: MPLS-RT review of mpls psc documents
> 
> Eric, Eric, Kenji, Yacoov, Sam, Mach, Kamran, Wim, Thomas and Markus,
> 
> You been selected as MPLS-RT reviewers for a set of psc document that 
> we will start progress through the mpls working group.
> 
> The normal rules and question for an MPLS-RT apply:
> 
> ---------- quote from a standard mail initiating MPLS-RT review 
> -------
> 
> 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).
> 
> 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.
> 
> ---------------------- end quote -------------------------
> 
> The only difference is that we take on more than one document and that 
> there are such inter-dependencies that we want to coordinate how they 
> are progressed through IETF.
> 
> Please respond (at least) to the wg chairs and Martin that you are 
> willing/un-willing to undertake the reviews.
> 
> Since we are starting MPLS-RT reviews of 5 documents, with 4 reviewers 
> for each document and each reviewer have two documents, you'll need to 
> send the review with the draft name in the subject line, i.e. do not 
> respond to this mail with your review comments.
> 
> There is also a "PSC modes" document in the pipe, currently it is our 
> opinion that this document is necessary when we will start the wglc's 
> but is not necessary to make the other drafts wg documents.
> 
> Can you please finish your reviews eob August 23, 2013.
> 
> Here is the list of reviewers per document:
> 
> draft-rhd-mpls-tp-psc-priority
> ------------------------------
> mach chen
> thomas morin
> yacoov weingarten
> Wim Henderickx
> 
> draft-cdh-mpls-tp-psc-non-revertive
> -----------------------------------
> mach chen
> eric osborne
> eric gray
> yacoov weingarten
> 
> draft-rhd-mpls-tp-psc-sd
> ------------------------
> eric osborne
> sam aldrin
> eric gray
> kamran raza
> 
> draft-dj-mpls-tp-exer-psc
> -------------------------
> sam aldrin
> markus jork
> kamran raza
> kenji fuhira
> 
> draft-osborne-mpls-psc-updates
> ------------------------------
> markus jork
> thomas morin
> kenji fuhira
> Wim Henderickx
> 
> 
> Authors,
> 
> Please do not update the documents during the review period, we will 
> tell you when the review period has ended.
> 
> When we close the review period you'll need to address the comments 
> from the reviewers and communicate with them (preferably on the mpls 
> wg mailing list) to make sure that they are comfortable with how the 
> comments has been addressed.
> 
> 
> /Loa
> for the mpls wg chairs
> 
> --
> 
> 
> Loa Andersson                        email: loa@mail01.huawei.com
> Senior MPLS Expert                          loa@pi.nu
> Huawei Technologies (consultant)     phone: +46 739 81 21 64
_______________________________________________
mpls mailing list
mpls@ietf.org
https://www.ietf.org/mailman/listinfo/mpls