Alon Bar-Lev has uploaded a new change for review.

Change subject: host-deploy: use tar instead of python tarfile
......................................................................

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 <alo...@redhat.com>
---
M 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/76/11776/1

diff --git 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
index ed38129..64a191d 100644
--- 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
+++ 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
@@ -1155,7 +1155,7 @@
         "trap \"chmod -R u+rwX \\\"${MYTMP}\\\" > /dev/null 2>&1; rm -fr 
\\\"${MYTMP}\\\" > /dev/null 2>&1\" 0; " +
         "rm -fr \"${MYTMP}\" && " +
         "mkdir \"${MYTMP}\" && " +
-        "python -c \"import sys, tarfile; tarfile.open(fileobj=sys.stdin, 
mode='r|').extractall(path='${MYTMP}')\" && " +
+        "tar -C \"${MYTMP}\" -x && " +
         "@ENVIRONMENT@ \"${MYTMP}\"/setup DIALOG/dialect=str:machine 
DIALOG/customization=bool:True"
     )
     BootstrapCommand(373),


--
To view, visit http://gerrit.ovirt.org/11776
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9ece009af7d0c1800eb69b1ca99930892903fe9f
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alo...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to