c5b0b3
From: Cole Robinson <crobinso@redhat.com>
c5b0b3
Date: Mon, 6 Apr 2015 13:59:46 -0400
c5b0b3
Subject: [PATCH] virsh: Improve change-media success message
c5b0b3
c5b0b3
$ sudo virsh change-media f19 hdc /mnt/data/devel/media/Fedora-16-x86_64-Live-KDE.iso
c5b0b3
succeeded to complete action update on media
c5b0b3
c5b0b3
Change the message to:
c5b0b3
c5b0b3
  Successfully {inserted,ejected,changed} media.
c5b0b3
c5b0b3
https://bugzilla.redhat.com/show_bug.cgi?id=967946
c5b0b3
(cherry picked from commit e3aa4c91c8b54cdfb1c312a142fd9fb79daec65a)
c5b0b3
---
c5b0b3
 tools/virsh-domain.c | 6 +++++-
c5b0b3
 1 file changed, 5 insertions(+), 1 deletion(-)
c5b0b3
c5b0b3
diff --git a/tools/virsh-domain.c b/tools/virsh-domain.c
c5b0b3
index 9bbb964..a735649 100644
c5b0b3
--- a/tools/virsh-domain.c
c5b0b3
+++ b/tools/virsh-domain.c
c5b0b3
@@ -12295,6 +12295,7 @@ cmdChangeMedia(vshControl *ctl, const vshCmd *cmd)
c5b0b3
     bool ret = false;
c5b0b3
     vshUpdateDiskXMLType update_type;
c5b0b3
     const char *action = NULL;
c5b0b3
+    const char *success_msg = NULL;
c5b0b3
     bool config = vshCommandOptBool(cmd, "config");
c5b0b3
     bool live = vshCommandOptBool(cmd, "live");
c5b0b3
     bool current = vshCommandOptBool(cmd, "current");
c5b0b3
@@ -12314,16 +12315,19 @@ cmdChangeMedia(vshControl *ctl, const vshCmd *cmd)
c5b0b3
     if (eject) {
c5b0b3
         update_type = VSH_UPDATE_DISK_XML_EJECT;
c5b0b3
         action = "eject";
c5b0b3
+        success_msg = _("Successfully ejected media.");
c5b0b3
     }
c5b0b3
 
c5b0b3
     if (insert) {
c5b0b3
         update_type = VSH_UPDATE_DISK_XML_INSERT;
c5b0b3
         action = "insert";
c5b0b3
+        success_msg = _("Successfully inserted media.");
c5b0b3
     }
c5b0b3
 
c5b0b3
     if (update || (!eject && !insert)) {
c5b0b3
         update_type = VSH_UPDATE_DISK_XML_UPDATE;
c5b0b3
         action = "update";
c5b0b3
+        success_msg = _("Successfully updated media.");
c5b0b3
     }
c5b0b3
 
c5b0b3
     VSH_EXCLUSIVE_OPTIONS_VAR(current, live);
c5b0b3
@@ -12367,7 +12371,7 @@ cmdChangeMedia(vshControl *ctl, const vshCmd *cmd)
c5b0b3
             goto cleanup;
c5b0b3
         }
c5b0b3
 
c5b0b3
-        vshPrint(ctl, _("succeeded to complete action %s on media\n"), action);
c5b0b3
+        vshPrint(ctl, "%s", success_msg);
c5b0b3
     }
c5b0b3
 
c5b0b3
     ret = true;