Change in vdsm[master]: Deprecated string.join() replaced
danken at redhat.com
danken at redhat.com
Tue Aug 26 09:18:39 UTC 2014
Dan Kenigsberg has posted comments on this change.
Change subject: Deprecated string.join() replaced
......................................................................
Patch Set 1: Code-Review-1
(1 comment)
http://gerrit.ovirt.org/#/c/31897/1/vds_bootstrap/setup
File vds_bootstrap/setup:
Line 188: else:
Line 189: execfn += [random_num]
Line 190: execfn.append(str(int(rebootAfterInstallation)))
Line 191: logging.debug("trying to run %s script cmd = '%s'",
Line 192: script, execfn.join(" "))
" ".join(execfn)
Line 193: proc = subprocess.Popen(execfn)
Line 194: proc.communicate()
Line 195: if proc.returncode == 0:
Line 196: return True
--
To view, visit http://gerrit.ovirt.org/31897
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d16939a5d1d4518d1d91a525d8b403b3be12787
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
More information about the vdsm-patches
mailing list