[pve-devel] [PATCH pve-container] allow to load configs from CTs	located on other nodes
    Thomas Lamprecht 
    t.lamprecht at proxmox.com
       
    Wed Aug 26 15:36:28 CEST 2015
    
    
  
This allows load_config to load also configs from LXC container
which arent't located on the same node.
This is needed to fix the bug that doesn't let view the noVNC
console from an CT located on another node, whereas viewing it on
the same node as the container is located works fine.
Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
---
 src/PVE/API2/LXC.pm | 2 +-
 src/PVE/LXC.pm      | 5 +++--
 2 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/src/PVE/API2/LXC.pm b/src/PVE/API2/LXC.pm
index 8431125..e9bd5a7 100644
--- a/src/PVE/API2/LXC.pm
+++ b/src/PVE/API2/LXC.pm
@@ -593,7 +593,7 @@ __PACKAGE__->register_method ({
 	my $remcmd = $remip ?
 	    ['/usr/bin/ssh', '-t', $remip] : [];
 
-	my $conf = PVE::LXC::load_config($vmid);
+	my $conf = PVE::LXC::load_config($vmid, $node);
 	my $concmd = PVE::LXC::get_console_command($vmid, $conf);
 
 	my $shcmd = [ '/usr/bin/dtach', '-A',
diff --git a/src/PVE/LXC.pm b/src/PVE/LXC.pm
index 5e21df6..abd6ff0 100644
--- a/src/PVE/LXC.pm
+++ b/src/PVE/LXC.pm
@@ -439,9 +439,10 @@ sub config_file {
 }
 
 sub load_config {
-    my ($vmid) = @_;
+    my ($vmid, $node) = @_;
 
-    my $cfspath = cfs_config_path($vmid);
+    $node = $nodename if !$node;
+    my $cfspath = cfs_config_path($vmid, $node);
 
     my $conf = PVE::Cluster::cfs_read_file($cfspath);
     die "container $vmid does not exists\n" if !defined($conf);
-- 
2.1.4
    
    
More information about the pve-devel
mailing list