Bug 1702020

Summary: ConversionHost ansible_playbook method calls broken
Product: Red Hat CloudForms Management Engine Reporter: Satoe Imaishi <simaishi>
Component: V2VAssignee: Fabien Dupont <fdupont>
Status: CLOSED ERRATA QA Contact: Yadnyawalk Tale <ytale>
Severity: high Docs Contact: Red Hat CloudForms Documentation <cloudforms-docs>
Priority: high    
Version: 5.10.2CC: bthurber, dberger, mshriver, sshveta
Target Milestone: GAKeywords: ZStream
Target Release: 5.10.4   
Hardware: Unspecified   
OS: Unspecified   
URL: https://github.com/ManageIQ/manageiq/pull/18323
Whiteboard:
Fixed In Version: 5.10.4.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1695767 Environment:
Last Closed: 2019-05-21 05:46:54 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: V2V Target Upstream Version:
Embargoed:
Bug Depends On: 1695767    
Bug Blocks:    

Comment 2 CFME Bot 2019-04-24 19:48:44 UTC
New commit detected on ManageIQ/manageiq/hammer:

https://github.com/ManageIQ/manageiq/commit/3f30559bc27438bf078fbcab866bd5473f6bcee2
commit 3f30559bc27438bf078fbcab866bd5473f6bcee2
Author:     Adam Grare <agrare>
AuthorDate: Thu Jan 17 09:05:16 2019 -0500
Commit:     Adam Grare <agrare>
CommitDate: Thu Jan 17 09:05:16 2019 -0500

    Merge pull request #18323 from djberg96/conversion_host_ansible_playbook

    Alter ansible_playbook method so that some arguments are optional

    (cherry picked from commit 658581253e0b061eb3266e5d2030b6f391d08ea6)

    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1702020

 app/models/conversion_host.rb | 18 +-
 1 file changed, 9 insertions(+), 9 deletions(-)

Comment 3 Shveta 2019-05-03 20:35:46 UTC
Tried to call the method disable_conversion_host_role.

conversion_host = ConversionHost.find_by(:name => 'create converison host')
PostgreSQLAdapter#log_after_checkout, connection_pool: size: 5, connections: 1, in use: 1, waiting_in_queue: 0
=> #<ConversionHost id: 6, name: "create converison host", address: nil, type: nil, resource_type: "Host", resource_id: 3, version: nil, max_concurrent_tasks: nil, vddk_transport_supported: false, ssh_transport_supported: false, created_at: "2019-05-03 20:29:46", updated_at: "2019-05-03 20:29:46", concurrent_transformation_limit: nil, cpu_limit: nil, memory_limit: nil, network_limit: nil, blockio_limit: nil>

irb(main):002:0> conversion_host.disable_conversion_host_role
=> #<AwesomeSpawn::CommandResult:0x000000000bf69ae0 @exit_status=2>


Daniel , please if this is good enough to verify the call to method ansible_playbook.

Comment 4 Daniel Berger 2019-05-03 21:18:13 UTC
Well, it's getting called and returning successfully, so I call that a win.

Comment 5 Shveta 2019-05-03 21:18:50 UTC
Great , Thanks

Comment 7 errata-xmlrpc 2019-05-21 05:46:54 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2019:1247