Re: [CDNi] [E] Re: I-D Action: draft-ietf-cdni-request-routing-extensions-07.txt

sanjay.mishra@verizon.com Fri, 11 October 2019 17:45 UTC

Return-Path: <sanjay.mishra@verizon.com>
X-Original-To: cdni@ietfa.amsl.com
Delivered-To: cdni@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 145C3120045; Fri, 11 Oct 2019 10:45:32 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.599
X-Spam-Level:
X-Spam-Status: No, score=-2.599 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, HTTPS_HTTP_MISMATCH=0.1, RCVD_IN_DNSWL_LOW=-0.7, SPF_HELO_NONE=0.001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=verizon.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 1LBofeTyS4u5; Fri, 11 Oct 2019 10:45:27 -0700 (PDT)
Received: from smtpout2-tdc.verizon.com (smtpout2-tdc.verizon.com [137.188.104.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 8E7A212004D; Fri, 11 Oct 2019 10:45:27 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=verizon.com; i=@verizon.com; q=dns/txt; s=corp; t=1570815927; x=1602351927; h=to:cc:subject:date:message-id:references:in-reply-to: mime-version:from; bh=fuZaw8ios+6dhpPKjR4IbebIfFof37exR4/wC0bT5K8=; b=ccAfUCx82sumCdr1vZeth5saf0uSKIA+Ru4sJBwKX4r/CjbXq5HVYezT 0kPgIvJwfRycCstuHL/WvRyXGm3nSVG/LSbQNqZWwW5PRxVevcy2eUluo stj+CG3gLr9rQkhUz+SJxdZrsG/lBgd4ekS1E0EUALVzIZiWNjuCncaRp Q=;
From: sanjay.mishra@verizon.com
Received: from tbwexch04apd.uswin.ad.vzwcorp.com ([153.114.162.28]) by smtpout2-tdc.verizon.com with ESMTP/TLS/ECDHE-RSA-AES256-SHA384; 11 Oct 2019 17:45:24 +0000
Received: from tbwexch13apd.uswin.ad.vzwcorp.com (153.114.162.37) by tbwexch04apd.uswin.ad.vzwcorp.com (153.114.162.28) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Fri, 11 Oct 2019 13:45:23 -0400
Received: from tbwexch02apd.uswin.ad.vzwcorp.com (153.114.162.26) by tbwexch13apd.uswin.ad.vzwcorp.com (153.114.162.37) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Fri, 11 Oct 2019 13:45:23 -0400
Received: from tbwexch02apd.uswin.ad.vzwcorp.com ([153.114.162.26]) by tbwexch02apd.uswin.ad.vzwcorp.com ([153.114.162.26]) with mapi id 15.00.1473.003; Fri, 11 Oct 2019 13:45:23 -0400
To: Kevin Ma <kevin.j.ma.ietf@gmail.com>, "Ori Finkelman (IETF)" <ori.finkelman.ietf@gmail.com>
CC: "<cdni@ietf.org>" <cdni@ietf.org>, "draft-ietf-cdni-request-routing-extensions.all@ietf.org" <draft-ietf-cdni-request-routing-extensions.all@ietf.org>
Thread-Topic: [E] Re: [CDNi] I-D Action: draft-ietf-cdni-request-routing-extensions-07.txt
Thread-Index: AQHVgDyWJkXHu5qDH0mpKk3T9hyh8KdVtozA
Date: Fri, 11 Oct 2019 17:45:23 +0000
Message-ID: <94d8e790a1be49b8856e27c8113e4d2f@tbwexch02apd.uswin.ad.vzwcorp.com>
References: <156933292629.15651.2153181471854597685@ietfa.amsl.com> <CAM8emGX8zbq663EpuQDt_B0Riu3mp60NaUXOPWQemjsmsTXHRA@mail.gmail.com> <CAMrHYE3bFUhsV2RVhJ84hoN0forf8F-AM6T3oMbGT6_g4OiBNQ@mail.gmail.com>
In-Reply-To: <CAMrHYE3bFUhsV2RVhJ84hoN0forf8F-AM6T3oMbGT6_g4OiBNQ@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: [10.144.60.250]
Content-Type: multipart/alternative; boundary="_000_94d8e790a1be49b8856e27c8113e4d2ftbwexch02apduswinadvzwc_"
MIME-Version: 1.0
Archived-At: <https://mailarchive.ietf.org/arch/msg/cdni/M188zn8TTM4t_iqjRV-0czSityg>
Subject: Re: [CDNi] [E] Re: I-D Action: draft-ietf-cdni-request-routing-extensions-07.txt
X-BeenThere: cdni@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: "This list is to discuss issues associated with the Interconnection of Content Delivery Networks \(CDNs\)" <cdni.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/cdni>, <mailto:cdni-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/cdni/>
List-Post: <mailto:cdni@ietf.org>
List-Help: <mailto:cdni-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/cdni>, <mailto:cdni-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 11 Oct 2019 17:45:32 -0000

Hi Kevin – Comments noted. Will make the updates.

Thank you
Sanjay

From: Kevin Ma [mailto:kevin.j.ma.ietf@gmail.com]
Sent: Friday, October 11, 2019 10:03 AM
To: Ori Finkelman (IETF) <ori.finkelman.ietf@gmail.com>
Cc: <cdni@ietf.org> <cdni@ietf.org>; draft-ietf-cdni-request-routing-extensions.all@ietf.org
Subject: [E] Re: [CDNi] I-D Action: draft-ietf-cdni-request-routing-extensions-07.txt

Hi Ori,

  A couple of nits that you may want to cleanup in AUTH48 (final author review):
  - in all places where AAAA was added, add the Oxford comma: "a DNS A record, AAAA record, or CNAME record"
  - section 3.4: missing comma "Using the MI" -> "Using the MI,"

thanx!

--  Kevin J. Ma

On Tue, Sep 24, 2019 at 10:00 AM Ori Finkelman (IETF) <ori.finkelman.ietf@gmail.com<mailto:ori.finkelman.ietf@gmail.com>> wrote:
Hi All,
This submission fixes the comments from all reviewers, I would like to thank all reviewers for taking the time and sharing their comments.
Please note the diff should be against revision 05 rather than 06 as most of the changes were submitted in 06.


Here is the full list of comments that were fixed in this version:
=====================================================================

****************************************************************
Reviewer: Barry Leiba (AD)

1. Please use the new BCP 14 boilerplate and references: see RFC 8174.
>> fixed

2. Abstract vs Introduction:  The sentence about the SVA seems out of
place in the Abstract, and is oddly missing from the Introduction.  I
would add the first two sentences of the Abstract to the Introduction.
Then remove the first sentence from the Abstract and also remove “In
that aspect,” from the second sentence.

>> fixed

3. RFC 6707 defines necessary terminology, so it probably should be
normative.  I will note a downref in the last-call notice in
anticipation of that.
>> fixed


***************************************************
Reviewer: Dan Romascanu (Genart)

1. Several non-obvious acronyms are not expanded: FCI, FQDN
>> fixed and removed the ones not used
2. Section 3 - typo in the first paragraph '...the uCDN MUST be differnet ...'
>> fixed

*************************************************
Reviewer: Zitao Wang (Opsdir)

#1: There are a lot of abbreviations that are not provided with explanations or
citations, such as uCDN, dCDN, CFI, etc.
>> fixed and remove the ones not necessary

#2: The example of of a Redirect Target capability object serialization, is it
encoded as json? Please present its encoding format.
>> fixed

#3: In section 2.1, the "Mandatory-to-Specify" attributes of dns-target and
http-target, it describes that "No, but at least one of dns-target or
http-target MUST be present and non-empty." I wonder whether there should be a
detection mechanism. For example, if the requirements are not met, an error
message will be returned. And if there are existing mechanisms, please briefly
introduce them.

>> That one is a great catch, thanks. I have changed it so it is not an error anymore. Instead we have defined a default behavior for the case
it is not present or empty, see the fixed draft.

******************************************************************
Reviewer: Linda Dunbar (Secdir)

The terminology RR (Request Router) and CP (Content Provider) specified by the
Terminology are not used for the entire document. I assume that RR would be the
one request content, isn't? is RR same as Client?  Is RR part of Downstream CDN
Provider? is the CP same as Downstream CDN provider or Upstream CDN Provider?
>> In the new version RR appears in a few locations. I have added more explanations and references to the relevant CDNI docs where it was defined.

who issued the Redirect Target?

It would be good for the document to clearly specify the relationship of all
the entities, such as who makes request and who respond, and who use the
Redirect Target capability, etc.

>> we have added drawings of sequences that explain it all, I hope it will be clearer now.

*******************************************************************
Reviewer: Michael Tüxen (Tsvart)

To improve readability, you might want to
* resolve acronyms on first occurence like CDN, CDNI,...
>> fixed
* Remove section 1.1, since the introduced abbreviations are not used in the text.
>> fixed
* add a graphical representation of the involved nodes and the messages being exchanged between them
>> Added sequence diagrams sections for both extensions.

Typos:
* Section 3: the uCDN provide a fallback -> the uCDN provides a fallback
* Section 6: Nir B.  Sopher -> Nir B. Sopher (no double space after period)
* Section 6: Kevin J.  Ma -> Kevin J. Ma (no double space after period)
>> fixed

*******************************************************************
Reviewer: Ben Niven-Jenkins (CDNI)

Is there a reason that IPv6 addresses (AAAA records) are excluded from being allowed as DnsTargets, or is this an oversight?

>> fixed. references to ipv6 and AAAA record were added in the relevant places.

*******************************************************************

Thanks,
Ori


---------- Forwarded message ---------
From: <internet-drafts@ietf.org<mailto:internet-drafts@ietf.org>>
Date: Tue, Sep 24, 2019 at 4:49 PM
Subject: [CDNi] I-D Action: draft-ietf-cdni-request-routing-extensions-07.txt
To: <i-d-announce@ietf.org<mailto:i-d-announce@ietf.org>>
Cc: <cdni@ietf.org<mailto:cdni@ietf.org>>



A New Internet-Draft is available from the on-line Internet-Drafts directories.
This draft is a work item of the Content Delivery Networks Interconnection WG of the IETF.

        Title           : CDNI Request Routing Extensions
        Authors         : Ori Finkelman
                          Sanjay Mishra
        Filename        : draft-ietf-cdni-request-routing-extensions-07.txt
        Pages           : 17
        Date            : 2019-09-24

Abstract:
   Open Caching is a use case of Content Delivery Networks
   Interconnetion (CDNI) in which the commercial Content Delivery
   Network (CDN) is the upstream CDN (uCDN) and the ISP caching layer
   serves as the downstream CDN (dCDN).  The extensions specified in
   this document to the CDNI Metadata and FCI interfaces are derived
   from requirements raised by Open Caching but are also applicable to
   CDNI use cases in general.



The IETF datatracker status page for this draft is:
https://datatracker.ietf.org/doc/draft-ietf-cdni-request-routing-extensions/<https://urldefense.proofpoint.com/v2/url?u=https-3A__datatracker.ietf.org_doc_draft-2Dietf-2Dcdni-2Drequest-2Drouting-2Dextensions_&d=DwMFaQ&c=udBTRvFvXC5Dhqg7UHpJlPps3mZ3LRxpb6__0PomBTQ&r=XniVbishGiO2Ao9hKqSc-hTVIWCi3T-x6GdHR4ZTgoM&m=zbEqPHalMUFdduKwQVYCLjgZbpJn87HbiZbVuGfJ89s&s=WSO4j-4uZ9QPEqeq1_A3EoCckH9byhldiYlHGqTFGqU&e=>

There are also htmlized versions available at:
https://tools.ietf.org/html/draft-ietf-cdni-request-routing-extensions-07<https://urldefense.proofpoint.com/v2/url?u=https-3A__tools.ietf.org_html_draft-2Dietf-2Dcdni-2Drequest-2Drouting-2Dextensions-2D07&d=DwMFaQ&c=udBTRvFvXC5Dhqg7UHpJlPps3mZ3LRxpb6__0PomBTQ&r=XniVbishGiO2Ao9hKqSc-hTVIWCi3T-x6GdHR4ZTgoM&m=zbEqPHalMUFdduKwQVYCLjgZbpJn87HbiZbVuGfJ89s&s=n6b7gAqSY0Fk2e8N1uPNNQDizSEfMg8hXDPUO8vI33s&e=>
https://datatracker.ietf.org/doc/html/draft-ietf-cdni-request-routing-extensions-07<https://urldefense.proofpoint.com/v2/url?u=https-3A__datatracker.ietf.org_doc_html_draft-2Dietf-2Dcdni-2Drequest-2Drouting-2Dextensions-2D07&d=DwMFaQ&c=udBTRvFvXC5Dhqg7UHpJlPps3mZ3LRxpb6__0PomBTQ&r=XniVbishGiO2Ao9hKqSc-hTVIWCi3T-x6GdHR4ZTgoM&m=zbEqPHalMUFdduKwQVYCLjgZbpJn87HbiZbVuGfJ89s&s=b3-Bddguz3AaazcPp8Ha61l3xPLj4htWpUriAgqGhjI&e=>

A diff from the previous version is available at:
https://www.ietf.org/rfcdiff?url2=draft-ietf-cdni-request-routing-extensions-07<https://urldefense.proofpoint.com/v2/url?u=https-3A__www.ietf.org_rfcdiff-3Furl2-3Ddraft-2Dietf-2Dcdni-2Drequest-2Drouting-2Dextensions-2D07&d=DwMFaQ&c=udBTRvFvXC5Dhqg7UHpJlPps3mZ3LRxpb6__0PomBTQ&r=XniVbishGiO2Ao9hKqSc-hTVIWCi3T-x6GdHR4ZTgoM&m=zbEqPHalMUFdduKwQVYCLjgZbpJn87HbiZbVuGfJ89s&s=bkC2zKwNCq0rQULOsbwXfpnJOHvrx2KMfmovR3AY9wM&e=>


Please note that it may take a couple of minutes from the time of submission
until the htmlized version and diff are available at tools.ietf.org<https://urldefense.proofpoint.com/v2/url?u=http-3A__tools.ietf.org&d=DwMFaQ&c=udBTRvFvXC5Dhqg7UHpJlPps3mZ3LRxpb6__0PomBTQ&r=XniVbishGiO2Ao9hKqSc-hTVIWCi3T-x6GdHR4ZTgoM&m=zbEqPHalMUFdduKwQVYCLjgZbpJn87HbiZbVuGfJ89s&s=M4_3NmPQqDT9wa35NzA9j8xHI9D05o34YZ1FDrmMdI0&e=>.

Internet-Drafts are also available by anonymous FTP at:
ftp://ftp.ietf.org/internet-drafts/<https://urldefense.proofpoint.com/v2/url?u=ftp-3A__ftp.ietf.org_internet-2Ddrafts_&d=DwMFaQ&c=udBTRvFvXC5Dhqg7UHpJlPps3mZ3LRxpb6__0PomBTQ&r=XniVbishGiO2Ao9hKqSc-hTVIWCi3T-x6GdHR4ZTgoM&m=zbEqPHalMUFdduKwQVYCLjgZbpJn87HbiZbVuGfJ89s&s=r3luW2sc5iaZWKIF7EWZQcxiaca9PFOGKa0VIIXdK_A&e=>

_______________________________________________
CDNi mailing list
CDNi@ietf.org<mailto:CDNi@ietf.org>
https://www.ietf.org/mailman/listinfo/cdni<https://urldefense.proofpoint.com/v2/url?u=https-3A__www.ietf.org_mailman_listinfo_cdni&d=DwMFaQ&c=udBTRvFvXC5Dhqg7UHpJlPps3mZ3LRxpb6__0PomBTQ&r=XniVbishGiO2Ao9hKqSc-hTVIWCi3T-x6GdHR4ZTgoM&m=zbEqPHalMUFdduKwQVYCLjgZbpJn87HbiZbVuGfJ89s&s=YASpC8io0WjhtifyTb2Gn_6JjeKjM264aMYxIwHqhl0&e=>