Re: [Gen-art] [spring] Genart last call review of draft-ietf-spring-segment-routing-ldp-interop-11

"Les Ginsberg (ginsberg)" <ginsberg@cisco.com> Mon, 14 May 2018 20:32 UTC

Return-Path: <ginsberg@cisco.com>
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 6A0AF124319; Mon, 14 May 2018 13:32:08 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -14.511
X-Spam-Level:
X-Spam-Status: No, score=-14.511 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, RCVD_IN_DNSWL_HI=-5, SPF_PASS=-0.001, T_DKIMWL_WL_MED=-0.01, USER_IN_DEF_DKIM_WL=-7.5] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=cisco.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 YN5DKNSis59T; Mon, 14 May 2018 13:32:06 -0700 (PDT)
Received: from rcdn-iport-2.cisco.com (rcdn-iport-2.cisco.com [173.37.86.73]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id D2ED2126D05; Mon, 14 May 2018 13:32:05 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=6332; q=dns/txt; s=iport; t=1526329925; x=1527539525; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-transfer-encoding:mime-version; bh=Ng6nkVbI+DPSKGeOf0z0id/HvUC4yGR9d7nsbItiYgQ=; b=Q10L+yQmSwd8OJSCVgr2j5grgF96BOM0NHZ6Zcts5sqNEXJwS418JOBl JwgiHxbXCXycoixQjZJv9eH+IriR6SW/u6ri4rIujl2GJCtQrTf/LMVDm QUpbpaI0jyO+aA0fcIjfXzi28GMKiJX0rNQ2ThB1nDEHhehbfPZVXZywX 4=;
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: A0CTAQBM8fla/4YNJK1cGQEBAQEBAQEBAQEBAQcBAQEBAYNDYXsoCoNoiASMcoF5gQ+TMoF4CxgLhANGAhqCdyE0GAECAQEBAQEBAmwcDIUoAQEBBAEBIRE6CwwEAgEIEQMBAQEBAgImAgICJQsVCAgCBAENBQiDHYF/D6tVghyIQoIiBYEJhxyBVD+BDgGDC4MRAQEDgUc8glqCVAKHXJBaCQKFZYhigT6DZYdUiVWGZwIREwGBJAEcOIFScBU7gkOCIBcRg3qEToU+b49RgRgBAQ
X-IronPort-AV: E=Sophos;i="5.49,401,1520899200"; d="scan'208";a="398322814"
Received: from alln-core-12.cisco.com ([173.36.13.134]) by rcdn-iport-2.cisco.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 14 May 2018 20:32:04 +0000
Received: from XCH-ALN-005.cisco.com (xch-aln-005.cisco.com [173.36.7.15]) by alln-core-12.cisco.com (8.14.5/8.14.5) with ESMTP id w4EKW3P9029079 (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=FAIL); Mon, 14 May 2018 20:32:03 GMT
Received: from xch-aln-001.cisco.com (173.36.7.11) by XCH-ALN-005.cisco.com (173.36.7.15) with Microsoft SMTP Server (TLS) id 15.0.1320.4; Mon, 14 May 2018 15:32:03 -0500
Received: from xch-aln-001.cisco.com ([173.36.7.11]) by XCH-ALN-001.cisco.com ([173.36.7.11]) with mapi id 15.00.1320.000; Mon, 14 May 2018 15:32:03 -0500
From: "Les Ginsberg (ginsberg)" <ginsberg@cisco.com>
To: Joel Halpern Direct <jmh.direct@joelhalpern.com>, Joel Halpern <jmh@joelhalpern.com>, "gen-art@ietf.org" <gen-art@ietf.org>
CC: "draft-ietf-spring-segment-routing-ldp-interop.all@ietf.org" <draft-ietf-spring-segment-routing-ldp-interop.all@ietf.org>, "spring@ietf.org" <spring@ietf.org>, "ietf@ietf.org" <ietf@ietf.org>
Thread-Topic: [spring] Genart last call review of draft-ietf-spring-segment-routing-ldp-interop-11
Thread-Index: AQHT675r7HqBmiVlEkS/VLoESDZWzKQvpz+wgABV2gD//693EA==
Date: Mon, 14 May 2018 20:32:03 +0000
Message-ID: <749d7ac055aa40c0a1e149b755f16c01@XCH-ALN-001.cisco.com>
References: <152632807068.10078.4478550408904407310@ietfa.amsl.com> <e53fa35538f042d98bde5e4e82f621d6@XCH-ALN-001.cisco.com> <8050bc23-d648-4ac8-ff4c-1978d6ecc9d9@joelhalpern.com>
In-Reply-To: <8050bc23-d648-4ac8-ff4c-1978d6ecc9d9@joelhalpern.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-ms-exchange-transport-fromentityheader: Hosted
x-originating-ip: [10.24.122.133]
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/SyYteITSe1EpW9mAojNLSF2n8dA>
Subject: Re: [Gen-art] [spring] Genart last call review of draft-ietf-spring-segment-routing-ldp-interop-11
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 14 May 2018 20:32:08 -0000

Joel -

I don’t fully understand the rest of your comment then. You said:

" And that document does appear to define the  SRMS."

(where "that document" refers to draft-ietf-spring-conflict-resolution).

But the conflict resolution document never defined an SRMS - it merely described how SRMS advertisements were used in the context of conflict resolution.
So if you are unsatisfied with the "SRMS definition" in ldp-interop draft I think you need to be more clear as to what you think is lacking.

I leave it to the draft authors to resolve this issue with you.

    Les


> -----Original Message-----
> From: Joel Halpern Direct <jmh.direct@joelhalpern.com>
> Sent: Monday, May 14, 2018 1:16 PM
> To: Les Ginsberg (ginsberg) <ginsberg@cisco.com>; Joel Halpern
> <jmh@joelhalpern.com>; gen-art@ietf.org
> Cc: draft-ietf-spring-segment-routing-ldp-interop.all@ietf.org;
> spring@ietf.org; ietf@ietf.org
> Subject: Re: [spring] Genart last call review of draft-ietf-spring-segment-
> routing-ldp-interop-11
> 
> Thanks Les.  I wondered if that were the case.
> 
> Looking again at the draft, the problem then is that section 4.2 of the subject
> draft is not a normative definition of an SRMS.  It states the general
> functionality, and then provides an example of how it would work in the
> given scenario.
> 
> If the text were enhanced to be an effective normative definition of an
> SRMS, then that would also resolve the quesiton of the intended status of
> the draft.
> 
> Yours,
> Joel
> 
> On 5/14/18 4:12 PM, Les Ginsberg (ginsberg) wrote:
> > Joel -
> >
> > I am not an author of this draft - but I am an author on the referenced IS-IS
> draft - which I assume is one of the drafts mentioned in  your comment:
> >
> >>      Server).  Looking at the relevant routing protocol document, they point
> to
> >>      https://tools.ietf.org/html/draft-ietf-spring-conflict-resolution-05 as
> the
> >>      defining source for the SRMS.
> >
> > The IGP document references in the ldp-interop draft are stale. Newer
> versions of the IGP drafts have been published and they no longer reference
> draft-ietf-spring-conflict-resolution - a draft which is no longer active.
> >
> > HTH
> >
> >      Les
> >
> >
> >> -----Original Message-----
> >> From: spring <spring-bounces@ietf.org> On Behalf Of Joel Halpern
> >> Sent: Monday, May 14, 2018 1:01 PM
> >> To: gen-art@ietf.org
> >> Cc: draft-ietf-spring-segment-routing-ldp-interop.all@ietf.org;
> >> spring@ietf.org; ietf@ietf.org
> >> Subject: [spring] Genart last call review of
> >> draft-ietf-spring-segment-routing-
> >> ldp-interop-11
> >>
> >> Reviewer: Joel Halpern
> >> Review result: Ready with Issues
> >>
> >> I am the assigned Gen-ART reviewer for this draft. The General Area
> >> Review Team (Gen-ART) reviews all IETF documents being processed by
> >> the IESG for the IETF Chair.  Please treat these comments just like
> >> any other last call comments.
> >>
> >> For more information, please see the FAQ at
> >>
> >> <https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.
> >>
> >> Document: draft-ietf-spring-segment-routing-ldp-interop-11
> >> Reviewer: Joel Halpern
> >> Review Date: 2018-05-14
> >> IETF LC End Date: 2018-05-24
> >> IESG Telechat date: Not scheduled for a telechat
> >>
> >> Summary: This document appears to be ready for publication as an RFC.
> >> The question of whether it is an Informational RFC or a Proposed
> >> Standards track RFC is one that the ADs should examine.
> >>
> >> Major issues:
> >>      This document is quite readable, and quite useful.  If my reading below
> >>      (minor comment about section 4.2) is wrong, then everything is fine.
> >>      However, reading the text, it does not appear to define SRMS.  Rather,
> it
> >>      describes a good way to use SRMS to achive smooth SR - LDP
> >> integration and
> >>      migration.  As such, this seems to me to be a really good Informational
> >>      Document.
> >>
> >> Minor issues:
> >>      Section 4.2 states that it defines the SRMS (Segment Routing Mapping
> >>      Server).  Looking at the relevant routing protocol document, they point
> to
> >>      https://tools.ietf.org/html/draft-ietf-spring-conflict-resolution-05 as
> the
> >>      defining source for the SRMS.  And that document does appear to
> >> define the
> >>      SRMS.
> >>
> >> Nits/editorial comments:
> >>
> >>
> >> _______________________________________________
> >> spring mailing list
> >> spring@ietf.org
> >> https://www.ietf.org/mailman/listinfo/spring