Ofer Schreiber has posted comments on this change.

Change subject: Add Fedora support to vdsm-bootstrap
......................................................................


Patch Set 1: (2 inline comments)

....................................................
File vds_bootstrap/vds_bootstrap_complete.py
Line 2: 
Sure....


Tue, 27 Dec 2011 14:30:52 ERROR    Traceback (most recent call last):
  File "/tmp/vds_installer_85fdd02e-b128-49bd-8f24-dd55bf0cf540.py", line 247, 
in runInstaller
    subprocess.Popen(execfn).communicate()
  File "/usr/lib64/python2.7/subprocess.py", line 679, in __init__
    errread, errwrite)
  File "/usr/lib64/python2.7/subprocess.py", line 1228, in _execute_child
    raise child_exception
OSError: [Errno 8] Exec format error

....................................................
File vdsm_reg/deployUtil.py.in
Line 86: MGT_BRIDGE_NAME = "engine"
Your patch is nice, but it will require more work from the engine side.

Lets make this work first, and change it to the right name afterwards.

--
To view, visit http://gerrit.ovirt.org/767
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I17e9fd3550c5db5865b88ef4cfc93953b99827b6
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ofer Schreiber <[email protected]>
Gerrit-Reviewer: Barak Azulay <[email protected]>
Gerrit-Reviewer: Douglas Schilling Landgraf <[email protected]>
Gerrit-Reviewer: Ofer Schreiber <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to