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

[1LP][RFR] Fixing conversion host api for OSP #9977

Merged
merged 1 commit into from
Mar 12, 2020

Conversation

mnadeem92
Copy link
Contributor

@mnadeem92 mnadeem92 commented Mar 11, 2020

Signed-off-by: mnadeem92 [email protected]

Correcting conversion_entity value, the conversion_entity is always 'vms' for Openstack provider and it is true for all cfme version, however for RHV provider it is "hosts" till 5.11.4.X version and "vms" from 5.11.5 version.

At the moment the conversion_entity is set to "hosts" which breaks OSP related all tests, This PR will correct the same.

conversion_entity = (
        "hosts"
        if target_provider.one_of(RHEVMProvider) and appliance.version < '5.11.5'
        else "vms"
    )

{{ pytest: cfme/tests/v2v/test_conversion_host_ui.py -k "test_add_conversion_host_ui_crud" --use-template-cache --provider-limit 2 --use-provider osp13-ims --use-provider vsphere67-ims }}

The above PRT failing due to issue fixed in this #PR https://github.com/ManageIQ/integration_tests/pull/9979/files

@sshveta
Copy link
Contributor

sshveta commented Mar 11, 2020

PRT isn't running as the commit is not signed .

@nachandr
Copy link
Contributor

I ran this PR locally and the test passed for both RHV and OSP.

@mnadeem92 mnadeem92 force-pushed the fix_conversion_host_for_osp branch from 1ee1c2a to 07fc3b7 Compare March 12, 2020 06:58
@mnadeem92 mnadeem92 changed the title [WIPTEST] Fixing conversion host api for OSP [RFR] Fixing conversion host api for OSP Mar 12, 2020
Copy link
Contributor

@valaparthvi valaparthvi left a comment

Choose a reason for hiding this comment

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

Looking at the comments from other reviewers and PRT results, moving this 1LP.

@valaparthvi valaparthvi changed the title [RFR] Fixing conversion host api for OSP [1LP][RFR] Fixing conversion host api for OSP Mar 12, 2020
@jawatts jawatts merged commit bb6deb9 into ManageIQ:master Mar 12, 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.

6 participants