[node-patches] Change in ovirt-node[master]: ipmi: Add OpenIPMI Requires and start ipmi service

[fabiand at fedoraproject.org]

Fabian Deutsch has posted comments on this change.

Change subject: ipmi: Add OpenIPMI Requires and start ipmi service
......................................................................


Patch Set 3:

(1 comment)

....................................................
File src/ovirt/node/setup/ipmi/ipmi_page.py
Line 82:         return self._call_ipmi(["-I", "open", "sdr", "elist", "full"])
Line 83: 
Line 84:     def check_status(self):
Line 85:         try:
Line 86:             system.service("ipmi", "start")
We should probably start ipmi during ovirt-init-functions.
Line 87:             process.check_call(["ipmitool", "-I", "open", "chassis", "status"])
Line 88:             return True
Line 89:         except CalledProcessError:


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ide755987aea32f082369037e7f036eb2db6fd375
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: hadong <hadong0720 at gmail.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Ryan Barry <rbarry at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes