Stop and Delete operations should give the Guest a chance to shutdown
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack Compute (nova) |
Fix Released
|
Undecided
|
Phil Day | ||
nova (Ubuntu) |
Fix Released
|
Undecided
|
Liang Chen | ||
Trusty |
Fix Released
|
Undecided
|
Liang Chen |
Bug Description
This feature will cause an ACPI event to be sent to the system while shutting down, and the acpid running inside the system can catch the event, thus giving the system a chance to shutdown cleanly.
[Impact]
* VMs being shutdown with any signal/notification from the The hypervisor level, services running inside VMs have no chance to perform a clean shutoff
[Test Case]
* 1. stop a VM
2. the VM is shutdown without any notification
The can be easily seen by ssh into the system before shutting down. With the patch in place, the ssh session will be close during shutdown, because the sshd has the chance to close the connection before being brought down. Without the patch, the ssh session will just hang there for a while until timeout, because the connection is not promptly closed.
To leverage the clean shutdown feature, one can create a file named /etc/acpi/
Then create a file named /etc/acpi/power.sh that contains whatever required to gracefully shutdown a particular server (VM).
With the apicd running, shutdown of the VM will cause the rule in /etc/acpi/
[Regression Potential]
* none
Currently in libvirt stop and delete operations simply destroy the underlying VM. Some GuestOS's do not react well to this type of power failure, and it would be better if these operations followed the same approach a a soft_reboot and give the guest a chance to shutdown gracefully. Even where VM is being deleted, it may be booted from a volume which will be reused on another server.
Related branches
- James Page: Pending requested
-
Diff: 653 lines (+628/-0)4 files modifieddebian/changelog (+9/-0)
debian/patches/add-support-for-syslog-connect-retries.patch (+115/-0)
debian/patches/clean-shutdown.patch (+502/-0)
debian/patches/series (+2/-0)
Changed in nova: | |
assignee: | nobody → Phil Day (philip-day) |
Changed in nova: | |
milestone: | none → icehouse-2 |
Changed in nova: | |
status: | Fix Committed → Fix Released |
Changed in nova: | |
milestone: | icehouse-2 → icehouse-3 |
Changed in nova: | |
milestone: | icehouse-3 → icehouse-rc1 |
tags: | added: compute |
Changed in nova: | |
milestone: | icehouse-rc1 → next |
tags: | added: icehouse-rc-potential |
tags: |
added: icehouse-backport-potential removed: icehouse-rc-potential |
Changed in nova: | |
assignee: | Phil Day (philip-day) → Claudiu Belu (cbelu) |
Changed in nova: | |
milestone: | next → none |
status: | Fix Committed → Fix Released |
Changed in nova (Ubuntu): | |
assignee: | nobody → Liang Chen (cbjchen) |
description: | updated |
Changed in nova (Ubuntu): | |
status: | New → In Progress |
description: | updated |
Changed in nova (Ubuntu Trusty): | |
assignee: | nobody → Liang Chen (cbjchen) |
Changed in nova (Ubuntu Trusty): | |
status: | New → In Progress |
Changed in nova (Ubuntu): | |
status: | In Progress → Fix Released |
Fix proposed to branch: master /review. openstack. org/35303
Review: https:/