address reviews

Author: JoaoJandre

core/src/main/java/com/cloud/agent/api/storage/DeleteDiskOnlyVmSnapshotCommand.java

@@ -39,6 +39,6 @@ public List<DataTO> getSnapshots() {
 
     @Override
     public boolean executeInSequence() {
-        return true;
+        return false;
     }
 }

core/src/main/java/com/cloud/agent/api/storage/MergeDiskOnlyVmSnapshotCommand.java

@@ -49,7 +49,7 @@ public String getVmName() {
 
     @Override
     public boolean executeInSequence() {
-        return true;
+        return false;
     }
 
 }

core/src/main/java/com/cloud/agent/api/storage/RevertDiskOnlyVmSnapshotCommand.java

@@ -44,7 +44,7 @@ public String getVmName() {
 
     @Override
     public boolean executeInSequence() {
-        return true;
+        return false;
     }
 
 }

plugins/hypervisors/kvm/src/main/java/org/apache/cloudstack/utils/qemu/QemuImg.java

@@ -835,17 +835,17 @@ public void resize(final QemuImgFile file, final long size) throws QemuImgExcept
      *            The file to be commited.
      * @param base
      *            If base is not specified, the immediate backing file of the top image (which is {@code file}) will be used.
-     * @param deleteFile
-     *            If true, the commited file(s) will be deleted.
+     * @param skipEmptyingFiles
+     *            If true, the commited file(s) will not be emptied. If base is informed, skipEmptyingFiles is implied.
      */
-    public void commit( QemuImgFile file, QemuImgFile base, boolean deleteFile) throws QemuImgException {
+    public void commit(QemuImgFile file, QemuImgFile base, boolean skipEmptyingFiles) throws QemuImgException {
         if (file == null) {
             throw new QemuImgException("File should not be null");
         }
 
         final Script s = new Script(_qemuImgPath, timeout);
         s.add("commit");
-        if (deleteFile) {
+        if (skipEmptyingFiles) {
             s.add("-d");
         }