IETF Logo Mail Archive

Re: [Ice] Benjamin Kaduk's Discuss on draft-ietf-ice-pac-04: (with DISCUSS and COMMENT) - DISCUSS

Christer Holmberg <christer.holmberg@ericsson.com> Wed, 22 April 2020 10:45 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 66FB83A00C0; Wed, 22 Apr 2020 03:45:31 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.92
X-Spam-Level:
X-Spam-Status: No, score=-2.92 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, HTML_MESSAGE=0.001, RCVD_IN_MSPIKE_H2=-0.82, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=ericsson.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 fVDDMjgr4YUG; Wed, 22 Apr 2020 03:45:29 -0700 (PDT)
Received: from EUR04-VI1-obe.outbound.protection.outlook.com (mail-eopbgr80044.outbound.protection.outlook.com [40.107.8.44]) (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 8F5433A0063; Wed, 22 Apr 2020 03:45:28 -0700 (PDT)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=MxcSYMz/zkwESDHxmTKM0dNT5UH6FmVhJJcx7ZEFUShh70dJhyuqo8qVkIxc860tVQHqgsWxAaoZ1+Q32qt/1lgAx3F7uXSXG9q7atJJX/SZ3dol2pa1D+hLh0G4NAT36XFA9bFhRczC/72wFVnF8AxmdT/3oFU/HiW2xA5bWpMwP1u+3+CR3avRgc7M3SRqEi9wtiRH0D7CRlDG+JAQY8O1/AN9A4YFWmIIhuPmYhjNlb54D6EgFpy2d3zaASoH/96DBRmLCdK9rNEdowZqZ5pCE048d3PQjr9L/fFVszujZtsG23A7M2O0GkX6Jz9MKAq+MqNFtRJ+NNvFZ21ipw==
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=rKMttq53sIWwYKXBeI/FdUWCMPS02/+GcVaGQwI/UzY=; b=guS+wfY73XgvdPc8v8qD8r9gVtjnRObvoVcaI8ksScIrHF/geG6QkX96ILm5V5YbjHKhjnfpEZ6X/EpWHx2gMmElxuJgczo2smZgtc0wau5gWF84M+gIuRRPzp1cYljWSE5KtG22Pi67ni3C/5jD+NHGqnMeyH0OCwS0/U8y7d8sdcFsg20HpVmtFyGGp625p2xlLD+Jdyys3kfVNcggQyC95VeMWfVPaVnOAMTszCiioZC+5lZN+a4ULuu5JHsq8GAxsP7/LYGfw6EOwNVeen9LktAcouzfjvDir2tk6deZCE+T1ZPdkj60uezt7+zy+4sBb/HSPOtduvFnk/z0Ww==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=ericsson.com; dmarc=pass action=none header.from=ericsson.com; dkim=pass header.d=ericsson.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ericsson.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=rKMttq53sIWwYKXBeI/FdUWCMPS02/+GcVaGQwI/UzY=; b=S4mrTbkXQ+w8nYs8PZWld/9XwhgnO+dsPDCsWSKWyByOgkA+aqQ+qotjDZ5X46aLR25HEEgt9szfh9BNQvNj7qrDIJcxwKjk8i0xI+S7qCvr5xDIwPUiryPqlmoduIvsolArw8RZZ48zqaiV++YCLeAa1YknuXB70Kw6B4+AnXA=
Received: from AM0PR07MB3987.eurprd07.prod.outlook.com (2603:10a6:208:46::31) by AM0PR07MB4802.eurprd07.prod.outlook.com (2603:10a6:208:f6::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2937.6; Wed, 22 Apr 2020 10:45:25 +0000
Received: from AM0PR07MB3987.eurprd07.prod.outlook.com ([fe80::b929:4e5c:6b46:3ccc]) by AM0PR07MB3987.eurprd07.prod.outlook.com ([fe80::b929:4e5c:6b46:3ccc%7]) with mapi id 15.20.2937.012; Wed, 22 Apr 2020 10:45:25 +0000
From: Christer Holmberg <christer.holmberg@ericsson.com>
To: Benjamin Kaduk <kaduk@mit.edu>, The IESG <iesg@ietf.org>
CC: "draft-ietf-ice-pac@ietf.org" <draft-ietf-ice-pac@ietf.org>, "ice-chairs@ietf.org" <ice-chairs@ietf.org>, "ice@ietf.org" <ice@ietf.org>, Nils Ohlmeier <nohlmeier@mozilla.com>
Thread-Topic: Benjamin Kaduk's Discuss on draft-ietf-ice-pac-04: (with DISCUSS and COMMENT) - DISCUSS
Thread-Index: AQHWGJMh51smdeuAxkK5VW50oM5mAQ==
Date: Wed, 22 Apr 2020 10:45:25 +0000
Message-ID: <ECD8AE15-61B3-49DD-9744-9779D6C58634@ericsson.com>
Accept-Language: en-US
Content-Language: en-GB
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
user-agent: Microsoft-MacOutlook/10.1e.0.191013
authentication-results: spf=none (sender IP is ) smtp.mailfrom=christer.holmberg@ericsson.com;
x-originating-ip: [188.127.223.154]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: f422d6e2-99a9-4be9-73f8-08d7e6aa43d9
x-ms-traffictypediagnostic: AM0PR07MB4802:
x-microsoft-antispam-prvs: <AM0PR07MB480210D2AC0C419274B22E4E93D20@AM0PR07MB4802.eurprd07.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:10000;
x-forefront-prvs: 03818C953D
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:AM0PR07MB3987.eurprd07.prod.outlook.com; PTR:; CAT:NONE; SFTY:; SFS:(4636009)(136003)(39860400002)(396003)(376002)(366004)(346002)(186003)(8936002)(66446008)(91956017)(36756003)(6512007)(66946007)(64756008)(66556008)(6486002)(66476007)(33656002)(76116006)(86362001)(8676002)(81156014)(71200400001)(2906002)(2616005)(54906003)(44832011)(6506007)(5660300002)(316002)(110136005)(4326008)(478600001)(26005); DIR:OUT; SFP:1101;
received-spf: None (protection.outlook.com: ericsson.com does not designate permitted sender hosts)
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: OyBij0dnzNJIkwSj/4q9IMg7KBqGZ0pKudGX+w8dZp3AMWriev81U9WG2apVx4nUzW3m9u4U7CpNeRogWLn7Zi2ZMPerEW7IOY7BoXmShTuWzr1E44CzaXWx0AxwcdR3+Fj50GaBlCGEkNz4gN62rgV4U8gjbJtsr80dHFtnJyoPELdrg9Q1O6xQKdRhBA7gM6DwpDonzKd2wLwQjNUa/xJpiwQDVyRtyj7ntr3KucDwx/yfBIiwLi+nA1gh9F2oil5+5s1c0VzcBv/bQ9Uu6CNhcOhHo7RNbM657gK+46g7htXJ5gHCcgfccslNL67OrrIYvV+1+/kB7U95cFbRTYP0vnQHTTfEjMonCpghbol0BxEdXEBthE8j1Qobjn9NMTTlJQKpw47cMoowlcOHUT5VKJtZSbh/qpcghPQNmIu+uIU6E9vi9T9ndbAhqMaN
x-ms-exchange-antispam-messagedata: mzpWpiWRWvdQd3//svqFu97kVqb1Qy6EQJisiIvCwQv5K8dppxWSmtudMvHbozLK4A0tiuH9DE4yyIsq0FMcCp7LyrrqbT4VTA07LxHWLqTZ5BKjI1Mfjfyb77MxYHyg7VeEc8KYXDzixoWp6EJ9OA==
x-ms-exchange-transport-forked: True
Content-Type: multipart/alternative; boundary="_000_ECD8AE1561B349DD97449779D6C58634ericssoncom_"
MIME-Version: 1.0
X-OriginatorOrg: ericsson.com
X-MS-Exchange-CrossTenant-Network-Message-Id: f422d6e2-99a9-4be9-73f8-08d7e6aa43d9
X-MS-Exchange-CrossTenant-originalarrivaltime: 22 Apr 2020 10:45:25.3047 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 92e84ceb-fbfd-47ab-be52-080c6b87953f
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: eOur0fkXsgMR3c9KYCR79VTJUC2JhG2mS/pDKu6I1QUzOcy4TiKK/3Ps0wICRKc0uVSnIp+Dnie0K62Dn/vIHBNA/JyHRsP+8D0w86a4oZM=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR07MB4802
Archived-At: <https://mailarchive.ietf.org/arch/msg/ice/6XbffVTHvTQgrllNHTdU1b4-9bM>
Subject: Re: [Ice] Benjamin Kaduk's Discuss on draft-ietf-ice-pac-04: (with DISCUSS and COMMENT) - DISCUSS
X-BeenThere: ice@ietf.org
X-Mailman-Version: 2.1.29
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: Wed, 22 Apr 2020 10:45:32 -0000

Hi Benjamin,



Thank You for the review! In this reply I will give some initial thoughts on your DISCUSS. I will address your COMMENT issues in another reply.



    ----------------------------------------------------------------------

    DISCUSS:

    ----------------------------------------------------------------------



>    I think we may have to be more specific about updates to the RFC 8445 state

>    machine, namely whether we are specifying a new state for a checklist to be

>    in (vs. keeping it somehow in the "Running" state and modifying the

>    procedures for that state) and describing what happens in Section 7.2.5.4

>    when all candidate pairs in the checklist are Failed or Succeeded but the

>    PAC timer has not expired.  In other words, the combination of 8445 and this

>    document need to be consistent about what the ICE state machine is.

>    In contrast, Trickle is pretty clear about which conditions in which

>    sections of [rfc5245bis] are updated and how, but we don't seem to provide

>    the same level of detail.



I don't think we need a new state, but instead stay in the Running state while the PAC timer is still running.



So, I guess we can make it more explicit that we stay in the Running state, as the text currently only say that the state is not set to Failed.



Perhaps something like:



OLD:



   "While the timer is running, the ICE agent MUST NOT set the state of a

   checklist to Failed, even if the checklist has no pairs left to

   check.  As a result, the ICE agent will not remove any data streams

   or set the state of the ICE session to Failed as long as the timer is

   running."



NEW:



  “While the timer is still running, the ICE agent MUST NOT update a checklist state

  from Running to Failed, even if there are no pairs left in the checklist to check.

  As a result, the ICE agent will not remove any data streams or set the state of the

  ICE session to Failed as long as the timer is running. As a result, the ICE agent will

  not remove any data streams or set the state of the ICE session to Failed as long as

  the timer is running."



Regards,



Christer

v2.23.0 | Report a Bug | By Email