Repository: cloudstack Updated Branches: refs/heads/4.5 bbaf7d2d9 -> b462a56f2
CLOUDSTACK-8282: startSystemVM should override consoleproxy.start setting Signed-off-by: Rohit Yadav <[email protected]> Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/b462a56f Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/b462a56f Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/b462a56f Branch: refs/heads/4.5 Commit: b462a56f27b21455b70415ea03663af5cd62647e Parents: bbaf7d2 Author: Rohit Yadav <[email protected]> Authored: Wed Feb 25 14:21:11 2015 +0530 Committer: Rohit Yadav <[email protected]> Committed: Wed Feb 25 14:21:11 2015 +0530 ---------------------------------------------------------------------- .../cloud/consoleproxy/AgentBasedConsoleProxyManager.java | 2 +- server/src/com/cloud/consoleproxy/ConsoleProxyManager.java | 2 +- .../src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java | 8 ++++---- server/src/com/cloud/server/ManagementServerImpl.java | 2 +- 4 files changed, 7 insertions(+), 7 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/cloudstack/blob/b462a56f/server/src/com/cloud/consoleproxy/AgentBasedConsoleProxyManager.java ---------------------------------------------------------------------- diff --git a/server/src/com/cloud/consoleproxy/AgentBasedConsoleProxyManager.java b/server/src/com/cloud/consoleproxy/AgentBasedConsoleProxyManager.java index 3a1c387..6824b9c 100755 --- a/server/src/com/cloud/consoleproxy/AgentBasedConsoleProxyManager.java +++ b/server/src/com/cloud/consoleproxy/AgentBasedConsoleProxyManager.java @@ -175,7 +175,7 @@ public class AgentBasedConsoleProxyManager extends ManagerBase implements Consol } @Override - public ConsoleProxyVO startProxy(long proxyVmId) { + public ConsoleProxyVO startProxy(long proxyVmId, boolean ignoreRestartSetting) { return null; } http://git-wip-us.apache.org/repos/asf/cloudstack/blob/b462a56f/server/src/com/cloud/consoleproxy/ConsoleProxyManager.java ---------------------------------------------------------------------- diff --git a/server/src/com/cloud/consoleproxy/ConsoleProxyManager.java b/server/src/com/cloud/consoleproxy/ConsoleProxyManager.java index 9d22b7f..8496301 100755 --- a/server/src/com/cloud/consoleproxy/ConsoleProxyManager.java +++ b/server/src/com/cloud/consoleproxy/ConsoleProxyManager.java @@ -40,7 +40,7 @@ public interface ConsoleProxyManager extends Manager, ConsoleProxyService { public void resumeLastManagementState(); - public ConsoleProxyVO startProxy(long proxyVmId); + public ConsoleProxyVO startProxy(long proxyVmId, boolean ignoreRestartSetting); public boolean stopProxy(long proxyVmId); http://git-wip-us.apache.org/repos/asf/cloudstack/blob/b462a56f/server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java ---------------------------------------------------------------------- diff --git a/server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java b/server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java index bb64fa3..ec23e95 100755 --- a/server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java +++ b/server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java @@ -520,7 +520,7 @@ public class ConsoleProxyManagerImpl extends ManagerBase implements ConsoleProxy } @Override - public ConsoleProxyVO startProxy(long proxyVmId) { + public ConsoleProxyVO startProxy(long proxyVmId, boolean ignoreRestartSetting) { try { ConsoleProxyVO proxy = _consoleProxyDao.findById(proxyVmId); if (proxy.getState() == VirtualMachine.State.Running) { @@ -528,7 +528,7 @@ public class ConsoleProxyManagerImpl extends ManagerBase implements ConsoleProxy } String restart = _configDao.getValue(Config.ConsoleProxyRestart.key()); - if (restart != null && restart.equalsIgnoreCase("false")) { + if (!ignoreRestartSetting && restart != null && restart.equalsIgnoreCase("false")) { return null; } @@ -912,7 +912,7 @@ public class ConsoleProxyManagerImpl extends ManagerBase implements ConsoleProxy if (proxy != null) { long proxyVmId = proxy.getId(); - proxy = startProxy(proxyVmId); + proxy = startProxy(proxyVmId, false); if (proxy != null) { if (s_logger.isInfoEnabled()) { @@ -1134,7 +1134,7 @@ public class ConsoleProxyManagerImpl extends ManagerBase implements ConsoleProxy return false; } } else { - return startProxy(proxyVmId) != null; + return startProxy(proxyVmId, false) != null; } } http://git-wip-us.apache.org/repos/asf/cloudstack/blob/b462a56f/server/src/com/cloud/server/ManagementServerImpl.java ---------------------------------------------------------------------- diff --git a/server/src/com/cloud/server/ManagementServerImpl.java b/server/src/com/cloud/server/ManagementServerImpl.java index 85ce413..cbf7925 100755 --- a/server/src/com/cloud/server/ManagementServerImpl.java +++ b/server/src/com/cloud/server/ManagementServerImpl.java @@ -2182,7 +2182,7 @@ public class ManagementServerImpl extends ManagerBase implements ManagementServe } private ConsoleProxyVO startConsoleProxy(long instanceId) { - return _consoleProxyMgr.startProxy(instanceId); + return _consoleProxyMgr.startProxy(instanceId, true); } private ConsoleProxyVO stopConsoleProxy(VMInstanceVO systemVm, boolean isForced) throws ResourceUnavailableException, OperationTimedoutException, ConcurrentOperationException {
