Alon Bar-Lev has posted comments on this change.

Change subject: host-deploy: use cpio instead of tar for bundle transfer
......................................................................


Patch Set 1: (4 inline comments)

....................................................
File 
backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/archivers/CachedCpio.java
Line 13: /**
Line 14:  * Handles cache tar file based on directory.
Line 15:  *
Line 16:  * Cache tar based on directory structure. If files are changed
Line 17:  * recreate tar file. Test file change once per interval.
Done
Line 18:  */
Line 19: public class CachedCpio {
Line 20: 
Line 21:     private static final Log log = LogFactory.getLog(CachedCpio.class);


Line 110:     }
Line 111: 
Line 112:     /**
Line 113:      * Constructor.
Line 114:      * @param archive name of tar to cache.
Done
Line 115:      * @param dir base directory.
Line 116:      */
Line 117:     public CachedCpio(File archive, File dir) {
Line 118:         _archive = archive;


....................................................
File 
backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/archivers/Cpio.java
Line 79:      *  @param os output stream to write into.
Line 80:      *  @param base base directory.
Line 81:      *
Line 82:      *  Only regular files and directories are supported.
Line 83:      *  Files will be owner rw and optional execute bit.
Done
Line 84:      */
Line 85:     public static void doArchive(
Line 86:         OutputStream os,
Line 87:         File base


....................................................
File 
backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/archivers/CpioTest.java
Line 63:         File tmpDir1 = null;
Line 64:         File tmpDir2 = null;
Line 65: 
Line 66:         try {
Line 67:             tmpCpio = File.createTempFile("test1", "tar");
Done
Line 68:             tmpDir1 = File.createTempFile("test1", "tmp");
Line 69:             tmpDir1.delete();
Line 70:             tmpDir1.mkdir();
Line 71:             tmpDir2 = File.createTempFile("test1", "tmp");


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I1386e7d688a9ec7e28519fb407478fd17cbab4ca
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to