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

[1LP][RFR] Add uncollect for more than 2 hosts and RHEVM and SCVMM providers #9854

Merged
merged 1 commit into from
Jan 27, 2020

Conversation

prichard77
Copy link
Contributor

@prichard77 prichard77 commented Jan 19, 2020

Purpose or Intent

  • Updating tests so test_infrastructure_hosts_refresh_multi will not run with rhevm and scvmm providers for more than 2 hosts. These providers will not have the needed hosts and the tests are getting uploaded into Polarion and must be marked blocked, manually.

PRT Run

{{ pytest: --long-running cfme/tests/infrastructure/test_host.py::test_infrastructure_hosts_refresh_multi }}

@prichard77 prichard77 changed the title [WIP] Add uncollect for more than 2 hosts and RHEVM and SCVMM providers [WIPTEST] Add uncollect for more than 2 hosts and RHEVM and SCVMM providers Jan 19, 2020
@dajoRH
Copy link
Contributor

dajoRH commented Jan 19, 2020

I detected some fixture changes in commit aeadda4

The local fixture setup_provider_min_hosts is used in the following files:

  • cfme/tests/infrastructure/test_host.py
    • test_infrastructure_hosts_refresh_multi
    • test_compare_hosts_from_provider_allhosts
    • test_infrastructure_hosts_crud

Please, consider creating a PRT run to make sure your fixture changes do not break existing usage 😃

@dajoRH dajoRH added WIP-testing and removed WIP labels Jan 19, 2020
@prichard77 prichard77 changed the title [WIPTEST] Add uncollect for more than 2 hosts and RHEVM and SCVMM providers [WIP] Add uncollect for more than 2 hosts and RHEVM and SCVMM providers Jan 19, 2020
@dajoRH dajoRH added WIP and removed WIP-testing labels Jan 19, 2020
@prichard77 prichard77 changed the title [WIP] Add uncollect for more than 2 hosts and RHEVM and SCVMM providers [WIPTEST] Add uncollect for more than 2 hosts and RHEVM and SCVMM providers Jan 20, 2020
@dajoRH dajoRH added WIP-testing and removed WIP labels Jan 20, 2020
@prichard77
Copy link
Contributor Author

prichard77 commented Jan 20, 2020

PRT is showing as still running, but the report and artifacts(cfme.log) from the run say the tests all passed.

@prichard77 prichard77 changed the title [WIPTEST] Add uncollect for more than 2 hosts and RHEVM and SCVMM providers [RFR] Add uncollect for more than 2 hosts and RHEVM and SCVMM providers Jan 20, 2020
@ganeshhubale ganeshhubale changed the title [RFR] Add uncollect for more than 2 hosts and RHEVM and SCVMM providers [WIPTEST] Add uncollect for more than 2 hosts and RHEVM and SCVMM providers Jan 21, 2020
@ganeshhubale
Copy link
Member

Rerunning PRT...

@prichard77
Copy link
Contributor Author

The PRT tests are running and passing, but the run is showing as invalid. You can see from the report and artifacts in Ostriz that the tests passed.

@prichard77 prichard77 changed the title [WIPTEST] Add uncollect for more than 2 hosts and RHEVM and SCVMM providers [RFR] Add uncollect for more than 2 hosts and RHEVM and SCVMM providers Jan 22, 2020
@ganeshhubale
Copy link
Member

Test cases are passed on PRT. But it shows invalid. Need to go into report and check. With that result moving this PR to 1LP.

@ganeshhubale ganeshhubale changed the title [RFR] Add uncollect for more than 2 hosts and RHEVM and SCVMM providers [1LP][RFR] Add uncollect for more than 2 hosts and RHEVM and SCVMM providers Jan 23, 2020
@mshriver mshriver merged commit 72ea02e into ManageIQ:master Jan 27, 2020
@prichard77 prichard77 deleted the uncollectFour branch March 10, 2020 18:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants