Merge changes from topic 'blueprint'

* changes:
  Remove inventory manager config yang and XML
  Remove statistics manager config yang and XML

Remove inventory manager config yang and XML

Change-Id: I47b550deadd08e002fe4388f707f9cbfe3b3bb64
Signed-off-by: Tom Pantelis <tpanteli@brocade.com>

Merge "BUG-2771: Converting String.split() to Guava Splitter for Inventory Utils"

Remove statistics manager config yang and XML

Change-Id: I56c38dcb3ca57e9ccc7a4d37e2873ea8b29cb515
Signed-off-by: Tom Pantelis <tpanteli@brocade.com>

Merge changes from topic 'blueprint'

* changes:
  Add blueprint wiring for the drop-test app
  Add blueprint wiring for the bulk-o-matic app
  Add blueprint wiring for the lldp-speaker app
  Add blueprint wiring for the of-switch-config-pusher app
  Add blueprint wiring for the table-miss-enforcer app

Merge "BUG:5888 - Changing FRM from clustered DCN to clustered DTCN"

Merge changes from topic 'blueprint'

* changes:
  Add blueprint wiring for the topology manager app
  Add blueprint wiring for the TopologyLldpDiscovery app
  Add blueprint wiring for the ForwardingRulesManager app

Bug 5920 Do not wipe out device when there is not configuration

   When device appears in operational DS either syncup is done or the event is skipped:
    - syncup in case there is record for the device in config DS
    - if there is not, device should remain untouched
   (+update tests)

Change-Id: I0b9f579f9987f886ad5d1dc4307d3d053aa53cea
Signed-off-by: Andrej Leitner <anleitne@cisco.com>

Refator names & imports (frsync)

  - fix typos, guard & executor factory rename
  - junit test for guard decorator
  - add missing parameter descriptions
  - remove unused local variables and suppress warnings
  - remove commented usage of incremental strategy
  - edit access level modifiers
  - organize and remove unused imports

Change-Id: I5e1767df2aa1dce77001161b0148d0717aefef6c
Signed-off-by: Andrej Leitner <anleitne@cisco.com>

Merge "Bug 5953: Fixed NPE in FlowRemovedTranslator."

BUG:5888 - Changing FRM from clustered DCN to clustered DTCN

Change-Id: I60435b5216036b66791251639f0537ed2ef832f7
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

Fix unused import warnings

This is just a pass at reorganizing imports where eclipse
warns of unused import.

Change-Id: I143fc2662dde58de9c40068563f37fb544363c5d
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Eliminate the use of CompositeObjectRegistration"

Add blueprint wiring for the drop-test app

Also removed yhe config yang and XML file as this is a test app and
backeards compatibility is not needed.

Change-Id: I2415b81872ada6fd0d603cbe115b8cff02745d04
Signed-off-by: Tom Pantelis <tpanteli@brocade.com>

Add blueprint wiring for the bulk-o-matic app

Change-Id: I6a2454fde67c776854b91a804ffa54d025d73e89
Signed-off-by: Tom Pantelis <tpanteli@brocade.com>

Add blueprint wiring for the lldp-speaker app

Change-Id: Iab911edca42540835c668cf20d7b00dd673e9428
Signed-off-by: Tom Pantelis <tpanteli@brocade.com>

Add blueprint wiring for the of-switch-config-pusher app

Change-Id: I38c0cbac5882468853a73882d717bb57b93d9c63
Signed-off-by: Tom Pantelis <tpanteli@brocade.com>

Add blueprint wiring for the table-miss-enforcer app

Change-Id: Ic257f017d95ff5c58c74a6a6abcb0045b31b68b1
Signed-off-by: Tom Pantelis <tpanteli@brocade.com>

Add blueprint wiring for the topology manager app

Change-Id: I83906f6125c61f34654b729754ed65527a200b9e
Signed-off-by: Tom Pantelis <tpanteli@brocade.com>

Add blueprint wiring for the TopologyLldpDiscovery app

Change-Id: Iae04ce7a7dc6b6f2389d4792a8c16de1605c9cdd
Signed-off-by: Tom Pantelis <tpanteli@brocade.com>

Add blueprint wiring for the ForwardingRulesManager app

Change-Id: Iaa552cd6361031ca9dc96def6976c762d3985ac1
Signed-off-by: Tom Pantelis <tpanteli@brocade.com>

Add the openflowplugin-blueprint-config bundles to the features xml

This turns on the blueprint wiring for the openflow helium plugin

Change-Id: I5725495f3a40e636bc35cb1a9281c593ad072840
Signed-off-by: Tom Pantelis <tpanteli@brocade.com>

Add blueprint wiring for the lithium OpenflowPluginProvider

Change-Id: I95d721e85f0116b7d83c6b2db475ddfc27aa70c9
Signed-off-by: Tom Pantelis <tpanteli@brocade.com>

Add blueprint wiring for the helium inventory manager

Change-Id: Ie6685cdf636c6f707943460bc6dbcac3483442c8
Signed-off-by: Tom Pantelis <tpanteli@brocade.com>

Add blueprint wiring for the helium StatisticsManager

Change-Id: I81ac14b8ea9e4d76d3088ae512a52cfc758c8688
Signed-off-by: Tom Pantelis <tpanteli@brocade.com>

Add blueprint wiring for the helium OpenflowPluginProvider

Added a new bundle, openflowplugin-blueprint-config-he, containing the
blueprint XML that instantiates the SwitchConnectionProvider
and OpenflowPluginProvider instances for the helium plugin. It uses
the new SwitchConnectionProviderFactory interface added to the
openflowjava SPI.

This replaces the ConfigurableOpenFlowProviderModule which is now
deprecated but remains for backwards compatibility. The createInstance
method now obtains the OpenflowPluginProvider created via blueprint from
the OSGi registry and returns a proxy instance that delegates to the
actual instance but overrides the close method appropriately.

Patch 3 : Resolved conflict locally

Change-Id: I315444847f5a95d905a5e6adc602445c1101da5b
Signed-off-by: Tom Pantelis <tpanteli@brocade.com>
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

Ignore maven-metadata-local.xml produced by maven-invoker-plugin in Git

Also remove two redundant (non-root) .gitignore files from projects

Change-Id: I21366c5a4dc96365cf34215835724e07a5f3b9ca
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Merge "Remove M2E lifecycle-mapping <execute/> for yang-maven-plugin"

Merge "Fix bad generated path for generated sources"

Merge "BUG-2771: String.split() replaced by Guava's Splitter for"

Merge "BUG-2771: String.split() replaced by Guava's splitter in nicira-extensions Added tests, reorganised packages"

BUG-2771: Converting String.split() to Guava Splitter
for Inventory Utils

Change-Id: I70eaaebae0a9ad879aba5fa3313d01450af6194d
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

Merge "BUG-2188 :To report (TCP) port number (for the OpenFlow connection) for switches as the IP address is not guaranteed to be unique—especially with mininet"

BUG-2771: String.split() replaced by Guava's Splitter for

1. statistics-manager - following commit
https://git.opendaylight.org/gerrit/#/c/39019/
2. lithium ofplugin test
3. inventoryDataServiceUtil - portNoStringfromNodeConnectorID

Change-Id: If12d09a2b2fb03912cefd6a2851eee77c568f941
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

Eliminate the use of CompositeObjectRegistration

The class has been long-deprecated, remove its users and substitute
them with proper registration subclasses.

Change-Id: Ie7534d69efb117b7d0469dce54e25a3708f065d8
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-2771: String.split() replaced by Guava's splitter in nicira-extensions
Added tests, reorganised packages

Change-Id: I4942c5a60ef46f3aa07ec51f9760f253d88f1629
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

Correcting test-failures during openflowplugin-periodic builds

Change-Id: Ia94fc4a405e3c128b6c5660fe1a629d47a48e8fc
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

BUG-5888: Registering entity-ownership listerner globally

Change-Id: I9f3208ffb718a16da34b4baaaf220629784d0333
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

Merge "BUG-5636: making table features configurable for the He plugin. DEFAULT will be ON, can be TURNED OFF."

BUG-5636: making table features configurable for the He plugin.
DEFAULT will be ON, can be TURNED OFF.

Change-Id: I16403590f068cd53806c15acde6e6ccbb3ade866
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

Bug 5953: Fixed NPE in FlowRemovedTranslator.

table-id in OF10 FlowRemoved should be ignored because OF10 FLOW_REMOVED
message doesn't contain table ID.

Change-Id: I5e9b1a69219b361bbe3e20210ff04221416f1039
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Bug 5575 introduced counters for sync strategies

    - flat-batch & incremental
    - added more unit tests + fixed counters

Change-Id: I66d27d6d6615bdd66e9f0f2831bfb3ea51c5a691
Signed-off-by: Andrej Leitner <anleitne@cisco.com>

Bug 5575 flat-bulk strategy wired

  - FR-sync currently using flat-batch strategy
  - renaming, time

Change-Id: Id59ff6c15ac25649df16db9b786b1d6b85b75806
Signed-off-by: Andrej Leitner <anleitne@cisco.com>

Bug 5575 added strategy implementation for flat-batch + tests

Change-Id: I667485c13d4fffd166648c339f8554a2680ccd38
Signed-off-by: Andrej Leitner <anleitne@cisco.com>

Bug 5575 pulled out extraction of diff items

  - added common synchronization plan execution strategy
  - added wrapper for diff input
  - removing, renaming & utils

Change-Id: If7876aac23560bb050c1ce2803bf6b732cfba544
Signed-off-by: Andrej Leitner <anleitne@cisco.com>

Bug 5575 added RpcResultBuilder & comments

Change-Id: I44e6a879b7a3fe5b53a73b4a37c1dc0ac5b4d8db
Signed-off-by: Andrej Leitner <anleitne@cisco.com>

Bug 5575 added JUnit tests

  - basic tests for forwarders, listeners and syncreactor

Change-Id: I11fb3c9ca4bd049a170ce6e04ff754399cdeb7ef
Signed-off-by: Andrej Leitner <anleitne@cisco.com>

Bug 5575 added listeners config/operational

  - listeners for node changes in DS
  - syncup starting

Change-Id: I568620061892e3fb997771f272f2beca4c160f81
Signed-off-by: Andrej Leitner <anleitne@cisco.com>

Bug 5575 added SyncReactorDecorators and thread pool

  - Guard, Future, Future w/compression decorators
  - thread pool and semaphore keeper

Change-Id: I4d8d3327ef3f318cc17534e789134204977550f7
Signed-off-by: Andrej Leitner <anleitne@cisco.com>

Bug 5575 main SyncReactor + utils

  - synchronization reactor implementation, applicable for both - syncup and reconciliation
  - useful util methods

Change-Id: I4a0ef8043e3e9b66a92c49deb86a83b083782ccc
Signed-off-by: Andrej Leitner <anleitne@cisco.com>

Bug 5575 added config/operational DAOs

  - implemented pair of cached and non-cached FCN dao

Change-Id: Ic68e48d4e9a9b651c1db19dcc2f279fb5159a0ca
Signed-off-by: Andrej Leitner <anleitne@cisco.com>

Bug 5575 ForwardingRulesCommiters, Flow/Group/Table Forwarder

  (coming from FRM)
  - added interfaces and implementation of add, update and remove
    of Flows, Groups, Meters and TableFeatures

Change-Id: I0168434e2ac56959ff6086952d2a2ad152e252ee
Signed-off-by: Andrej Leitner <anleitne@cisco.com>

Bug 5575 forwardingrules-sync (initial commit)

 - initial commit
 - added FRS bluerpint and provider
 - added pom and feature files

Change-Id: I95695e4a7a01c43c6a7ba0bc161762f61cea82fd
Signed-off-by: Andrej Leitner <anleitne@cisco.com>

Merge "BUG-4117: notification supplier - rename"

Merge "BUG-5602: OFP-Li implements a YANG notification which notifies its applications of the removal of a flow entry on the event of receiving FLOW_REMOVED message from a switch."

BUG-4117: notification supplier - rename

 - renamed from old-notification-supplier to notification-supplier
 - reason: prefix 'old' usually gets strongly misleading after
     a couple of releases

Bringing Michal's draft to master

Change-Id: I0fd387571714cda943042671f9aa454003fe5bed
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

Merge "Bug 5692: Treat null store-stats in direct stats RPC input as false."

Merge "BUG-4117: add support of Old Notif. for Statistics"

Merge "BUG-4117: add support of Old Notif. for Statistics"

Merge "BUG-4117: add support of Old Notif. for Flow Statistics"

Merge "BUG-4117: add support for meter and group old Notif"

Merge "BUG-4117: add support for flow old Notif"

Merge changes from topic 'BUG-4117'

* changes:
  BUG-4117: add impl for FlowCapableNode Connector notif.
  BUG-4117: add impl for FlowCapableNode notification

Merge "BUG-4117: preparation for FlowCapableNode notification impl."

Bug 5692: Treat null store-stats in direct stats RPC input as false.

Change-Id: I19676bd3285975d345ca29b44845cf60e9670b55
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Merge "Bug 5914 - Flow statistics don't show up on the same flow id, if flow uses IPv6 match with subnet mask"

Bug 5692 - Direct statistics RPC

- Added new RPCs for direct gathering of statistics
  with optional flag of storing the statistics to DS/operational

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

Bug 5692 - Statistics ON/OFF

- Removed DeviceState.getRole
- Statistics are properly shut down when device changes to state other than Master
- Statistics are properly started only when device becomes Master
- LifecycleConductor now have access to StatisticsManager

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

Bug 5914 - Flow statistics don't show up on the same flow id,
if flow uses IPv6 match with subnet mask

Patch2: Added unit test

Change-Id: I566c84334a096229274925d223bb906be643f326
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

Remove M2E lifecycle-mapping <execute/> for yang-maven-plugin

This appears to be broken anyway (causing:
org.apache.maven.plugin.MojoExecutionException: yang-to-sources: Unable
to parse yang files from
/home/vorburger/dev/ODL/git/openflowplugin/applications/forwardingrules-manager/src/main/yang;
Caused by:
org.opendaylight.yangtools.yang.parser.spi.meta.InferenceException:
Imported module [config] was not found. [at null:6:1])

In https://github.com/vorburger/opendaylight-eclipse-setup, the
workspace wide lifecycle mapping for yang-maven-plugin is ignore.

In the future, I'm hoping that I'll be able to bundle yangide in
opendaylight-eclipse-setup, and it will correctly configure code
generation.

Change-Id: I8c1640482912eb05ea3fb07e9d55d82f90ab6ebd
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Fix bad generated path for generated sources

The yang-maven-plugin appears to generate code into
target/generated-sources/sal and target/generated-sources/config, and
not (anymore?) into src/main/yang-gen-sal & src/main/yang-gen-config.

Without making this change, there is a lot of red in Eclipse at least,
because generate code is not seen on the classpath.

I wonder in fact how this could ever have even worked in CLI mvn build,
or other IDEs.

Change-Id: Ib2c67bd7b52f1e038626161d29886457c3db8566
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Merge "Bug 5591 - Changed DeviceContext to DeviceState in translate methods of MessageTranslator classes - Added util static method createWithOook to AggregateFlowsInTableService   and to OpendaylightFlowStatisticsServiceImpl (calls DeviceContext.oook() before   creating instance of the service)"

Bug 5590 - writeToTransaction+submitTransaction
- New interface TxFacade, DeviceContext now implements this interface
- Changed constructor of SalTableServiceImpl to accept TxFacade and NodeId
- Changed MultipartRequestOnTheFlyCallback to not need the DeviceContext and accept TxFacade
- Updated some methods in StatisticGatheringUtils to not depend on DeviceContext (because of change above)
- Updated usages of these classes according to changes.
- Updated tests

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

Bug 5591
- Changed DeviceContext to DeviceState in translate methods of MessageTranslator classes
- Added util static method createWithOook to AggregateFlowsInTableService
  and to OpendaylightFlowStatisticsServiceImpl (calls DeviceContext.oook() before
  creating instance of the service)

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

Merge "BUG-5573: Altering the HandshakePool to have a LinkedBlockingQueue instead of an ArrayBlockingQueue with size 20. The main difference between ArrayBlockingQueue and LinkedBlockingQueue is clear from constructor one has underlying data structure Array and other linkedList."

Bug 5543 - JUnit tests update

Change-Id: I7bfcec5ad87213870d3043a3ea00c098e21391b9
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.sk>
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>

Merge "BUG 5680 - TxManager init submit fix"

BUG-5573: Altering the HandshakePool to have a LinkedBlockingQueue instead of an
ArrayBlockingQueue with size 20. The main difference between
ArrayBlockingQueue and LinkedBlockingQueue is clear from constructor one
has underlying data structure Array and other linkedList.

ArrayBlockingQueue uses single-lock double condition algorithm and
LinkedBlockingQueue is variant of the "two lock queue" algorithm and it
has 2 locks 2 conditions ( takeLock , putLock)

In scenarios that involve adding and removing elemenst LinkedBlockingQueue
performs better than ArrayBlockingQueue.

Change-Id: I08f077552599d663857db2ca6d33a86e3b5eb01e
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

Merge "Bug 5592 - NodeId+DatapathId in Services (remove dependency on DeviceContext) - Changed AbstractService.getVersion and AbstractService.getDatapathId to return data from DeviceState - Added AbstractService.getNodeId what returns nodeId from DeviceState - Updated all ocurences of getting above 3 values in subclasses to use these methods - Updated tests (in most of them, DeviceState was not mocked)"

Merge "BUG-4148: Improving the logging levels, adding more information wherever required"

BUG-2188 :To report (TCP) port number (for the OpenFlow connection)
for switches as the IP address is not guaranteed to be unique—especially
with mininet

Bringing https://git.opendaylight.org/gerrit/#/c/14317 to master

Change-Id: I2a74b8760829447c22aa92fb933be4810f31bf99
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

Merge "BUG-5841: Enhances bulk-o-matic to stress test Data Store and Openflowplugin RPC Added asciidoc for the same"

BUG-5841: Enhances bulk-o-matic to stress test Data Store and Openflowplugin RPC
Added asciidoc for the same

- Takes JSON input from REST to provision flows
- Different options to push to DS or Openflowplugin
- Results are printed to karaf.log

Change-Id: I231e9278a7292ac2424751a0689f632d3382c497
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

Merge "BUG-5839: Removing groups and meters on being stale-marked"

Merge "BUG-4148: Improving the logging in flow/group/meter to provide more contextual information as to the success/failure of the operation"

Merge "Add custom compare for ArpMatch objects"

Bug 5592 - NodeId+DatapathId in Services (remove dependency on DeviceContext)
- Changed AbstractService.getVersion and AbstractService.getDatapathId to return data from DeviceState
- Added AbstractService.getNodeId what returns nodeId from DeviceState
- Updated all ocurences of getting above 3 values in subclasses to use these methods
- Updated tests (in most of them, DeviceState was not mocked)

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

Add custom compare for ArpMatch objects

The problem here was that layer3MatchEquals was falling through to
the default compare where the generated equals() method is called
which was comparing the mac addresses case sensitive. This caused
an IT test we're working on in netvirt to fail because certain ARP
flows were not making their way into the operational store. This
only occurs on the beryllium branch but not on master as this
method has a fully different implementation there.

Change-Id: I3aa6e79b48741f656ae036b16743e0e097623593
Signed-off-by: Josh <jhershbe@redhat.com>

BUG 5680 - TxManager init submit fix

Initial submit could fail with unexpected reason and we have to catch it, because we could have distributed DS without corect Node path and sub-path initialization. Added initCommit property.
TODO: think about alternative solution (try to reinit whole Node)

Change-Id: I6a12fb1a9542312408d76a774dd7c0ff66328e47
Signed-off-by: andrej.leitner <anleitne@cisco.com>

BUG-5602: OFP-Li implements a YANG notification which notifies its
applications of the removal of a flow entry on the event of receiving
FLOW_REMOVED message from a switch.

Change-Id: I3496effc0313e38702907e0e0eaf202f460d2406
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

Bug 5595 - RpcContext nodeId+xidSequencer (remove dependency on DeviceContext)
- Changed RpcContextImpl to not depend on DeviceContext.
- Moved isStatisticsRpcEnabled to DeviceManager and DeviceContext
- Updated RpcContextManagerImpl to use new RpcContextImpl constructor
- Updated tests

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

Bug-5523 Lifecycle conductor changes

- changed lifecycle conductor from static thread safe singleton
to singleton created in OpenFlowPluginImpl as are managers defined

Change-Id: I1bd21b2e3a842ce3152882a746a2b9efd3f2cb83
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.sk>

Bug-5523 CloseConnection changes

- dependency of deviceContext removed from starting
- lifecycle conductor thread safe now
- role manager was not removing context from map
- test update

Change-Id: Iadafdfb8f377326bfb76e9163ad66656b16c15f3
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>

Bug-5523 RoleManager changes

- all functionality moved to role manager
- role context hold only information about status
- life cycle conductor defined as listener to role change
- able to register listener to start/stop master services
- timer for initialization device disabled
- added thrown exception when not able to write to transaction
- added junit tests

Change-Id: I741cde5c8161dde8c45f27c6a2e1f99617a613bf
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>

Merge "Remove nexusproxy property"

BUG-5839: Removing groups and meters on being stale-marked

Change-Id: I18f752b7e099a7c423e569de23d957e4f32779c7
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

BUG-4148: Improving the logging levels, adding more information wherever required

Change-Id: I165db99bf76c52dc5c7a20ce84b0058e6a45058d
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

BUG-4148: Improving the logging in flow/group/meter to provide more contextual
information as to the success/failure of the operation

Change-Id: Ica94dbca7559f0afff1f8dfe5695ed8d0bf6376e
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

WIP: BUG-5464: scaling issue

    - pull up table-features to flow-cabable-node
    - fix adjanced logic
    - remove augmentation of flow-capable-node (redundant table-features)
    - fixed unit test

Change-Id: I798f87420330cfd71045eac3de61291a10abdafa
Signed-off-by: Michal Rehak <mirehak@cisco.com>
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>

Merge "Derive distributuon/karaf pom from karaf-parent"