Re: [Gluster-devel] Fwd: Change in glusterfs[master]: nlm: use virtual ip of server to connect to client

[Rajesh Amaravathi]

Most probably, yes. We will need to test out the actual scenario to confirm it.
This change makes sure we connect to NLM clients using the virtual IP, so if the
problems are only with NLM, and virtual IP's, it should work now.

Regards, 
Rajesh Amaravathi, 
Software Engineer, GlusterFS 
RedHat Inc. 

----- Original Message -----
> From: "John Mark Walker" <address@hidden>
> To: address@hidden
> Sent: Tuesday, December 18, 2012 6:47:42 PM
> Subject: [Gluster-devel] Fwd: Change in glusterfs[master]: nlm: use virtual 
> ip of server to connect to client
> 
> Question: does this change how Glusterfs can be utilized in a
> virtualized context, including with VMware? My understanding was
> that the virtual IP handling in VMware had previously presented
> problems.
> 
> -------- Original Message --------
> Subject: Change in glusterfs[master]: nlm: use virtual ip of server
> to connect to client
> From: "Rajesh Amaravathi (Code Review)" <address@hidden>
> To:
> CC:
> 
> 
> 
> 
> Rajesh Amaravathi has uploaded a new change for review.
> 
> Change subject: nlm: use virtual ip of server to connect to client
> ......................................................................
> 
> nlm: use virtual ip of server to connect to client
> 
> In cases where the servers use virtual ip's, this commit
> makes sure we use them and not the physical ip.
> 
> This change also refactors code around nlm4_establish_callback
> by sending granted msg only after a connection establishment.
> 
> Change-Id: I087362c547a25aa52ef7fc6653845a3863466ee6
> BUG: 888283
> Signed-off-by: Rajesh Amaravathi <address@hidden>
> ---
> M rpc/rpc-transport/socket/src/socket.c
> M xlators/nfs/server/src/nlm4.c
> M xlators/nfs/server/src/nlm4.h
> 3 files changed, 83 insertions(+), 69 deletions(-)
> 
> 
>   git pull ssh://git.gluster.org/glusterfs refs/changes/26/4326/1
> --
> To view, visit http://review.gluster.org/4326
> To unsubscribe, visit http://review.gluster.org/settings
> 
> Gerrit-MessageType: newchange
> Gerrit-Change-Id: I087362c547a25aa52ef7fc6653845a3863466ee6
> Gerrit-PatchSet: 1
> Gerrit-Project: glusterfs
> Gerrit-Branch: master
> Gerrit-Owner: Rajesh Amaravathi <address@hidden>
> 
> _______________________________________________
> Gluster-devel mailing list
> address@hidden
> https://lists.nongnu.org/mailman/listinfo/gluster-devel
>