Re: [AVTCORE] AD Evaluation of draft-ietf-avtcore-rfc5764-mux-fixes-10

"Gonzalo Salgueiro (gsalguei)" <gsalguei@cisco.com> Tue, 19 July 2016 13:50 UTC

Return-Path: <gsalguei@cisco.com>
X-Original-To: avt@ietfa.amsl.com
Delivered-To: avt@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 9A19512E17D; Tue, 19 Jul 2016 06:50:07 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -15.808
X-Spam-Level:
X-Spam-Status: No, score=-15.808 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=-1.287, 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 NSPF6CleFmEo; Tue, 19 Jul 2016 06:50:03 -0700 (PDT)
Received: from rcdn-iport-4.cisco.com (rcdn-iport-4.cisco.com [173.37.86.75]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 3BE1312E139; Tue, 19 Jul 2016 06:24:41 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=5448; q=dns/txt; s=iport; t=1468934681; x=1470144281; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-id:content-transfer-encoding: mime-version; bh=YSvQwWUztBe0CjW9Gc68yZbON+Oqxko0UEaysVeKaFc=; b=BK3mMbf2/gs5Me3nxhJJ995PWZFZKTtyFdVwvQQW4u8m8qc97HUnDy2n 60el4AfmDElvV5kVHm7USUZVRZQEfv2/uul82bsoNTjQ40xnR3PmVaLQH 6ZgIhIpmN8fVQO+1H7oIC+9GKIJLnkwQDojZwPgd67aeoCFHfbQrRatZp 8=;
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: A0BPAgDYKI5X/4cNJK1cgz+BUga4YYF6hhoCHIETOBQBAQEBAQEBZSeEXAEBBAEjEUUFCwIBCBgCAiYCAgIwFRACBA4FiCgIrkKOCgEBAQEBAQEBAQEBAQEBAQEBAQEegQGFKYF4CIFKgQOEEhEBHIMBK4IvBZNjhUEBjmGBa41Mhl+JPgEeNoNzboZwNn8BAQE
X-IronPort-AV: E=Sophos;i="5.28,389,1464652800"; d="scan'208";a="127502958"
Received: from alln-core-2.cisco.com ([173.36.13.135]) by rcdn-iport-4.cisco.com with ESMTP/TLS/DHE-RSA-AES256-SHA; 19 Jul 2016 13:24:40 +0000
Received: from XCH-ALN-010.cisco.com (xch-aln-010.cisco.com [173.36.7.20]) by alln-core-2.cisco.com (8.14.5/8.14.5) with ESMTP id u6JDOe2h014515 (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=FAIL); Tue, 19 Jul 2016 13:24:40 GMT
Received: from xch-aln-009.cisco.com (173.36.7.19) by XCH-ALN-010.cisco.com (173.36.7.20) with Microsoft SMTP Server (TLS) id 15.0.1210.3; Tue, 19 Jul 2016 08:24:39 -0500
Received: from xch-aln-009.cisco.com ([173.36.7.19]) by XCH-ALN-009.cisco.com ([173.36.7.19]) with mapi id 15.00.1210.000; Tue, 19 Jul 2016 08:24:39 -0500
From: "Gonzalo Salgueiro (gsalguei)" <gsalguei@cisco.com>
To: Ben Campbell <ben@nostrum.com>
Thread-Topic: AD Evaluation of draft-ietf-avtcore-rfc5764-mux-fixes-10
Thread-Index: AQHR36zkTe5Ongs7J0C60x5OulaS0KAgDZeAgAADvICAAATygA==
Date: Tue, 19 Jul 2016 13:24:39 +0000
Message-ID: <F7CB7135-A7DD-4058-B400-F7CB8BB9FD2F@cisco.com>
References: <5F4C7382-41E2-4791-BEC6-51481BE1D917@nostrum.com> <A042ADC3-2400-4412-9278-E5333A1E56B3@cisco.com> <872DF567-F773-400D-BED1-669A6A05FD1F@nostrum.com>
In-Reply-To: <872DF567-F773-400D-BED1-669A6A05FD1F@nostrum.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-ms-exchange-messagesentrepresentingtype: 1
x-ms-exchange-transport-fromentityheader: Hosted
x-originating-ip: [10.82.177.69]
Content-Type: text/plain; charset="utf-8"
Content-ID: <B8512D8686330745B7036728C3162E1F@emea.cisco.com>
Content-Transfer-Encoding: base64
MIME-Version: 1.0
Archived-At: <https://mailarchive.ietf.org/arch/msg/avt/XXg6PG_VOms2pYC-TIPXm9MJ8SY>
Cc: IETF AVTCore WG <avt@ietf.org>, "draft-ietf-avtcore-rfc5764-mux-fixes.all@ietf.org" <draft-ietf-avtcore-rfc5764-mux-fixes.all@ietf.org>
Subject: Re: [AVTCORE] AD Evaluation of draft-ietf-avtcore-rfc5764-mux-fixes-10
X-BeenThere: avt@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: Audio/Video Transport Core Maintenance <avt.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/avt>, <mailto:avt-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/avt/>
List-Post: <mailto:avt@ietf.org>
List-Help: <mailto:avt-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/avt>, <mailto:avt-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 19 Jul 2016 13:50:07 -0000

> On Jul 19, 2016, at 3:06 PM, Ben Campbell <ben@nostrum.com> wrote:
> 
> Thanks, comments also inline:
> 
> Ben.
> 
> 
> On 19 Jul 2016, at 14:53, Gonzalo Salgueiro (gsalguei) wrote:
> 
>> Hi Ben -
>> 
>> Comments inline…
>> 
>>> On Jul 16, 2016, at 11:56 PM, Ben Campbell <ben@nostrum.com> wrote:
>>> 
>>> Hi,
>>> 
>>> This is my AD Evaluation of draft-ietf-avtcore-rfc5764-mux-fixes-10. I think these comments can be addressed in parallel to the IETF last call, so I'm going to request that shortly.
>>> 
>>> Thanks!
>>> 
>>> Ben.
>>> 
>>> Substantive Comments:
>>> 
>>> 
>>> - This draft has a pre-RFC5378 disclaimer. Is that really needed? Am I correct it is due to the excerpted "OLD" text from RFC5764? If so, has anyone checked with the authors of that (Ekr and David McGrew) about whether they would agree to let this draft progress without the disclaimer?
>> 
>> We sent an email to David McGrew and EKR asking to publish without the pre-RFC5378 disclaimer. McGrew was in favor but EKR stated "I am not sure that all the text in 5764 is pre-5378, so probably better to be safe.”  This is the reason we have proceeded with the disclaimer.
>> 
> 
> I'm okay leaving the disclaimer if there is a reason. But I'm a bit confused about the quote from ERK. Text that is _not_ pre-5378 shouldn't matter.

I can add you to the thread and perhaps you can shed light on that for EKR.

>>> -1, last paragraph:
>>> 
>>> Are there any plans to fix 7345 or bundle-negotiation?
>> 
>> I co-authored 7345 with Christer, so we can have a discussion about fixing that one.
> 
> Okay
> 
>> 
>>> I see that 7345 replicates language from 5764 rather than references it, but it looks like a fairly easy fix.
>> 
>> Are you think bis effort?
> 
> ... or an update.

I was thinking bis=update, so we can get that going once mux-fix gets published.

>> or filing an errata?
> 
> I don't think errata is the correct approach. That's more for minor errors where a draft doesn't reflect the authors' intent at the time of writing. That doesn't seem to be the case here.

Agreed.

>>> I wonder if bundle-negotiation really needs a fix beyond the fact that it's reference to 5764 will effectively inherit any fixes due to the obsolescence of 5764.
>> 
>> Bundle isn’t published, so presumably they are in time to fix it.  I’ll leave that one to you to handle in the best way possible.
> 
> Good point :-)
> 
>> 
>>> This paragraph will likely be overtaken by events at some point. Remember that an RFC lasts forever. Language to the effect of "at the time of this writing" might be helpful.
>> 
>> OK, we can prepend to the first sentence of the last paragraph something like what you suggest, namely "at the time of this writing”.  Would that work
> 
> Yes.

Done.

>>> Editorial Comments:
>>> 
>>> - Section 1, paragraph 6 (First paragraph after numbered list)
>>> 
>>> This paragraph goes a bit far into value judgements. Can we dispense terms like "bad design" and "socially undesirable”?
>> 
>> This text came more or less verbatim from Stephen Farrell during the TLS WG review of the document and is what they signed off on. I hesitate to undo that at this point, but we can discuss it as part of IESG review.
> 
> Okay.
> 
>> 
>>> Also, I'm a little confused by the last sentence--what is the point of "even if a codepoint is not initially thought to be useful”?
>> 
>> Good catch.  Perhaps it would be clearer to say "even if the feature related to a codepoint is not initially thought to be useful in the context of demultiplexing”?
> 
> I'm still a little confused. " by not useful" do you mean codepoints that do not affect demuxing? Why does this draft need to talk about such codepoints?

Well the added text was meant to reduce scope to "useful in the context of demultiplexing”.  So it is my intent to specifically refer to codepoints that might make use of demux.

-G