summaryrefslogtreecommitdiffstats
path: root/geo-replication
diff options
context:
space:
mode:
authorVenky Shankar <vshankar@redhat.com>2014-01-20 19:38:44 +0530
committerVijay Bellur <vbellur@redhat.com>2014-02-06 01:58:28 -0800
commit692202e1ef7a5d009970e96e4933edb3231716b5 (patch)
tree0fc22e9b32d0a816c933b33824459b161cea2a6a /geo-replication
parenta2355deb62f647e257f3ffc8a33f9d831f211964 (diff)
gsyncd / geo-rep: cleanup the "tar" process
A missing cleanup for the "tar" process (when tar+ssh is used as the sync engine). Change-Id: Ib9599b43e7ec606c70b7c5598793417142be3c0b BUG: 1036539 Signed-off-by: Venky Shankar <vshankar@redhat.com> Reviewed-on: http://review.gluster.org/6794 Reviewed-on: http://review.gluster.org/6861 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'geo-replication')
-rw-r--r--geo-replication/syncdaemon/resource.py4
1 files changed, 4 insertions, 0 deletions
diff --git a/geo-replication/syncdaemon/resource.py b/geo-replication/syncdaemon/resource.py
index 32aa7d1c7e0..41add6fb287 100644
--- a/geo-replication/syncdaemon/resource.py
+++ b/geo-replication/syncdaemon/resource.py
@@ -791,7 +791,11 @@ class SlaveRemote(object):
p0.stdin.write(f)
p0.stdin.write('\n')
p0.stdin.close()
+
+ # wait() for tar to terminate, collecting any errors, further
+ # waiting for transfer to complete
p0.wait()
+ p0.terminate_geterr(fail_on_err = False)
p1.wait()
p1.terminate_geterr(fail_on_err = False)