Skip to content
This repository has been archived by the owner on Apr 7, 2022. It is now read-only.

Commit

Permalink
Merge pull request #9838 from john-dupuy/actually-fix-hardware-reconf…
Browse files Browse the repository at this point in the history
…igured-test

[1LP][RFR] Actually fix hardware reconfigured alert test
  • Loading branch information
jawatts authored Jan 13, 2020
2 parents 73a425e + f0e18b8 commit 962d5a1
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions cfme/tests/control/test_alerts.py
Original file line number Diff line number Diff line change
Expand Up @@ -404,9 +404,11 @@ def test_alert_hardware_reconfigured(request, appliance, configure_fleecing, smt

for i in range(1, 3):
do_scan(vm, rediscover=False)
vm.reconfigure(
new_configuration={"cpu": True, "sockets": str(sockets_count + i), "disks": ()}
)
vm.reconfigure(changes={
"cpu": True, "sockets": str(sockets_count + i),
"disks": (),
"network_adapters": ()
})
service_request = appliance.collections.requests.instantiate(
description=service_request_desc.format(vm.name, sockets_count + i))
service_request.wait_for_request(method="ui", num_sec=300, delay=10)
Expand Down

0 comments on commit 962d5a1

Please sign in to comment.