*/
package org.opendaylight.controller.configpusherfeature.internal;
+import com.google.common.base.Optional;
+import com.google.common.base.Preconditions;
+import com.google.common.io.Files;
+import java.io.File;
import java.util.LinkedHashSet;
import java.util.List;
-import java.util.Map;
-
import javax.xml.bind.JAXBException;
-
+import javax.xml.stream.XMLStreamException;
import org.apache.karaf.features.BundleInfo;
import org.apache.karaf.features.Conditional;
import org.apache.karaf.features.ConfigFileInfo;
+import org.apache.karaf.features.ConfigInfo;
import org.apache.karaf.features.Dependency;
import org.apache.karaf.features.Feature;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-import com.google.common.base.Preconditions;
-
/*
* Wrap a Feature for the purposes of extracting the FeatureConfigSnapshotHolders from
* its underlying ConfigFileInfo's
* Delegates the the contained feature and provides additional methods.
*/
public class AbstractFeatureWrapper implements Feature {
- private static final Logger LOGGER = LoggerFactory.getLogger(AbstractFeatureWrapper.class);
+ private static final Logger LOG = LoggerFactory.getLogger(AbstractFeatureWrapper.class);
+
+ private static final String CONFIG_FILE_PATH_SUFFIX = "opendaylight" + File.separator + "karaf";
+ protected static final String CONFIG_FILE_SUFFIX = "xml";
+
protected Feature feature = null;
protected AbstractFeatureWrapper() {
/*
* @param f Feature to wrap
*/
- public AbstractFeatureWrapper(Feature f) {
+ public AbstractFeatureWrapper(final Feature f) {
Preconditions.checkNotNull(f,"FeatureWrapper requires non-null Feature in constructor");
this.feature = f;
}
* from the underlying Feature Config files
*/
public LinkedHashSet<FeatureConfigSnapshotHolder> getFeatureConfigSnapshotHolders() throws Exception {
- LinkedHashSet <FeatureConfigSnapshotHolder> snapShotHolders = new LinkedHashSet<FeatureConfigSnapshotHolder>();
- for(ConfigFileInfo c: getConfigurationFiles()) {
- try {
- snapShotHolders.add(new FeatureConfigSnapshotHolder(c,this));
- } catch (JAXBException e) {
- LOGGER.debug("{} is not a config subsystem config file",c.getFinalname());
+ final LinkedHashSet <FeatureConfigSnapshotHolder> snapShotHolders = new LinkedHashSet<>();
+ for(final ConfigFileInfo c: getConfigurationFiles()) {
+ // Skip non xml files
+ if(isConfigXMLFile(c.getFinalname())) {
+ final Optional<FeatureConfigSnapshotHolder> featureConfigSnapshotHolder = getFeatureConfigSnapshotHolder(c);
+ if(featureConfigSnapshotHolder.isPresent()) {
+ snapShotHolders.add(featureConfigSnapshotHolder.get());
+ }
}
}
return snapShotHolders;
}
+ private static boolean isConfigXMLFile(String fullName) {
+ String path = new File(fullName).getPath();
+ return path.contains(CONFIG_FILE_PATH_SUFFIX) &&
+ Files.getFileExtension(fullName).equals(CONFIG_FILE_SUFFIX);
+ }
+
+ protected Optional<FeatureConfigSnapshotHolder> getFeatureConfigSnapshotHolder(final ConfigFileInfo c) {
+ try {
+ return Optional.of(new FeatureConfigSnapshotHolder(c, this));
+ } catch (final JAXBException e) {
+ LOG.warn("Unable to parse configuration snapshot. Config from '{}' will be IGNORED. " +
+ "Note that subsequent config files may fail due to this problem. " +
+ "Xml markup in this file needs to be fixed, for detailed information see enclosed exception.",
+ c.getFinalname(), e);
+ } catch (final XMLStreamException e) {
+ // Files that cannot be loaded are ignored as non config subsystem files e.g. jetty.xml
+ LOG.debug("Unable to read configuration file '{}'. Not a configuration snapshot",
+ c.getFinalname(), e);
+ }
+ return Optional.absent();
+ }
+
@Override
public int hashCode() {
final int prime = 31;
int result = 1;
- result = prime * result + ((feature == null) ? 0 : feature.hashCode());
+ result = prime * result + (feature == null ? 0 : feature.hashCode());
return result;
}
@Override
- public boolean equals(Object obj) {
+ public boolean equals(final Object obj) {
if (this == obj) {
return true;
}
if (getClass() != obj.getClass()) {
return false;
}
- AbstractFeatureWrapper other = (AbstractFeatureWrapper) obj;
+ final AbstractFeatureWrapper other = (AbstractFeatureWrapper) obj;
if (feature == null) {
if (other.feature != null) {
return false;
* @return
* @see org.apache.karaf.features.Feature#getId()
*/
+ @Override
public String getId() {
return feature.getId();
}
* @return
* @see org.apache.karaf.features.Feature#getName()
*/
+ @Override
public String getName() {
return feature.getName();
}
* @return
* @see org.apache.karaf.features.Feature#getDescription()
*/
+ @Override
public String getDescription() {
return feature.getDescription();
}
* @return
* @see org.apache.karaf.features.Feature#getDetails()
*/
+ @Override
public String getDetails() {
return feature.getDetails();
}
* @return
* @see org.apache.karaf.features.Feature#getVersion()
*/
+ @Override
public String getVersion() {
return feature.getVersion();
}
* @return
* @see org.apache.karaf.features.Feature#hasVersion()
*/
+ @Override
public boolean hasVersion() {
return feature.hasVersion();
}
* @return
* @see org.apache.karaf.features.Feature#getResolver()
*/
+ @Override
public String getResolver() {
return feature.getResolver();
}
* @return
* @see org.apache.karaf.features.Feature#getInstall()
*/
+ @Override
public String getInstall() {
return feature.getInstall();
}
* @return
* @see org.apache.karaf.features.Feature#getDependencies()
*/
+ @Override
public List<Dependency> getDependencies() {
return feature.getDependencies();
}
* @return
* @see org.apache.karaf.features.Feature#getBundles()
*/
+ @Override
public List<BundleInfo> getBundles() {
return feature.getBundles();
}
* @return
* @see org.apache.karaf.features.Feature#getConfigurations()
*/
- public Map<String, Map<String, String>> getConfigurations() {
+ @Override
+ public List<ConfigInfo> getConfigurations() {
return feature.getConfigurations();
}
* @return
* @see org.apache.karaf.features.Feature#getConfigurationFiles()
*/
+ @Override
public List<ConfigFileInfo> getConfigurationFiles() {
return feature.getConfigurationFiles();
}
* @return
* @see org.apache.karaf.features.Feature#getConditional()
*/
+ @Override
public List<? extends Conditional> getConditional() {
return feature.getConditional();
}
* @return
* @see org.apache.karaf.features.Feature#getStartLevel()
*/
+ @Override
public int getStartLevel() {
return feature.getStartLevel();
}
* @return
* @see org.apache.karaf.features.Feature#getRegion()
*/
+ @Override
public String getRegion() {
return feature.getRegion();
}
-}
\ No newline at end of file
+}