[Bugs] [Bug 1382236] glusterfind pre session hangs indefinitely

[bugzilla at redhat.com]

https://bugzilla.redhat.com/show_bug.cgi?id=1382236



--- Comment #6 from Worker Ant <bugzilla-bot at gluster.org> ---
COMMIT: http://review.gluster.org/15609 committed in master by Aravinda VK
(avishwan at redhat.com) 
------
commit feea851fad4f89b48bfe89fe3b75250cc7bd6501
Author: Milind Changire <mchangir at redhat.com>
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()

    Change-Id: Ia8d96e2cd47bf2a64416bece312e67631a1dbf29
    BUG: 1382236
    Signed-off-by: Milind Changire <mchangir at redhat.com>
    Reviewed-on: http://review.gluster.org/15609
    Smoke: Gluster Build System <jenkins at build.gluster.org>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.org>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    Reviewed-by: Aravinda VK <avishwan at redhat.com>

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are on the CC list for the bug.