Updated Branches: refs/heads/4.3 78b9d5cda -> 88ce5b804 refs/heads/master 79dc74dcb -> 700184f3b
Added config to enable/disable router version check Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/700184f3 Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/700184f3 Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/700184f3 Branch: refs/heads/master Commit: 700184f3ba270be7a493d088990d8791ce80206d Parents: 79dc74d Author: Kishan Kavala <kis...@cloud.com> Authored: Wed Jan 1 17:32:00 2014 +0530 Committer: Kishan Kavala <kis...@cloud.com> Committed: Wed Jan 1 17:32:00 2014 +0530 ---------------------------------------------------------------------- .../router/VirtualNetworkApplianceManagerImpl.java | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/cloudstack/blob/700184f3/server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java ---------------------------------------------------------------------- diff --git a/server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java b/server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java index ee412b3..8904185 100755 --- a/server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java +++ b/server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java @@ -665,6 +665,10 @@ public class VirtualNetworkApplianceManagerImpl extends ManagerBase implements V static final ConfigKey<Boolean> UseExternalDnsServers = new ConfigKey<Boolean>(Boolean.class, "use.external.dns", "Advanced", "false", "Bypass internal dns, use external dns1 and dns2", true, ConfigKey.Scope.Zone, null); + static final ConfigKey<Boolean> routerVersionCheckEnabled = new ConfigKey<Boolean>("Advanced", Boolean.class, "router.version.check", "true", + "If true, router minimum required version is checked before sending command", false); + + @Override public boolean configure(final String name, final Map<String, Object> params) throws ConfigurationException { @@ -4156,6 +4160,10 @@ public class VirtualNetworkApplianceManagerImpl extends ManagerBase implements V //Checks if the router is at the required version // Compares MS version and router version private boolean checkRouterVersion(VirtualRouter router){ + if(!routerVersionCheckEnabled.value()){ + //Router version check is disabled. + return true; + } if(router.getTemplateVersion() == null){ return false; } @@ -4198,6 +4206,6 @@ public class VirtualNetworkApplianceManagerImpl extends ManagerBase implements V @Override public ConfigKey<?>[] getConfigKeys() { - return new ConfigKey<?>[] {UseExternalDnsServers}; + return new ConfigKey<?>[] {UseExternalDnsServers, routerVersionCheckEnabled}; } }