[pve-devel] [PATCH v2 guest-common] abstractconfig: add prototype for get_backup_volumes
Aaron Lauterer
a.lauterer at proxmox.com
Wed Mar 4 16:08:47 CET 2020
On 2/27/20 11:01 AM, Aaron Lauterer wrote:
> Signed-off-by: Aaron Lauterer <a.lauterer at proxmox.com>
> ---
>
> v1[0] -> v2:
> given the feedback from Fabian I
> * changed name of the method
> * added more details as to what implementations should return
>
> [0] https://pve.proxmox.com/pipermail/pve-devel/2020-January/041267.html
>
>
> PVE/AbstractConfig.pm | 18 ++++++++++++++++++
> 1 file changed, 18 insertions(+)
>
> diff --git a/PVE/AbstractConfig.pm b/PVE/AbstractConfig.pm
> index 782714f..aab84ff 100644
> --- a/PVE/AbstractConfig.pm
> +++ b/PVE/AbstractConfig.pm
> @@ -366,6 +366,24 @@ sub get_replicatable_volumes {
> die "implement me - abstract method\n";
> }
>
> +# Returns whether the guests volumes are included in a vzdump job
> +# Return Format:
> +# referenced array with hashes as elements:
> +# [
> +# {
> +# key, key in the config, e.g. mp0, scsi0,...
> +# included, boolean
> +# reason, string
> +# volume, volid / mountpoint
That line slipped through here. It's not present in the actual code.
> +# data volume object as returned from foreach_drive/foreach_mountpoint
> +# },
> +# ]
> +sub get_backup_volumes {
> + my ($class, $conf) = @_;
> +
> + die "implement me - abstract method\n";
> +}
> +
> # Internal snapshots
>
> # NOTE: Snapshot create/delete involves several non-atomic
>
More information about the pve-devel
mailing list