Bug 908309 - [host-deploy] use tar instead of python tarfile
Summary: [host-deploy] use tar instead of python tarfile
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Virtualization Manager
Classification: Red Hat
Component: ovirt-engine
Version: unspecified
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: 3.2.0
Assignee: Alon Bar-Lev
QA Contact: Tareq Alayan
URL:
Whiteboard: infra
Depends On:
Blocks: 917401
TreeView+ depends on / blocked
 
Reported: 2013-02-06 11:51 UTC by Alon Bar-Lev
Modified: 2016-02-10 19:06 UTC (History)
11 users (show)

Fixed In Version: sf9
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed:
oVirt Team: Infra
Target Upstream Version:
Embargoed:

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 11776 0 None None None Never

Description Alon Bar-Lev 2013-02-06 11:51:22 UTC 
commit 5a95730e64b8a9cae8932386d2023ac69bafa3bb
Author: Alon Bar-Lev <alonbl>
Date:   Wed Feb 6 01:35:26 2013 +0200

    host-deploy: use tar instead of python tarfile
    
    python usage was introduced when fedora-18 alpha lacked tar
    utility, now fedora-18 has tar utility and the python
    implementation is too buggy. It does not treat trailing
    zeros of tarfile correctly, causing the deployment to fail.
    
    Change-Id: I9ece009af7d0c1800eb69b1ca99930892903fe9f
    Signed-off-by: Alon Bar-Lev <alonbl>

http://gerrit.ovirt.org/#/c/11776/
Comment 1 Alon Bar-Lev 2013-02-06 11:53:26 UTC 
This was reported in the past by one of the users, the suspected component was commons-compress. It was reproduced at my environment using the lastest commons-compress, the conclusion is that the python tarfile implementation is invalid in some cases.

Please approve this for 3.2.

Thanks.
Comment 3 Tareq Alayan 2013-03-22 08:16:33 UTC 
verified.
couldn't reproduce.
Comment 4 Itamar Heim 2013-06-11 08:55:50 UTC 
3.2 has been released
Comment 5 Itamar Heim 2013-06-11 08:55:50 UTC 
3.2 has been released
Comment 6 Itamar Heim 2013-06-11 08:56:03 UTC 
3.2 has been released
Comment 7 Itamar Heim 2013-06-11 08:58:35 UTC 
3.2 has been released
Comment 8 Itamar Heim 2013-06-11 09:28:05 UTC 
3.2 has been released
Note You need to log in before you can comment on or make changes to this bug.