[PATCH qemu-server 6/8] block job: monitor: rename 'op' argument to 'operation'

[Fiona Ebner <f.ebner@proxmox.com>]

Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
---
 src/PVE/QemuServer/BlockJob.pm | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/PVE/QemuServer/BlockJob.pm b/src/PVE/QemuServer/BlockJob.pm
index 3b105c37..d4998370 100644
--- a/src/PVE/QemuServer/BlockJob.pm
+++ b/src/PVE/QemuServer/BlockJob.pm
@@ -114,7 +114,7 @@ my sub print_job_status {
 # 'skip': wait until all jobs are ready, return with block jobs in ready state
 # 'auto': wait until all jobs disappear, only use for jobs which complete automatically
 sub monitor {
-    my ($qmp_peer, $vmiddst, $jobs, $completion, $qga, $op) = @_;
+    my ($qmp_peer, $vmiddst, $jobs, $completion, $qga, $operation) = @_;
 
     die "drive mirror: different destination is only supported when peer is main QEMU instance\n"
         if $vmiddst && $qmp_peer->{type} ne 'qmp';
@@ -127,14 +127,14 @@ sub monitor {
 
         my $starttime = time();
         while (1) {
-            die "block job ('$op') timed out\n" if $err_complete > 300;
+            die "block job ('$operation') timed out\n" if $err_complete > 300;
 
             my $stats = qmp_cmd($qmp_peer, "query-block-jobs");
             my $ctime = time();
 
             my $running_jobs = {};
             for my $stat (@$stats) {
-                next if $stat->{type} ne $op;
+                next if $stat->{type} ne $operation;
                 $running_jobs->{ $stat->{device} } = $stat;
             }
 
@@ -147,12 +147,12 @@ sub monitor {
                 my $vanished = !defined($job);
                 my $complete = defined($jobs->{$job_id}->{complete}) && $vanished;
                 if ($complete || ($vanished && $completion eq 'auto')) {
-                    print "$job_id: $op-job finished\n";
+                    print "${job_id}: ${operation}-job finished\n";
                     delete $jobs->{$job_id};
                     next;
                 }
 
-                die "$job_id: '$op' has been cancelled\n" if !defined($job);
+                die "$job_id: '$operation' has been cancelled\n" if !defined($job);
 
                 if ($job && $job->{status} eq 'concluded') {
                     qemu_handle_concluded_blockjob($qmp_peer, $job_id, $job, $jobs->{$job_id});
@@ -172,7 +172,7 @@ sub monitor {
                 $wait_time = 100_000; # everything is ready, query more often
 
                 if (!$ready_message_printed) {
-                    print "all '$op' jobs are ready\n";
+                    print "all '$operation' jobs are ready\n";
                     $ready_message_printed = 1;
                 }
 
@@ -262,7 +262,7 @@ sub monitor {
 
     if ($err) {
         eval { qemu_blockjobs_cancel($qmp_peer, $jobs) };
-        die "block job ($op) error: $err";
+        die "block job ($operation) error: $err";
     }
 }
 
-- 
2.47.3