Re: [spring] draft-ietf-spring-ipv6-use-cases - shepherd review

"Roberta Maglione (robmgl)" <robmgl@cisco.com> Fri, 23 December 2016 09:37 UTC

Return-Path: <robmgl@cisco.com>
X-Original-To: spring@ietfa.amsl.com
Delivered-To: spring@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 79FDD12961D; Fri, 23 Dec 2016 01:37:06 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -17.622
X-Spam-Level:
X-Spam-Status: No, score=-17.622 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, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, RP_MATCHES_RCVD=-3.1, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001, 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 xLqQRm7rLJ9Z; Fri, 23 Dec 2016 01:37:04 -0800 (PST)
Received: from rcdn-iport-3.cisco.com (rcdn-iport-3.cisco.com [173.37.86.74]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 4A8EE1295F4; Fri, 23 Dec 2016 01:37:04 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=9229; q=dns/txt; s=iport; t=1482485824; x=1483695424; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-transfer-encoding:mime-version; bh=M+XgfiJeDx5v/iPmGxMjch/bqs6ce39tjcWZu9QPCzc=; b=BWpV6V3SNcjzON0HWdp7WhsW8Zucc0hNx8kHRuhvRsF5lgVQV6ukPdYf rJm80t9PmL2uAiUFMcu1g9YbseSWfXBf7pxi5saDPlkLyH8S1PXLgyLft Sb9jbQg6b3sr32YrtkJU3MzR1QzppOdM6/gdKgScgWPszY78ugYYzXGGC A=;
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: A0BhBADe71xY/4YNJK1eGQEBAQEBAQEBAQEBBwEBAQEBgzcBAQEBAR9dTzoHjUyWUZUXggkqhS5KAoFyPxQBAgEBAQEBAQFiKIRoAQEBBG4LDAQCAQgRAQMBASgHMhQDBggBAQQBDQUIE4hUDq4siwABAQEBAQEBAQEBAQEBAQEBAQEBAQEYBYsqhBwQAgFNhS8FiF6CBYQiPoEDijMBiWSHToF+hQmJVod7hi+EDgEfNwFoQhYYg20XgV1yAYd6gQ0BAQE
X-IronPort-AV: E=Sophos;i="5.33,392,1477958400"; d="scan'208";a="188822374"
Received: from alln-core-12.cisco.com ([173.36.13.134]) by rcdn-iport-3.cisco.com with ESMTP/TLS/DHE-RSA-AES256-SHA; 23 Dec 2016 09:37:02 +0000
Received: from XCH-RCD-008.cisco.com (xch-rcd-008.cisco.com [173.37.102.18]) by alln-core-12.cisco.com (8.14.5/8.14.5) with ESMTP id uBN9b2Co020524 (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=FAIL); Fri, 23 Dec 2016 09:37:02 GMT
Received: from xch-rcd-009.cisco.com (173.37.102.19) by XCH-RCD-008.cisco.com (173.37.102.18) with Microsoft SMTP Server (TLS) id 15.0.1210.3; Fri, 23 Dec 2016 03:37:02 -0600
Received: from xch-rcd-009.cisco.com ([173.37.102.19]) by XCH-RCD-009.cisco.com ([173.37.102.19]) with mapi id 15.00.1210.000; Fri, 23 Dec 2016 03:37:02 -0600
From: "Roberta Maglione (robmgl)" <robmgl@cisco.com>
To: "bruno.decraene@orange.com" <bruno.decraene@orange.com>, "draft-ietf-spring-ipv6-use-cases@ietf.org" <draft-ietf-spring-ipv6-use-cases@ietf.org>, "spring@ietf.org" <spring@ietf.org>
Thread-Topic: draft-ietf-spring-ipv6-use-cases - shepherd review
Thread-Index: AdJXqlEYjoPjwSyiQZm/+o0EBVlxZgFUu9GA
Date: Fri, 23 Dec 2016 09:37:01 +0000
Message-ID: <ba6958cb55454eafb52b9712b1e7828a@XCH-RCD-009.cisco.com>
References: <17317_1481899619_5853FE63_17317_744_1_53C29892C857584299CBF5D05346208A1ECC6721@OPEXCLILM21.corporate.adroot.infra.ftgroup>
In-Reply-To: <17317_1481899619_5853FE63_17317_744_1_53C29892C857584299CBF5D05346208A1ECC6721@OPEXCLILM21.corporate.adroot.infra.ftgroup>
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.60.123.211]
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Archived-At: <https://mailarchive.ietf.org/arch/msg/spring/dh84s1qCJosDcMUbZngUIiyIMYw>
Cc: "Christian Martin (martincj)" <martincj@cisco.com>, "spring-chairs@ietf.org" <spring-chairs@ietf.org>, "Stefano Previdi (sprevidi)" <sprevidi@cisco.com>
Subject: Re: [spring] draft-ietf-spring-ipv6-use-cases - shepherd review
X-BeenThere: spring@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: "Stacked Tunnels for Source Routing \(STATUS\)." <spring.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/spring>, <mailto:spring-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/spring/>
List-Post: <mailto:spring@ietf.org>
List-Help: <mailto:spring-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/spring>, <mailto:spring-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 23 Dec 2016 09:37:06 -0000

Hello Bruno,
Thanks for your comments, we are going to address them and post an updated version of the document after the Holidays.
Please see  initial answers inline.
Regards
Roberta

-----Original Message-----
From: bruno.decraene@orange.com [mailto:bruno.decraene@orange.com] 
Sent: Friday, December 16, 2016 3:47 PM
To: draft-ietf-spring-ipv6-use-cases@ietf.org; spring@ietf.org
Cc: spring-chairs@ietf.org
Subject: draft-ietf-spring-ipv6-use-cases - shepherd review

Hi authors,

As the document shepherd, I have reviewed draft-ietf-spring-ipv6-use-cases.
I'm generally fine with the document, but do have some comments. Please find them below.

Best regards,
Bruno

===== Major comments
Section 2.3.0 (intro of 2.3) and 2.3.1 are related to Service Function Chaining and Data Center Network Overlay use cases.
Hence they have an adherence with the SFC and NVO3 WG.
Have those WG been presented this document, kept up to date, and are aligned with the requirements as written?
Besides, those texts are unchanged since early 2014, while the situation may have change since then, especially since both SFC and NVO3 WG are "recent". Especially SFC which has been formed at the same period (23/12/2013). A priori, their work is likely to have progressed since then, which does not seem reflected in this draft.

[RM] we are going the re-work theses sections to update the text in there.

===== Minor comments

§2 IPv6 SPRING use cases
There are a few paragraphs ("In addition....in the above use case.") which describes that the lack of MPLS support for IPv6 only networks is a use case for the IPv6 SR dataplane. However it seems to me that MPLS SR support IPv6 FEC/segment hence would have been a solution for such use case. So it does not seem to me a use case mandating a new dataplane (IPv6 SR).

[RM] Agree with you that  when MPLS is present MPLS SR with IPv6 FEC/segment is a viable solution. However it was pointed out in the mailing list by Wes George (https://www.ietf.org/mail-archive/web/spring/current/msg00832.html ) that MPLS in IPv6 only environment still has some gaps as described in RFC 7439. Specific text was added to address that comment.
We can definitely re-word the text in this section, to clarify the meaning.

On a related point, the term "IPv6-only" does not seem well defined as it could sometime means "non-IPv4" and sometimes "non-MPLS", which is different.
[RM] will clarify this point thanks

---
  "In addition it is worth to note that in today's MPLS dual-stack
   networks IPv4 traffic is labeled while IPv6 traffic is usually
   natively routed, not label-switched.  Therefore in order to be able
   to provide Traffic Engineering "like" capabilities for IPv6 traffic
   additional/alternative encapsulation mechanisms would be required."
   
The first observation does not seem enough to require the new of a new data-plane. As discussed above, this may be caused by a lack of support for IPv6 in existing MPLS signaling protocols. MPLS SR seems to be a valid option. So it does not seem to be a use case mandating a new dataplane (IPv6 SR).


[RM] agree with you, the intention here was to capture current common deployments for dual-stack networks where most of the times IPv4 is labeled switched while IPv6 is natively routed, given the current gaps for MPLS on IPv6 only networks.


---
"   2.  There is a strict lack of an MPLS dataplane"
May be adding "in a portion of the end to end path". (as some may have MPLS in the core, yet not in the access/DC/home network...)

[RM] okay

---
   "This section will describe some scenarios where MPLS may not be
   present and it will highlight how the SPRING architecture could be
   used to address such use cases, particularly, when an MPLS data plane
   is neither present nor desired."

May be rephrasing to avoid saying twice in the same sentence that MPLS is not present. 

[RM] will do

---
"   In any environment with requirements such as those listed above, an
   IPv6 data plane provides a powerful combination of capabilities for a
   network operator to realize benefits in explicit routing, protection
   and restoration, high routing scalability, traffic engineering,
   service chaining, service differentiation and application flexibility
   via programmability."

[...]
   
"   In addition to the use cases described in this document the SPRING
   architecture can be applied to all the use cases described in
   [RFC7855] for the SPRING MPLS data plane, when an IPv6 data plane is
   present.  Here there is a summary of those use cases:

   1.  Traffic Engineering

   2.  Disjoint paths in dual-plane networks

   3.  Fast Reroute: Protecting node and adjacency segments

   4.  OAM/monitoring

   5.  Egress Peering Engineering"


I feel that there is redundancy with those 2 paragraphs. In particular the catalog of usages/benefits is duplicated.

[RM] will simplify the text and remove redundant text

---
§2.2
It's not clear to me what "egress vectoring" is. (May be because this Access Network section seems to refer to a specific access techno (DOCSIS) which I'm not familiar with.) Could you add a reference?  

[RM] will add a reference

---
§2.3

   "A service provider may choose to have these service functions
   performed external to the routing infrastructure, specifically on
   either dedicated physical servers or within VMs running on a
   virtualization platform."
   
It's not clear to me what is meant by "routing infrastructure", especially when opposed to servers/VM. Indeed, routers can now run in servers/VM. So may be rephrasing the whole point would help.

[RM] will re-phrase the sentence

---
Introduction part (2.3.0) is mostly related to service chaining, although this is not reflected by the title. It is referencing two WG documents from the SFC WG. Does this mean that the SFC WG is presenting those requirements in the SPRING WG?  This may require involving the SFC WG in this last call. And some elaboration why the SFC WG solutions are not adequate/enough. (e.g. is this a need to combine both routing instructions and service instruction in order to both choose the route and the sequence of services function? In particular when NHS meta data is not needed?)

This service chaining text seems to originate from draft-martin-spring-segment-routing-ipv6-use-cases-00 i.e .from march 2014. And the text is unchanged since draft-martin-spring-segment-routing-ipv6-use-cases-01 i.e from April 4, 2014.This seems like a long time in this domain: mostly the whole duration of the SFC WG. Is this still aligned with the work that the SFC WG has done in that 2.5 years period?

[RM] will re-work the section and update the text

---
2.3.1 is mostly related to VPN overlay. In the IETF, this seem in scope to the NVO3 WG. Does this mean that the NVO3 WG is presenting those requirements in the SPRING WG? This may require involving the NVO3 WG in this last call. Especially since NVO3 seems to have enough encapsulation techniques to deal with, and it's not clear to me that NVO3 needs one more.

Besides, this section seems to come from 2014 and draft-baker-openstack-ipv6-model which has expired 18 month ago. 2014 may be a long time ago in the DC environment. Is this approach still up to date/relevant?

[RM] will re-work the section and update the text

----
"The 128-bit PE Ingress ID in the Segment Router Header (SRH) policy list"
Name of the field and structure seems to have changed in the 6MAN draft. possibly:
:s/PE Ingress ID/Ingress Node TLV
:s/policy list/Optional TLV objects

[RM] will fix this

===== Nits:
 
Reference:
Some references are outdated (cf idnits), in particular [I-D.previdi-6man-segment-routing-header].

§2.3
"The term "Service Function Chain", as defined in [RFC7498], it is used"
:s/it is/is

[RM] ok will fix both of them

Thanks again for the comments.
Regards
Roberta
_________________________________________________________________________________________________________________________

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.