proxmox-mirrors/pve-manager
1
0
Fork 0
mirror of https://git.proxmox.com/git/pve-manager synced 2025-08-06 22:42:30 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

api/services: whitespace cleanup, modern closure call

Browse Source 
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
This commit is contained in:
Thomas Lamprecht 2020-07-24 13:47:39 +02:00
parent e51cd02b5c
commit 05d5632db2
1 changed files with 19 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

43
PVE/API2/Services.pm
View File

@ -17,7 +17,7 @@ use IO::File;
use base qw(PVE::RESTHandler); use base qw(PVE::RESTHandler);
my $service_name_list = [ my $service_name_list = [
'pveproxy', 'pveproxy',
'pvedaemon', 'pvedaemon',
'spiceproxy', 'spiceproxy',
'pvestatd', 'pvestatd',
@ -33,7 +33,7 @@ my $service_name_list = [
'postfix', 'postfix',
'ksmtuned', 'ksmtuned',
'systemd-timesyncd', 'systemd-timesyncd',
]; ];
# since postfix package 3.1.0-3.1 the postfix unit is only here to # since postfix package 3.1.0-3.1 the postfix unit is only here to
# manage subinstances, of which the default is called "-". # manage subinstances, of which the default is called "-".
@ -46,7 +46,7 @@ my $get_full_service_state = sub {
my ($service) = @_; my ($service) = @_;
$service = $unit_extra_names->{$service} if $unit_extra_names->{$service}; $service = $unit_extra_names->{$service} if $unit_extra_names->{$service};
my $res; my $res;
my $parser = sub { my $parser = sub {
my $line = shift; my $line = shift;
if ($line =~ m/^([^=\s]+)=(.*)$/) { if ($line =~ m/^([^=\s]+)=(.*)$/) {
@ -54,8 +54,8 @@ my $get_full_service_state = sub {
} }
}; };
PVE::Tools::run_command(['systemctl', 'show', $service], outfunc => $parser); PVE::Tools::run_command(['systemctl', 'show', $service], outfunc => $parser);
return $res; return $res;
}; };
@ -64,7 +64,7 @@ my $static_service_list;
sub get_service_list { sub get_service_list {
return $static_service_list if $static_service_list; return $static_service_list if $static_service_list;
my $list = {}; my $list = {};
foreach my $name (@$service_name_list) { foreach my $name (@$service_name_list) {
my $ss; my $ss;
@ -76,7 +76,7 @@ sub get_service_list {
} }
$static_service_list = $list; $static_service_list = $list;
return $static_service_list; return $static_service_list;
} }
@ -269,7 +269,7 @@ __PACKAGE__->register_method ({
syslog('info', "starting service $param->{service}: $upid\n"); syslog('info', "starting service $param->{service}: $upid\n");
&$service_cmd($param->{service}, 'start'); $service_cmd->($param->{service}, 'start');
}; };
@ -308,7 +308,7 @@ __PACKAGE__->register_method ({
syslog('info', "stopping service $param->{service}: $upid\n"); syslog('info', "stopping service $param->{service}: $upid\n");
&$service_cmd($param->{service}, 'stop'); $service_cmd->($param->{service}, 'stop');
}; };
@ -316,8 +316,8 @@ __PACKAGE__->register_method ({
}}); }});
__PACKAGE__->register_method ({ __PACKAGE__->register_method ({
name => 'service_restart', name => 'service_restart',
path => '{service}/restart', path => '{service}/restart',
method => 'POST', method => 'POST',
description => "Restart service.", description => "Restart service.",
permissions => { permissions => {
@ -326,7 +326,7 @@ __PACKAGE__->register_method ({
proxyto => 'node', proxyto => 'node',
protected => 1, protected => 1,
parameters => { parameters => {
additionalProperties => 0, additionalProperties => 0,
properties => { properties => {
node => get_standard_option('pve-node'), node => get_standard_option('pve-node'),
service => $service_prop_desc, service => $service_prop_desc,
@ -337,26 +337,23 @@ __PACKAGE__->register_method ({
}, },
code => sub { code => sub {
my ($param) = @_; my ($param) = @_;
my $rpcenv = PVE::RPCEnvironment::get();
my $rpcenv = PVE::RPCEnvironment::get();
my $user = $rpcenv->get_user(); my $user = $rpcenv->get_user();
my $realcmd = sub { my $realcmd = sub {
my $upid = shift; my $upid = shift;
syslog('info', "re-starting service $param->{service}: $upid\n"); syslog('info', "re-starting service $param->{service}: $upid\n");
&$service_cmd($param->{service}, 'restart'); $service_cmd->($param->{service}, 'restart');
}; };
return $rpcenv->fork_worker('srvrestart', $param->{service}, $user, $realcmd); return $rpcenv->fork_worker('srvrestart', $param->{service}, $user, $realcmd);
}}); }});
__PACKAGE__->register_method ({ __PACKAGE__->register_method ({
name => 'service_reload', name => 'service_reload',
path => '{service}/reload', path => '{service}/reload',
method => 'POST', method => 'POST',
description => "Reload service.", description => "Reload service.",
permissions => { permissions => {
@ -365,7 +362,7 @@ __PACKAGE__->register_method ({
proxyto => 'node', proxyto => 'node',
protected => 1, protected => 1,
parameters => { parameters => {
additionalProperties => 0, additionalProperties => 0,
properties => { properties => {
node => get_standard_option('pve-node'), node => get_standard_option('pve-node'),
service => $service_prop_desc, service => $service_prop_desc,
@ -376,17 +373,15 @@ __PACKAGE__->register_method ({
}, },
code => sub { code => sub {
my ($param) = @_; my ($param) = @_;
my $rpcenv = PVE::RPCEnvironment::get();
my $rpcenv = PVE::RPCEnvironment::get();
my $user = $rpcenv->get_user(); my $user = $rpcenv->get_user();
my $realcmd = sub { my $realcmd = sub {
my $upid = shift; my $upid = shift;
syslog('info', "reloading service $param->{service}: $upid\n"); syslog('info', "reloading service $param->{service}: $upid\n");
&$service_cmd($param->{service}, 'reload'); $service_cmd->($param->{service}, 'reload');
}; };