Re: [nvo3] WG Last Call for draft-ietf-nvo3-hpvr2nve-cp-req-10
worley@ariadne.com (Dale R. Worley) Tue, 12 December 2017 10:34 UTC
Return-Path: <worley@alum.mit.edu>
X-Original-To: nvo3@ietfa.amsl.com
Delivered-To: nvo3@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 36D12129418 for <nvo3@ietfa.amsl.com>; Tue, 12 Dec 2017 02:34:52 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.934
X-Spam-Level:
X-Spam-Status: No, score=-1.934 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HEADER_FROM_DIFFERENT_DOMAINS=0.001, RCVD_IN_DNSWL_LOW=-0.7, SPF_SOFTFAIL=0.665] autolearn=no 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 FLFDFB1mjBKI for <nvo3@ietfa.amsl.com>; Tue, 12 Dec 2017 02:34:46 -0800 (PST)
Received: from resqmta-ch2-07v.sys.comcast.net (resqmta-ch2-07v.sys.comcast.net [IPv6:2001:558:fe21:29:69:252:207:39]) (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 78969129417 for <nvo3@ietf.org>; Tue, 12 Dec 2017 02:34:43 -0800 (PST)
Received: from resomta-ch2-15v.sys.comcast.net ([69.252.207.111]) by resqmta-ch2-07v.sys.comcast.net with ESMTP id OhtYe4QvH27mvOhtaeDNMK; Tue, 12 Dec 2017 10:34:42 +0000
Received: from hobgoblin.ariadne.com ([65.96.206.41]) by resomta-ch2-15v.sys.comcast.net with SMTP id OhtZe85GgLPDOOhtae8Gxl; Tue, 12 Dec 2017 10:34:42 +0000
Received: from hobgoblin.ariadne.com (hobgoblin.ariadne.com [127.0.0.1]) by hobgoblin.ariadne.com (8.14.7/8.14.7) with ESMTP id vBCAYfkp018439 for <nvo3@ietf.org>; Tue, 12 Dec 2017 05:34:41 -0500
Received: (from worley@localhost) by hobgoblin.ariadne.com (8.14.7/8.14.7/Submit) id vBCAYenT018436; Tue, 12 Dec 2017 05:34:40 -0500
X-Authentication-Warning: hobgoblin.ariadne.com: worley set sender to worley@alum.mit.edu using -f
From: worley@ariadne.com
To: nvo3@ietf.org
In-Reply-To: <43911A9C-E1FA-4A6D-8E97-40ABC10BAC29@nokia.com> (matthew.bocci@nokia.com)
Sender: worley@ariadne.com
Date: Tue, 12 Dec 2017 05:34:40 -0500
Message-ID: <87zi6oz133.fsf@hobgoblin.ariadne.com>
X-CMAE-Envelope: MS4wfLG1VtI2JNtRx23Ws4cD/rrS9PfUqKY4LQdSzZjJQ7DGu3V4mEImoG+XHGg+FBUhZQwL9nm7W8jLIakzP7UwfAWZG4fYj70ZdKUmUjL7KiJbyyOOjFps 1WFEQ71N4+FUgeatbGC9G6CSkqzIxHbn3KvAmQtN0mmPb9WYzWqWVbSA
Archived-At: <https://mailarchive.ietf.org/arch/msg/nvo3/VB_JG5gofv3_AR9sqEXdoNx4Bc4>
Subject: Re: [nvo3] WG Last Call for draft-ietf-nvo3-hpvr2nve-cp-req-10
X-BeenThere: nvo3@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: "Network Virtualization Overlays \(NVO3\) Working Group" <nvo3.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/nvo3>, <mailto:nvo3-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/nvo3/>
List-Post: <mailto:nvo3@ietf.org>
List-Help: <mailto:nvo3-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/nvo3>, <mailto:nvo3-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 12 Dec 2017 10:34:52 -0000
The draft looks good to me. A few nits: 1. Introduction Then the high level requirements to be fulfilled are satisfied. I think s/satisfied/stated/ is intended here. 1.1 Terminology The key words "MUST", "MUST NOT", "REQUIRED", "SHALL", "SHALL NOT", "SHOULD", "SHOULD NOT", "RECOMMENDED", "MAY", and "OPTIONAL" in this document are to be interpreted as described in RFC 2119 [RFC2119]. RFC 2119 has been updated by RFC 8174 and the newer version should be used. Hypervisor/Container: the logical collection of software, firmware and/or hardware that allows the creation and running of server or service appliance virtualization. tNVE is located on Hypervisor/Container. It is loosely used in this document to refer to the end device supporting the virtualization. For simplicity, we also use Hypervisor in this document to represent both hypervisor and container. I think this would be clearer if the term you intend to use (hypervisor) was indexed and described as such. You could also index "container" or "hypervisor/container" and point it to "hypervisor". (Better would be to use a generic word throughout and not overload a term which also has a use of much narrower scope, but it's late to make that change. The use of "tenant system" is a good example of this style, as it doesn't carry much baggage about what *type* of tenant it is. OTOH, "tenant system" isn't used consistently in the document.) 1.2 Target Scenarios This implies that if a given TSI disassociates from one VN, all the MAC and/or IP addresses are also disassociated. There is no need to signal the deletion of every MAC or IP when the TSI is brought down or deleted. This sentence is very detailed for the context in which it appears. To me, it reads more as a requirement (about the dissociation action) than part of the introduction. And I don't see (on one reading) a clear statement of this property among the listed requirements. Then again, is this intended as a firm requirement? 2.2 VM Live Migration Event If we've intiated a migration from hypervisor 1 to hypervisor 2, before it is finished, can we initiate a migration from hypervisor 2 to hypervisor 3? That is, does the CP have to support chained migrations-in-progress? 2.4 VM Pause, Suspension and Resumption Events The VM pause event leads to the VM transiting from Running state to Paused state. The Paused state indicates that the VM is resident in memory but no longer scheduled to execute by the hypervisor [I- D.ietf-opsawg-vmm-mib]. The VM can be easily re-activated from Paused state to Running state. The VM suspension event leads to the VM transiting from Running state to Suspended state. The VM resumption event leads to the VM transiting state from Suspended state to Running state. Suspended state means the memory and CPU execution state of the virtual machine are saved to persistent store. During this state, the virtual machine is not scheduled to execute by the hypervisor [I-D.ietf- opsawg-vmm-mib]. >From the split-NVe point of view, is there any difference between Paused and Suspended? 5. VDP Applicability and Enhancement Needs +------+-----------+-----------------------------------------------+ | Req | VDP | remarks | | | supported?| | +------+-----------+-----------------------------------------------+ I think "VDP supported?" would be clearer as "Supported by VDP?". As written now, it means "Does this requirement support VDP?". But my suggested wording won't fit well into this format. Perhaps "VDP supports?" would work. Dale
- [nvo3] WG Last Call for draft-ietf-nvo3-hpvr2nve-… Bocci, Matthew (Nokia - GB)
- Re: [nvo3] WG Last Call for draft-ietf-nvo3-hpvr2… Liyizhou
- Re: [nvo3] WG Last Call for draft-ietf-nvo3-hpvr2… Linda Dunbar
- Re: [nvo3] WG Last Call for draft-ietf-nvo3-hpvr2… ao.ting
- Re: [nvo3] WG Last Call for draft-ietf-nvo3-hpvr2… Dale R. Worley
- Re: [nvo3] WG Last Call for draft-ietf-nvo3-hpvr2… Liyizhou
- Re: [nvo3] WG Last Call for draft-ietf-nvo3-hpvr2… Black, David
- Re: [nvo3] WG Last Call for draft-ietf-nvo3-hpvr2… Liyizhou
- Re: [nvo3] WG Last Call for draft-ietf-nvo3-hpvr2… Dale R. Worley
- Re: [nvo3] WG Last Call for draft-ietf-nvo3-hpvr2… Donald Eastlake
- Re: [nvo3] WG Last Call for draft-ietf-nvo3-hpvr2… Bocci, Matthew (Nokia - GB)
- Re: [nvo3] WG Last Call for draft-ietf-nvo3-hpvr2… Liyizhou
- Re: [nvo3] WG Last Call for draft-ietf-nvo3-hpvr2… Bocci, Matthew (Nokia - GB)
- Re: [nvo3] WG Last Call for draft-ietf-nvo3-hpvr2… Liyizhou
- Re: [nvo3] WG Last Call for draft-ietf-nvo3-hpvr2… Bocci, Matthew (Nokia - GB)
- Re: [nvo3] WG Last Call for draft-ietf-nvo3-hpvr2… Liyizhou