Comment 8 for bug 1783565

Reviewed: https://review.opendev.org/692252
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=1c93ca82b8fb8e39787020decfe335d45f8732e7
Submitter: Zuul
Branch: master

commit 1c93ca82b8fb8e39787020decfe335d45f8732e7
Author: melanie witt <email address hidden>
Date: Thu Oct 31 01:57:41 2019 +0000

    Replace time.sleep(10) with service forced_down in tests

    The server group functional tests are doing time.sleep(10) in order
    to make sure a stopped compute service is considered "down" by the nova
    compute API.

    Instead of sleeping, we can set the service as "forced_down" to get the
    desired "down" compute service status and avoid unnecessary delays in
    these tests.

    Unnecessary service start() calls are also removed in this change. They
    appear at the end of tests and services are started during each test
    setUp() and killed during each test tearDown() via the ServiceFixture.

    Closes-Bug: #1783565

    Change-Id: I74f64b68e4b33ee0f8c45fdc5f570c7e12e05d3b