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