Bug 1744606

Summary: V2V - Support local payload file download
Product: Red Hat CloudForms Management Engine Reporter: Adam Grare <agrare>
Component: Migration AnalyticsAssignee: Adam Grare <agrare>
Status: CLOSED CURRENTRELEASE QA Contact: Shveta <sshveta>
Severity: high Docs Contact: Red Hat CloudForms Documentation <cloudforms-docs>
Priority: high    
Version: 5.11.0CC: bthurber, dmetzger, fdupont, mturley, simaishi
Target Milestone: GA   
Target Release: 5.11.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 5.11.0.22 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2019-12-13 14:57:06 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: CFME Core Target Upstream Version:
Embargoed:
Attachments:
Description Flags
Payload can be copied none

Description Adam Grare 2019-08-22 14:11:15 UTC
Description of problem:
Update payload generation so that v2v migration_analytics can support local payload download by the user

Version-Release number of selected component (if applicable):
5.11

Comment 3 CFME Bot 2019-08-23 20:11:52 UTC
New commit detected on RedHatCloudForms/cfme-cloud_services/ivanchuk:

https://github.com/RedHatCloudForms/cfme-cloud_services/commit/c953eb00f194d1f43f8ebee8c12e75fb60f067e9
commit c953eb00f194d1f43f8ebee8c12e75fb60f067e9
Author:     Alberto Bellotti <abellotti.github.com>
AuthorDate: Fri Aug 16 09:26:59 2019 -0400
Commit:     Alberto Bellotti <abellotti.github.com>
CommitDate: Fri Aug 16 09:26:59 2019 -0400

    Merge pull request #41 from agrare/fix_file_size_issue

    Fix an integer overflow error for large files

    (cherry picked from commit bda65acc228b81d6286a8e67378900dda325575d)

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

 lib/cfme/cloud_services/data_packager.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


https://github.com/RedHatCloudForms/cfme-cloud_services/commit/8adfb97c7823e2e0de55861dfda8f19255146fff
commit 8adfb97c7823e2e0de55861dfda8f19255146fff
Author:     Alberto Bellotti <abellotti.github.com>
AuthorDate: Mon Aug 19 11:39:21 2019 -0400
Commit:     Alberto Bellotti <abellotti.github.com>
CommitDate: Mon Aug 19 11:39:21 2019 -0400

    Merge pull request #42 from agrare/allow_file_permissions_to_be_set

    Allow file permissions to be set by caller

    (cherry picked from commit 77b85429fb36cb8ea09e10948d6dc89e60df35df)

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

 lib/cfme/cloud_services/data_packager.rb | 8 +-
 lib/cfme/cloud_services/inventory_sync.rb | 10 +-
 2 files changed, 11 insertions(+), 7 deletions(-)


https://github.com/RedHatCloudForms/cfme-cloud_services/commit/a9f9f526edffda74f75adf1451bc25154fbb95a4
commit a9f9f526edffda74f75adf1451bc25154fbb95a4
Author:     Adam Grare <agrare>
AuthorDate: Thu Aug 22 10:03:05 2019 -0400
Commit:     Adam Grare <agrare>
CommitDate: Thu Aug 22 10:03:05 2019 -0400

    Merge pull request #43 from mzazrivec/add_ip_address_to_context_data_of_the_bundle_task_results

    Add ip address to context data

    (cherry picked from commit d83e00ef5022772c61666fad7dfaf2dda04113ae)

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

 lib/cfme/cloud_services/inventory_sync.rb | 1 +
 1 file changed, 1 insertion(+)

Comment 4 Shveta 2019-08-29 18:43:40 UTC
Created attachment 1609643 [details]
Payload can be copied

In 5.11.0.22.20190827200559_e618ece Payload can be copied by scp but not downloaded .

Will verify the BZ if this what is expected ?

Comment 5 Adam Grare 2019-08-29 19:09:09 UTC
Scp was chosen over a download link for simplicity and security

Comment 6 Brett Thurber 2019-08-29 19:15:44 UTC
(In reply to Shveta from comment #4)
> Created attachment 1609643 [details]
> Payload can be copied
> 
> In 5.11.0.22.20190827200559_e618ece Payload can be copied by scp but not
> downloaded .
> 
> Will verify the BZ if this what is expected ?

Working as designed.  We will revisit the download button in a future release.

Comment 7 Shveta 2019-09-05 04:23:10 UTC
@Brett : I think you moved it to ON_QA by mistake .