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

[1LP][RFR] Update set_conversion_host_api() to use CA certs#10047

Merged
jawatts merged 8 commits intoManageIQ:masterfrom
nachandr:update_set_conversion_host_api
May 4, 2020
Merged

[1LP][RFR] Update set_conversion_host_api() to use CA certs#10047
jawatts merged 8 commits intoManageIQ:masterfrom
nachandr:update_set_conversion_host_api

Conversation

@nachandr
Copy link
Contributor

@nachandr nachandr commented Apr 10, 2020

Purpose or Intent

PRT Run

{{ pytest: cfme/tests/v2v/test_v2v_smoke.py --use-template-cache --provider-limit 2 --use-provider rhv-ims --use-provider vsphere67-ims }}

PRT results:
510 - PASS
511 - PASS

Copy link
Contributor

@sshveta sshveta 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 Nandini . Let's run its against 5.11.4 and upstream (locally) and post results

Copy link
Contributor

Choose a reason for hiding this comment

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

Remove comment

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@nachandr nachandr changed the title [WIP] Update set_conversion_host_api() to use CA certs [WIPTEST] Update set_conversion_host_api() to use CA certs Apr 13, 2020
@dajoRH dajoRH added WIP-testing and removed WIP labels Apr 13, 2020
@mnadeem92
Copy link
Contributor

@nachandr as UCI is not coming with 5.11.5 release, can you update this PR only for VM refresh issue for upstream, we will enable UCI code only after UCI gets released.

@dajoRH dajoRH changed the title [WIPTEST] Update set_conversion_host_api() to use CA certs [WIP] Update set_conversion_host_api() to use CA certs Apr 21, 2020
@nachandr nachandr force-pushed the update_set_conversion_host_api branch from 7c56ac6 to 7303efd Compare April 24, 2020 18:31
@dajoRH dajoRH added lint-ok and removed needs-lint labels Apr 27, 2020
@nachandr nachandr changed the title [WIP] Update set_conversion_host_api() to use CA certs [RFR] Update set_conversion_host_api() to use CA certs Apr 29, 2020
@dajoRH dajoRH removed the WIP label Apr 29, 2020
@nachandr nachandr force-pushed the update_set_conversion_host_api branch from 7c0f87d to eb832c6 Compare April 29, 2020 23:14
@sshveta sshveta changed the title [RFR] Update set_conversion_host_api() to use CA certs [1LP][RFR] Update set_conversion_host_api() to use CA certs Apr 30, 2020
@jawatts jawatts merged commit fa06359 into ManageIQ:master May 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