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

Fix check if vm is ha managed in get_start_stop_list

Browse Source 
The check used was completely wrong and so the startall and stopall
API calls touched HA managed service, which they shouldn't.

Using the vm_is_ha_managed call from the HA stack fixes that.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
This commit is contained in:
Thomas Lamprecht 2015-10-19 18:02:47 +02:00 committed by Dietmar Maurer
parent 04b2004b36
commit 3625a09137
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
PVE/API2/Nodes.pm
View File

@ -1132,7 +1132,6 @@ __PACKAGE__->register_method({
my $get_start_stop_list = sub { my $get_start_stop_list = sub {
my ($nodename, $autostart) = @_; my ($nodename, $autostart) = @_;
my $haconf = PVE::HA::Config::read_resources_config();
my $vmlist = PVE::Cluster::get_vmlist(); my $vmlist = PVE::Cluster::get_vmlist();
my $resList = {}; my $resList = {};
@ -1164,8 +1163,8 @@ my $get_start_stop_list = sub {
} }
# skip ha managed VMs (started by pve-ha-manager) # skip ha managed VMs (started by pve-ha-manager)
return if defined($haconf->{ids}->{"pvevm:$vmid"}); return if PVE::HA::Config::vm_is_ha_managed($vmid);
$resList->{$startup->{order}}->{$vmid} = $startup; $resList->{$startup->{order}}->{$vmid} = $startup;
$resList->{$startup->{order}}->{$vmid}->{type} = $d->{type}; $resList->{$startup->{order}}->{$vmid}->{type} = $d->{type};
}; };