[pve-devel] applied: [PATCH container 1/3] prefer storage_check_enabled over storage_check_node

Thomas Lamprecht t.lamprecht at proxmox.com
Mon Jun 21 10:42:29 CEST 2021


On 18.06.21 12:59, Fabian Ebner wrote:
> storage_check_enabled simply checks for the 'disable' option and then calls
> storage_check_node.
> 
> While not strictly necessary for a second call where only the storage differs,
> it is more future-proof: if support for a target storage is added at some point,
> it might be easy to miss adapting the call.
> 
> For the migration checks, disabled storages are now always caught.
> 
> Signed-off-by: Fabian Ebner <f.ebner at proxmox.com>
> ---
>  src/PVE/API2/LXC.pm    |  4 ++--
>  src/PVE/LXC/Migrate.pm | 10 +++++-----
>  2 files changed, 7 insertions(+), 7 deletions(-)
> 
>

applied, thanks!





More information about the pve-devel mailing list