Bug 1343926 - port-map: let brick choose its own port
Summary: port-map: let brick choose its own port
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: rpc
Version: mainline
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Prasanna Kumar Kalever
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-06-08 10:43 UTC by Prasanna Kumar Kalever
Modified: 2019-03-25 16:30 UTC (History)
2 users (show)

Fixed In Version: glusterfs-6.0
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-03-25 16:30:11 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Prasanna Kumar Kalever 2016-06-08 10:43:07 UTC
Description of problem:
In the current implementation, glusterd will choose a free port and pass
that to brick which need to bind on it, if you observe carefully there is window
open for race here. In case, if the passed port was consumed by some other
processes then brick will fail to bind. In the recent past we have addressed
this issue by returning EADDRINUSE from brick to glusterd after which glusterd
will allocate a port from refreshed free ports list, but I feel that's kind of
too much of automation and also a hacky way.


Version-Release number of selected component (if applicable):
mainline

Comment 1 Vijay Bellur 2016-06-08 10:55:30 UTC
REVIEW: http://review.gluster.org/14670 (port-map: let brick choose its own port) posted (#1) for review on master by Prasanna Kumar Kalever (pkalever)

Comment 2 Vijay Bellur 2016-06-08 12:02:17 UTC
REVIEW: http://review.gluster.org/14670 (port-map: let brick choose its own port) posted (#2) for review on master by Prasanna Kumar Kalever (pkalever)

Comment 3 Vijay Bellur 2016-07-18 07:56:30 UTC
REVIEW: http://review.gluster.org/14670 (port-map: let brick choose its own port) posted (#3) for review on master by Prasanna Kumar Kalever (pkalever)

Comment 4 Vijay Bellur 2016-07-19 11:38:35 UTC
REVIEW: http://review.gluster.org/14670 (port-map: let brick choose its own port) posted (#4) for review on master by Prasanna Kumar Kalever (pkalever)

Comment 5 Worker Ant 2016-08-24 11:05:24 UTC
REVIEW: http://review.gluster.org/14670 (port-map: let brick choose its own port) posted (#5) for review on master by Prasanna Kumar Kalever (pkalever)

Comment 6 Worker Ant 2016-08-30 11:16:56 UTC
REVIEW: http://review.gluster.org/14670 (port-map: let brick choose its own port) posted (#6) for review on master by Prasanna Kumar Kalever (pkalever)

Comment 7 Worker Ant 2018-09-19 11:24:21 UTC
REVIEW: https://review.gluster.org/14670 (port-map: let brick choose its own port) posted (#7) for review on master by Amar Tumballi

Comment 8 Worker Ant 2018-09-23 18:36:32 UTC
REVIEW: https://review.gluster.org/21249 (rpc: make binding to port 0 as the default if no option is provided) posted (#1) for review on master by Amar Tumballi

Comment 9 Worker Ant 2018-10-02 11:08:07 UTC
COMMIT: https://review.gluster.org/21249 committed in master by "Raghavendra G" <rgowdapp> with a commit message- rpc: make binding to port 0 as the default if no option is provided

Right now, if no option is provided, the default port is assumed,
which is 24007. Ideally, for 'glusterfsd' processes, it is better
to not assume there are any ports given, so it can start listening
on any port which is available.

This helps us to cleanup the dependencies on glusterd from glusterfsd
at the moment. No changes would be done to glusterd code, but making
the right defaults helps to make glusterfsd more independent process
later.

NOTE: This patch is a reduced version of below set of patches:
 * https://review.gluster.org/14613/ &
 * https://review.gluster.org/14670/ &
 * https://review.gluster.org/14671/

Credits: Prasanna Kumar Kalever <pkalever>

updates: bz#1343926
Change-Id: Ib874e10505e7366dc56ba754458252b67052e653
Signed-off-by: Amar Tumballi <amarts>

Comment 10 Shyamsundar 2019-03-25 16:30:11 UTC
This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-6.0, please open a new bug report.

glusterfs-6.0 has been announced on the Gluster mailinglists [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution.

[1] https://lists.gluster.org/pipermail/announce/2019-March/000120.html
[2] https://www.gluster.org/pipermail/gluster-users/


Note You need to log in before you can comment on or make changes to this bug.