[GitHub] [cloudstack] GabrielBrascher commented on pull request #4978: KVM High Availability regardless of storage

[GitBox <gi...@apache.org>]

GabrielBrascher commented on pull request #4978:
URL: https://github.com/apache/cloudstack/pull/4978#issuecomment-911728760


   Thanks for the tests @DaanHoogland. I am checking on them.
   Also, I am constantly hammering the HA implementation on a staging CloudStack DC with Redfish OOBM + KVM nodes.
   
   This is interesting. A different result but also impacting the HA tests.
   
   #### Prior tests had:
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_hostha_enable_ha_when_host_disabled | `Error` | 1.11 | test_hostha_kvm.py
   test_hostha_enable_ha_when_host_in_maintenance | `Error` | 303.92 | test_hostha_kvm.py
   
   - case 1, disabled: it was on a "wrong" state (maintenance) which did not allow to disable.
   ` updatehost failed, due to: errorCode: 530, errorText:Failed to update host:2,No next resource state found for current state = Maintenance event = Disable`
   - case 2, Maintenance: These tests were mainly related to a host already in maintenance that could not be replaced as in maintenance.
   `Failed to prepare host for maintenance due to: Host is already in state Maintenance.`
   
   I assume that some of the tests in the CI put a host in maintenance and this was not cleaned in time for these 2 tests that needed a host not in Maintenance
   
   
   #### Recent CI tests
   For the new CI spin, the test failing is a different one (I still need to check the logs):
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_hostha_kvm_host_degraded | `Failure` | 768.71 | test_hostha_kvm.py


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org