cloudstack-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From wid...@apache.org
Subject [1/2] git commit: updated refs/heads/master to 34caa69
Date Tue, 15 Sep 2015 08:57:41 GMT
Repository: cloudstack
Updated Branches:
  refs/heads/master 2bd61c35b -> 34caa694d


CLOUDSTACK-8645: Improve logging of RBD functionality in KVM

A simple commit which changes a couple of log lines.


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/4f409a7c
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/4f409a7c
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/4f409a7c

Branch: refs/heads/master
Commit: 4f409a7c173d2a2dd0e400dcd1ba426a1af66425
Parents: d091b91
Author: Wido den Hollander <wido@widodh.nl>
Authored: Mon Sep 14 13:50:53 2015 +0200
Committer: Wido den Hollander <wido@widodh.nl>
Committed: Tue Sep 15 10:10:04 2015 +0200

----------------------------------------------------------------------
 .../kvm/storage/LibvirtStorageAdaptor.java          | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/4f409a7c/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/LibvirtStorageAdaptor.java
----------------------------------------------------------------------
diff --git a/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/LibvirtStorageAdaptor.java
b/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/LibvirtStorageAdaptor.java
index 6ef5b24..f656982 100644
--- a/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/LibvirtStorageAdaptor.java
+++ b/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/LibvirtStorageAdaptor.java
@@ -273,7 +273,7 @@ public class LibvirtStorageAdaptor implements StorageAdaptor {
                         s.undefine();
                         s.free();
                     } catch (LibvirtException l) {
-                        s_logger.debug("Failed to undefine the libvirt secret: " + l.toString());
+                        s_logger.error("Failed to undefine the libvirt secret: " + l.toString());
                     }
                 }
                 return null;
@@ -288,7 +288,7 @@ public class LibvirtStorageAdaptor implements StorageAdaptor {
             sp = conn.storagePoolCreateXML(spd.toString(), 0);
             return sp;
         } catch (LibvirtException e) {
-            s_logger.debug("Failed to create RBD storage pool: " + e.toString());
+            s_logger.error("Failed to create RBD storage pool: " + e.toString());
             if (sp != null) {
                 try {
                     if (sp.isPersistent() == 1) {
@@ -299,17 +299,17 @@ public class LibvirtStorageAdaptor implements StorageAdaptor {
                     }
                     sp.free();
                 } catch (LibvirtException l) {
-                    s_logger.debug("Failed to undefine RBD storage pool: " + l.toString());
+                    s_logger.error("Failed to undefine RBD storage pool: " + l.toString());
                 }
             }
 
             if (s != null) {
                 try {
-                    s_logger.debug("Failed to create the RBD storage pool, cleaning up the
libvirt secret");
+                    s_logger.error("Failed to create the RBD storage pool, cleaning up the
libvirt secret");
                     s.undefine();
                     s.free();
                 } catch (LibvirtException se) {
-                    s_logger.debug("Failed to remove the libvirt secret: " + se.toString());
+                    s_logger.error("Failed to remove the libvirt secret: " + se.toString());
                 }
             }
 
@@ -615,7 +615,7 @@ public class LibvirtStorageAdaptor implements StorageAdaptor {
         try {
             s = conn.secretLookupByUUIDString(uuid);
         } catch (LibvirtException e) {
-            s_logger.debug("Storage pool " + uuid + " has no corresponding secret. Not removing
any secret.");
+            s_logger.info("Storage pool " + uuid + " has no corresponding secret. Not removing
any secret.");
         }
 
         try {
@@ -877,8 +877,8 @@ public class LibvirtStorageAdaptor implements StorageAdaptor {
                 rbd.close(image);
                 r.ioCtxDestroy(io);
 
-                s_logger.debug("Succesfully unprotected and removed any snapshots of " +
pool.getSourceDir() + "/" + uuid +
-                        " Continuing to remove the RBD image");
+                s_logger.info("Succesfully unprotected and removed any remaining snapshots
(" + snaps.size() + ") of "
+                              + pool.getSourceDir() + "/" + uuid + " Continuing to remove
the RBD image");
             } catch (RadosException e) {
                 throw new CloudRuntimeException(e.toString());
             } catch (RbdException e) {


Mime
View raw message