[pve-devel] [RFC v3 pve-ha-manager 0/8] Do not freeze services on shutdown

Dietmar Maurer dietmar at proxmox.com
Wed Dec 16 09:04:42 CET 2015


This is a rework, trying to simplify code.

Changes since v2:
* simplify code
* always stop VM at node shutdown (no freeze)
* freeze state is only used for lrm service restart
* shutdown_all_remaining_services() is not included - I will send a
  modified version later
* added my own test (tests from v2 are not included)

Dietmar Maurer (7):
  TestHardware: add 'reboot' and 'shutdown' command
  lrm: implement different behavior for shutdown and restart
  rename is_poweroff to is_node_shutdown
  lrm: only freeze services if lrm is in 'restart' mode
  correctly reset node status on popwer on/off
  new test for node shutdown
  implement restart-lrm test

Thomas Lamprecht (1):
  HA Env: add 'is_poweroff' function

 src/PVE/HA/Env.pm                         |  8 +++++
 src/PVE/HA/Env/PVE2.pm                    | 17 ++++++++++
 src/PVE/HA/LRM.pm                         | 32 +++++++++++++-----
 src/PVE/HA/Manager.pm                     | 12 +++----
 src/PVE/HA/Sim/Env.pm                     |  5 +++
 src/PVE/HA/Sim/RTHardware.pm              |  3 +-
 src/PVE/HA/Sim/TestEnv.pm                 | 11 ++++++
 src/PVE/HA/Sim/TestHardware.pm            | 56 +++++++++++++++++++++++++++----
 src/test/test-basic5/log.expect           |  2 ++
 src/test/test-reboot1/cmdlist             |  4 +++
 src/test/test-reboot1/hardware_status     |  5 +++
 src/test/test-reboot1/log.expect          | 35 +++++++++++++++++++
 src/test/test-reboot1/manager_status      |  1 +
 src/test/test-reboot1/service_config      |  3 ++
 src/test/test-restart-lrm/cmdlist         |  4 +++
 src/test/test-restart-lrm/hardware_status |  5 +++
 src/test/test-restart-lrm/log.expect      | 30 +++++++++++++++++
 src/test/test-restart-lrm/manager_status  |  1 +
 src/test/test-restart-lrm/service_config  |  3 ++
 src/test/test-shutdown1/cmdlist           |  4 +++
 src/test/test-shutdown1/hardware_status   |  5 +++
 src/test/test-shutdown1/log.expect        | 40 ++++++++++++++++++++++
 src/test/test-shutdown1/manager_status    |  1 +
 src/test/test-shutdown1/service_config    |  3 ++
 24 files changed, 268 insertions(+), 22 deletions(-)
 create mode 100644 src/test/test-reboot1/cmdlist
 create mode 100644 src/test/test-reboot1/hardware_status
 create mode 100644 src/test/test-reboot1/log.expect
 create mode 100644 src/test/test-reboot1/manager_status
 create mode 100644 src/test/test-reboot1/service_config
 create mode 100644 src/test/test-restart-lrm/cmdlist
 create mode 100644 src/test/test-restart-lrm/hardware_status
 create mode 100644 src/test/test-restart-lrm/log.expect
 create mode 100644 src/test/test-restart-lrm/manager_status
 create mode 100644 src/test/test-restart-lrm/service_config
 create mode 100644 src/test/test-shutdown1/cmdlist
 create mode 100644 src/test/test-shutdown1/hardware_status
 create mode 100644 src/test/test-shutdown1/log.expect
 create mode 100644 src/test/test-shutdown1/manager_status
 create mode 100644 src/test/test-shutdown1/service_config

-- 
2.1.4




More information about the pve-devel mailing list