[PATCH] virsh: Fix job status indicator for 0 length block jobs

[Peter Krempa <pkrempa@xxxxxxxxxx>]

Although 0 length block jobs aren't entirely useful, the output of virsh
blockjob is empty due to the condition that suppresses the output for
migration jobs that did not start. Add a flag for virshPrintJobProgress
that specifies that the job was started so that the function can be used
both in migration where the job starts delayed and for block jobs where
we already know that the job was started.

Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1196711
---
 tools/virsh-domain.c | 24 +++++++++++++++++-------
 1 file changed, 17 insertions(+), 7 deletions(-)

diff --git a/tools/virsh-domain.c b/tools/virsh-domain.c
index b029b65..7837974 100644
--- a/tools/virsh-domain.c
+++ b/tools/virsh-domain.c
@@ -1683,13 +1683,23 @@ cmdBlkiotune(vshControl * ctl, const vshCmd * cmd)
 }


+/**
+ * virshPrintJobProgress:
+ * @label: name of the block job
+ * @remaining: amount of remaining work
+ * @total: amount of total work
+ * @started: The block job was started already
+ *
+ * Prints a human friendly percentage string decribing the state of the given
+ * job. The output is omitted if the job was not started.
+ */
 static void
 virshPrintJobProgress(const char *label, unsigned long long remaining,
-                      unsigned long long total)
+                      unsigned long long total, bool started)
 {
     int progress;

-    if (total == 0)
+    if (total == 0 && !started)
         /* migration has not been started */
         return;

@@ -1921,7 +1931,7 @@ virshBlockJobWait(virshBlockJobWaitDataPtr data)

         if (data->verbose)
             virshPrintJobProgress(data->job_name, info.end - info.cur,
-                                  info.end);
+                                  info.end, true);

         if (data->timeout && virTimeMillisNow(&curr) < 0) {
             vshSaveLibvirtError();
@@ -1949,7 +1959,7 @@ virshBlockJobWait(virshBlockJobWaitDataPtr data)
     if (data->verbose &&
         (ret == VIR_DOMAIN_BLOCK_JOB_COMPLETED ||
          ret == VIR_DOMAIN_BLOCK_JOB_READY))
-        virshPrintJobProgress(data->job_name, 0, 1);
+        virshPrintJobProgress(data->job_name, 0, 1, true);

     sigaction(SIGINT, &old_sig_action, NULL);
     return ret;
@@ -2621,7 +2631,7 @@ virshBlockJobInfo(vshControl *ctl,
                  info.bandwidth, info.cur, info.end);
     } else {
         virshPrintJobProgress(virshDomainBlockJobToString(info.type),
-                              info.end - info.cur, info.end);
+                              info.end - info.cur, info.end, true);
         if (speed) {
             const char *unit;
             double val = vshPrettyCapacity(speed, &unit);
@@ -4356,7 +4366,7 @@ virshWatchJob(vshControl *ctl,
                 retchar == '0') {
                 if (verbose) {
                     /* print [100 %] */
-                    virshPrintJobProgress(label, 0, 1);
+                    virshPrintJobProgress(label, 0, 1, true);
                 }
                 break;
             }
@@ -4392,7 +4402,7 @@ virshWatchJob(vshControl *ctl,
             if (ret == 0) {
                 if (verbose)
                     virshPrintJobProgress(label, jobinfo.dataRemaining,
-                                          jobinfo.dataTotal);
+                                          jobinfo.dataTotal, false);

                 if (!jobStarted &&
                     (jobinfo.type == VIR_DOMAIN_JOB_BOUNDED ||
-- 
2.4.5

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list