[RTG-DIR] RtgDir review: draft-ietf-idr-as-migration-02

<thomas.morin@orange.com> Wed, 03 June 2015 09:21 UTC

Message-ID: <13866_1433323275_556EC70B_13866_4773_1_556EC70B.6040400@orange.com>
Date: Wed, 03 Jun 2015 11:21:15 +0200
From: thomas.morin@orange.com
Organization: Orange
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0
MIME-Version: 1.0
To: idr@ietf.org, "rtg-ads@tools.ietf.org" <rtg-ads@tools.ietf.org>
Content-Type: text/plain; charset="utf-8"; format="flowed"
Content-Transfer-Encoding: 8bit
Archived-At: <http://mailarchive.ietf.org/arch/msg/rtg-dir/3eo3554_xJ3hFWWzARltpgiZQ8U>
Cc: "rtg-dir@ietf.org" <rtg-dir@ietf.org>, draft-ietf-idr-as-migration.all@tools.ietf.org
Subject: [RTG-DIR] RtgDir review: draft-ietf-idr-as-migration-02
Precedence: list

Hello,

In the context of Routing Area QA reviews (see
https://trac.tools.ietf.org/area/rtg/trac/wiki/RtgDirDocQa ), I have
been selected as the Routing Directorate reviewer for this draft. The
Routing Directorate seeks to review all routing or routing-related
drafts as they pass through IETF last call and IESG review, and
sometimes on special request. The purpose of the review is to provide
assistance to the Routing ADs. For more information about the Routing
Directorate, please see
http://trac.tools.ietf.org/area/rtg/trac/wiki/RtgDir

Although these comments are primarily for the use of the Routing ADs, it
would be helpful if you could consider them along with any other IETF
Last Call comments that you receive, and strive to resolve them through
discussion or by updating the draft.

//// Note well: this review comes as a complement to a first review I
did last September. ////

Document: http://tools.ietf.org/html/draft-ietf-idr-as-migration-05
Reviewer: Thomas Morin
Review Date: 2015-06-02
Intended Status: standards track

*Summary:* This document is I think ready for publication.

*Comments:*

This is overall a very well written document, with a clear goal and
addressing its target goal.

Although it is all about local behavior, it makes sense to make a
standard track document that (a) can be used as a reference for
implementors and deployers to implement this properly, and (b) keep
track of these features in future evolutions of the protocol (as
motivated in the Conclusion section of the document).

The document has taken into account my comments made on -02, and has
gone to additional changes that bring significant improvements to the
document.

*Major Issues:* No major issues found

*Minor Issues:*

- it sounds weird to name "customer C" the customer attached to ISP B,
not naming it, or naming it CE-B would better match figures 4 and 5
- figures 1 and 2 could be left-right reversed to match fig 4 and 5
- "are service interrupting to the eBGP sessions of the PE", maybe could
be worded as "result in a service interruption of eBGP sessions of the
PE" ? (or, at least, "service-interrupting", with a dash)

_________________________________________________________________________________________________________________________

Ce message et ses pieces jointes peuvent contenir des informations confidentielles ou privilegiees et ne doivent donc
pas etre diffuses, exploites ou copies sans autorisation. Si vous avez recu ce message par erreur, veuillez le signaler
a l'expediteur et le detruire ainsi que les pieces jointes. Les messages electroniques etant susceptibles d'alteration,
Orange decline toute responsabilite si ce message a ete altere, deforme ou falsifie. Merci.

This message and its attachments may contain confidential or privileged information that may be protected by law;
they should not be distributed, used or copied without authorisation.
If you have received this email in error, please notify the sender and delete this message and its attachments.
As emails may be altered, Orange is not liable for messages that have been modified, changed or falsified.
Thank you.

[RTG-DIR] RtgDir review: draft-ietf-idr-as-migrat… Thomas Morin
Re: [RTG-DIR] [Idr] RtgDir review: draft-ietf-idr… Jeffrey Haas
Re: [RTG-DIR] [Idr] RtgDir review: draft-ietf-idr… George, Wes
Re: [RTG-DIR] [Idr] RtgDir review: draft-ietf-idr… George, Wes
Re: [RTG-DIR] [Idr] RtgDir review: draft-ietf-idr… Thomas Morin
[RTG-DIR] RtgDir review: draft-ietf-idr-as-migrat… thomas.morin
Re: [RTG-DIR] RtgDir review: draft-ietf-idr-as-mi… Susan Hares