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

[1LP][RFR] Move distributed and replicated appliance configuration to fixtures #10000

Merged
merged 1 commit into from
Mar 25, 2020
Merged

[1LP][RFR] Move distributed and replicated appliance configuration to fixtures #10000

merged 1 commit into from
Mar 25, 2020

Conversation

tpapaioa
Copy link
Contributor

@tpapaioa tpapaioa commented Mar 19, 2020

The configuration of database replication (global/remote appliances) or distributed (primary/secondary) appliances in cfme/tests/distributed/test_appliance_replication.py has been moved to the replicated_appliances and distributed_appliances fixtures in cfme/fixtures/cli.py.

The existing fixture replicated_appliances_with_providers has uses the new fixture replicated_appliances, and the new fixtures replicated_appliances_preupdate and replicated_appliances_preupdate_with_providers have been added for use by test_appliance_console_restore_db_replicated.

{{ pytest: --long-running -k 'test_appliance_replicate_between_regions or test_external_database_appliance or test_appliance_console_restore_db_replicated or test_update_replicated_webui' cfme/tests/distributed/test_appliance_replication.py cfme/tests/cli/test_appliance_console_db_restore.py cfme/tests/cli/test_appliance_update.py -vv }}

@dajoRH
Copy link
Contributor

dajoRH commented Mar 19, 2020

I detected some fixture changes in commit 5b0c2c4

The global fixture distributed_appliances is used in the following files:

  • cfme/tests/distributed/test_appliance_replication.py
    • test_external_database_appliance
    • test_appliance_httpd_roles

The global fixture replicated_appliances is used in the following files:

  • cfme/tests/distributed/test_appliance_replication.py
    • test_appliance_replicate_between_regions
    • test_appliance_replicate_database_disconnection
    • test_appliance_replicate_database_disconnection_with_backlog
    • test_replication_connect_to_vm_in_region
    • test_appliance_replicate_zones
    • test_appliance_replicate_remote_down

The global fixture replicated_appliances_preupdate was changed, but I didn't find where it's used.
The global fixture replicated_appliances_with_providers is used in the following files:

  • cfme/tests/cli/test_appliance_console_db_restore.py

The global fixture replicated_appliances_preupdate_with_providers is used in the following files:

  • cfme/tests/cli/test_appliance_update.py
    • test_update_replicated_webui

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

@tpapaioa tpapaioa changed the title [WIPTEST] Move distributed and replicated appliance configuration to fixtures [RFR] Move distributed and replicated appliance configuration to fixtures Mar 23, 2020
Copy link
Contributor

@john-dupuy john-dupuy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM to me and PRT is green 👍 Congrats on the 10000th PR!!!

@john-dupuy john-dupuy changed the title [RFR] Move distributed and replicated appliance configuration to fixtures [1LP][RFR] Move distributed and replicated appliance configuration to fixtures Mar 24, 2020
@digitronik digitronik self-assigned this Mar 25, 2020
@digitronik
Copy link
Contributor

Nice PR @tpapaioa LGTM 👍

@digitronik digitronik merged commit 371ddd8 into ManageIQ:master Mar 25, 2020
@tpapaioa tpapaioa deleted the appliance_replication_cleanup branch March 26, 2020 23:46
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.

5 participants