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

[1LP][RFR] Continue swapping EC2 with OpenStack and fix HTTP 503 issues.#10061

Merged
mshriver merged 6 commits intoManageIQ:masterfrom
jarovo:swap-the-ec2-with-other-cloud
May 6, 2020
Merged

[1LP][RFR] Continue swapping EC2 with OpenStack and fix HTTP 503 issues.#10061
mshriver merged 6 commits intoManageIQ:masterfrom
jarovo:swap-the-ec2-with-other-cloud

Conversation

@jarovo
Copy link
Contributor

@jarovo jarovo commented Apr 17, 2020

Another piece to saga about stabilization of the db restore tests.

  • There was a PR 9959, but more has to be done to see more tests running in release runs as EC2 is disabled on these.
  • Also, some tests were still quite flaky. The problem is that the API is probably handled by other process/thread then the web-ui and we may get HTTP 503 while the web UI might be already loaded and responding with HTTP 2xx.

I believe this PR, greatly reduces the problem by blocking until the API responds with some OK result. The API client object cached in the appliance had to be invalidated.

The errors currently on the PRT are not related. Related issue is an exception saying HTTP 503.

@jarovo jarovo changed the title Continue swapping EC2 with OpenStack [WIPTEST] Continue swapping EC2 with OpenStack Apr 17, 2020
Jaroslav Henner added 5 commits April 30, 2020 19:25
More methods are using EC2, which is disabled on the release
runs.
We were getting HTTP 503 sometimes due to API not yet started though
WebUI available.

This adds simple check using the requests library.
@jarovo jarovo changed the title [WIPTEST] Continue swapping EC2 with OpenStack [WIPTEST] Continue swapping EC2 with OpenStack and fix HTTP 503 issues. May 5, 2020
@jarovo jarovo changed the title [WIPTEST] Continue swapping EC2 with OpenStack and fix HTTP 503 issues. [RFR] Continue swapping EC2 with OpenStack and fix HTTP 503 issues. May 5, 2020
@dajoRH dajoRH removed the WIP-testing label May 5, 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, thanks for this PR 👍

try:
# There are 2 hard problems in computer science: cache
# invalidation, naming things, and off-by-1 errors.
# -- Leon Bambrick
Copy link
Contributor

Choose a reason for hiding this comment

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

😄

Copy link
Member

Choose a reason for hiding this comment

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

Hilarious, and great implementation of the wait/check pattern here @JaryN !

@john-dupuy john-dupuy changed the title [RFR] Continue swapping EC2 with OpenStack and fix HTTP 503 issues. [1LP][RFR] Continue swapping EC2 with OpenStack and fix HTTP 503 issues. May 5, 2020
@john-dupuy john-dupuy added fix-test test-cleanup Test removal, collection changes, re-organization labels May 5, 2020
@dajoRH
Copy link
Contributor

dajoRH commented May 5, 2020

I detected some fixture changes in commit f493032

The local fixture get_appliances_with_providers is used in the following files:

  • cfme/tests/cli/test_appliance_console_db_restore.py
    • test_appliance_console_dump_restore_db_local

The local fixture get_appliance_with_ansible is used in the following files:

  • cfme/tests/cli/test_appliance_console_db_restore.py
    • test_appliance_console_restore_pg_basebackup_ansible

The local fixture get_ext_appliances_with_providers is used in the following files:

  • cfme/tests/cli/test_appliance_console_db_restore.py
    • test_appliance_console_restore_db_external

The local fixture get_ha_appliances_with_providers is used in the following files:

  • cfme/tests/cli/test_appliance_console_db_restore.py

The local fixture two_appliances_one_with_providers is used in the following files:

  • cfme/tests/cli/test_appliance_console_db_restore.py
    • test_appliance_console_backup_restore_db_local
    • test_appliance_console_restore_db_nfs
    • test_appliance_console_restore_db_samba

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

@mshriver
Copy link
Member

mshriver commented May 6, 2020

I expect to see Leon Bambrick on all of our cache invalidation calls from now on :)

@mshriver mshriver merged commit a69c982 into ManageIQ:master May 6, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

fix-test lint-ok test-cleanup Test removal, collection changes, re-organization

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants