Re: [MMUSIC] [Gen-art] Gen-ART Last Call review of draft-ietf-mmusic-dtls-sdp-27

Christer Holmberg <christer.holmberg@ericsson.com> Mon, 31 July 2017 17:20 UTC

Return-Path: <christer.holmberg@ericsson.com>
X-Original-To: mmusic@ietfa.amsl.com
Delivered-To: mmusic@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id D83AF132702; Mon, 31 Jul 2017 10:20:31 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.22
X-Spam-Level:
X-Spam-Status: No, score=-4.22 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham 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 GcFMz90-LqJO; Mon, 31 Jul 2017 10:20:29 -0700 (PDT)
Received: from sessmg23.ericsson.net (sessmg23.ericsson.net [193.180.251.45]) (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 99124132701; Mon, 31 Jul 2017 10:20:28 -0700 (PDT)
X-AuditID: c1b4fb2d-c3b2c9c000005f66-fa-597f66daa973
Received: from ESESSHC015.ericsson.se (Unknown_Domain [153.88.183.63]) by sessmg23.ericsson.net (Symantec Mail Security) with SMTP id 88.41.24422.AD66F795; Mon, 31 Jul 2017 19:20:26 +0200 (CEST)
Received: from ESESSMB109.ericsson.se ([169.254.9.91]) by ESESSHC015.ericsson.se ([153.88.183.63]) with mapi id 14.03.0352.000; Mon, 31 Jul 2017 19:19:39 +0200
From: Christer Holmberg <christer.holmberg@ericsson.com>
To: Paul Kyzivat <pkyzivat@alum.mit.edu>, Ben Campbell <ben@nostrum.com>
CC: "draft-ietf-mmusic-dtls-sdp.all@ietf.org" <draft-ietf-mmusic-dtls-sdp.all@ietf.org>, General Area Review Team <gen-art@ietf.org>, IETF MMUSIC WG <mmusic@ietf.org>
Thread-Topic: [Gen-art] Gen-ART Last Call review of draft-ietf-mmusic-dtls-sdp-27
Thread-Index: AQHTB0VTgKnrsM/ZrUePhRU1wUhw6qJp3c2ggAAC0pD///QEgIAAQsKAgAEBLjCAABUpAIAAKPpAgAGEIyD//+oVAIAA0u4ggABlFwCAADbsMA==
Date: Mon, 31 Jul 2017 17:19:39 +0000
Message-ID: <7594FB04B1934943A5C02806D1A2204B4CCA52CE@ESESSMB109.ericsson.se>
References: <fb41df24-ffeb-2cdd-6233-858f886ee6ec@alum.mit.edu> <7594FB04B1934943A5C02806D1A2204B4CCA1616@ESESSMB109.ericsson.se> <7594FB04B1934943A5C02806D1A2204B4CCA1662@ESESSMB109.ericsson.se> <e0e715ef-322a-e060-2d1e-86ca9650df2a@alum.mit.edu> <73BB780D-2A87-4C6E-BF8A-9976BCB953E1@nostrum.com> <7594FB04B1934943A5C02806D1A2204B4CCA2017@ESESSMB109.ericsson.se> <af3442ce-712d-4391-bf26-05a860211062@alum.mit.edu> <7594FB04B1934943A5C02806D1A2204B4CCA234E@ESESSMB109.ericsson.se> <7594FB04B1934943A5C02806D1A2204B4CCA35FF@ESESSMB109.ericsson.se> <2aac65f2-ea15-3825-b9f9-917ddc935578@alum.mit.edu> <7594FB04B1934943A5C02806D1A2204B4CCA42A8@ESESSMB109.ericsson.se> <6cf88d1c-526a-ccf2-0103-7822d81a2690@alum.mit.edu>
In-Reply-To: <6cf88d1c-526a-ccf2-0103-7822d81a2690@alum.mit.edu>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [153.88.183.154]
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFprAIsWRmVeSWpSXmKPExsUyM2K7ve6ttPpIgxl9lhbzO0+zW+y4u4PN 4uqrzywWU5c/ZrFYseEAqwOrx9/3H5g8liz5yeQxa+cTlgDmKC6blNSczLLUIn27BK6M3WuX MhfMc6p4tXMbYwPjEYcuRk4OCQETicX7t7N3MXJxCAkcYZRY9nYhK4SzmFFi67JdTF2MHBxs AhYS3f+0QRpEBDwkVkx5DFbDLLCKUWJ9awMjSEJYIEji69+VjBBFwRJND59C2XUSH6ZOYAex WQRUJW4+/skMYvMK+Eo8OLSaEWLZE1aJjomzwRo4BRwk7hz5DGYzCohJfD+1hgnEZhYQl7j1 ZD4TxNkCEkv2nGeGsEUlXj7+xwphK0ksuv0Z7GhmAU2J9bv0IVoVJaZ0P2SH2CsocXLmE5YJ jKKzkEydhdAxC0nHLCQdCxhZVjGKFqcWF+emGxnrpRZlJhcX5+fp5aWWbGIExtPBLb91dzCu fu14iFGAg1GJh9ferz5SiDWxrLgy9xCjBAezkgivRTxQiDclsbIqtSg/vqg0J7X4EKM0B4uS OK/DvgsRQgLpiSWp2ampBalFMFkmDk6pBsbaDeIX6v/usJS8caDkStf5z28qXN8YpW/mjr7K dW5tyuOjZpFd2wNKc9dbfpXf4vbgR0eClTXj/m9XGdm0fCR6U5Zb8P/r/fEoWnrG1ksVW+XX yHc2Vna5eaZyvmh8dHeugfUZ470Jgib/bv74x3hx7jRTl/9h078X/uJ5seLh5MMB06a97j6u xFKckWioxVxUnAgAXUhAN6MCAAA=
Archived-At: <https://mailarchive.ietf.org/arch/msg/mmusic/_ABpp7XMJGJegoMN54L2n6nxtFo>
Subject: Re: [MMUSIC] [Gen-art] Gen-ART Last Call review of draft-ietf-mmusic-dtls-sdp-27
X-BeenThere: mmusic@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: Multiparty Multimedia Session Control Working Group <mmusic.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/mmusic>, <mailto:mmusic-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/mmusic/>
List-Post: <mailto:mmusic@ietf.org>
List-Help: <mailto:mmusic-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/mmusic>, <mailto:mmusic-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 31 Jul 2017 17:20:32 -0000

PR updated.

Regards,

Christer

-----Original Message-----
From: Paul Kyzivat [mailto:pkyzivat@alum.mit.edu] 
Sent: 31 July 2017 18:03
To: Christer Holmberg <christer.holmberg@ericsson.com>; Ben Campbell <ben@nostrum.com>
Cc: draft-ietf-mmusic-dtls-sdp.all@ietf.org; General Area Review Team <gen-art@ietf.org>; IETF MMUSIC WG <mmusic@ietf.org>
Subject: Re: [Gen-art] Gen-ART Last Call review of draft-ietf-mmusic-dtls-sdp-27

On 7/31/17 4:05 AM, Christer Holmberg wrote:
> Hi Paul,
> 
>>> PR created:
>>>
>>> https://github.com/cdh4u/draft-dtls-sdp/pull/34
>>
>> This leaves RFC5763 in an inconsistent state:
>> - the reference to 8122 in section 5 isn't backed up with an entry in 
>> the references section
> 
> In the PR, I DO add 8122 to the reference section of 5763 :)

Oh, sorry.

>> - there is still a reference to 4572 in the introduction.
> 
> I could add a statement, saying that the reference in the Introduction is updated.

That works for me.

	Thanks,
	Paul

> Regards,
> 
> Christer
> 
> 
> 
> 
> Otherwise looks right.
> 
> 	Thanks,
> 	Paul
> 
>> Regards,
>>
>> Christer
>>
>> -----Original Message-----
>> From: Christer Holmberg [mailto:christer.holmberg@ericsson.com]
>> Sent: 29 July 2017 23:38
>> To: Paul Kyzivat <pkyzivat@alum.mit.edu>; Ben Campbell 
>> <ben@nostrum.com>
>> Cc: draft-ietf-mmusic-dtls-sdp.all@ietf.org; General Area Review Team 
>> <gen-art@ietf.org>; IETF MMUSIC WG <mmusic@ietf.org>
>> Subject: RE: [Gen-art] Gen-ART Last Call review of
>> draft-ietf-mmusic-dtls-sdp-27
>>
>> Hi,
>>
>>>>>>> Regarding the reference to RFC 4572, the new text in section
>>>>>>> 10.2.1 references RFC 4572. We earlier agreed we were not going to update that text, and keep an informative reference to RFC 4572.
>>>>>>
>>>>>> OK, I guess I remember that now. Is it considered acceptable to 
>>>>>> issue a new document with a reference to an obsolete document when it isn't to highlight a difference from the current document?
>>>>>>
>>>>>> Since this is a review for the teleconference, I'll just leave that for the IESG folk to decide.
>>>>>
>>>>> As far as I know, there’s no hard and fast rule about this. It 
>>>>> really depends on whether the difference between the new and 
>>>>> obsolete dependencies are material to the draft. I do think we (i.e.
>>>>> the IESG) would favor referencing the new RFC, but would be open 
>>>>> to arguments about why a WG chose to reference the obsolete 
>>>>> version
>>>>>
>>>>> Does anyone recall the reasoning in this instance?
>>>>
>>>> Just to make sure we are on the same page, there are TWO references to RFC 4572 in the draft.
>>>>
>>>> The FIRST reference is in section 8, where it is used to reference 
>>>> an example in RFC 4572. The same example exists in RFC 8122, so we can change that reference.
>>>>
>>>> The SECOND reference is in section 10.2.1, as part of the updated 
>>>> text for RFC 5763. Now, RFC 5763 references RFC 4572 in 4 
>>>> difference places, so if we change the >reference to RFC 8122 in 
>>>> the text updated by the draft we would also have to do it in every other place. That was the reason we decided not to do it (I have no problem doing it that's what IESG wants, though).
>>>
>>> Thanks for pointing that out. I just looked at that to size up the 
>>> situation. Of those four references, three of them are in section 5 
>>> and will all be replaced by the new text in this document. The remaining reference is simply a general one in the introduction. And then in addition there is the actual reference text in the normative references.
>>>
>>> ISTM that it would be sufficient to update the reference in the new 
>>> text for section 5 and then add a general statement to update all references to 4572 to refer to 8122.
>>>
>>> But again, this is really an IESG issue at this point.
>>
>> Or, we could just go ahead and do it :)
>>
>> Regards,
>>
>> Christer
>>
>>>
>>>
>>>
>>>>> -----Original Message-----
>>>>> From: Christer Holmberg [mailto:christer.holmberg@ericsson.com]
>>>>> Sent: 29 July 2017 01:07
>>>>> To: Paul Kyzivat <pkyzivat@alum.mit.edu>; 
>>>>> draft-ietf-mmusic-dtls-sdp.all@ietf.org
>>>>> Cc: General Area Review Team <gen-art@ietf.org>; IETF MMUSIC WG 
>>>>> <mmusic@ietf.org>
>>>>> Subject: RE: [Gen-art] Gen-ART Last Call review of
>>>>> draft-ietf-mmusic-dtls-sdp-27 Hi Paul, Thanks for the review. I'll 
>>>>> fix references.
>>>>> Regards,
>>>>> Christer
>>>>> -----Original Message-----
>>>>> From: Paul Kyzivat [mailto:pkyzivat@alum.mit.edu]
>>>>> Sent: 28 July 2017 04:01
>>>>> To: draft-ietf-mmusic-dtls-sdp.all@ietf.org
>>>>> Cc: General Area Review Team <gen-art@ietf.org>; IETF MMUSIC WG 
>>>>> <mmusic@ietf.org>
>>>>> Subject: [Gen-art] Gen-ART Last Call review of
>>>>> draft-ietf-mmusic-dtls-sdp-27 I am the assigned Gen-ART reviewer for this draft. The General Area Review Team (Gen-ART) reviews all IETF documents being processed by the IESG for the IETF Chair. Please wait for direction from your document shepherd or AD before posting a new version of the draft. For more information, please see the FAQ at <​http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.
>>>>> Document: draft-ietf-mmusic-dtls-sdp-27
>>>>> Reviewer: Paul Kyzivat
>>>>> Review Date: 2017-07-07
>>>>> IETF LC End Date: 2017-07-24
>>>>> IESG Telechat date: 2017-08-15
>>>>> Summary:
>>>>> This draft is basically ready for publication, but has nits that should be fixed before publication.
>>>>> (These nits were reported by IdNits. I apologize for not noticing 
>>>>> these during my Last Call review.)
>>>>> Issues:
>>>>> Major: 0
>>>>> Minor: 0
>>>>> Nits:  2
>>>>> (1) NIT: Unused Reference: 'RFC5245' is defined on line 1065, but 
>>>>> no explicit reference was found in the text This is now redundant because all the references in the text have been changed to draft-ietf-ice-rfc5245bis.
>>>>> (2) NIT: Obsolete informational reference (is this intentional?):
>>>>> RFC
>>>>> 4572 This is now obsolete because it has been replaced by RFC8122. This draft should now be referencing that.
>>>>
>>>
>>
>