Merge pull request ManageIQ#17057 from agrare/remove_snapshot_logging

Enhance logging around remove snapshot operations
oslegl · Feb 27, 2018 · 071270a · 071270a
2 parents 533fb20 + a475155
commit 071270a
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/app/models/vm_or_template/operations/snapshot.rb b/app/models/vm_or_template/operations/snapshot.rb
@@ -63,6 +63,8 @@ def raw_remove_snapshot(snapshot_id)
     snapshot = snapshots.find_by(:id => snapshot_id)
     raise _("Requested VM snapshot not found, unable to remove snapshot") unless snapshot
     begin
+      _log.info("removing snapshot ID: [#{snapshot.id}] uid_ems: [#{snapshot.uid_ems}] ems_ref: [#{snapshot.ems_ref}] name: [#{snapshot.name}] description [#{snapshot.description}]")
+
       run_command_via_parent(:vm_remove_snapshot, :snMor => snapshot.uid_ems)
     rescue => err
       create_notification(:vm_snapshot_failure, :error => err.to_s, :snapshot_op => "remove")