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

[1LP][RFR]Add migration plan via API #9846

Merged
merged 2 commits into from
Jan 17, 2020

Conversation

sshveta
Copy link
Contributor

@sshveta sshveta commented Jan 15, 2020

{{ pytest: cfme/tests/v2v/test_rest_migrations.py --use-provider rhv43 --use-provider vsphere67-ims --provider-limit 2 -vvvv --long-running -k test_rest_plan_create }}

Test to create migration plan via API

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.

Test case looks good, just one required comment.

Comment on lines 247 to 252
assignee: sshveta
casecomponent: V2V
testtype: functional
initialEstimate: 1/8h
startsin: 5.9
tags: V2V
Copy link
Contributor

Choose a reason for hiding this comment

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

This needs to be indented

@john-dupuy john-dupuy changed the title [RFR]Add migration plan via API [1LP][WIPTEST]Add migration plan via API Jan 16, 2020
@sshveta sshveta changed the title [1LP][WIPTEST]Add migration plan via API [1LP][RFR]Add migration plan via API Jan 16, 2020
Copy link
Member

@mshriver mshriver left a comment

Choose a reason for hiding this comment

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

Looks good - but the PRT run 227 is a false positive.

Please add a pytest command to the PR description, specifying v2v_group providers.

@mshriver mshriver self-assigned this Jan 16, 2020
@mshriver mshriver changed the title [1LP][RFR]Add migration plan via API [1LP][WIPTEST]Add migration plan via API Jan 16, 2020
@sshveta sshveta changed the title [1LP][WIPTEST]Add migration plan via API [1LP][RFR]Add migration plan via API Jan 17, 2020
@jawatts jawatts merged commit 6c8cbe9 into ManageIQ:master Jan 17, 2020
spusateri pushed a commit to spusateri/integration_tests that referenced this pull request Jan 27, 2020
* Add migration plan via API

* Add migration plan via API
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