Re: [Speermint] AD review: draft-ietf-speermint-architecture-16.txt

"Livingood, Jason" <Jason_Livingood@cable.comcast.com> Wed, 15 December 2010 22:14 UTC

Return-Path: <jason_livingood@cable.comcast.com>
X-Original-To: speermint@core3.amsl.com
Delivered-To: speermint@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id 237DB28C0EB for <speermint@core3.amsl.com>; Wed, 15 Dec 2010 14:14:04 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -104.678
X-Spam-Level:
X-Spam-Status: No, score=-104.678 tagged_above=-999 required=5 tests=[AWL=-2.943, BAYES_00=-2.599, HELO_MISMATCH_COM=0.553, HOST_MISMATCH_NET=0.311, USER_IN_WHITELIST=-100]
Received: from mail.ietf.org ([64.170.98.32]) by localhost (core3.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id KnbcHf+sxNH6 for <speermint@core3.amsl.com>; Wed, 15 Dec 2010 14:14:03 -0800 (PST)
Received: from cable.comcast.com (copdcimo01.potomac.co.ndcwest.comcast.net [76.96.32.251]) by core3.amsl.com (Postfix) with ESMTP id 19A1328C0CE for <speermint@ietf.org>; Wed, 15 Dec 2010 14:14:03 -0800 (PST)
Received: from ([24.40.55.41]) by copdcimo01.cable.comcast.com with ESMTP with TLS id 5503630.18928297; Wed, 15 Dec 2010 15:25:04 -0700
Received: from PACDCEXMB06.cable.comcast.com ([fe80::6134:ea50:286a:c0]) by PACDCEXHUB02.cable.comcast.com ([fe80::11d4:f530:37a0:9f4e%12]) with mapi id 14.01.0255.000; Wed, 15 Dec 2010 17:15:38 -0500
From: "Livingood, Jason" <Jason_Livingood@cable.comcast.com>
To: Gonzalo Camarillo <Gonzalo.Camarillo@ericsson.com>, "speermint@ietf.org" <speermint@ietf.org>
Thread-Topic: [Speermint] AD review: draft-ietf-speermint-architecture-16.txt
Thread-Index: AQHLhl0MkDKCH3qosUCwiyz13QaXzJOiPnUA
Date: Wed, 15 Dec 2010 22:15:35 +0000
Message-ID: <C92E455F.10B12%jason_livingood@cable.comcast.com>
In-Reply-To: <4CE3DB58.3070203@ericsson.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
user-agent: Microsoft-MacOutlook/14.1.0.101012
x-originating-ip: [147.191.227.151]
Content-Type: text/plain; charset="us-ascii"
Content-ID: <CD522DB9EE030645B9B2108449F5A1C4@cable.comcast.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Subject: Re: [Speermint] AD review: draft-ietf-speermint-architecture-16.txt
X-BeenThere: speermint@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
List-Id: Mailing list for the speermint working group <speermint.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/speermint>, <mailto:speermint-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/speermint>
List-Post: <mailto:speermint@ietf.org>
List-Help: <mailto:speermint-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/speermint>, <mailto:speermint-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 15 Dec 2010 22:14:04 -0000

Responses inline below. There is only one open question for you and I can
update the I-D.


- Jason

On 11/17/10 8:40 AM, "Gonzalo Camarillo"
>Title: I leave it up to the editors whether or not they want to expand
>the acronym SPEERMINT. Given that it is exanded in the Introduction,
>it should probably be expanded also in the title.

Done. Changed it to "Session PEERing for Multimedia INTerconnect
Architecture" 

>
>First sentence in the Abstract and first sentence in the Introduction:
>
>s/it's/its/

correction made

>
>First paragraph in the Introduction:
>
>"...of a SIP Service providers [RFC5486] network."
>
>It should be Provider's. Also, when capitalizing Provider, expand the
>SSP acronym there, and not in the next sentence.

Both corrections made.

>When the reader reaches Figure 1, there are a lot of new
>acronyms. Having a terminology section after the Introduction would
>help readers understand the figure. Otherwise, readers need to consult
>other RFCs or check the meaning of the acronyms later in this
>document.

While we'd prefer to stick with the reference to the terminology
document (RFC 5486) noted at the top of the figure (that document was
quite difficult to finalized), I have included a brief terminology section
before Figure 1 that defines each of the key terms.

>Acronyms such as UA should also be expanded on their first use or
>added to the Terminology section.

Correction made.

>Section 3. Why is the sending of the BYE (step 5) more likely to be
>performed by the target SSP than by the originating SSP?

It just seemed logical as we envisioned the call flow between two people
in our head. 

QUESTION: Would you like a modification there?

>Section 5.1.1.2
>
>s/chosen for based on/chosen based on/

correction made

>
>Section 5.1.2:
>
>"not all of them may be used". Does this mean that the LRF MUST NOT
>use all of them or that the LRF does not need to necessarily use all
>of them?

It means that the sections that follow (5.1.2.1 - 5.1.2.3) are typical
parts of the LRF but that based on the considerations of each SSP, that
particular SSP may not implement all of them (and it is not an RFC 2119
statement).

>
>Section 5.1.3
>
>   When communicating with another SSP, entities compliant to this
>   document should select a TLS-protected transport for communication
>   from the originating (or indirect) SSP to the receiving SSP if
>   available, as described further in Section 5.2.1.
>
>This "should" sounds like a normative SHOULD. If this normatively
>defined somewhere else (i.e., in another document)?
>
>In general, there are a set of instructions that are given without
>using normative language. What is the intention of those instructions?

This is intended to be information and to use non-normative language.
While words like "should" are still appropriate, they are not 2119-style
words, and as such we have removed any reference to 2119.

>
>ID nits gives the following warning:
>
> == Unused Reference: 'I-D.ietf-speermint-voip-consolidated-usecases' is
>     defined on line 609, but no explicit reference was found in the
>     text

We thought it was a good informative reference for further reading. I have
added text to that effect in the next version to clear the nit check.

Jason
>