Re: [PATCH pve-storage 1/2] fix #6050: nfs: strip trailing slashes from mount status check

["Jakob Klocker" <j.klocker@proxmox.com>]

Tested this by adding leading and trailing slashes to the `export` 
property in an NFS storage. Before the patch, the storage was
displayed as inactive, even though it was usable. After the patch 
it shows correctly as active.

Reviewed-by: Jakob Klocker <j.klocker@proxmox.com>
Tested-by: Jakob Klocker <j.klocker@proxmox.com>
On Fri Aug 7, 2026 at 11:36 AM CEST, David Riley wrote:
> Manual edits to the storage configuration bypass strict API
> validation. If an NFS export path includes a trailing slash, the share
> mounts successfully but continuously reports as 'inactive' in both the
> web UI and pvesm status.
>
> This occurs because mount status verification compares the
> configuration string against the entries in /proc/mounts.
>
> Normalize the export path by stripping trailing slashes prior to
> verifying the mount status.
>
> Link: https://bugzilla.proxmox.com/show_bug.cgi?id=6050
> Signed-off-by: David Riley <d.riley@proxmox.com>
> ---
>  src/PVE/Storage/NFSPlugin.pm | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/src/PVE/Storage/NFSPlugin.pm b/src/PVE/Storage/NFSPlugin.pm
> index 4cc02c9..6c67848 100644
> --- a/src/PVE/Storage/NFSPlugin.pm
> +++ b/src/PVE/Storage/NFSPlugin.pm
> @@ -2,9 +2,11 @@ package PVE::Storage::NFSPlugin;
>  
>  use strict;
>  use warnings;
> +
> +use File::Path;
> +use File::Spec;
>  use IO::File;
>  use Net::IP;
> -use File::Path;
>  
>  use PVE::Network;
>  use PVE::Tools qw(run_command);
> @@ -19,6 +21,8 @@ use base qw(PVE::Storage::Plugin);
>  sub nfs_is_mounted {
>      my ($server, $export, $mountpoint, $mountdata) = @_;
>  
> +    $export = File::Spec->canonpath($export);
> +
>      $server = "[$server]" if Net::IP::ip_is_ipv6($server);
>      my $source = "$server:$export";
>