CLOUDSTACK-8956: Remove assert(false) on opaque network and ping method on NiciraNvpApiVersion

This commit is contained in:
nvazquez 2015-10-23 06:10:27 -07:00 committed by nvazquez
parent e1d1a40325
commit b20c5be1ce
3 changed files with 6 additions and 22 deletions

View File

@ -190,7 +190,7 @@ public class NiciraNvpResource implements ServerResource {
ClusterRoleConfig[] configuredRoles = ccs.getConfiguredRoles();
if (configuredRoles != null){
String apiProviderMajorityVersion = searchApiProvider(configuredRoles);
NiciraNvpApiVersion.pingNiciraApiVersion(apiProviderMajorityVersion);
NiciraNvpApiVersion.setNiciraApiVersion(apiProviderMajorityVersion);
NiciraNvpApiVersion.logNiciraApiVersion();
}
}

View File

@ -26,34 +26,19 @@ import com.cloud.maint.Version;
public class NiciraNvpApiVersion {
private static final Logger s_logger = Logger.getLogger(NiciraNvpApiVersion.class);
private static String niciraApiVersion = null;
private static String niciraApiVersion;
public static void pingNiciraApiVersion(String apiVersion){
if (apiVersion == null){
niciraApiVersion = null;
}
else{
if (niciraApiVersion == null){
setNiciraApiVersion(apiVersion);
}
else {
if (! apiVersion.equals(niciraApiVersion)){
setNiciraApiVersion(apiVersion);
}
}
}
}
private static void setNiciraApiVersion(String apiVersion){
public static synchronized void setNiciraApiVersion(String apiVersion){
niciraApiVersion = apiVersion;
}
public static boolean isApiVersionLowerThan(String apiVersion){
public static synchronized boolean isApiVersionLowerThan(String apiVersion){
if (niciraApiVersion == null) return false;
int compare = Version.compare(niciraApiVersion, apiVersion);
return (compare < 0);
}
public static void logNiciraApiVersion(){
public static synchronized void logNiciraApiVersion(){
s_logger.info("NSX API VERSION: " + ((niciraApiVersion != null) ? niciraApiVersion : " NOT PRESENT"));
}

View File

@ -103,7 +103,6 @@ public class VmwareHelper {
break;
default:
assert (false);
nic = new VirtualE1000();
}