Return-Path: X-Original-To: gen-art@ietfa.amsl.com Delivered-To: gen-art@ietfa.amsl.com Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 1CC7C12B032; Thu, 30 Jun 2016 00:56:23 -0700 (PDT) X-Virus-Scanned: amavisd-new at amsl.com X-Spam-Flag: NO X-Spam-Score: -2.699 X-Spam-Level: X-Spam-Status: No, score=-2.699 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_LOW=-0.7, SPF_PASS=-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 YoDNBVTG1X2W; Thu, 30 Jun 2016 00:56:21 -0700 (PDT) Received: from mail-wm0-x233.google.com (mail-wm0-x233.google.com [IPv6:2a00:1450:400c:c09::233]) (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 D3DAE12B009; Thu, 30 Jun 2016 00:56:19 -0700 (PDT) Received: by mail-wm0-x233.google.com with SMTP id v199so209682367wmv.0; Thu, 30 Jun 2016 00:56:19 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=from:to:cc:subject:date:message-id:mime-version:thread-index :content-language; bh=1G0uxvVWymDlp5+s5UYd8EXy778poOgAL0k7/6AnLZ8=; b=iZqdQg0N5rOxZxviG66Iuoe/WTXDKUZVr6TQUY+f5WyqIO79lrDGjoiCdNJ7wcCvfY tKfIw2/0sGBoBpN63N9Rnh2HV5IInxCqauV8mVozvKyK1S9uZXJVd0vM9REjbgtV6Qdb knf6GFsPv7cjroRVG+SLo0a/wM2p3IJRiQii7lgek5m1jKVEfwMH3ROqq0RD3WLbGKIA tBe49+oTa0z+7GL1C86kttWtjlNacHwubMF1FnH3VN1f1ZJSZsrP/5qbSos1AAMBQlot oQC2o6cQdkNxl8vI4byl9Ad/3yszV7wnyGeJepi2ETLd149XY8yHfx1tADmW1F/qVC9i H08w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :thread-index:content-language; bh=1G0uxvVWymDlp5+s5UYd8EXy778poOgAL0k7/6AnLZ8=; b=FOz1kd9a4fT8Q7W0MTJ5a3P9m+YeR9emYK2cgpVdnCwpa7d6QHd12VPJ6yyU2SGzWj p5WXmeqPDSPRAtNR7N5rG4xLYhmf8zY2ehZKno6NcQV3Ree96EUPWqhibrUztTSfpBBI uOWssvjc7up4yoznx5+hBKKKpg1WleuCz6JsTTXLhKSHQcSZOyclhMTDmLzatOc0q0b7 KhhMBvaxxnThSMd4g38q+AyB47lOgZHjSsViqzyV7IeZNEYGEVe+xHvCJxfuO7WhU0P/ sus6sBhUCU8UeWZZdKflcrGm2WVvaKAJzC3OaW5sqxU2Wxp1dAh9sus5tixfximMhM7D zSfw== X-Gm-Message-State: ALyK8tLeHKkv3aen2aLXOhZ6LwZCdjF0Rr1JPsemmN2UDUavG8koRsyxhHTOOriVU1rHkw== X-Received: by 10.28.7.20 with SMTP id 20mr13743460wmh.52.1467273378346; Thu, 30 Jun 2016 00:56:18 -0700 (PDT) Received: from RoniPC (bzq-79-178-255-108.red.bezeqint.net. [79.178.255.108]) by smtp.gmail.com with ESMTPSA id c142sm1159867wme.18.2016.06.30.00.56.16 (version=TLS1 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Thu, 30 Jun 2016 00:56:17 -0700 (PDT) From: "Roni Even" To: , Date: Thu, 30 Jun 2016 10:54:13 +0300 Message-ID: <003b01d1d2a4$988704f0$c9950ed0$@gmail.com> MIME-Version: 1.0 Content-Type: multipart/alternative; boundary="----=_NextPart_000_003C_01D1D2BD.BDD43CF0" X-Mailer: Microsoft Outlook 14.0 Thread-Index: AdHSpHs6/fCHt3+gQPyUb5xSik2nfQ== Content-Language: he Archived-At: Cc: gen-art@ietf.org Subject: [Gen-art] Gen-Art LC review of draft-ietf-pals-mpls-tp-pw-over-bidir-lsp-08 X-BeenThere: gen-art@ietf.org X-Mailman-Version: 2.1.17 Precedence: list List-Id: "GEN-ART: General Area Review Team" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 30 Jun 2016 07:56:23 -0000 This is a multipart message in MIME format. ------=_NextPart_000_003C_01D1D2BD.BDD43CF0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit I am the assigned Gen-ART reviewer for this draft. For background on Gen-ART, please see the FAQ at . Please resolve these comments along with any other Last Call comments you may receive. Document: draft-ietf-pals-mpls-tp-pw-over-bidir-lsp-08 Reviewer: Roni Even Review Date:2016-6-30 IETF LC End Date: 2016-7-4 IESG Telechat date: 2016-7-7 Summary: This draft is ready for publication as standard track RFC. Major issues: Minor issues: 1. In section 2.1 "C (Co-routed path) bit: This informs the remote T-PE/S-PEs about the properties of the underlying LSPs. When set, the remote T-PE/S-PEs need to select co-routed LSP (as the forwarding tunnel) as the reverse PSN tunnel. If there is no such tunnel available, it may trigger the remote T-PE/S-PEs to establish a new LSP." Why are you using non normative language here "need to" instead of MUST or SHOULD while for the S bit normative language is used? Nits/editorial comments: 1. In the abstract "TE" is not expanded, only later in the document ------=_NextPart_000_003C_01D1D2BD.BDD43CF0 Content-Type: text/html; charset="us-ascii" Content-Transfer-Encoding: quoted-printable

I am the assigned Gen-ART = reviewer for this draft. For background on Gen-ART, please see the FAQ = at <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq= >.

Please resolve these = comments along with any other Last Call comments you may = receive.

Document:   = draft-ietf-pals-mpls-tp-pw-over-bidir-lsp-08

 

Reviewer: = Roni Even

Review = Date:2016–6-30

IETF LC End = Date: 2016–7-4

IESG Telechat = date: 2016-7-7

 =

Summary: = This draft is ready for publication as standard track  = RFC.

 =

 =

Major = issues:

 =

Minor = issues:

1.  In section = 2.1 “C (Co-routed path) bit: = This informs the remote T-PE/S-PEs about the properties of the = underlying LSPs.  When set, the remote T-PE/S-PEs need to select = co-routed LSP (as the forwarding tunnel) as the reverse PSN = tunnel.  If there is no such tunnel available, it may trigger the = remote T-PE/S-PEs to establish a new LSP.” Why are you using non = normative language here “need to” instead of MUST or SHOULD = while for the S bit normative language is used?

 =

 

Nits/editorial comments:

  1. In the abstract “TE” is = not expanded, only later in the document

 

------=_NextPart_000_003C_01D1D2BD.BDD43CF0--