[pve-devel] applied-series: Re: [PATCH v3 manager 1/2] vzdump: make guest include logic testable

Thomas Lamprecht t.lamprecht at proxmox.com
Mon Jun 8 16:03:06 CEST 2020


On 6/8/20 3:00 PM, Aaron Lauterer wrote:
> As a first step to make the whole guest include logic more testable the
> part from the API endpoint has been moved to its own method with as
> little changes as possible.
> 
> Everything concerning `all` and `exclude` logic is still in the
> PVE::VZDump->exec_backup() method.
> 
> Signed-off-by: Aaron Lauterer <a.lauterer at proxmox.com>
> ---
> 
> v2 -> v3:
> * remove $self from get_included_guests
> * added `use` statement for API2Tools
> * fixed a regression introduced in the previous revision that would
>   skip the backup job completely if the --all flag was set
> 
> v1 -> v2:
> * fixed return value. Array refs inside an array lead to nested
>   arrays not working with `my ($foo, $bar) = method();`
> 
> 
> As talked with thomas on[0] and off list, this patch series is meant to
> have more confidence in the ongoing changes.
> 
> My other ongoing patch series [1] will move the all the logic, even the
> one in the `exec_backup()` method into one single method.
> 
> [0] https://pve.proxmox.com/pipermail/pve-devel/2020-April/042795.html
> [1] https://pve.proxmox.com/pipermail/pve-devel/2020-April/042753.html
> (new revision is on the way)
> 
>  PVE/API2/VZDump.pm | 36 ++++++------------------------------
>  PVE/VZDump.pm      | 37 +++++++++++++++++++++++++++++++++++++
>  2 files changed, 43 insertions(+), 30 deletions(-)
> 
>

applied series, thanks!

Small followup, we normally do not use `${$vmids}[0]` to access elements of array
references, but rather `$vmids->[0]` both OK.




More information about the pve-devel mailing list