git.opendaylight Code Review - openflowplugin.git/commit

Code Review / openflowplugin.git / commit

author	Tomas Slusny <tomas.slusny@pantheon.tech>
	Mon, 19 Jun 2017 12:45:35 +0000 (14:45 +0200)
committer	Tomas Slusny <tomas.slusny@pantheon.tech>
	Mon, 26 Jun 2017 10:59:48 +0000 (12:59 +0200)
commit	bb5487ecc9d866af16d2f80e3fae7e35b9a8bf2e
tree	49e6e0019fe11fcf7a0cd7483a92b1816689d9da	tree \| snapshot
parent	39633bcebf3b3eb1bd1c92d37ba3c8b510916027	commit \| diff

Encapsulate OpenFlowPlugin configuration

- Stop sending configuration to OpenFlow managers in long constructors
  and send just aggregated configuration in one class
- Add ability to register configuration listener to configuration
service
- Separate configuration service from OpenFlowPluginProviderImpl
- Fix FRM configuration breaking OpenFlowPlugin configuration by trying
  to load configuration from openflowplugin.cfg directly, what is only
  accesible to main OpenFlowPlugin project by creating configuration
  service and passing it to FRM
- Add missing 'disable-reconciliation' flag to FRM yang config, and use
  default values from YANG file for FRM configuration

Resolves: bug 8705

Change-Id: I791925bb1964c9eb5a9f1db77fcc25f6425f053f
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

46 files changed:

applications/forwardingrules-manager/pom.xml		diff \| blob \| history
applications/forwardingrules-manager/src/main/java/org/opendaylight/openflowplugin/applications/frm/ForwardingRulesManager.java		diff \| blob \| history
applications/forwardingrules-manager/src/main/java/org/opendaylight/openflowplugin/applications/frm/ForwardingRulesProperty.java	[new file with mode: 0644]	blob
applications/forwardingrules-manager/src/main/java/org/opendaylight/openflowplugin/applications/frm/impl/ForwardingRulesManagerImpl.java		diff \| blob \| history
applications/forwardingrules-manager/src/main/resources/org/opendaylight/blueprint/forwardingrules-manager.xml		diff \| blob \| history
applications/forwardingrules-manager/src/main/yang/forwardingrules-manager-config.yang		diff \| blob \| history
applications/forwardingrules-manager/src/test/java/test/mock/FlowListenerTest.java		diff \| blob \| history
applications/forwardingrules-manager/src/test/java/test/mock/GroupListenerTest.java		diff \| blob \| history
applications/forwardingrules-manager/src/test/java/test/mock/MeterListenerTest.java		diff \| blob \| history
applications/forwardingrules-manager/src/test/java/test/mock/NodeListenerTest.java		diff \| blob \| history
applications/forwardingrules-manager/src/test/java/test/mock/TableFeaturesListenerTest.java		diff \| blob \| history
applications/forwardingrules-manager/src/test/java/test/mock/util/FRMTest.java		diff \| blob \| history
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/OpenFlowPluginConfigurationService.java	[deleted file]	blob \| history
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/OpenFlowPluginProviderFactory.java		diff \| blob \| history
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/configuration/ConfigurationListener.java	[new file with mode: 0644]	blob
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/configuration/ConfigurationProperty.java	[new file with mode: 0644]	blob
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/configuration/ConfigurationService.java	[new file with mode: 0644]	blob
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/configuration/ConfigurationServiceFactory.java	[new file with mode: 0644]	blob
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/connection/ConnectionManager.java		diff \| blob \| history
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/device/DeviceContext.java		diff \| blob \| history
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/device/DeviceManager.java		diff \| blob \| history
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/lifecycle/ContextChainHolder.java		diff \| blob \| history
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/rpc/RpcManager.java		diff \| blob \| history
openflowplugin-api/src/main/java/org/opendaylight/openflowplugin/api/openflow/statistics/StatisticsManager.java		diff \| blob \| history
openflowplugin-blueprint-config/src/main/resources/initial/openflowplugin.cfg		diff \| blob \| history
openflowplugin-blueprint-config/src/main/resources/org/opendaylight/blueprint/openflowplugin.xml		diff \| blob \| history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/OpenFlowPluginProviderFactoryImpl.java		diff \| blob \| history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/OpenFlowPluginProviderImpl.java		diff \| blob \| history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/configuration/ConfigurationServiceFactoryImpl.java	[new file with mode: 0644]	blob
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/configuration/OpenFlowProviderConfigImpl.java	[new file with mode: 0644]	blob
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/connection/ConnectionManagerImpl.java		diff \| blob \| history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/device/DeviceContextImpl.java		diff \| blob \| history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/device/DeviceManagerImpl.java		diff \| blob \| history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/device/initialization/OF13DeviceInitializer.java		diff \| blob \| history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/lifecycle/ContextChainHolderImpl.java		diff \| blob \| history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/rpc/RpcManagerImpl.java		diff \| blob \| history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/statistics/StatisticsManagerImpl.java		diff \| blob \| history
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/util/DeviceInitializationUtil.java		diff \| blob \| history
openflowplugin-impl/src/main/resources/org/opendaylight/blueprint/openflowplugin-impl.xml		diff \| blob \| history
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/OpenFlowPluginProviderImplTest.java		diff \| blob \| history
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/connection/ConnectionManagerImplTest.java		diff \| blob \| history
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/device/DeviceContextImplTest.java		diff \| blob \| history
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/device/DeviceManagerImplTest.java		diff \| blob \| history
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/lifecycle/ContextChainHolderImplTest.java		diff \| blob \| history
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/rpc/RpcManagerImplTest.java		diff \| blob \| history
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/statistics/StatisticsManagerImplTest.java		diff \| blob \| history

OpenFlow Plugin

RSS Atom