public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: Andreas Neufert via pve-devel <pve-devel@lists.proxmox.com>
To: Dominik Csapak <d.csapak@proxmox.com>,
	Proxmox VE development discussion <pve-devel@lists.proxmox.com>,
	Pavel Tide <Pavel.TIde@veeam.com>
Cc: Andreas Neufert <Andreas.Neufert@veeam.com>
Subject: Re: [pve-devel] [Veeam] Veeam change requests?
Date: Thu, 19 Sep 2024 08:40:19 +0000	[thread overview]
Message-ID: <mailman.11.1726735814.332.pve-devel@lists.proxmox.com> (raw)
In-Reply-To: <39504c85-1fbb-44a3-850c-0d8132c01c09@proxmox.com>

[-- Attachment #1: Type: message/rfc822, Size: 17185 bytes --]

From: Andreas Neufert <Andreas.Neufert@veeam.com>
To: Dominik Csapak <d.csapak@proxmox.com>, Proxmox VE development discussion <pve-devel@lists.proxmox.com>, Pavel Tide <Pavel.TIde@veeam.com>
Subject: Re: [pve-devel] [Veeam] Veeam change requests?
Date: Thu, 19 Sep 2024 08:40:19 +0000
Message-ID: <PH0PR14MB42646286C0090066DDDB9EFCF4632@PH0PR14MB4264.namprd14.prod.outlook.com>

Hi Dominik, thanks for your input and feedback.

I CC @Pavel Tide<mailto:Pavel.TIde@veeam.com>, who can answer the code questions.

For the support. Veeam customers can open a support case at https://support.veeam.com<https://support.veeam.com/>

I would like to ask the PVE group to do the same when you identify a bug and select the evaluation product choice from the dropdown.
Then please send me or Pavel the support ticket number so that we can route the ticket differently (out of evaluation support).
The advantage of using the Veeam support portal is that you can upload logs, and we can share fixes with you for testing or similar things.
To start the support ticket with the evaluation product selection, you need to create a free account on veeam.com

If nothing works, Pavel and I can help directly (send mail). We also monitor this list for Veeam issues and questions.

Best regards… Andreas

From: Dominik Csapak <d.csapak@proxmox.com>
Date: Thursday, 19. September 2024 at 09:59
To: Proxmox VE development discussion <pve-devel@lists.proxmox.com>
Cc: Andreas Neufert <Andreas.Neufert@veeam.com>
Subject: Re: [pve-devel] [Veeam] Veeam change requests?
This is the first time you've received an email from this sender d.csapak @ proxmox.com, please exercise caution when clicking on links or opening attachments.

On 9/17/24 09:20, Andreas Neufert via pve-devel wrote:
>
> Hi Proxmox Dev team,
>

Hi,

> Tim Marx mentioned that you have some insights and change wishes for the Veeam backup processing and that we should reach out to this list. We would be happy to get this feedback here to be able to address it in our code or join a call if this helps.

Thanks for reaching out!

During (very basic & short) testing, i discovered a few things that are problematic from our point
of view:

* During backup, there is often a longer running connection open to our QMP socket of running VMs
(/var/run/qemu-server/XXXX.qmp, where XXXX is the vmid). This blocks our management stack from
doing certain tasks, like start/stop (probably does not matter during backup) but also
things like the VNC console, etc.

a better way would be to close the connections as soon as possible instead of keeping them
open. (Alternatively using our API/CLI could also be done, but I don't know what
exact QMP commands you're running)

if you absolutely need a longer running socket, please open a bug report on
https://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fbugzilla.proxmox.com%2F&data=05%7C02%7CAndreas.Neufert%40veeam.com%7C32edd9e9d4034ac3e8f308dcd880fbd1%7Cba07baab431b49edadd7cbc3542f5140%7C1%7C0%7C638623295714596487%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C0%7C%7C%7C&sdata=SF75C1yogf8n9EPMYzwv%2BSFRDx2Y0%2BSruqkgF%2FuDcQU%3D&reserved=0 so we can discuss and track that there, how we could make
a socket available that is not used by our stack

* Another thing that I noticed was that it's not really visible if a backup is running
for a particular VM, so users might accidentally them down (or pause, etc.). Especially
i think it's bad if the VM is placed under a HA policy that has 'stopped' as target, as
that will try to stop the VM by itself. (Though this might be a configuration error in itself?)

A quick way to fix this would be to have a (custom) lock in our VMs. For longer running tasks
that block a guest, we have a line 'lock: XXXX' in the config that prevents our stack
from doing most operations.

Putting that in would be a very short call to our perl code that locks the config locally
( `PVE::QemuConfig->lock_config($vmid, $updatefn) ), checks for existing locks,
updates the config with a new (custom) lock and writes it again.

Though i must admit, I'm not sure if custom locks outside of our defined ones would work,
but I'm sure we could add a 'custom' lock that you could use, should my mentioned
approach not work properly.

* Also, I noticed that when a guest is started from your stack, you modify the QEMU command line a
bit, namely removing some options that would be necessary to start the VM during the backup.
Is there a specific reason why you do it this way, instead of starting the VM through
our API/CLI?


A more general question last: What is the process for our/your users and us if they/we
find a bug? Where can they be reported to you?

I hope this helps

Best regards
Dominik

[-- Attachment #2: Type: text/plain, Size: 160 bytes --]

_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

      reply	other threads:[~2024-09-19  8:50 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-09-17  7:20 Andreas Neufert via pve-devel
2024-09-19  7:53 ` Dominik Csapak
2024-09-19  8:40   ` Andreas Neufert via pve-devel [this message]

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=mailman.11.1726735814.332.pve-devel@lists.proxmox.com \
    --to=pve-devel@lists.proxmox.com \
    --cc=Andreas.Neufert@veeam.com \
    --cc=Pavel.TIde@veeam.com \
    --cc=d.csapak@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal