if not os.path.isdir (path):
raise Exception
if not is_mounted(root_ssh):
- # xxx perform mount
- subprocess.call("mount --bind -o ro %s %s"%(root_ssh,sliver_ssh),shell=True)
- logger.log("_manage_ssh_dir: mounted %s into slice %s"%(root_ssh,slicename))
+ command=['mount','--bind','-o','ro',root_ssh,sliver_ssh]
+ mounted=logger.log_call (command)
+ msg="OK" if mounted else "WARNING: FAILED"
+ logger.log("_manage_ssh_dir: mounted %s into slice %s - %s"%(root_ssh,slicename,msg))
else:
- if is_mounted (root_ssh):
- # xxx perform umount
- subprocess.call("umount %s"%(root_ssh),shell=True)
- logger.log("_manage_ssh_dir: umounted %s"%(root_ssh))
+ if is_mounted (sliver_ssh):
+ command=['umount',sliver_ssh]
+ umounted=logger.log_call(command)
+ msg="OK" if umounted else "WARNING: FAILED"
+ logger.log("_manage_ssh_dir: umounted %s - %s"%(sliver_ssh,msg))
except:
- logger.log_exc("_manage_ssh_dir with slice %s failed"%slicename)
+ logger.log_exc("_manage_ssh_dir failed",name=slicename)
class Worker:
if not isinstance(self._acct, next_class): self._acct = next_class(rec)
logger.verbose("account.Worker.ensure_created: %s, running=%r"%(self.name,self.is_running()))
- # reservation_alive is set on reervable nodes, and its value is a boolean
+ # reservation_alive is set on reservable nodes, and its value is a boolean
if 'reservation_alive' in rec:
# reservable nodes
if rec['reservation_alive']: