Re: [i2rs] comments on draft-atlas-i2rs-architecture-01

Alia Atlas <akatlas@gmail.com> Tue, 13 August 2013 20:45 UTC

Return-Path: <akatlas@gmail.com>
X-Original-To: i2rs@ietfa.amsl.com
Delivered-To: i2rs@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 3D0D821F9F45 for <i2rs@ietfa.amsl.com>; Tue, 13 Aug 2013 13:45:10 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.549
X-Spam-Level:
X-Spam-Status: No, score=-2.549 tagged_above=-999 required=5 tests=[AWL=0.050, BAYES_00=-2.599, HTML_MESSAGE=0.001, NO_RELAYS=-0.001]
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 F+xUKDOL4Jai for <i2rs@ietfa.amsl.com>; Tue, 13 Aug 2013 13:45:09 -0700 (PDT)
Received: from mail-ob0-x22c.google.com (mail-ob0-x22c.google.com [IPv6:2607:f8b0:4003:c01::22c]) by ietfa.amsl.com (Postfix) with ESMTP id 4663521F9F3A for <i2rs@ietf.org>; Tue, 13 Aug 2013 13:45:09 -0700 (PDT)
Received: by mail-ob0-f172.google.com with SMTP id er7so9959726obc.31 for <i2rs@ietf.org>; Tue, 13 Aug 2013 13:45:08 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type; bh=HsS1ubWmnXLEqBGrx6U11MJAdsah73W3zEefi3nSXyo=; b=LdiFY6zBIK8ccGhv+FiHRA89KaBi+JQBSSyPGNvV0xtW673HQSuZKsZ4j7uPZik9x6 sm5g3q3aVop6scILMEIehBwMgoFvKYiMVJdieahmU7b0so+HNMudyWFC7j++osUVVzQg Hs5joRjzbLOlJEC2/cyUaW+HmlX9iThwE5S4PjVIIRL2tduyA/Dw2fJHDUWT/2NShKOL aHuTIC9aPVFIRBqYBs+HZofWZNzM25V1ZamdSpHcW4GVU9rWt7N6rqOwe25tcXRVRTuk i5m44xxGnMV1uwz57P4ur/BUtBWBUk8tDlyrDkgFG2uhXtNz1B5Xy3kALDmmRoLD4zhR s8PQ==
MIME-Version: 1.0
X-Received: by 10.182.76.38 with SMTP id h6mr2382147obw.74.1376426708818; Tue, 13 Aug 2013 13:45:08 -0700 (PDT)
Received: by 10.182.221.98 with HTTP; Tue, 13 Aug 2013 13:45:08 -0700 (PDT)
In-Reply-To: <20130801120229.GB25597@elstar.local>
References: <20130801120229.GB25597@elstar.local>
Date: Tue, 13 Aug 2013 16:45:08 -0400
Message-ID: <CAG4d1rfPswVXPhpw+VJJCd6+L41joHY-AMLRvuyPc2URPHCJsQ@mail.gmail.com>
From: Alia Atlas <akatlas@gmail.com>
To: Juergen Schoenwaelder <j.schoenwaelder@jacobs-university.de>, "i2rs@ietf.org" <i2rs@ietf.org>
Content-Type: multipart/alternative; boundary="047d7b4503ca09dfdd04e3da5113"
Subject: Re: [i2rs] comments on draft-atlas-i2rs-architecture-01
X-BeenThere: i2rs@ietf.org
X-Mailman-Version: 2.1.12
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: Tue, 13 Aug 2013 20:45:10 -0000

Hi Juergen,

Thanks for the comments and review.  Responses are in-line.

Alia

On Thu, Aug 1, 2013 at 8:02 AM, Juergen Schoenwaelder <
j.schoenwaelder@jacobs-university.de> wrote:

> Hi,
>
> I have a few comments on draft-atlas-i2rs-architecture-01 I like to
> share. Overall, I found the document clearly written and well worked
> out.
>
> a) I suggest to remove the last two paragraphs from section 1.1 (the
>    paragraphs that try to make a statement that SNMP, NETCONF, and
>    ForCES do not work for i2rs).
>
>    Rationale: This is an architectural document and it does not need
>    to explain why certain protocols may not work as i2rs protocols. It
>    is not the job of this document - so simply avoid discussions
>    around this by removing these two paragraphs.
>

[Alia] Sure - I'd tried to trim it down a lot and avoid protocol bashing,
but am happy to remove it.


> b) Is it necessary to describe an example in the middle of the
>    architecture document, given that there is a separate use cases
>    document? My suggestion would be to simply remove section 4.1 (and
>    perhaps consider merging the text left in section 4 into some of
>    the other sections).
>

[Alia] I can certainly understand how that section seems a bit odd.
 However, the topology manager is a key focus point and use-case for I2RS -
and it is important architecturally to clarify the initial scope of I2RS in
the context of that use-case.  Certainly, those are the two reasons that we
put the section in there.  Similarly, we have bits of what might be
info-models to give a sense of what architecturally is in scope.


> c) In section 6.9, I suggest to rename 'Perform all' to 'Perform all
>    ignoring errors' or something like that which better highlights the
>    behaviour.
>
[Alia] Sure - renamed to "Perform all storing errors"


> Editorial: I think the NETCONF community has settled on spelling
>            NETCONF as NETCONF, not as NetConf.


[Alia] Sure - made the change.

Thanks again,
Alia



> /js
>
> --
> Juergen Schoenwaelder           Jacobs University Bremen gGmbH
> Phone: +49 421 200 3587         Campus Ring 1, 28759 Bremen, Germany
> Fax:   +49 421 200 3103         <http://www.jacobs-university.de/>
> _______________________________________________
> i2rs mailing list
> i2rs@ietf.org
> https://www.ietf.org/mailman/listinfo/i2rs
>