This repository was archived by the owner on Apr 7, 2022. It is now read-only.
[1LP][RFR] Fixing conversion host timeout issue#10257
Merged
digitronik merged 1 commit intoManageIQ:masterfrom Jul 17, 2020
Merged
[1LP][RFR] Fixing conversion host timeout issue#10257digitronik merged 1 commit intoManageIQ:masterfrom
digitronik merged 1 commit intoManageIQ:masterfrom
Conversation
Signed-off-by: mnadeem92 <mnadeem@redhat.com>
mnadeem92
changed the title
valaparthvi
approved these changes
Jul 17, 2020
valaparthvi
changed the title
Contributor
|
Also, can you add the blocker to that test, in case you haven't yet? |
Contributor
Author
There is no need to add blocker as it hits all most all of the migration TC and the issue has been fixed and released shortly in the subversion of 5.11.7 |
digitronik
approved these changes
Jul 17, 2020
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Signed-off-by: mnadeem92 mnadeem@redhat.com
This PR will fix the timeout failure for conversion host configuration.
4 minutes is a very marginal limit for the time taken by the ansible-playbook to configure the conversion host. So I have increase the max limit to 10 minutes, though it keeps checking the status at every 3 seconds.
{{ pytest: cfme/tests/v2v/test_conversion_host_ui.py -k "test_add_conversion_host_ui_crud" --use-provider osp13-ims --use-provider vsphere67-ims --provider-limit 2 }}
Note: **Please ignore the PRT result, as in 5.11.7 we hit a major bug in configuring the conversion hosts:
https://bugzilla.redhat.com/show_bug.cgi?id=1857523
I have tested it locally by applying the workaround of above BZ and it is working fine, Also this PR won't change anything other than extending the time interval, So I guess it is safe to merge this PR.**