Re: [pve-devel] [PATCH container] mountpoint mount: activate PVE-managed volumes during preparation

["Fabian Grünbichler" <f.gruenbichler@proxmox.com>]

On March 25, 2024 12:00 pm, Fiona Ebner wrote:
> Otherwise it was not possible to hotplug a volume that was previously
> deactivated and requires activation, e.g. an LVM LV that was detached
> after shutting down the container couldn't be hotplugged anymore
> later.
> 
> Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
> ---
> 
> Requires the following patch to be tested on recent kernels:
> https://lists.proxmox.com/pipermail/pve-devel/2024-January/061260.html
> 
>  src/PVE/LXC.pm | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/PVE/LXC.pm b/src/PVE/LXC.pm
> index 7db4833..29ff418 100644
> --- a/src/PVE/LXC.pm
> +++ b/src/PVE/LXC.pm
> @@ -1849,6 +1849,7 @@ sub __mountpoint_mount {
>  
>  	my $scfg = PVE::Storage::storage_config($storage_cfg, $storage);
>  
> +	PVE::Storage::activate_volumes($storage_cfg, [$volid]);

shouldn't this atually pass $snapname as well? for some storages this
does make quite the difference ;)

there is also one more call in pct.pm (for the `pct fsck` command) that
seems to lack activation before mapping.

>  	my $path = PVE::Storage::map_volume($storage_cfg, $volid, $snapname);
>  
>  	$path = PVE::Storage::path($storage_cfg, $volid, $snapname) if !defined($path);
> -- 
> 2.39.2
> 
> 
> 
> _______________________________________________
> pve-devel mailing list
> pve-devel@lists.proxmox.com
> https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
> 
> 
>