ACPI is missing on fuel nodes

Bug #1470172 reported by Stepan Rogov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Medium
Bartłomiej Piotrowski
7.0.x
Won't Fix
Medium
Mykola Grygoriev
8.0.x
Fix Released
Medium
Bartłomiej Piotrowski

Bug Description

Many customers and peoples are running various Fuel nodes(i.e. controller) as virtual machines.
There are situations when you need to shutdown node gracefully without login on it - ACPI is better solution for it.
It also would be useful on baremetal nodes.

I propose to install acpid service on Fuel nodes(master and slaves) and add it to autostart.

Stepan Rogov (srogov)
description: updated
Changed in fuel:
milestone: none → 7.0
assignee: nobody → Fuel Library Team (fuel-library)
importance: Undecided → Medium
status: New → Confirmed
tags: added: low-hanging-fruit
Changed in fuel:
assignee: Fuel Library Team (fuel-library) → Mykola Grygoriev (mgrygoriev)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

Fix proposed to branch: master
Review: https://review.openstack.org/200083

Changed in fuel:
status: Confirmed → In Progress
Changed in fuel:
assignee: Mykola Grygoriev (mgrygoriev) → Sergii Golovatiuk (sgolovatiuk)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-library (master)

Change abandoned by Fuel DevOps Robot (<email address hidden>) on branch: master
Review: https://review.openstack.org/200083
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Revision history for this message
Matthew Mosesohn (raytrac3r) wrote :

Patch https://review.openstack.org/200083 needs to be rebased and committed for 8.0

tags: added: feature
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

Fix proposed to branch: master
Review: https://review.openstack.org/231350

Revision history for this message
Vladimir Kuklin (vkuklin) wrote :

I would say the following - enabled acpid is a problem when someone pushes the button on HW server. I myself prefer to disable it on hw servers in order to not suffer from someone pushing the button accidentally in the data center.

So I would do the following:

1) Introduce a configuration flag to enable acpid
2) Set it to enabled by default for VMs and disabled by default for HW servers
3) Introduce it in the UI and let user decide

Revision history for this message
Bartłomiej Piotrowski (bpiotrowski) wrote :

Current patchset enables acpid only if /proc/acpi/event exists and node is not a physical one.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/231350
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=be35055538478491f563408dc76203830225d229
Submitter: Jenkins
Branch: master

commit be35055538478491f563408dc76203830225d229
Author: Mykola Grygoriev <email address hidden>
Date: Thu Jul 9 15:22:45 2015 +0300

    Install and start acpid on virtual nodes

    Some users may be running various nodes (including master node since
    7.0) as virtual machines – acpid allows them to do graceful shutdown or
    reboot without using ssh.

    The commit introduces custom fact to check if ACPI is available via
    procfs on machine. acpid itself is only installed and started if node is
    not physical and has ACPI.

    Closes-Bug: 1470172
    Change-Id: I4186b4a2fbb57a0c63123377f986f888c27aa13e
    Co-authored-by: Bartłomiej Piotrowski <email address hidden>

Changed in fuel:
status: In Progress → Fix Committed
Dmitry Pyzhov (dpyzhov)
tags: added: area-library
Dmitry Pyzhov (dpyzhov)
Changed in fuel:
milestone: 7.0 → 8.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.