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

[hadong0720 at gmail.com]

hadong has uploaded a new change for review.

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

ipmi: Add OpenIPMI Requires and start ipmi service

Change-Id: Ide755987aea32f082369037e7f036eb2db6fd375
Signed-off-by: hadong <hadong0720 at gmail.com>
---
M ovirt-node.spec.in
M src/ovirt/node/setup/ipmi/ipmi_page.py
2 files changed, 3 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-node refs/changes/29/20129/1

diff --git a/ovirt-node.spec.in b/ovirt-node.spec.in
index a259744..7ab1167 100644
--- a/ovirt-node.spec.in
+++ b/ovirt-node.spec.in
@@ -130,6 +130,7 @@
 Summary:        Ipmi plugin for %{product_family} image
 Group:          Applications/System
 Requires:       ipmitool
+Requires:       OpenIPMI
 
 %description plugin-ipmi
 This package provides a ipmi plugin for use with %{product_family} image.
diff --git a/src/ovirt/node/setup/ipmi/ipmi_page.py b/src/ovirt/node/setup/ipmi/ipmi_page.py
index 10b9616..54c4394 100644
--- a/src/ovirt/node/setup/ipmi/ipmi_page.py
+++ b/src/ovirt/node/setup/ipmi/ipmi_page.py
@@ -19,7 +19,7 @@
 # MA  02110-1301, USA.  A copy of the GNU General Public License is
 # also available at http://www.gnu.org/copyleft/gpl.html.
 from ovirt.node import base, plugins, ui
-from ovirt.node.utils import process
+from ovirt.node.utils import process, system
 from subprocess import CalledProcessError
 """
 IPMI Status
@@ -83,6 +83,7 @@
 
     def check_status(self):
         try:
+            system.service("ipmi", "start")
             process.check_call(["ipmitool", "-I", "open", "chassis", "status"])
             return True
         except CalledProcessError:


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ide755987aea32f082369037e7f036eb2db6fd375
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: hadong <hadong0720 at gmail.com>