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

[1LP][RFR] Fixing multiple jenkins failures#10159

Merged
digitronik merged 1 commit intoManageIQ:masterfrom
mnadeem92:fix_plan_started_method
Jun 4, 2020
Merged

[1LP][RFR] Fixing multiple jenkins failures#10159
digitronik merged 1 commit intoManageIQ:masterfrom
mnadeem92:fix_plan_started_method

Conversation

@mnadeem92
Copy link
Contributor

@mnadeem92 mnadeem92 commented Jun 2, 2020

Signed-off-by: mnadeem92 mnadeem@redhat.com

In 5.11.6, some locators gets changed and migration In_progress is visible as list where as in earlier version it listed as progress-bar

Further, in earlier version in progress-bar, there was no spinner visible, however in 5.11.6, the spinner visible on list.

This PR fix those issue to resolve multiple TC failures.

{{ pytest: cfme/tests/v2v/test_post_migrations.py -k "test_migration_post_attribute" --use-provider osp13-ims --use-provider vsphere67-ims --provider-limit 2 -v }}

Signed-off-by: mnadeem92 <mnadeem@redhat.com>
@mnadeem92 mnadeem92 force-pushed the fix_plan_started_method branch from a830424 to 615591e Compare June 4, 2020 07:46
@mnadeem92 mnadeem92 changed the title [WIPTEST] Fixing multiple jenkins failures [RFR] Fixing multiple jenkins failures Jun 4, 2020
@dajoRH dajoRH removed the WIP-testing label Jun 4, 2020
Copy link
Member

@ganeshhubale ganeshhubale left a comment

Choose a reason for hiding this comment

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

LGTM :)

@ganeshhubale ganeshhubale changed the title [RFR] Fixing multiple jenkins failures [1LP][RFR] Fixing multiple jenkins failures Jun 4, 2020
Copy link
Contributor

@digitronik digitronik left a comment

Choose a reason for hiding this comment

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

Thanks for quick fix

@digitronik digitronik merged commit c5dbbf2 into ManageIQ:master Jun 4, 2020
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