Re: [Gen-art] [alto] Genart last call review of draft-ietf-alto-cdni-request-routing-alto-16
Russ Housley <housley@vigilsec.com> Tue, 07 September 2021 16:27 UTC
Return-Path: <housley@vigilsec.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 A4DBD3A12AB for <gen-art@ietfa.amsl.com>; Tue, 7 Sep 2021 09:27:47 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.897
X-Spam-Level:
X-Spam-Status: No, score=-1.897 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, SPF_HELO_NONE=0.001, SPF_NONE=0.001] autolearn=unavailable autolearn_force=no
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 be8kvWwadY-k for <gen-art@ietfa.amsl.com>; Tue, 7 Sep 2021 09:27:44 -0700 (PDT)
Received: from mail.smeinc.net (mail.smeinc.net [209.135.209.11]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 945DF3A12A9 for <gen-art@ietf.org>; Tue, 7 Sep 2021 09:27:44 -0700 (PDT)
Received: from localhost (localhost [127.0.0.1]) by mail.smeinc.net (Postfix) with ESMTP id B9EA7300C68 for <gen-art@ietf.org>; Tue, 7 Sep 2021 12:27:44 -0400 (EDT)
X-Virus-Scanned: amavisd-new at mail.smeinc.net
Received: from mail.smeinc.net ([127.0.0.1]) by localhost (mail.smeinc.net [127.0.0.1]) (amavisd-new, port 10026) with ESMTP id 3E7kjnn44Bnp for <gen-art@ietf.org>; Tue, 7 Sep 2021 12:27:40 -0400 (EDT)
Received: from a860b60074bd.fios-router.home (pool-141-156-161-153.washdc.fios.verizon.net [141.156.161.153]) by mail.smeinc.net (Postfix) with ESMTPSA id 20AB5300C57; Tue, 7 Sep 2021 12:27:40 -0400 (EDT)
From: Russ Housley <housley@vigilsec.com>
Message-Id: <98E4C570-9A37-48AD-A3C5-35FACD7B5000@vigilsec.com>
Content-Type: multipart/alternative; boundary="Apple-Mail=_FF2F5653-EB7D-45DA-AE06-C14A52DD5D17"
Mime-Version: 1.0 (Mac OS X Mail 12.4 \(3445.104.21\))
Date: Tue, 07 Sep 2021 12:27:38 -0400
In-Reply-To: <CAAbpuyoLtgjjOjtzKzecXTbmMmzumtabSBCgJh7_FZO=QCd3ww@mail.gmail.com>
Cc: IETF Gen-ART <gen-art@ietf.org>, last-call@ietf.org, IETF ALTO <alto@ietf.org>, draft-ietf-alto-cdni-request-routing-alto.all@ietf.org
To: Jensen Zhang <jingxuan.n.zhang@gmail.com>
References: <162938388879.9100.14963385308969703713@ietfa.amsl.com> <CAAbpuyoLtgjjOjtzKzecXTbmMmzumtabSBCgJh7_FZO=QCd3ww@mail.gmail.com>
X-Mailer: Apple Mail (2.3445.104.21)
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/CiZcbPRpkAayT2h1texJFW8fLWw>
Subject: Re: [Gen-art] [alto] Genart last call review of draft-ietf-alto-cdni-request-routing-alto-16
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.29
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: Tue, 07 Sep 2021 16:27:48 -0000
Hi. The vast bulk of the changes resolve my comments. I'm dropping the resolved comments from my reply. > > Section 5.6, 3rd paragraph after bullets: I do not understand the > second MUST statement in this paragraph. The sentence seems to contain > a mix of defining the superset and a MUST statement. I cannot suggest > a rewording. > > Yes, the original sentence mixed a definition and a MUST statement. To make it easy to read, > we would like to propose the following change to separate the definition and the MUST statement: > > OLD: > > The returned CDNI Advertisement resource MUST contain only > BaseAdvertisementObject objects whose CDNI capability object is the > superset of one of CDNI capability object in "cdni-fci-capabilities". > Specifically, that a CDNI capability object A is the superset of > another CDNI capability object B means that these two CDNI capability > objects have the same capability type and mandatory properties in > capability value of A MUST include mandatory properties in capability > value of B semantically. See Section 5.7.2 for a concrete example. > > NEW: > > The returned filtered CDNI Advertisement resource MUST contain all the > BaseAdvertisementObject objects satisfying the following condition: The > CDNI capability object of each included BaseAdvertisementObject object > MUST follow two constraints: > > o The "cdni-capabilities" field of the input includes a CDNI capability object > X having the same capability type as it. > o All the mandatory properties in its capability value is a superset of > mandatory properties in capability value of X semantically. > > See Section 5.7.2 for a concrete example. The nested colons make this rather awkward. I think it is even less clear than the original. > > > Section 4.2.4 includes: > > data: "/cdni-advertisement/capabilities-with-footprints > /0/footprints/0/footprint-value/-", > data: "value": "germany" > > Since Section 6.1.2.2 says that a countrycode domain is encoded > as an ISO 3166-1 alpha-2 code in lowercase, I was surprised to see > "germany" in this example. > > If you check the example in Sec 4.2.3, you will find "germany" here is not a country code but an ALTO PID name. > If the name is confusing, we can change it to make it more like a PID name. I see. No need for a change, but you might add a note so that others do not make the mistake that I did. Perhaps in the introduction paragraph to Section 4.2.4: ... second one is to remove the "south-france" PID from the footprints ... > > Section 2.2, 1st bullet: please make two bullets, one for > Application Layer-oriented, and another for CDNI. > > This bullet explains that ALTO is can provide application layer-oriented information and therefore is a good match for CDNI. > I am not quite sure what you mean by separating this bullet. Could you explain more? Thanks. I just reread it, and the use of nested colons and the placement of the page break confused me: o Application Layer-oriented: ALTO is a protocol specifically ... <page break> CDNI: a uCDN wants to improve application layer CDN request ... I suggest: s/CDNI: a uCDN wants/CDNI, where a uCDN wants/ Russ
- [Gen-art] Genart last call review of draft-ietf-a… Russ Housley via Datatracker
- Re: [Gen-art] [alto] Genart last call review of d… Jensen Zhang
- Re: [Gen-art] [alto] Genart last call review of d… Russ Housley