Re: [Gen-art] Gen-ART telechat review for draft-ietf-mmusic-sdp-mux-attributes-14

Dan Romascanu <dromasca@gmail.com> Wed, 26 October 2016 19:55 UTC

Return-Path: <dromasca@gmail.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 0BD7012953C for <gen-art@ietfa.amsl.com>; Wed, 26 Oct 2016 12:55:23 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.699
X-Spam-Level:
X-Spam-Status: No, score=-2.699 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_LOW=-0.7, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.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 yPRiNbkOIJc8 for <gen-art@ietfa.amsl.com>; Wed, 26 Oct 2016 12:55:21 -0700 (PDT)
Received: from mail-qk0-x234.google.com (mail-qk0-x234.google.com [IPv6:2607:f8b0:400d:c09::234]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id DC21112949D for <gen-art@ietf.org>; Wed, 26 Oct 2016 12:55:20 -0700 (PDT)
Received: by mail-qk0-x234.google.com with SMTP id n189so17588886qke.0 for <gen-art@ietf.org>; Wed, 26 Oct 2016 12:55:20 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=TjmKMQaTpalux9RL2kwb6WM5HV6ZkpGpb/ago7WkzQU=; b=cl97CLSkTARIYAazWE2PmJMHrPEkfSHpQw21SiJC4tydXeuwCHmOC2VFtxRC+3ibn2 SdvMBiTA8AyKP+yjFEJCUiJvea/KgdM4y21q5Ydid3/e8Ft56VrKTrJYY1+6sW8DXX9Y GuPHv5GkkP4y1o7mVN0sUSIOCAPw1tlCQqp95A7YRat5q3Ws23RrPrYvTauJEhHlPkIw DgcTUckW3OxxCJjLBYMk4TjhbvySnYEBVMkt+kO4JjG0Tai+2RahELNNnQdH120KQe9J SV633W0AgAlfRAlaj6oN43e7GUr0mLd9WIIHv7dAVfN2+SAA3q7GZ+xj3pqWFtaIA0Mm xF/Q==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=TjmKMQaTpalux9RL2kwb6WM5HV6ZkpGpb/ago7WkzQU=; b=hudmq1DcxJMl6VA08XKBdwCoJfJA0AiDX+FQPZR7EpQDJO4+8k8rUEMhy6jQmmoLPW oJZMBHm4gKdBSlfUJ5Q/iTP+HGp2+Rhy/rZHWPaW1Z8X6+/j0/JcE0EhJPweSDZb5QcZ H1VhZzNQObEStFiJGl/CwkTN/7jJAVmZW6Ol7CoZ/YC5XjSIEkGVIhujGpWN7t6MfRz2 qzuMqpEL1unzid3YVUa8qxqkEjF8asIL+6jeuWOSY478bkHZJttRpSSue5bXEES7Zrzr HvQxYPBchhd/MBjAKOfLo+wCHUlDKsE3SGCDFrUX00wWtkMcOovkIMEY7FeEqKqCCg8Q TiUg==
X-Gm-Message-State: ABUngvfsi/06uGDD0mtbfA+KcEwJU04KOipOy/+fzYDHXAssNqUNIGKvvzH35t1ndViVC5hXdOBsU6M7Vq82EA==
X-Received: by 10.55.135.1 with SMTP id j1mr3029281qkd.46.1477511720025; Wed, 26 Oct 2016 12:55:20 -0700 (PDT)
MIME-Version: 1.0
Received: by 10.140.21.228 with HTTP; Wed, 26 Oct 2016 12:55:19 -0700 (PDT)
In-Reply-To: <1477505577801.11203@cisco.com>
References: <CAFgnS4UMNCbACqJ8C0yn6H8piPzuEfRxgmr4Z4RZxfeVEa72dQ@mail.gmail.com> <1477505577801.11203@cisco.com>
From: Dan Romascanu <dromasca@gmail.com>
Date: Wed, 26 Oct 2016 22:55:19 +0300
Message-ID: <CAFgnS4WWxDk3mD=QGu7BqMdDrAT+jdRkgdtyDccLhmAvisVXXQ@mail.gmail.com>
To: "Suhas Nandakumar (snandaku)" <snandaku@cisco.com>
Content-Type: multipart/alternative; boundary="94eb2c073c1039456c053fca0126"
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/WGN3jjczewfbL9W8guRtsObYr70>
Cc: "draft-ietf-mmusic-sdp-mux-attributes.all@tools.ietf.org" <draft-ietf-mmusic-sdp-mux-attributes.all@tools.ietf.org>, "gen-art@ietf.org" <gen-art@ietf.org>
Subject: Re: [Gen-art] Gen-ART telechat review for draft-ietf-mmusic-sdp-mux-attributes-14
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.17
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: Wed, 26 Oct 2016 19:55:23 -0000

Hi Suhas,

The proposed edits would be fine with me. Thank you for addressing all my
concerns.

Regards,

Dan


On Wed, Oct 26, 2016 at 9:12 PM, Suhas Nandakumar (snandaku) <
snandaku@cisco.com> wrote:

> Hello Dan
>
>
>   My apologies for missing on the updates. For each concern raised , here
> are my responses.
>
>
> .1.       The use of B – ‘Both’ terminology used to indicate that an
> attribute is specified S – Session Level and M – Medial Level (e.g. in
> Section 5) may be confusing, as there is a third possible level SR – Source
> Level. Actually S + M would probably be more clear.
>
>
> [Suhas] - As discussed in our earlier email , i will be updating the
> description of 'B' to imply the attribute applies to both Session and Media
> level
>
>
> 2.Section 5.54 includes a note referring to the TBD content. ‘As per
> section 9.1 of [I-D.ietf-mmusic-sdp-bundle-negotiation],  there exists no
> publicly available specification that defines procedures for
> multiplexing/demultiplexing fax protocols flows over a single 5-tuple.
> Once such a specification is available, the multiplexing category
> assignments for the attributes in this section could be revisited.’
> Assuming the missing specification will be publicly available sometime in
> the future – how will this information be added? Revise this RFC? The
> question applies to other TBD marked in the ‘Mux Category’ column of the
> tables in Section 5 (in 5.42, 5.44, …)
>
>
> [Suhas] Section 15.2 of the latest version does address how to deal with
> registry updates for the categories. Excerpt below
>
> "
>
>    Any future updates to the "Mux Category" column values needs to
>    follow the existing registration policy of the affected table
>    (Section 8.2.4.2 of [I-D.ietf-mmusic-rfc4566bis]).
>
>    Also, the procedures from Section 8.2.4.1 of
>    [I-D.ietf-mmusic-rfc4566bis] needs to be followed when assigning "Mux
>    Category" value for the newly defined SDP attributes.
>
>
> "
>
>
> Please let me know your thoughts. I can produce a new version along wth
> IESG Evaluation comments next week.
>
>
>
> Thanks
>
> Suhas
>
>
> ------------------------------
> *From:* Dan Romascanu <dromasca@gmail.com>
> *Sent:* Wednesday, October 26, 2016 2:59 AM
> *To:* gen-art@ietf.org; draft-ietf-mmusic-sdp-mux-
> attributes.all@tools.ietf.org
> *Subject:* Gen-ART telechat review for draft-ietf-mmusic-sdp-mux-
> attributes-14
>
> 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-sdp-mux-attributes-14
> Reviewer: Dan Romascanu
> Review Date: 10/26/16
> IETF LC End Date: 8/10/16
> IESG Telechat date: 10/28/16
>
> Summary:
> Ready.
>
> The more important issue in my initial review was clarified in draft-14.
> Two minor issues were not, but these are not essential.
>
> Major issues:
>
> Minor issues:
>
> 1.       The use of B – ‘Both’ terminology used to indicate that an
> attribute is specified S – Session Level and M – Medial Level (e.g. in
> Section 5) may be confusing, as there is a third possible level SR –
> Source Level. Actually S + M would probably be more clear.
>
> 2. Section 5.54 includes a note referring to the TBD content. ‘As per
> section 9.1 of [I-D.ietf-mmusic-sdp-bundle-negotiation],  there exists no
> publicly available specification that defines procedures for
> multiplexing/demultiplexing fax protocols flows over a single 5-tuple.
> Once such a specification is available, the multiplexing category
> assignments for the attributes in this section could be revisited.’
> Assuming the missing specification will be publicly available sometime in
> the future – how will this information be added? Revise this RFC? The
> question applies to other TBD marked in the ‘Mux Category’ column of the
> tables in Section 5 (in 5.42, 5.44, …)
>
> Nits/editorial comments:
>
>
> Regards,
>
> Dan
>
>