X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?p=controller.git;a=blobdiff_plain;f=opendaylight%2Fconfig%2Fconfig-persister-feature4-adapter%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fconfigpusherfeature%2Finternal%2FFeatureConfigPusher.java;h=7f254d539337970a68a58c759b43c001e9bd6f7e;hp=50757fb7b90961df577d1dc1d7675bbf56ea8bb5;hb=65defe208374fd0e3688f0653c46aac086241904;hpb=3354f45e32faaab5452f95f9931274a3580de912 diff --git a/opendaylight/config/config-persister-feature4-adapter/src/main/java/org/opendaylight/controller/configpusherfeature/internal/FeatureConfigPusher.java b/opendaylight/config/config-persister-feature4-adapter/src/main/java/org/opendaylight/controller/configpusherfeature/internal/FeatureConfigPusher.java index 50757fb7b9..7f254d5393 100644 --- a/opendaylight/config/config-persister-feature4-adapter/src/main/java/org/opendaylight/controller/configpusherfeature/internal/FeatureConfigPusher.java +++ b/opendaylight/config/config-persister-feature4-adapter/src/main/java/org/opendaylight/controller/configpusherfeature/internal/FeatureConfigPusher.java @@ -75,9 +75,12 @@ public class FeatureConfigPusher { private Set pushConfig(final Feature feature) throws Exception { Set configs = new LinkedHashSet<>(); - if(isInstalled(feature)) { + // Ignore feature conditions — these encode conditions on other features and shouldn't be processed here + if (feature.getName().contains("-condition-")) { + LOG.debug("Ignoring conditional feature {}", feature); + } else if (isInstalled(feature)) { // FIXME Workaround for BUG-2836, features service returns null for feature: standard-condition-webconsole_0_0_0, 3.0.1 - if(featuresService.getFeature(feature.getName(), feature.getVersion()) == null) { + if (featuresService.getFeature(feature.getName(), feature.getVersion()) == null) { LOG.debug("Feature: {}, {} is missing from features service. Skipping", feature.getName(), feature.getVersion()); } else { ChildAwareFeatureWrapper wrappedFeature = new ChildAwareFeatureWrapper(feature, featuresService);