Bug 763046 (GLUSTER-1314)

Summary: portmapper functionality
Product: [Community] GlusterFS Reporter: Anand Avati <aavati>
Component: coreAssignee: Anand Avati <aavati>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: high Docs Contact:
Priority: low    
Version: mainlineCC: amarts, chrisw, gluster-bugs, shehjart
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Shehjar Tikoo 2010-08-10 02:57:32 UTC
I hope it is clear that portmapper will only allow a single GlusterFS (program, version) pair registration on a single machine, preventing us from running multiple bricks on the same server. Unless, every glusterfsd has a different (program, version) pair, but that basically voids the backward compatibility support that was to be brought in using the (program,vers) support.

To do what I think we're trying to do, we'll need a GlusterFS specific portmapper that allows connecting to a specific brick on the server. We could do this through a new tool that uses a triple like (program, vers, brick-id/vol-id or whatever else), instead of a tuple like the (program. version).

Comment 1 Anand Avati 2010-08-10 05:43:58 UTC
functionality for bricks to be assigned port numbers at run time and clients resolve it on the fly

Comment 2 Anand Avati 2010-08-12 07:55:15 UTC
PATCH: http://patches.gluster.com/patch/4091 in master (portmap functionality in glusterd)

Comment 3 Anand Avati 2010-08-12 07:55:19 UTC
PATCH: http://patches.gluster.com/patch/4092 in master (rpc-clnt/socket: rpc_clnt_reconfig() API to support overriding of port numbers)

Comment 4 Anand Avati 2010-08-12 07:55:23 UTC
PATCH: http://patches.gluster.com/patch/4093 in master (protocol/client: query portmap and reconnect on new port)

Comment 5 Anand Avati 2010-08-21 09:17:30 UTC
PATCH: http://patches.gluster.com/patch/4231 in master (portmapper program added to glusterfsd mgmt rpc)

Comment 6 Anand Avati 2010-08-26 12:51:25 UTC
PATCH: http://patches.gluster.com/patch/4329 in master (glusterd-pmap: trigger signout by RPC_DISCONNECT event)

Comment 7 Anand Avati 2013-07-19 03:56:40 UTC
REVIEW: http://review.gluster.org/5345 (glusterfsd: is_mgmt_rpc_reconnect should not be `char` but gf_boolean_t) posted (#1) for review on master by Harshavardhana (harsha)

Comment 8 Anand Avati 2013-07-25 07:32:32 UTC
COMMIT: http://review.gluster.org/5345 committed in master by Vijay Bellur (vbellur) 
------
commit 333872c8ad93cb9da682d27e89bc2a58cf86aba0
Author: Harshavardhana <harsha>
Date:   Thu Jul 18 20:52:36 2013 -0700

    glusterfsd: is_mgmt_rpc_reconnect should not be `char` but gf_boolean_t
    
    Change-Id: I35c4dc040ebb5e926c7f54d823d1c81b1228bd4f
    BUG: 763046
    Signed-off-by: Harshavardhana <harsha>
    Reviewed-on: http://review.gluster.org/5345
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Vijay Bellur <vbellur>