[pve-devel] applied: [PATCH storage] fix #1598: use glusterfs daemon default port for online check

[Wolfgang Bumiller]

applied

I'm not sure you could change the main port even if you wanted to
without using port redirection... Otherwise the only better approach
would be to do whatever libglusterfs does (and/or have the port in the
config...)

On Mon, Jan 28, 2019 at 02:37:05PM +0100, Thomas Lamprecht wrote:
> use the port where the main glusterfs daemon listens on as ping port,
> this one is also used by QEMU as default.
> 
> Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
> ---
>  PVE/Storage/GlusterfsPlugin.pm | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/PVE/Storage/GlusterfsPlugin.pm b/PVE/Storage/GlusterfsPlugin.pm
> index b6ede44..1f9465f 100644
> --- a/PVE/Storage/GlusterfsPlugin.pm
> +++ b/PVE/Storage/GlusterfsPlugin.pm
> @@ -39,9 +39,8 @@ my $get_active_server = sub {
>  	my $status = 0;
>  
>  	if ($server && $server ne 'localhost' && $server ne '127.0.0.1' && $server ne '::1') {
> -
> -	    # ping the echo port (7) without service check
> -	    $status = PVE::Network::tcp_ping($server, undef, 2);
> +	    # ping the gluster daemon default port (24007) as heuristic
> +	    $status = PVE::Network::tcp_ping($server, 24007, 2);
>  
>  	} else {
>  
> -- 
> 2.20.1