[node-patches] Change in ovirt-node[master]: /dev/live doesn't exist on F17 or F18. Use /dev/initramfs/li...

rbarry at redhat.com rbarry at redhat.com
Fri Jun 28 14:11:46 UTC 2013


Ryan Barry has posted comments on this change.

Change subject: /dev/live doesn't exist on F17 or F18. Use /dev/initramfs/livedev
......................................................................


Patch Set 5: (1 inline comment)

....................................................
File src/ovirtnode/ovirtfunctions.py
Line 64: PRODUCT_RELEASE = aug.get("/files/etc/default/version/RELEASE")
Line 65: 
Line 66: DEV_LIVE = "/dev/{live}".format(live=process.check_output(
Line 67:                                 "/usr/libexec/ovirt-functions get_live_disk")
Line 68:                                 .strip())
Not calling ovirt-boot-functions, since I didn't want to also figure out where dracut scripts are living. /usr/libexec/ovirt-functions sources ovirt-boot-functions, and get_live_disk is available there instead, which saves some overhead in figuring it out.
Line 69: 
Line 70: OVIRT_VARS = {}
Line 71: # Parse all OVIRT_* variables
Line 72: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I6cee5e5dfe70f38f06bda24066c757872289430d
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Ryan Barry <rbarry at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Michael Burns <mburns at redhat.com>
Gerrit-Reviewer: Ryan Barry <rbarry at redhat.com>



More information about the node-patches mailing list