path: root/tools/glusterfind/src/
diff options
authorMilind Changire <>2016-10-17 12:16:36 +0530
committerAravinda VK <>2016-10-26 02:09:34 -0700
commit915ae56a65d5a96bfddf977193dca60535ac7c11 (patch)
treec7d60135c7f8ff9e252738332fc2d559fa7e759f /tools/glusterfind/src/
parenta55a780349615a0e5dc2d97fec61c143d5cd8256 (diff)
tools/glusterfind: kill remote processes and separate run-time directoriesv3.9.0rc2
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 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 Fixed spurious trailing space getting written if second path is empty in Fixed repetitive changelog processing in > Reviewed-on: > Smoke: Gluster Build System <> > CentOS-regression: Gluster Build System <> > NetBSD-regression: NetBSD Build System <> > Reviewed-by: Aravinda VK <> (cherry picked from commit feea851fad4f89b48bfe89fe3b75250cc7bd6501) Change-Id: Ia8d96e2cd47bf2a64416bece312e67631a1dbf29 BUG: 1388731 Signed-off-by: Milind Changire <> Reviewed-on: Smoke: Gluster Build System <> NetBSD-regression: NetBSD Build System <> CentOS-regression: Gluster Build System <> Reviewed-by: Aravinda VK <>
Diffstat (limited to 'tools/glusterfind/src/')
1 files changed, 3 insertions, 1 deletions
diff --git a/tools/glusterfind/src/ b/tools/glusterfind/src/
index f707449..07d8282 100644
--- a/tools/glusterfind/src/
+++ b/tools/glusterfind/src/
@@ -26,7 +26,8 @@ logger = logging.getLogger()
def mode_cleanup(args):
working_dir = os.path.join(conf.get_opt("working_dir"),
- args.volume)
+ args.volume,
+ args.tmpfilename)
mkdirp(os.path.join(conf.get_opt("log_dir"), args.session, args.volume),
@@ -98,6 +99,7 @@ def _get_args():
parser_cleanup = subparsers.add_parser('cleanup')
parser_cleanup.add_argument("session", help="Session Name")
parser_cleanup.add_argument("volume", help="Volume Name")
+ parser_cleanup.add_argument("tmpfilename", help="Temporary File Name")
parser_cleanup.add_argument("--debug", help="Debug", action="store_true")
parser_session_create = subparsers.add_parser('create')