Permalink
Browse files

Fix arguments to injectAsHead CLI callout

  • Loading branch information...
1 parent 0bed2f7 commit 890df914320805f079b43ee86f1dcbe62b7820d7 @NicolasT NicolasT committed Aug 27, 2012
Showing with 3 additions and 2 deletions.
  1. +3 −2 extension/server/ArakoonManagement.py
@@ -821,10 +821,11 @@ def injectAsHead(self, nodeName, newHead):
@return void
"""
self._requireLocal(nodeName)
- r = [self._binary,'--inject-as-head', nodeName, '-config',
+ r = [self._binary,'--inject-as-head', newHead, nodeName, '-config',
'%s/%s.cfg' % (self._clusterPath, self._clusterId) ]
#output = subprocess.check_output(r, shell= True) # starting from python 2.7
- p = subprocess.Popen(r, shell= True, stdout = subprocess.PIPE)
+ rs = ' '.join(r)
+ p = subprocess.Popen(rs, shell= True, stdout = subprocess.PIPE)
output = p.communicate()[0]
logging.debug("injectAsHead returned %s", output)
return

0 comments on commit 890df91

Please sign in to comment.