Bug 1539701 - gsyncd is running gluster command to get config file path is not required
Summary: gsyncd is running gluster command to get config file path is not required
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: geo-replication
Version: 4.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Aravinda VK
QA Contact:
URL:
Whiteboard:
Depends On: 1539545
Blocks:
TreeView+ depends on / blocked
 
Reported: 2018-01-29 13:32 UTC by Aravinda VK
Modified: 2018-03-15 11:26 UTC (History)
1 user (show)

Fixed In Version: glusterfs-4.0.0
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1539545
Environment:
Last Closed: 2018-03-15 11:26:04 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Aravinda VK 2018-01-29 13:32:33 UTC
+++ This bug was initially created as a clone of Bug #1539545 +++

Description of problem:

With the enhanced Config management(Issue https://github.com/gluster/glusterfs/issues/73), gsyncd should not run gluster command to get config path when spawned.

--- Additional comment from Worker Ant on 2018-01-29 02:33:45 EST ---

REVIEW: https://review.gluster.org/19359 (geo-rep: Remove unused working directory check in gsyncd) posted (#1) for review on master by Aravinda VK

--- Additional comment from Worker Ant on 2018-01-29 08:29:25 EST ---

COMMIT: https://review.gluster.org/19359 committed in master by $GERRIT_PATCHSET_UPLOADER with a commit message- geo-rep: Remove unused working directory check in gsyncd

To append the default config file path, gsyncd calls gluster
command to get the workdir path and constructs config file path.
This is not required now since the Config management in Geo-replication
is changed with patch 18257(Issue #73)

BUG: 1539545
Change-Id: Ia7eb39e36ed59ece4de65ea7ec71a0f615e338bb
Signed-off-by: Aravinda VK <avishwan>

Comment 1 Worker Ant 2018-01-29 13:34:27 UTC
REVIEW: https://review.gluster.org/19364 (geo-rep: Remove unused working directory check in gsyncd) posted (#1) for review on release-4.0 by Aravinda VK

Comment 2 Worker Ant 2018-01-30 18:36:40 UTC
COMMIT: https://review.gluster.org/19364 committed in release-4.0 by "Shyamsundar Ranganathan" <srangana> with a commit message- geo-rep: Remove unused working directory check in gsyncd

To append the default config file path, gsyncd calls gluster
command to get the workdir path and constructs config file path.
This is not required now since the Config management in Geo-replication
is changed with patch 18257(Issue #73)

BUG: 1539701
Change-Id: Ia7eb39e36ed59ece4de65ea7ec71a0f615e338bb
Signed-off-by: Aravinda VK <avishwan>
(cherry picked from commit 596143a286079338c30c8abc4b862f8d6ddddb3a)

Comment 3 Shyamsundar 2018-03-15 11:26:04 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-4.0.0, please open a new bug report.

glusterfs-4.0.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] http://lists.gluster.org/pipermail/announce/2018-March/000092.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.