Re: [spring] Comments on draft-filsfils-spring-sr-policy-considerations-01

"Ketan Talaulikar (ketant)" <ketant@cisco.com> Wed, 25 July 2018 15:59 UTC

Return-Path: <ketant@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 5B4D1130E78 for <spring@ietfa.amsl.com>; Wed, 25 Jul 2018 08:59:13 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -14.51
X-Spam-Level:
X-Spam-Status: No, score=-14.51 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_HI=-5, SPF_PASS=-0.001, T_DKIMWL_WL_HIGH=-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 Y68lsl9tV-vG for <spring@ietfa.amsl.com>; Wed, 25 Jul 2018 08:59:07 -0700 (PDT)
Received: from alln-iport-8.cisco.com (alln-iport-8.cisco.com [173.37.142.95]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id D1D0E130EA8 for <spring@ietf.org>; Wed, 25 Jul 2018 08:59:06 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=24588; q=dns/txt; s=iport; t=1532534346; x=1533743946; h=from:to:subject:date:message-id:references:in-reply-to: mime-version; bh=P8D3yCOpm9DhLI3Ww+TpY3nlUGHJo8txnrBKCosBREg=; b=jF/T4PjR0tr2QTWoFs5p0/3sJ7lzXLDYsIPDW+8kj97f0GmrQFy8DQ5X OhZIXCijugaBew7ep0vadY2XdULRr5EK2HVd3f59R1rdbtVBoMtjbx3HF 8kMj17t4J+1bkzmWQK/HSG1k/LhhcLul3QzM8s49mCgRhIkSDpJDBtMLB 4=;
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: A0CDAgDznFhb/4UNJK1cGgEBAQEBAgEBAQEIAQEBAYJXdmN/KAqDdGKTX4IMlUmBegsshEACF4JPITYWAQIBAQIBAQJtKIU2AQEBBCMKXAIBCBEEAQErAgICMB0IAgQBEggTgwaBG2SwGoEuilmJAheBQT+BEYMSh36CVQKZdQkCjyuBToQZiCCSBgIRFIEkIwExgVJwFTuCaYIlF44Xb4EWjEABgRoBAQ
X-IronPort-AV: E=Sophos;i="5.51,401,1526342400"; d="scan'208,217";a="147705779"
Received: from alln-core-11.cisco.com ([173.36.13.133]) by alln-iport-8.cisco.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 25 Jul 2018 15:59:05 +0000
Received: from XCH-ALN-008.cisco.com (xch-aln-008.cisco.com [173.36.7.18]) by alln-core-11.cisco.com (8.15.2/8.15.2) with ESMTPS id w6PFx5OT017205 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=FAIL); Wed, 25 Jul 2018 15:59:05 GMT
Received: from xch-aln-008.cisco.com (173.36.7.18) by XCH-ALN-008.cisco.com (173.36.7.18) with Microsoft SMTP Server (TLS) id 15.0.1320.4; Wed, 25 Jul 2018 10:59:05 -0500
Received: from xch-aln-008.cisco.com ([173.36.7.18]) by XCH-ALN-008.cisco.com ([173.36.7.18]) with mapi id 15.00.1320.000; Wed, 25 Jul 2018 10:59:05 -0500
From: "Ketan Talaulikar (ketant)" <ketant@cisco.com>
To: Rob Shakir <robjs=40google.com@dmarc.ietf.org>, SPRING WG List <spring@ietf.org>, "draft-filsfils-spring-sr-policy-considerations@tools.ietf.org" <draft-filsfils-spring-sr-policy-considerations@tools.ietf.org>
Thread-Topic: [spring] Comments on draft-filsfils-spring-sr-policy-considerations-01
Thread-Index: AQHUJC8gIgkUpf/WTkeLttnG75TNTqSgGGvA
Date: Wed, 25 Jul 2018 15:59:04 +0000
Message-ID: <b9932cf792dc49118a24898bd4db27f9@XCH-ALN-008.cisco.com>
References: <CAHd-QWv0E1skiiAV+L7AiDk78qjPH=RZ_vC-me94Cj_yC8eRCQ@mail.gmail.com>
In-Reply-To: <CAHd-QWv0E1skiiAV+L7AiDk78qjPH=RZ_vC-me94Cj_yC8eRCQ@mail.gmail.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: [161.44.213.21]
Content-Type: multipart/alternative; boundary="_000_b9932cf792dc49118a24898bd4db27f9XCHALN008ciscocom_"
MIME-Version: 1.0
X-Outbound-SMTP-Client: 173.36.7.18, xch-aln-008.cisco.com
X-Outbound-Node: alln-core-11.cisco.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/spring/mYcT_v6V-_J-ENb3ZYA5QzwwjP0>
Subject: Re: [spring] Comments on draft-filsfils-spring-sr-policy-considerations-01
X-BeenThere: spring@ietf.org
X-Mailman-Version: 2.1.27
Precedence: list
List-Id: "Source Packet Routing in NetworkinG \(SPRING\)" <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: Wed, 25 Jul 2018 15:59:21 -0000

Hi Rob,

Thanks a lot for your detail review and comments. I will work on them and get back.

Thanks,
Ketan

From: spring <spring-bounces@ietf.org> On Behalf Of Rob Shakir
Sent: 25 July 2018 11:49
To: SPRING WG List <spring@ietf.org>; draft-filsfils-spring-sr-policy-considerations@tools.ietf.org
Subject: [spring] Comments on draft-filsfils-spring-sr-policy-considerations-01

(As an individual contributor)

Hi Authors,

Per my comments in SPRING at IETF 102, I have a number of comments/concerns about the contents of this document. Please find them below. I look forward to discussing them in more detail..

  *   (2) What is the intention of the diagram shown in this section? It seems to be completely an implementation detail that an implementation has the "SRPM" that acts as a central resolution point. For instance, what should a reader learn from the fact that the SRPM is not a standard RIB resolution process? If there are suggestions that one wants this implementation - should there be some discussion of the complexity of this new API between say, the BGP daemon and a general RIB process?
  *   (2) My general feedback on this section is that this is implementation discussion, that does not add to the IETF content that we are publishing within SPRING. Like we have had discussion of use case drafts, I think this is similar but from the implementor side. I'd like to discuss the value that this content has.
  *   (3.1) I think that this section has some useful content, but it's buried by starting out by defining the algorithms.. Why not make this section be focused towards the constraints that must be considered when calculating a SID stack for a particular path. i.e., the key points seem to be:

     *   Use of the IGP metric as the metric for path optimisation is desirable, especially in constrained push or readable depth environments, because it allows the minimum number of deviations from the shortest path and therefore labels.
     *   If a different metric is used, then this implies that every time that metric differs from the IGP metric, then this will result in additional SIDs.

        *   There is no mention of flex-algorithm in this section. It seems relevant given that you can also mitigate the problem that is trying to be solved here by having a set of prefix SIDs per metric.

     *   It may be advantageous to sacrifice optimality of the path calculation solution by relaxing the optimisation constraints.

        *   The draft should talk about the operational considerations here - i.e., it implies that you can actually tolerate the margin in the optimisation objective for the service.
        *   The "just pick the best you can do within N SIDs" is dangerous, since it means that the network delivers a service that *isn't* what the operator asked for - which may result in service degradation (e.g., consider live/live where there is a maximum latency difference that is tolerable between the two feeds).

  *   (3.2) I'm unclear of the value of this text. It seems to me that we're restating some of the optimisation objectives that are known for general TE (and, for example, are described by - say RFC3209). What is it that we're trying to communicate to the reader here -- can it be covered by "existing path calculation considerations, such as resource affinity [rfc3209] can be applied to the path calculation of SR paths"?
  *   (3.4) I'm again going to question the value of this section -- it doesn't seem to give enough detail of the algorithm that you're proposing to be generally useful, and points out it is a node-local behaviour. If there's a desire to get to a common understanding of how to take a path and compress its SID stack, then let's write this out.
  *   (4) See my comments on Section 2 of this document, why is describing how the interaction between different processes within what sounds like a single implementation something that we should publish within the IETF?
  *   (5+5.1+5.2) The core point that seems to be being made here is that - within a single IGP area the head-end has all the visibility it requires; if there are multiple areas, there are ways that a head-end could get access to the areas that it is not part of (e.g., BGP-LS). Is anything more being said here? Do the implementation details that there are BGP-LS RRs actually matter?
  *   (5.3) Similarly to the above, this seems to assume one particular mechanism of building a centralised system, that doesn't need any new extensions in the IETF. Is this something that we need to document?
  *   (6.2) This section seems to imply that there can never be allocations from the SRLB that are not dynamically advertised via some other protocol. Is this really true?
  *   (8) Given that there is a separate draft discussing this -- what is the motivation to have this in this document?
Thanks,
r.