Re: [CCAMP] Comments on draft-ietf-ccamp-wson-signaling-05
"Margaria, Cyril (Coriant - DE/Munich)" <cyril.margaria@coriant.com> Fri, 19 July 2013 08:09 UTC
Return-Path: <cyril.margaria@coriant.com>
X-Original-To: ccamp@ietfa.amsl.com
Delivered-To: ccamp@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id EC6A211E81D5 for <ccamp@ietfa.amsl.com>; Fri, 19 Jul 2013 01:09:05 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.348
X-Spam-Level:
X-Spam-Status: No, score=-4.348 tagged_above=-999 required=5 tests=[AWL=-1.749, BAYES_00=-2.599]
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 Q8RkYDGWsxG0 for <ccamp@ietfa.amsl.com>; Fri, 19 Jul 2013 01:08:59 -0700 (PDT)
Received: from db9outboundpool.messaging.microsoft.com (mail-db9lp0251.outbound.messaging.microsoft.com [213.199.154.251]) by ietfa.amsl.com (Postfix) with ESMTP id 2D63D21F9684 for <ccamp@ietf.org>; Fri, 19 Jul 2013 01:08:59 -0700 (PDT)
Received: from mail156-db9-R.bigfish.com (10.174.16.232) by DB9EHSOBE033.bigfish.com (10.174.14.96) with Microsoft SMTP Server id 14.1.225.22; Fri, 19 Jul 2013 08:08:58 +0000
Received: from mail156-db9 (localhost [127.0.0.1]) by mail156-db9-R.bigfish.com (Postfix) with ESMTP id 1A0BA160223; Fri, 19 Jul 2013 08:08:58 +0000 (UTC)
X-Forefront-Antispam-Report: CIP:157.56.253.53; KIP:(null); UIP:(null); IPV:NLI; H:DB3PRD0411HT002.eurprd04.prod.outlook.com; RD:none; EFVD:NLI
X-SpamScore: -2
X-BigFish: PS-2(zzc89bh1432I31c5Izz1f42h208ch1ee6h1de0h1fdah2073h1202h1e76h1d1ah1d2ah1fc6hzzz2fh2a8h668h839h947hd25hf0ah1288h12a5h12a9h12bdh137ah13b6h1441h1504h1537h153bh15d0h162dh1631h1758h18e1h1946h19b5h19ceh1ad9h1b0ah1d07h1d0ch1d2eh1d3fh1de9h1dfeh1dffh1e1dh1155h)
Received-SPF: pass (mail156-db9: domain of coriant.com designates 157.56.253.53 as permitted sender) client-ip=157.56.253.53; envelope-from=cyril.margaria@coriant.com; helo=DB3PRD0411HT002.eurprd04.prod.outlook.com ; .outlook.com ;
Received: from mail156-db9 (localhost.localdomain [127.0.0.1]) by mail156-db9 (MessageSwitch) id 1374221335899803_23446; Fri, 19 Jul 2013 08:08:55 +0000 (UTC)
Received: from DB9EHSMHS011.bigfish.com (unknown [10.174.16.251]) by mail156-db9.bigfish.com (Postfix) with ESMTP id CC23812008E; Fri, 19 Jul 2013 08:08:55 +0000 (UTC)
Received: from DB3PRD0411HT002.eurprd04.prod.outlook.com (157.56.253.53) by DB9EHSMHS011.bigfish.com (10.174.14.21) with Microsoft SMTP Server (TLS) id 14.16.227.3; Fri, 19 Jul 2013 08:08:55 +0000
Received: from DB3PRD0411MB427.eurprd04.prod.outlook.com ([169.254.6.251]) by DB3PRD0411HT002.eurprd04.prod.outlook.com ([10.255.73.37]) with mapi id 14.16.0329.000; Fri, 19 Jul 2013 08:08:55 +0000
From: "Margaria, Cyril (Coriant - DE/Munich)" <cyril.margaria@coriant.com>
To: ext Leeyoung <leeyoung@huawei.com>, "Giovanni Martinelli (giomarti)" <giomarti@cisco.com>, "Margaria, Cyril (NSN - DE/Munich)" <cyril.margaria@nsn.com>
Thread-Topic: [CCAMP] Comments on draft-ietf-ccamp-wson-signaling-05
Thread-Index: Ac4f9fHWZRxXufGDRkGdO9Y5p2fdaAAeYW8AAAdWyjAY8n5cMA==
Date: Fri, 19 Jul 2013 08:08:53 +0000
Message-ID: <523C37072C291347B9730C9291CCA07D0C658B@DB3PRD0411MB427.eurprd04.prod.outlook.com>
References: <8DC6547C806B644F998A0566E79E15920F7DFFCC@DEMUMBX006.nsn-intra.net> <0D7F95913F470A4B83AB5F5833A4390D293158@xmb-rcd-x14.cisco.com> <7AEB3D6833318045B4AE71C2C87E8E17291183C3@dfweml511-mbs.china.huawei.com>
In-Reply-To: <7AEB3D6833318045B4AE71C2C87E8E17291183C3@dfweml511-mbs.china.huawei.com>
Accept-Language: fr-FR, en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [62.159.77.167]
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: coriant.com
X-FOPE-CONNECTOR: Id%0$Dn%*$RO%0$TLS%0$FQDN%$TlsDn%
Cc: CCAMP <ccamp@ietf.org>, "draft-ietf-ccamp-wson-signaling@tools.ietf.org" <draft-ietf-ccamp-wson-signaling@tools.ietf.org>
Subject: Re: [CCAMP] Comments on draft-ietf-ccamp-wson-signaling-05
X-BeenThere: ccamp@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: Discussion list for the CCAMP working group <ccamp.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ccamp>, <mailto:ccamp-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/ccamp>
List-Post: <mailto:ccamp@ietf.org>
List-Help: <mailto:ccamp-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ccamp>, <mailto:ccamp-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 19 Jul 2013 08:09:06 -0000
Dear Authors, I checked the revision -06 for the updates, I think you forgot one, and I have minors comments > > I have the following comments on draft-ietf-ccamp-wson-signaling-05: > > > > Comment 1: > > The document considers bidirection RB set and limit 2 RBInformation > per hop. However per RFC3473 section 8.1.1 and RFC3471, using > different interface id for the upstream and downstream direction. The > current definition does not support that. > > The following change would allow it. > > Section 4.2 > > > > OLD: > > "If more than two objects are encountered, two > > MUST be processed and the rest SHOULD be ignored." > > > > NEW: > > "If more than two objects per direction are encountered, two > > MUST be processed and the rest SHOULD be ignored." > > > > GM> ok > [[Margaria.C]] This was partially updated it seems: Section 4.3 "If more than two objects are encountered, two MUST be processed and the rest SHOULD be ignored." --> this is in the section of the RB information sub tlv, does this refers to the WSON_signaling object or RBInformation? --> from the text below and to address my comment, I would say --> WSON_signaling, it would be more clear to state it explicitly > > Section 4 > > > > OLD: > > "The usage of <WSON Processing> object for the bidirectional case is > > the same as per unidirectional. When an intermediate node uses > > information from this object to instruct a node about wavelength > > regeneration, the same information applies to both downstream and > > upstream directions." > > > > NEW: > > "The usage of <WSON Processing> object for the bidirectional case is > > the same as per unidirectional. When an intermediate node uses > > information from this object to instruct a node about wavelength > > regeneration, the same regeneration information should applies to > both downstream and > > upstream directions." > > GM> I assume you meant section 5. Ok. Just wondering if worth spending > GM> some words on usage of I|E flags vs directions > > YOUNG>> Simple grammar check with your suggested text: > > OLD: "...should applies to both downstream and upstream directions." > NEW: "...should apply to both downstream and upstream directions." > [[Margaria.C]] OK > > > > Comment-2: Section 4.2 : > > I think the restriction on the RB Set identifier processing is maybe > too strong. > > I would like to suggest the following modification: > > > > remove > > "RB Set Field MAY contain more than one RB Indetifier. Only the first > > one MUST be processed, the others SHOULD be ignored. > > " > > In all case, consider the Typo > > GM> fine with me no problem in relaxing constrain (btw thx for catching > GM> the typo) > [[Margaria.C]] the typo (and restriction) are still present > > Comment-3 : > > The restriction on the OIC and other parameters are also too strong, > only considering one does not allows signaling to pick the one RB among > several. > > Please consider the following change > > Remove > > " The Optical Interface Class List, Input Bit Range List and > > Processing Capability List MAY contain more than one element. Only > > the first MUST be processed, the others SHOULD be ignored. > > " > > > > GM> same as comment above [[Margaria.C]] OK Mit freundlichen Grüßen / Best Regards Cyril Margaria
- [CCAMP] Comments on draft-ietf-ccamp-wson-signali… Margaria, Cyril (NSN - DE/Munich)
- Re: [CCAMP] Comments on draft-ietf-ccamp-wson-sig… Giovanni Martinelli (giomarti)
- Re: [CCAMP] Comments on draft-ietf-ccamp-wson-sig… Leeyoung
- Re: [CCAMP] Comments on draft-ietf-ccamp-wson-sig… Margaria, Cyril (NSN - DE/Munich)
- Re: [CCAMP] Comments on draft-ietf-ccamp-wson-sig… Margaria, Cyril (Coriant - DE/Munich)