From: Stephen Mulcahy <stephen.mulcahy@hp.com>
Date: Wed, 27 Feb 2013 09:15:57 +0000 (+0000)
Subject: Changed to INFO level logging for main cinder-backup operations
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=2664ed38a3dbf54e468ec8a462765bfc63ab2889;p=openstack-build%2Fcinder-build.git

Changed to INFO level logging for main cinder-backup operations

Changed create backup, restore backup and delete backup operations to
log key operations at INFO level instead of DEBUG level (so environments
like devstack see main operations by default).

Fixes bug: 1132788

Change-Id: I3b4600be8fbc1869054f81752c92936d55395185
---

diff --git a/cinder/backup/manager.py b/cinder/backup/manager.py
index 96426f525..11843cd5b 100755
--- a/cinder/backup/manager.py
+++ b/cinder/backup/manager.py
@@ -116,8 +116,8 @@ class BackupManager(manager.SchedulerDependentManager):
         backup = self.db.backup_get(context, backup_id)
         volume_id = backup['volume_id']
         volume = self.db.volume_get(context, volume_id)
-        LOG.debug(_('create_backup started, backup: %(backup_id)s for '
-                    'volume: %(volume_id)s') % locals())
+        LOG.info(_('create_backup started, backup: %(backup_id)s for '
+                   'volume: %(volume_id)s') % locals())
         self.db.backup_update(context, backup_id, {'host': self.host,
                                                    'service':
                                                    FLAGS.backup_service})
@@ -157,14 +157,14 @@ class BackupManager(manager.SchedulerDependentManager):
                                                    'size': volume['size'],
                                                    'availability_zone':
                                                    self.az})
-        LOG.debug(_('create_backup finished. backup: %s'), backup_id)
+        LOG.info(_('create_backup finished. backup: %s'), backup_id)
 
     def restore_backup(self, context, backup_id, volume_id):
         """
         Restore volume backups from configured backup service.
         """
-        LOG.debug(_('restore_backup started, restoring backup: %(backup_id)s'
-                    ' to volume: %(volume_id)s') % locals())
+        LOG.info(_('restore_backup started, restoring backup: %(backup_id)s'
+                   ' to volume: %(volume_id)s') % locals())
         backup = self.db.backup_get(context, backup_id)
         volume = self.db.volume_get(context, volume_id)
         self.db.backup_update(context, backup_id, {'host': self.host})
@@ -217,15 +217,15 @@ class BackupManager(manager.SchedulerDependentManager):
 
         self.db.volume_update(context, volume_id, {'status': 'available'})
         self.db.backup_update(context, backup_id, {'status': 'available'})
-        LOG.debug(_('restore_backup finished, backup: %(backup_id)s restored'
-                    ' to volume: %(volume_id)s') % locals())
+        LOG.info(_('restore_backup finished, backup: %(backup_id)s restored'
+                   ' to volume: %(volume_id)s') % locals())
 
     def delete_backup(self, context, backup_id):
         """
         Delete volume backup from configured backup service.
         """
         backup = self.db.backup_get(context, backup_id)
-        LOG.debug(_('delete_backup started, backup: %s'), backup_id)
+        LOG.info(_('delete_backup started, backup: %s'), backup_id)
         self.db.backup_update(context, backup_id, {'host': self.host})
 
         expected_status = 'deleting'
@@ -258,4 +258,4 @@ class BackupManager(manager.SchedulerDependentManager):
 
         context = context.elevated()
         self.db.backup_destroy(context, backup_id)
-        LOG.debug(_('delete_backup finished, backup %s deleted'), backup_id)
+        LOG.info(_('delete_backup finished, backup %s deleted'), backup_id)