From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <t.lamprecht@proxmox.com>
Received: from firstgate.proxmox.com (firstgate.proxmox.com [212.224.123.68])
 (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)
 key-exchange X25519 server-signature RSA-PSS (2048 bits))
 (No client certificate requested)
 by lists.proxmox.com (Postfix) with ESMTPS id 463E876CA5
 for <pve-devel@lists.proxmox.com>; Wed, 20 Oct 2021 16:23:45 +0200 (CEST)
Received: from firstgate.proxmox.com (localhost [127.0.0.1])
 by firstgate.proxmox.com (Proxmox) with ESMTP id 3701E131CC
 for <pve-devel@lists.proxmox.com>; Wed, 20 Oct 2021 16:23:45 +0200 (CEST)
Received: from proxmox-new.maurer-it.com (proxmox-new.maurer-it.com
 [94.136.29.106])
 (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)
 key-exchange X25519 server-signature RSA-PSS (2048 bits))
 (No client certificate requested)
 by firstgate.proxmox.com (Proxmox) with ESMTPS id C57DF131C1
 for <pve-devel@lists.proxmox.com>; Wed, 20 Oct 2021 16:23:44 +0200 (CEST)
Received: from proxmox-new.maurer-it.com (localhost.localdomain [127.0.0.1])
 by proxmox-new.maurer-it.com (Proxmox) with ESMTP id 9B48245E89
 for <pve-devel@lists.proxmox.com>; Wed, 20 Oct 2021 16:23:44 +0200 (CEST)
Message-ID: <7209d9f5-9b43-a882-ea0c-ca010aa79f09@proxmox.com>
Date: Wed, 20 Oct 2021 16:23:43 +0200
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:94.0) Gecko/20100101
 Thunderbird/94.0
Content-Language: en-US
To: Proxmox VE development discussion <pve-devel@lists.proxmox.com>,
 Aaron Lauterer <a.lauterer@proxmox.com>,
 Dominik Csapak <d.csapak@proxmox.com>
References: <20210922092749.2386238-1-d.csapak@proxmox.com>
 <724abcda-3bda-dbe6-ac18-7b81339fe284@proxmox.com>
 <641ef788-8ede-d6dc-7697-2fa41ee00a5e@proxmox.com>
From: Thomas Lamprecht <t.lamprecht@proxmox.com>
In-Reply-To: <641ef788-8ede-d6dc-7697-2fa41ee00a5e@proxmox.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
X-SPAM-LEVEL: Spam detection results:  0
 AWL 1.331 Adjusted score from AWL reputation of From: address
 BAYES_00                 -1.9 Bayes spam probability is 0 to 1%
 KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment
 NICE_REPLY_A           -2.267 Looks like a legit reply (A)
 SPF_HELO_NONE           0.001 SPF: HELO does not publish an SPF Record
 SPF_PASS               -0.001 SPF: sender matches SPF record
Subject: Re: [pve-devel] partially-applied: [PATCH manager v2 00/12] multi
 tab disk panel & multi disk wizard
X-BeenThere: pve-devel@lists.proxmox.com
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Proxmox VE development discussion <pve-devel.lists.proxmox.com>
List-Unsubscribe: <https://lists.proxmox.com/cgi-bin/mailman/options/pve-devel>, 
 <mailto:pve-devel-request@lists.proxmox.com?subject=unsubscribe>
List-Archive: <http://lists.proxmox.com/pipermail/pve-devel/>
List-Post: <mailto:pve-devel@lists.proxmox.com>
List-Help: <mailto:pve-devel-request@lists.proxmox.com?subject=help>
List-Subscribe: <https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel>, 
 <mailto:pve-devel-request@lists.proxmox.com?subject=subscribe>
X-List-Received-Date: Wed, 20 Oct 2021 14:23:45 -0000

On 19.10.21 15:04, Aaron Lauterer wrote:
>>
>> applied=C2=A0above,=C2=A0delayed=C2=A0below;=C2=A0mostly=C2=A0because=C2=
=A0I'd=C2=A0like=C2=A0to=C2=A0have=C2=A0some=C2=A0feedback=C2=A0from
>> others=C2=A0here=C2=A0and=C2=A0I'd=C2=A0also=C2=A0like=C2=A0to=C2=A0ha=
ve=C2=A0the=C2=A0same=C2=A0for=C2=A0CT,=C2=A0to=C2=A0avoid=C2=A0a=C2=A0fe=
ature=C2=A0drift=C2=A0in
>> the=C2=A0wizards.
>=20
> I took the last two patches for a quick spin, and I have to say that I =
do
> like the functionality. Yes it does change the way one interacts with t=
he
> wizard quite a bit, but it really makes it so much easier to create VMs=
 with
> no disks at all (useful when preparing for qm importdisk) or more than =
one
> disk.

Thanks for the feedback, appreciated, for the record a v3 from that serie=
s was already
available (as you've now seen yourself, just wanted to ensure other devs =
see that this
is the reason the remaining v2 patches will not be used)