Re: [Ice] Trickle ICE review

Christer Holmberg <christer.holmberg@ericsson.com> Tue, 28 March 2017 18:20 UTC

Return-Path: <christer.holmberg@ericsson.com>
X-Original-To: ice@ietfa.amsl.com
Delivered-To: ice@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id AA8131297CC for <ice@ietfa.amsl.com>; Tue, 28 Mar 2017 11:20:00 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.221
X-Spam-Level:
X-Spam-Status: No, score=-4.221 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] 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 iyKTQe72-sgW for <ice@ietfa.amsl.com>; Tue, 28 Mar 2017 11:19:58 -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 4A2CA12947C for <ice@ietf.org>; Tue, 28 Mar 2017 11:19:58 -0700 (PDT)
X-AuditID: c1b4fb2d-25fff70000005be8-cd-58daa94a05cf
Received: from ESESSHC009.ericsson.se (Unknown_Domain [153.88.183.45]) by (Symantec Mail Security) with SMTP id 8E.69.23528.A49AAD85; Tue, 28 Mar 2017 20:19:56 +0200 (CEST)
Received: from ESESSMB109.ericsson.se ([169.254.9.242]) by ESESSHC009.ericsson.se ([153.88.183.45]) with mapi id 14.03.0339.000; Tue, 28 Mar 2017 20:19:54 +0200
From: Christer Holmberg <christer.holmberg@ericsson.com>
To: Ari Keränen <ari.keranen@ericsson.com>, Peter Saint-Andre <stpeter@stpeter.im>, Peter Thatcher <pthatcher@google.com>
CC: "ice@ietf.org" <ice@ietf.org>
Thread-Topic: [Ice] Trickle ICE review
Thread-Index: AQHSpd5vLfPb95agJ06E1TropZKa9qGnVe4AgAJB1wCAALtqgIAABEwAgAA6hcA=
Date: Tue, 28 Mar 2017 18:19:53 +0000
Message-ID: <7594FB04B1934943A5C02806D1A2204B4CB330A0@ESESSMB109.ericsson.se>
References: <CAJrXDUHzNT3v5oMPBQu5_OsXwonY7cogDQgTt5QPxN0=6DWQkQ@mail.gmail.com> <7ebb3254-a882-6e05-3159-0ec56614831b@stpeter.im> <CAJrXDUEi0n7P5mDuuLGj285AmQqr9HDFUPGLtLnU+BuJpws6Tw@mail.gmail.com> <7e9e8188-2add-0497-e94f-14ee41afe02d@stpeter.im> <BFB0CDEF-4572-41D5-A5D2-A5D210A1E175@ericsson.com>
In-Reply-To: <BFB0CDEF-4572-41D5-A5D2-A5D210A1E175@ericsson.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [153.88.183.148]
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFmpmkeLIzCtJLcpLzFFi42KZGbFdV9dn5a0Ig+n31C2+Xai1uLb8NavF sT39zA7MHgs2lXosWfKTyWPunhfMAcxRXDYpqTmZZalF+nYJXBm/f79hLbggUrHv/E6WBsat Al2MHBwSAiYSp/9ldzFycQgJrGeUaO5tYIVwljBK9DZcZAUpYhOwkOj+pw0SFxGYwCjx7uYs FpA4s4CixMu9al2MnBzCAqoS82ZeZgOxRQTUJC7O6IGy/SR2ftrFCGKzANWcWrSGFcTmFfCV ePFzJQuILSSwlUlibp87iM0p4CCx/ssSsDijgJjE91NrmEBsZgFxiVtP5oPZEgICEkv2nGeG sEUlXj7+xwphK0k0LnnCClGvJ3Fj6hQ2CFtbYtnC18wQewUlTs58wjKBUXQWkrGzkLTMQtIy C0nLAkaWVYyixanFxbnpRsZ6qUWZycXF+Xl6eaklmxiBUXNwy2/dHYyrXzseYhTgYFTi4V2w 5FaEEGtiWXFl7iFGCQ5mJRHeX3OAQrwpiZVVqUX58UWlOanFhxilOViUxHkd9l2IEBJITyxJ zU5NLUgtgskycXBKNTBmn04KbKsOMP4nlcu2SD7135mH5b1HEwQltqfMiRB+wKr0Kmu7fyJD bMirn0fFxS+u0/dvaX1z6aP8mpKuqOsa7AylsnH7r9y4t5pjqePan5YzTI+sZb6wt83ggrft yd8bmoRn682wmvZEh/PVyjXNLfKNTxhnhRRGyniECdzuCeObH+K8fLYSS3FGoqEWc1FxIgBq 2yp9lgIAAA==
Archived-At: <https://mailarchive.ietf.org/arch/msg/ice/Es4orWApBDQih_OzuZ7TBdEU4rE>
Subject: Re: [Ice] Trickle ICE review
X-BeenThere: ice@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: "Interactive Connectivity Establishment \(ICE\)" <ice.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ice>, <mailto:ice-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ice/>
List-Post: <mailto:ice@ietf.org>
List-Help: <mailto:ice-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ice>, <mailto:ice-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 28 Mar 2017 18:20:01 -0000

Hi,

I agree that adding a definition of "ICE session" to 5245bis is a good idea.

However, I am not sure simply saying "until a re-start would actually work". You may never do a re-start - often you simply terminate the session.

Also, in RFC 7675 we have the following text:

"That means that a new session, or an ICE restart,..."

...which seems to suggest that a new session and an ICE restart are not related. However, that's probably a minor issue, and we can fix that with an errata if needed.

Regards,

Christer




-----Original Message-----
From: Ice [mailto:ice-bounces@ietf.org] On Behalf Of Ari Keränen
Sent: 28 March 2017 19:43
To: Peter Saint-Andre <stpeter@stpeter.im>; Peter Thatcher <pthatcher@google.com>
Cc: ice@ietf.org
Subject: Re: [Ice] Trickle ICE review


> On 28 Mar 2017, at 11:27, Peter Saint-Andre <stpeter@stpeter.im> wrote:
> 
>>> - The name "ICE negotiation session" with definition of "A virtual
>>> session ..." is confusing.   I understand that we need a word for
>>> "period between restarts" and "period across restarts".  But what 
>>> does "virtual" mean in this context?  And what does "negotiation" 
>>> mean in "ICE negotiation session"?  Here's a possible suggestion: call it
>>> "single-exchange session".   A single-exchange session is the period
>>> from a single exchange of ICE description until the next exchange.  
>>> A "normal" ICE session is basically a multi-exchange session.
>> 
>>    This might be clearer:
>> 
>>       ICE Session:  All of the ICE-related interactions between ICE agents
>>          up until an ICE restart (if any).
>> 
>>    However, that definition probably belongs in 5245bis. The term as used
>>    in the Trickle specification is not limited to interactions between
>>    Trickle ICE agents.
>> 
>> 
>> I like that idea of using "ICE session" to mean the thing between 
>> restarts as long as it's compatible with 5245 and 5245bis and it 
>> doesn't cause confusion such that people think an "ICE session" is 
>> the time period across all ICE restarts.
>> 
>> I originally thought it should go in 5245bis also, but Ari thought it 
>> didn't make sense there since it would never use the term after 
>> defining it.
> 
> It's already used in 5245bis, isn't it? But we can continue to define 
> it in Trickle.

It's used but not defined actually. We'll add this to the terminology of 5245bis.


Cheers,
Ari
_______________________________________________
Ice mailing list
Ice@ietf.org
https://www.ietf.org/mailman/listinfo/ice