[system-tests] fuel-devops 3.0 on revert-resume does not start slaves, if it running before revert

[Aleksei Stepanov]

[system-tests] fuel-devops 3.0 on revert-resume does not start slaves, if it running before revert

Example:
(devops_vevns)ubuntu@pumphouse-hw:~/astepanov/fuel-qa$ virsh list --all
 Id Name State
----------------------------------------------------
 299 astepanov_admin running
 308 astepanov_slave-01 running
 309 astepanov_slave-02 running
 310 astepanov_slave-03 running
 311 astepanov_slave-04 running
 312 astepanov_slave-05 running
 313 astepanov_slave-06 running
 314 astepanov_slave-07 running
 315 astepanov_slave-08 running
 316 astepanov_slave-09 running

(devops_vevns)ubuntu@pumphouse-hw:~/astepanov/fuel-qa$ dos.py snapshot astepanov upgrade_first_cic

(devops_vevns)ubuntu@pumphouse-hw:~/astepanov/fuel-qa$ dos.py revert-resume astepanov upgrade_first_cic
Time synchronization is starting
Current time on 'admin' = Fri Jun 10 08:01:35 UTC 2016
Please wait for a few minutes while time is synchronized...
New time on 'admin' = Fri Jun 10 08:19:43 UTC 2016
(devops_vevns)ubuntu@pumphouse-hw:~/astepanov/fuel-qa$ virsh list --all
 Id Name State
----------------------------------------------------
 320 astepanov_admin running
 - astepanov_slave-01 shut off
 - astepanov_slave-02 shut off
 - astepanov_slave-03 shut off
 - astepanov_slave-04 shut off
 - astepanov_slave-05 shut off
 - astepanov_slave-06 shut off
 - astepanov_slave-07 shut off
 - astepanov_slave-08 shut off
 - astepanov_slave-09 shut off

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/328250
Committed: https://git.openstack.org/cgit/openstack/fuel-devops/commit/?id=d72d5856445e9359b6095b3353fd8781e83af870
Submitter: Jenkins
Branch: master

commit d72d5856445e9359b6095b3353fd8781e83af870
Author: Anton Studenov <email address hidden>
Date: Fri Jun 10 14:25:13 2016 +0300

    Fix libvirt node revert and resume

    - is_active check removed from resume
    - destroy() is not required in case of basic snapshot
    - is_active return valuet converted to bool

    Change-Id: I0e220de8665ae3b2a6a2d0b576e1a761d8c232d7
    Closes-Bug: #1591094