Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1388731

Summary: [GSS]glusterfind pre session hangs indefinitely in RHGS 3.1.3
Product: [Community] GlusterFS Reporter: Milind Changire <mchangir>
Component: glusterfindAssignee: Milind Changire <mchangir>
Status: CLOSED CURRENTRELEASE QA Contact: bugs <bugs>
Severity: high Docs Contact:
Priority: unspecified    
Version: 3.9CC: amukherj, ashah, avishwan, bugs, khiremat, mmalhotr, rhs-bugs, rnalakka, storage-qa-internal
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: All   
Whiteboard:
Fixed In Version: glusterfs-3.9.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1379790 Environment:
Last Closed: 2016-12-06 05:59:27 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1379790    

Comment 1 Worker Ant 2016-10-26 03:51:03 UTC
REVIEW: http://review.gluster.org/15729 (tools/glusterfind: kill remote processes and separate run-time directories) posted (#1) for review on release-3.9 by Milind Changire (mchangir)

Comment 2 Worker Ant 2016-10-26 09:09:38 UTC
COMMIT: http://review.gluster.org/15729 committed in release-3.9 by Aravinda VK (avishwan) 
------
commit 915ae56a65d5a96bfddf977193dca60535ac7c11
Author: Milind Changire <mchangir>
Date:   Mon Oct 17 12:16:36 2016 +0530

    tools/glusterfind: kill remote processes and separate run-time directories
    
    Problem #1:
    Hitting CTRL+C leaves stale processes on remote nodes if glusterfind pre
    has been initiated.
    
    Solution #1:
    Adding "-t -t" to ssh command-line forces pseudo-terminal to be assigned
    to remote process. When local process receives Keyboard Interrupt,
    SIGHUP is immediately conveyed to the remote terminal causing remote
    changelog.py process to terminate immediately.
    
    Problem #2:
    Concurrent glusterfind pre runs are not possible on the same glusterfind
    session in case of a runaway process.
    
    Solution #2:
    glusterfind pre runs now add random directory name to the working
    directory to store and manage temporary database and changelog
    processing.
    If KeyboardInterrupt is received, the function call
    run_cmd_nodes("cleanup", args, tmpfilename=gtmpfilename)
    cleans up the remote run specific directory.
    
    Patch:
    7571380 cli/xml: Fix wrong XML format in volume get command
    broke "gluster volume get <vol> changelog.rollover-time --xml"
    Now fixed function utils.py::get_changelog_rollover_time()
    
    Fixed spurious trailing space getting written if second path is empty in
    main.py::write_output()
    Fixed repetitive changelog processing in changelog.py::get_changes()
    
    > Reviewed-on: http://review.gluster.org/15609
    > Smoke: Gluster Build System <jenkins.org>
    > CentOS-regression: Gluster Build System <jenkins.org>
    > NetBSD-regression: NetBSD Build System <jenkins.org>
    > Reviewed-by: Aravinda VK <avishwan>
    (cherry picked from commit feea851fad4f89b48bfe89fe3b75250cc7bd6501)
    
    
    Change-Id: Ia8d96e2cd47bf2a64416bece312e67631a1dbf29
    BUG: 1388731
    Signed-off-by: Milind Changire <mchangir>
    Reviewed-on: http://review.gluster.org/15729
    Smoke: Gluster Build System <jenkins.org>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    CentOS-regression: Gluster Build System <jenkins.org>
    Reviewed-by: Aravinda VK <avishwan>

Comment 3 Aravinda VK 2016-10-27 05:22:25 UTC
glusterfs-3.9.0rc2 is released[1] and packages are available for different distributions[2] to test.

[1] http://www.gluster.org/pipermail/maintainers/2016-October/001601.html
[2] http://www.gluster.org/pipermail/maintainers/2016-October/001605.html and http://www.gluster.org/pipermail/maintainers/2016-October/001606.html

Comment 4 Aravinda VK 2016-12-06 05:59:27 UTC
Gluster 3.9 GA is released http://blog.gluster.org/2016/11/announcing-gluster-3-9/