diff options
Diffstat (limited to 'infra/northbound/bgp/src/main/java/io/fd/honeycomb/infra/bgp/BgpExtensionsModule.java')
-rw-r--r-- | infra/northbound/bgp/src/main/java/io/fd/honeycomb/infra/bgp/BgpExtensionsModule.java | 15 |
1 files changed, 4 insertions, 11 deletions
diff --git a/infra/northbound/bgp/src/main/java/io/fd/honeycomb/infra/bgp/BgpExtensionsModule.java b/infra/northbound/bgp/src/main/java/io/fd/honeycomb/infra/bgp/BgpExtensionsModule.java index d744065fb..b096e3dbd 100644 --- a/infra/northbound/bgp/src/main/java/io/fd/honeycomb/infra/bgp/BgpExtensionsModule.java +++ b/infra/northbound/bgp/src/main/java/io/fd/honeycomb/infra/bgp/BgpExtensionsModule.java @@ -16,9 +16,9 @@ package io.fd.honeycomb.infra.bgp; +import com.google.inject.AbstractModule; import com.google.inject.Singleton; import com.google.inject.multibindings.Multibinder; -import io.fd.honeycomb.northbound.NorthboundAbstractModule; import org.opendaylight.protocol.bgp.parser.spi.BGPExtensionConsumerContext; import org.opendaylight.protocol.bgp.parser.spi.BGPExtensionProviderActivator; import org.opendaylight.protocol.bgp.rib.spi.RIBExtensionConsumerContext; @@ -28,20 +28,13 @@ import org.slf4j.LoggerFactory; /** * Registers BGP extensions provided by ODL implementation. - * TODO add support for flowspec (requires some special initialization) + * TODO(HONEYCOMB-363): create module per BGP extension + * TODO(HONEYCOMB-378): add support for flowspec (requires some special initialization) */ -public final class BgpExtensionsModule extends NorthboundAbstractModule<BgpConfiguration> { +final class BgpExtensionsModule extends AbstractModule { private static final Logger LOG = LoggerFactory.getLogger(BgpExtensionsModule.class); - public BgpExtensionsModule() { - super(new BgpConfigurationModule(), BgpConfiguration.class); - } - protected void configure() { - if (!getConfiguration().isBgpEnabled()) { - LOG.debug("BGP disabled. Skipping initialization"); - return; - } LOG.debug("Initializing BgpExtensionsModule"); // This should be part of BgpModule, but that one is Private and Multibinders + private BASE_MODULES // do not work together, that's why there's a dedicated module here |