use PVE::ProcFSTools;
use PVE::RPCEnvironment;
use PVE::JSONSchema qw(get_standard_option);
+use PVE::Exception qw(raise_param_exc);
use PVE::AccessControl;
use base qw(PVE::RESTHandler);
__PACKAGE__->register_method({
- name => 'node_tasks',
- path => '',
+ name => 'node_tasks',
+ path => '',
method => 'GET',
- permissions => {
+ permissions => {
description => "List task associated with the current user, or all task the user has 'Sys.Audit' permissions on /nodes/<node> (the <node> the task runs on).",
- user => 'all'
+ user => 'all'
},
description => "Read task list for one node (finished tasks).",
proxyto => 'node',
start => {
type => 'integer',
minimum => 0,
+ default => 0,
optional => 1,
+ description => "List tasks beginning from this offset.",
},
limit => {
type => 'integer',
minimum => 0,
+ default => 50,
optional => 1,
+ description => "Only list this amount of tasks.",
},
userfilter => {
type => 'string',
optional => 1,
+ description => "Only list tasks from this user.",
},
+ typefilter => {
+ type => 'string',
+ optional => 1,
+ description => 'Only list tasks of this type (e.g., vzstart, vzdump).',
+ },
+ vmid => get_standard_option('pve-vmid', {
+ description => "Only list tasks for this VM.",
+ optional => 1,
+ }),
errors => {
type => 'boolean',
+ default => 0,
optional => 1,
},
+ source => {
+ type => 'string',
+ enum => ['archive', 'active', 'all'],
+ default => 'archive',
+ optional => 1,
+ description => 'List archived, active or all tasks.',
+ },
},
},
returns => {
items => {
type => "object",
properties => {
- upid => { type => 'string' },
+ upid => { type => 'string', title => 'UPID', },
+ node => { type => 'string', title => 'Node', },
+ pid => { type => 'integer', title => 'PID', },
+ pstart => { type => 'integer', },
+ starttime => { type => 'integer', title => 'Starttime', },
+ type => { type => 'string', title => 'Type', },
+ id => { type => 'string', title => 'ID', },
+ user => { type => 'string', title => 'User', },
+ endtime => { type => 'integer', optional => 1, title => 'Endtime', },
+ status => { type => 'string', optional => 1, title => 'Status', },
},
},
links => [ { rel => 'child', href => "{upid}" } ],
my $filename = "/var/log/pve/tasks/index";
my $node = $param->{node};
- my $start = $param->{start} || 0;
- my $limit = $param->{limit} || 50;
+ my $start = $param->{start} // 0;
+ my $limit = $param->{limit} // 50;
my $userfilter = $param->{userfilter};
- my $errors = $param->{errors};
+ my $typefilter = $param->{typefilter};
+ my $errors = $param->{errors} // 0;
+ my $source = $param->{source} // 'archive';
my $count = 0;
my $line;
my $auditor = $rpcenv->check($user, "/nodes/$node", [ 'Sys.Audit' ], 1);
+ my $filter_task = sub {
+ my $task = shift;
+
+ return 1 if $userfilter && $task->{user} !~ m/\Q$userfilter\E/i;
+ return 1 if !($auditor || $user eq $task->{user});
+
+ return 1 if $typefilter && $task->{type} ne $typefilter;
+
+ return 1 if $errors && $task->{status} && $task->{status} eq 'OK';
+ return 1 if $param->{vmid} && (!$task->{id} || $task->{id} ne $param->{vmid});
+
+ return 1 if $count++ < $start;
+ return 1 if $limit <= 0;
+
+ return 0;
+ };
+
my $parse_line = sub {
if ($line =~ m/^(\S+)(\s([0-9A-Za-z]{8})(\s(\S.*))?)?$/) {
my $upid = $1;
my $endtime = $3;
my $status = $5;
if ((my $task = PVE::Tools::upid_decode($upid, 1))) {
- return if $userfilter && $task->{user} !~ m/\Q$userfilter\E/i;
- next if !($auditor || $user eq $task->{user});
-
- return if $errors && $status && $status eq 'OK';
-
- return if $count++ < $start;
- return if $limit <= 0;
$task->{upid} = $upid;
$task->{endtime} = hex($endtime) if $endtime;
$task->{status} = $status if $status;
- push @$res, $task;
- $limit--;
+
+ if (!$filter_task->($task)) {
+ push @$res, $task;
+ $limit--;
+ }
}
}
};
- if (my $bw = File::ReadBackwards->new($filename)) {
- while (defined ($line = $bw->readline)) {
- &$parse_line();
+ if ($source eq 'active' || $source eq 'all') {
+ my $recent_tasks = PVE::INotify::read_file('active');
+ for my $task (@$recent_tasks) {
+ next if $task->{saved}; # archived task, already in index(.1)
+ if (!$filter_task->($task)) {
+ $task->{status} = 'RUNNING' if !$task->{status}; # otherwise it would be archived
+ push @$res, $task;
+ $limit--;
+ }
}
- $bw->close();
}
- if (my $bw = File::ReadBackwards->new("$filename.1")) {
- while (defined ($line = $bw->readline)) {
- &$parse_line();
+
+ if ($source ne 'active') {
+ if (my $bw = File::ReadBackwards->new($filename)) {
+ while (defined ($line = $bw->readline)) {
+ &$parse_line();
+ }
+ $bw->close();
+ }
+ if (my $bw = File::ReadBackwards->new("$filename.1")) {
+ while (defined ($line = $bw->readline)) {
+ &$parse_line();
+ }
+ $bw->close();
}
- $bw->close();
}
$rpcenv->set_result_attrib('total', $count);
}});
__PACKAGE__->register_method({
- name => 'upid_index',
- path => '{upid}',
+ name => 'upid_index',
+ path => '{upid}',
method => 'GET',
description => '', # index helper
permissions => { user => 'all' },
}});
__PACKAGE__->register_method({
- name => 'stop_task',
- path => '{upid}',
+ name => 'stop_task',
+ path => '{upid}',
method => 'DELETE',
description => 'Stop a task.',
- permissions => {
+ permissions => {
description => "The user needs 'Sys.Modify' permissions on '/nodes/<node>' if the task does not belong to him.",
- user => 'all',
+ user => 'all',
},
protected => 1,
proxyto => 'node',
$rpcenv->check($user, "/nodes/$node", [ 'Sys.Modify' ]);
}
- PVE::RPCEnvironment::check_worker($param->{upid}, 1);
+ PVE::RPCEnvironment->check_worker($param->{upid}, 1);
return undef;
}});
__PACKAGE__->register_method({
- name => 'read_task_log',
- path => '{upid}/log',
+ name => 'read_task_log',
+ path => '{upid}/log',
method => 'GET',
- permissions => {
+ permissions => {
description => "The user needs 'Sys.Audit' permissions on '/nodes/<node>' if the task does not belong to him.",
user => 'all',
},
start => {
type => 'integer',
minimum => 0,
+ default => 0,
optional => 1,
},
limit => {
type => 'integer',
minimum => 0,
+ default => 50,
optional => 1,
},
},
},
returns => {
type => 'array',
- items => {
+ items => {
type => "object",
properties => {
n => {
my ($task, $filename) = PVE::Tools::upid_decode($param->{upid}, 1);
raise_param_exc({ upid => "unable to parse worker upid" }) if !$task;
- my $lines = [];
-
my $rpcenv = PVE::RPCEnvironment::get();
my $user = $rpcenv->get_user();
my $node = $param->{node};
+ my $start = $param->{start} // 0;
+ my $limit = $param->{limit} // 50;
if ($user ne $task->{user}) {
$rpcenv->check($user, "/nodes/$node", [ 'Sys.Audit' ]);
}
- my $fh = IO::File->new($filename, "r");
- raise_param_exc({ upid => "no such task - unable to open file - $!" }) if !$fh;
-
- my $start = $param->{start} || 0;
- my $limit = $param->{limit} || 50;
- my $count = 0;
- my $line;
- while (defined ($line = <$fh>)) {
- next if $count++ < $start;
- next if $limit <= 0;
- chomp $line;
- push @$lines, { n => $count, t => $line};
- $limit--;
- }
-
- close($fh);
-
- # HACK: ExtJS store.guaranteeRange() does not like empty array
- # so we add a line
- if (!$count) {
- $count++;
- push @$lines, { n => $count, t => "no content"};
- }
+ my ($count, $lines) = PVE::Tools::dump_logfile($filename, $start, $limit);
$rpcenv->set_result_attrib('total', $count);
-
+
return $lines;
}});
+
+my $exit_status_cache = {};
+
__PACKAGE__->register_method({
- name => 'read_task_status',
- path => '{upid}/status',
+ name => 'read_task_status',
+ path => '{upid}/status',
method => 'GET',
- permissions => {
+ permissions => {
description => "The user needs 'Sys.Audit' permissions on '/nodes/<node>' if the task does not belong to him.",
user => 'all',
},
returns => {
type => "object",
properties => {
- pid => {
+ pid => {
type => 'integer'
},
- status => {
- type => 'string', enum => ['running', 'stopped'],
+ status => {
+ type => 'string', enum => ['running', 'stopped'],
},
},
},
$task->{upid} = $param->{upid}; # include upid
+ if ($task->{status} eq 'stopped') {
+ if (!defined($exit_status_cache->{$task->{upid}})) {
+ $exit_status_cache->{$task->{upid}} =
+ PVE::Tools::upid_read_status($task->{upid});
+ }
+ $task->{exitstatus} = $exit_status_cache->{$task->{upid}};
+ }
+
return $task;
}});
+
+1;