[1LP][WIP] Fix preemtivity in test_gce_preemptible_provision. #7674
Conversation
jarovo
changed the title
| @@ -97,9 +98,10 @@ def test_provision_from_template(provider, provisioned_instance): | |||
| @pytest.mark.provider([GCEProvider], required_fields=[['provisioning', 'image']], | |||
| override=True) | |||
| @pytest.mark.usefixtures('setup_provider') | |||
| @pytest.mark.meta(blockers=[BZ(1619298)]) | |||
There was a problem hiding this comment.
This BZ is for 5.9 only
@pytest.mark.meta(blockers=[BZ(1619298, forced_streams=['5.9'])])
sshveta
changed the title
jarovo
changed the title
izapolsk
changed the title
|
Modified pytest statement in PR description, since it needs to collect against GCE. The K filter that was there to select the test wasn't working either. |
| @@ -97,9 +98,10 @@ def test_provision_from_template(provider, provisioned_instance): | |||
| @pytest.mark.provider([GCEProvider], required_fields=[['provisioning', 'image']], | |||
| override=True) | |||
| @pytest.mark.usefixtures('setup_provider') | |||
| @pytest.mark.meta(blockers=[BZ(1619298, forced_streams=['5.9', '5.10'])]) | |||
There was a problem hiding this comment.
This isn't blocking on 59 in PRT, and the test fails with GCE Instance isn't Preemptible (cfme/tests/cloud_infra_common/test_provisioning.py:110)
There was a problem hiding this comment.
The problem is caused by #7661
mshriver
changed the title
|
@JaryN would you still like to have this changed merged? Please close or rebase. Thanks! |
|
@mshriver I rebased this, but I am not sure do I want to work on getting this in. I didn't see the any GCE test runing for quite a while. |
|
Lint report for commit f51ed1f:
Please, rectify these issues 😏 . No commit flaws detected. CFME QE Bot |
Purpose or Intent
{{ pytest: -v cfme/tests/cloud_infra_common/test_provisioning.py::test_gce_preemptible_provision --use-provider gce_central }}