all lists on lists.proxmox.com
 help / color / mirror / Atom feed
From: "DERUMIER, Alexandre" <Alexandre.DERUMIER@groupe-cyllene.com>
To: Fiona Ebner <f.ebner@proxmox.com>,
	"pve-devel@lists.proxmox.com" <pve-devel@lists.proxmox.com>,
	"aderumier@odiso.com" <aderumier@odiso.com>
Subject: Re: [pve-devel] [PATCH v7 qemu-server 00/10] cloudinit pending behaviour change
Date: Wed, 31 Aug 2022 09:40:31 +0000	[thread overview]
Message-ID: <dcf0bcaa-6fff-56bf-6f8a-5f70a010f49b@groupe-cyllene.com> (raw)
In-Reply-To: <bde6dd39-882c-5dff-1681-6118c302d63b@proxmox.com>

Hi,

Any news to apply this patch serie  ? Now that qemu 7.0 is pushed in repo.

Do you want a new patch serie with version check updated in 03/10 patch ?


Le 27/07/22 à 13:19, Fiona Ebner a écrit :
> Am 22.06.22 um 13:51 schrieb Alexandre Derumier:
>> Hi,
>>
>> This is an attempt to cleanup current behaviour of cloudinit online changes.
>>
>> Currently, we setup cloudinit options as pending, until we generate the config drive.
>>
>> This is not 100% true, because some option like vm name, nic mac address can be changed,
>> without going to pending, so user can't known if it need to regenerated it.
>>
>> Also, some can are very difficult to handle, if you hotplug a nic but it's failing,so pending,
>> then you defined an ipconfig, and then you revert hotplug.
>> or if you delete a nic, the ipconfig is no more displayed in the gui.
>>
>>
>> So, instead of setting cloudinit values in pending,
>> this patch serie copy the current cloudinit config in a new section [special:cloudinit],
>> when the config drive is generated.
>> This is only an hint, to allow to display diff between the generated cloudinit
>> drive, and the current vm config.
>>
>> A new specific cloudinit config api is added too displaying the diff between current and generated config.
>>
>> Reminder: This need pve-manager depency bump first to check the version for live migration.
> 
> Without 10/10 and the version check in 03/10 adapted in accordance with
> the above-mentioned bump (best done when applying):
> 
> Reviewed-by: Fiona Ebner <f.ebner@proxmox.com>
> 


  reply	other threads:[~2022-08-31  9:40 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-06-22 11:51 Alexandre Derumier
2022-06-22 11:51 ` [pve-devel] [PATCH v7 qemu-server 01/10] qemuconfig: load_current_config : delete cloudinit value Alexandre Derumier
2022-06-22 11:51 ` [pve-devel] [PATCH v7 qemu-server 02/10] vzdump : skip special:cloudinit section Alexandre Derumier
2022-06-22 11:51 ` [pve-devel] [PATCH v7 qemu-server 03/10] migration: test targetnode min version for cloudinit section Alexandre Derumier
2022-06-22 11:52 ` [pve-devel] [PATCH v7 qemu-server 04/10] cloudinit: add cloudinit section for current generated config Alexandre Derumier
2022-06-22 11:52 ` [pve-devel] [PATCH v7 qemu-server 05/10] generate cloudinit drive on offline plug Alexandre Derumier
2022-06-22 11:52 ` [pve-devel] [PATCH v7 qemu-server 06/10] cloudinit: make cloudnit options fastplug Alexandre Derumier
2022-06-22 11:52 ` [pve-devel] [PATCH v7 qemu-server 07/10] api2: add cloudinit config api Alexandre Derumier
2022-06-22 11:52 ` [pve-devel] [PATCH v7 qemu-server 08/10] api2: add cloudinit_update Alexandre Derumier
2022-06-22 11:52 ` [pve-devel] [PATCH v7 qemu-server 09/10] add cloudinit hotplug Alexandre Derumier
2022-06-22 11:52 ` [pve-devel] [PATCH v7 qemu-server 10/10] debian : control : add a break on pve-manager <= 7.2.4 Alexandre Derumier
2022-06-23  7:22   ` Fabian Ebner
2022-06-23  7:59     ` DERUMIER, Alexandre
2022-07-27 11:19 ` [pve-devel] [PATCH v7 qemu-server 00/10] cloudinit pending behaviour change Fiona Ebner
2022-08-31  9:40   ` DERUMIER, Alexandre [this message]
2022-09-02 12:54     ` Fiona Ebner
2022-09-02 16:42       ` DERUMIER, Alexandre
     [not found] ` <mailman.41.1662364805.354.pve-devel@lists.proxmox.com>
2022-09-06  7:24   ` DERUMIER, Alexandre
2022-11-08 16:44 ` [pve-devel] applied-series: " Thomas Lamprecht

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=dcf0bcaa-6fff-56bf-6f8a-5f70a010f49b@groupe-cyllene.com \
    --to=alexandre.derumier@groupe-cyllene.com \
    --cc=aderumier@odiso.com \
    --cc=f.ebner@proxmox.com \
    --cc=pve-devel@lists.proxmox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal