Re: [i2rs] Shepherd review on draft-ietf-i2rs-architecture-06

Mach Chen <mach.chen@huawei.com> Fri, 05 December 2014 07:33 UTC

Return-Path: <mach.chen@huawei.com>
X-Original-To: i2rs@ietfa.amsl.com
Delivered-To: i2rs@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 458201ACC72 for <i2rs@ietfa.amsl.com>; Thu, 4 Dec 2014 23:33:38 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.211
X-Spam-Level:
X-Spam-Status: No, score=-4.211 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, SPF_PASS=-0.001, T_RP_MATCHES_RCVD=-0.01] autolearn=ham
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 DRHt4lDXCwSY for <i2rs@ietfa.amsl.com>; Thu, 4 Dec 2014 23:33:34 -0800 (PST)
Received: from lhrrgout.huawei.com (lhrrgout.huawei.com [194.213.3.17]) (using TLSv1 with cipher RC4-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 2F5921AC82C for <i2rs@ietf.org>; Thu, 4 Dec 2014 23:33:33 -0800 (PST)
Received: from 172.18.7.190 (EHLO lhreml401-hub.china.huawei.com) ([172.18.7.190]) by lhrrg02-dlp.huawei.com (MOS 4.3.7-GA FastPath queued) with ESMTP id BMJ95104; Fri, 05 Dec 2014 07:33:30 +0000 (GMT)
Received: from SZXEMA411-HUB.china.huawei.com (10.82.72.70) by lhreml401-hub.china.huawei.com (10.201.5.240) with Microsoft SMTP Server (TLS) id 14.3.158.1; Fri, 5 Dec 2014 07:33:25 +0000
Received: from SZXEMA510-MBX.china.huawei.com ([169.254.3.51]) by szxema411-hub.china.huawei.com ([10.82.72.70]) with mapi id 14.03.0158.001; Fri, 5 Dec 2014 15:33:18 +0800
From: Mach Chen <mach.chen@huawei.com>
To: Joel Halpern <joel.halpern@ericsson.com>, "Joel M. Halpern" <jmh@joelhalpern.com>, "draft-ietf-i2rs-architecture@tools.ietf.org" <draft-ietf-i2rs-architecture@tools.ietf.org>
Thread-Topic: [i2rs] Shepherd review on draft-ietf-i2rs-architecture-06
Thread-Index: AdAPcZAjBj8H2Vo1SnSKKyPqx6sNiwAHt6EAACZIvsD//3u2gP//GC0w
Date: Fri, 05 Dec 2014 07:33:17 +0000
Message-ID: <F73A3CB31E8BE34FA1BBE3C8F0CB2AE28B2E7B4A@SZXEMA510-MBX.china.huawei.com>
References: <F73A3CB31E8BE34FA1BBE3C8F0CB2AE28B2E50A8@SZXEMA510-MBX.china.huawei.com> <548077CC.9090109@joelhalpern.com> <F73A3CB31E8BE34FA1BBE3C8F0CB2AE28B2E7A1C@SZXEMA510-MBX.china.huawei.com> <6BCE198E4EAEFC4CAB45D75826EFB076032BD962@eusaamb101.ericsson.se>
In-Reply-To: <6BCE198E4EAEFC4CAB45D75826EFB076032BD962@eusaamb101.ericsson.se>
Accept-Language: en-US, zh-CN
Content-Language: zh-CN
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.111.97.72]
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-CFilter-Loop: Reflected
Archived-At: http://mailarchive.ietf.org/arch/msg/i2rs/HE_bswODEcEOOF-cAduD8nztooE
Cc: "i2rs@ietf.org" <i2rs@ietf.org>
Subject: Re: [i2rs] Shepherd review on draft-ietf-i2rs-architecture-06
X-BeenThere: i2rs@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: "Interface to The Internet Routing System \(IRS\)" <i2rs.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/i2rs>, <mailto:i2rs-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/i2rs/>
List-Post: <mailto:i2rs@ietf.org>
List-Help: <mailto:i2rs-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/i2rs>, <mailto:i2rs-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 05 Dec 2014 07:33:38 -0000

Hi Joel,

In my understanding, an informational reference can be either WG or now-WG draft. But anyway, I believe that the traceability draft will be a WG draft before the publication of this document. 

BTW, so far, all the references of this document are informational, I do not think draft-traceability is different from other references. So let's make it as an informational reference. 

Best regards,
Mach

> -----Original Message-----
> From: Joel Halpern [mailto:joel.halpern@ericsson.com]
> Sent: Friday, December 05, 2014 9:26 AM
> To: Mach Chen; Joel M. Halpern; draft-ietf-i2rs-architecture@tools.ietf.org
> Cc: i2rs@ietf.org
> Subject: RE: [i2rs] Shepherd review on draft-ietf-i2rs-architecture-06
> 
> I have trouble constructing the sentence such that an informational reference is
> useful, but it does not become normative.
> I would note that even for an informational reference I would want to have a WG
> adopted draft.  I believe that hurdle will be cleared in sufficient time.
> 
> Yours,
> Joel
> 
> > -----Original Message-----
> > From: Mach Chen [mailto:mach.chen@huawei.com]
> > Sent: Thursday, December 04, 2014 8:24 PM
> > To: Joel M. Halpern; draft-ietf-i2rs-architecture@tools.ietf.org
> > Cc: i2rs@ietf.org
> > Subject: RE: [i2rs] Shepherd review on draft-ietf-i2rs-architecture-06
> >
> > Hi Joel,
> >
> > Thanks for your response!
> >
> > In my mind, I thought a informative reference is enough, which could
> > help readers to understand more about traceability but will not block
> > the publication of this document. How do you think?
> >
> > Best regards,
> > Mach
> >
> > > -----Original Message-----
> > > From: Joel M. Halpern [mailto:jmh@joelhalpern.com]
> > > Sent: Thursday, December 04, 2014 11:04 PM
> > > To: Mach Chen; draft-ietf-i2rs-architecture@tools.ietf.org
> > > Cc: i2rs@ietf.org
> > > Subject: Re: [i2rs] Shepherd review on
> > draft-ietf-i2rs-architecture-06
> > >
> > > Thanks for the review.  The editorial items we clealry should apply.
> > > If we put in a normative reference to
> > draft-clarke-i2rs-traceability
> > > (or even to the WG adopted version, which would be the minimum
> > > necessary) we would create a block to publication.  Given
> > that we are
> > > not trying to mandate the details here, I don't think we
> > need a reference.
> > >
> > > Yours,
> > > Joel
> > >
> > > On 12/3/14, 10:22 PM, Mach Chen wrote:
> > > > Hi Authors,
> > > >
> > > > I just finished the shepherd review on
> > > > draft-ietf-i2rs-architecture-06, it's well
> > > written and easy for reading. I have the following comments
> > for this
> > > version, most of them are editorial comments.
> > > >
> > > > 1.
> > > > Some (not well-known) of the acronyms may need to be expanded in
> > > > their first
> > > use. For example, DCCP, etc.
> > > >
> > > > 2.
> > > > The architecture document raises a lot of requirements to I2RS
> > > > protocol,
> > > Information model, Data model. It somehow can be treated as a
> > > requirement document. But I only found that there is only one place
> > > that uses the RFC2119 language and no reference to RFC2119 (idnits
> > > tool also pointed this). Do we need to use the RFC2219 language for
> > > all requirements or just change only one place to
> > > non-RFC2119 usage?
> > > >
> > > > 3.
> > > > Section 1.2 the last third paragraph
> > > >
> > > > "..., these these error cases should be
> > > >     resolved by the network applications and management systems."
> > > >
> > > > There is a redundant "these".
> > > >
> > > > 4.
> > > > Section 6.1
> > > >
> > > > "To facilitate operations, deployment and troubleshooting, it is
> > > >     important that traceability of the I2RS Agent's
> > requests and actions
> > > >     be supported via a common data model."
> > > >
> > > > Seems it's better to make a reference to
> > draft-clarke-i2rs-traceability here.
> > > >
> > > > 5.
> > > > Section 6.2.1
> > > >
> > > > "The I2RS Agent Agent must send a
> > > NOTIFICATION_I2RS_AGENT_TERMINATING to all
> > > >        its cached I2RS Clients."
> > > >
> > > > There is a redundant "Agent".
> > > >
> > > > 6.
> > > > Section 6.2.3
> > > >
> > > > "An I2RS Agent may decide that some state should no
> > longer be applied.
> > > >     An I2RS Client may instruct an Agent to remove state
> > it has applied.
> > > >     In all such cases, the state will revert to what it
> > would have been
> > > >     without the I2RS; that state is generally whatever
> > was specified via
> > > >     the CLI, NETCONF, SNMP, etc."
> > > >
> > > > An I2RS can only withdraws its own states that have been
> > applied to
> > > > the
> > > specific Routing Element, there may be other I2RS clients are in
> > > effect. So the decription "the state will revert to what it
> > would have been without the I2RS"
> > > may not be accuracy. How about changing it as:
> > > > "...the state will revert to what it would have been without the
> > > > I2RS Client; that
> > > state is generally whatever was specified via the CLI, NETCONF, SN,
> > > MP, other I2RS Clients etc."
> > > >
> > > > 7.
> > > > Section 6.4.1
> > > >
> > > > "...per-interface."  This..."
> > > >
> > > > There is a redundant " in between.
> > > >
> > > > s/per-platform-/per-platform
> > > >
> > > > 8.
> > > > Section 6.4.5.4
> > > >
> > > > Should the editors' note be removed before sending to IESG review?
> > > >
> > > >
> > > > 9.
> > > > Section 7.8
> > > >
> > > > s/it be possible/it is possible
> > > >
> > > > Hope this useful!
> > > >
> > > > Best regards,
> > > > Mach
> > > >
> > > > _______________________________________________
> > > > i2rs mailing list
> > > > i2rs@ietf.org
> > > > https://www.ietf.org/mailman/listinfo/i2rs
> > > >
> >