From 6677f6402ec334aa0ce1788301073567ebe41f2f Mon Sep 17 00:00:00 2001
From: Caspar Smit <casparsmit@supernas.eu>
Date: Fri, 18 Mar 2016 09:30:29 +0100
Subject: [PATCH] Skip templates when using the start all VMs button

Refactored and now using PVE::QemuConfig and PVE::LXC::Config
Moved the next if.. statements into the corresponding branches

Signed-off-by: Caspar Smit <casparsmit@supernas.eu>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
---
 PVE/API2/Nodes.pm | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/PVE/API2/Nodes.pm b/PVE/API2/Nodes.pm
index c807c79a..108b81e3 100644
--- a/PVE/API2/Nodes.pm
+++ b/PVE/API2/Nodes.pm
@@ -1274,6 +1274,18 @@ __PACKAGE__->register_method ({
 		foreach my $vmid (sort {$a <=> $b} keys %$vmlist) {
 		    my $d = $vmlist->{$vmid};
 
+		    # skip templates
+		    my $conf;
+		    if ($d->{type} eq 'lxc') {
+			$conf = PVE::LXC::Config->load_config($vmid);
+			next if PVE::LXC::Config->is_template($conf);
+		    } elsif ($d->{type} eq 'qemu') {
+			$conf = PVE::QemuConfig->load_config($vmid);
+			next if PVE::QemuConfig->is_template($conf);
+		    } else {
+			die "unknown VM type '$d->{type}'\n";
+		    }
+
 		    PVE::Cluster::check_cfs_quorum(); # abort when we loose quorum
 	    
 		    eval {