[pve-devel] [PATCH pve-ha-manager 0/7] V2 of service commands regression testing

Thomas Lamprecht t.lamprecht at proxmox.com
Mon Dec 21 16:38:01 CET 2015


Changes are:
* better spliced up and code reuse between RTHardware and TestHardware
* better order (at least in my opinion)
* add a test to show why the 'fix removing service from old node on migration'
  patch is needed. This test is in the last commit of this series and does not
  needs to be committed necessarily

Thomas Lamprecht (7):
  Sim/Hardware: move set_service_state to base class
  Sim/RTHardware: use set_service_state from baseclass
  add service disable/enable to regression tests
  Sim/Hardware: allow recursive locking in queue_crm_commands
  add service migrate/relocate to regression tests
  Sim/Env: fix removing service from old node on migration
  add test to for previous commit

 src/PVE/HA/Sim/Env.pm                          |  5 +--
 src/PVE/HA/Sim/Hardware.pm                     | 23 ++++++++++++--
 src/PVE/HA/Sim/RTHardware.pm                   |  8 +----
 src/PVE/HA/Sim/TestHardware.pm                 | 31 ++++++++++++++++---
 src/test/test-service-command1/README          |  2 ++
 src/test/test-service-command1/cmdlist         |  5 +++
 src/test/test-service-command1/hardware_status |  5 +++
 src/test/test-service-command1/log.expect      | 32 +++++++++++++++++++
 src/test/test-service-command1/manager_status  |  1 +
 src/test/test-service-command1/service_config  |  3 ++
 src/test/test-service-command2/README          |  1 +
 src/test/test-service-command2/cmdlist         |  4 +++
 src/test/test-service-command2/hardware_status |  5 +++
 src/test/test-service-command2/log.expect      | 34 ++++++++++++++++++++
 src/test/test-service-command2/manager_status  |  1 +
 src/test/test-service-command2/service_config  |  3 ++
 src/test/test-service-command3/README          |  1 +
 src/test/test-service-command3/cmdlist         |  5 +++
 src/test/test-service-command3/hardware_status |  5 +++
 src/test/test-service-command3/log.expect      | 43 ++++++++++++++++++++++++++
 src/test/test-service-command3/manager_status  |  1 +
 src/test/test-service-command3/service_config  |  3 ++
 22 files changed, 205 insertions(+), 16 deletions(-)
 create mode 100644 src/test/test-service-command1/README
 create mode 100644 src/test/test-service-command1/cmdlist
 create mode 100644 src/test/test-service-command1/hardware_status
 create mode 100644 src/test/test-service-command1/log.expect
 create mode 100644 src/test/test-service-command1/manager_status
 create mode 100644 src/test/test-service-command1/service_config
 create mode 100644 src/test/test-service-command2/README
 create mode 100644 src/test/test-service-command2/cmdlist
 create mode 100644 src/test/test-service-command2/hardware_status
 create mode 100644 src/test/test-service-command2/log.expect
 create mode 100644 src/test/test-service-command2/manager_status
 create mode 100644 src/test/test-service-command2/service_config
 create mode 100644 src/test/test-service-command3/README
 create mode 100644 src/test/test-service-command3/cmdlist
 create mode 100644 src/test/test-service-command3/hardware_status
 create mode 100644 src/test/test-service-command3/log.expect
 create mode 100644 src/test/test-service-command3/manager_status
 create mode 100644 src/test/test-service-command3/service_config

-- 
2.1.4





More information about the pve-devel mailing list