Merge "Eliminate the use of CompositeObjectRegistration"

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"

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 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"

Merge "Log at ERROR level when add/remove flow fail"

Log at ERROR level when add/remove flow fail

Change-Id: I78c14450e6ba235c99a74efff68c0a38ed4ce739
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Merge "Log at ERROR level when request context failed to close"

Log at ERROR level when request context failed to close

And also log the whole exception as the message can be null

 RequestContextUtil               | 190 - org.opendaylight.openflowplugin.impl - 0.3.0.SNAPSHOT | Request context wasn't closed. Exception message: null

Change-Id: Ib5d3f263db0baf83bc2d978a2c3e9f2cb1317b0c
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Log at ERROR level when the queue isn't set...

... as this results in failing the request.
This helps debug, and I believe should be turned
at error level

Change-Id: Ib5152371f449bc81d69b216a2061db7eed624d5a
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Derive distributuon/karaf pom from karaf-parent

This picks up the recent changes to the karaf distro from a common
place.

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

Merge "Bug 5589 - Deprecate PortNumberCache"

Merge changes from topic 'bug-5574'

* changes:
  BUG-5574: added flat-bulk service registration
  BUG-5574: flat batch proposal
  BUG-5574: bulk meters proposal
  BUG-5574: introduces sal-groups-batch service impl
  BUG-5574: sal-flows-batch proposal

BUG-5574: added flat-bulk service registration

    - added registration/unregistation for flat-batch routed service

Change-Id: I95d005aeb12d1b817c643ffc11a6f5343361d5a4
Signed-off-by: Michal Rehak <mirehak@cisco.com>
(cherry picked from commit b5ffe8525d85dcf92d38a1eab96e4f3d2836aad0)
Signed-off-by: andrej.leitner <anleitne@cisco.com>

BUG-5574: flat batch proposal

    - yang model
    - implementation proposal
    - flat batch util
    - unit tests

Change-Id: I4406eaa19024348e63e2cc6d07039949ba8aa941
Signed-off-by: Michal Rehak <mirehak@cisco.com>
(cherry picked from commit 60ddc33574168296cf1b59f59a1cbb998cecfc36)
Signed-off-by: andrej.leitner <anleitne@cisco.com>

BUG-5574: bulk meters proposal

    - added bulk API implementation for groups
    - with parametrized barrier-after support
    - MeterUtil
    - unit tests

Change-Id: I9146894ae0b1f630706b8aea6b85a13c8ce3364b
Signed-off-by: Michal Rehak <mirehak@cisco.com>
(cherry picked from commit 26fa4ba6f87c23e159b74526f6fb849f0ceb4783)
Signed-off-by: andrej.leitner <anleitne@cisco.com>

Bug 5589 - Deprecate PortNumberCache

- New class - Port to NodeConnectorRef converter
- New test - Conversion between port and NodeConnectorRef
- Methods in PortNumberCache are now deprecated
- Replaced use of above methods in PacketReceivedTranslator with new methods
- Modified tests for PacketReceivedTranslator to work with new methods

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

Bug 5747 Introduced snapshot gathering marks

Added timestamp of statistics collection start and finish
and successful collection flag.

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

BUG-5574: introduces sal-groups-batch service impl

 - added bulk API implementation for groups
 - with parametrized barrier-after support
 - added tests

Change-Id: If3e3d70d4ceb8eeb4a9902e2ea7f4bad26d9c934
Signed-off-by: Michal Rehak <mirehak@cisco.com>
(cherry picked from commit 134bd7a822a5a3d76ec82f3bd7ad6abf1a933a60)
Signed-off-by: andrej.leitner <anleitne@cisco.com>

BUG-5574: sal-flows-batch proposal

    - added common model for attaching barrier
    - added bulk API implementation for flows
      - with parametrized barrier-after support
    - added tests

Change-Id: I4e941d3654dc142e1d1d64a2258e629016e95a0e
Signed-off-by: Michal Rehak <mirehak@cisco.com>
(cherry picked from commit 79e8e834f0cbbccfbff99e51e4ddd854dd812d19)
Signed-off-by: andrej.leitner <anleitne@cisco.com>

Revert "Fix "Bug 5589 - Deprecate PortNumberCache""

This reverts commit 0cd77f8ef3c8045fba03dde20044a08ec44e29f5.
Signed-off-by: Michal Rehak <mirehak@cisco.com>
Change-Id: I852bf5dc979ef8a581b3d8326413451cd21cc83a

Remove nexusproxy property

Change-Id: Ie6cc211e3e4a5b39ef6a09c23cb737eab6c57a96
Signed-off-by: Luis Gomez <ecelgp@gmail.com>

Merge "Bug 5592 - DeviceContext -> DeviceState in *ToNotificationTransformer classes"

Merge "Fix "Bug 5589 - Deprecate PortNumberCache" - New class that can convert DeviceState to NodeConnectorRef and NodeConnectorRef to port number. - 1 test that will verify if conversion between DeviceState and NodeConnectorRef is valid. - Made methods in PortNumberCache deprecated and replaced use of them in PacketReceivedTranslator with new methods. - Modified tests for PacketReceivedTranslator to work with new methods."

Fix "Bug 5589 - Deprecate PortNumberCache"
- New class that can convert DeviceState to NodeConnectorRef and NodeConnectorRef to port number.
- 1 test that will verify if conversion between DeviceState and NodeConnectorRef is valid.
- Made methods in PortNumberCache deprecated and replaced use of them in PacketReceivedTranslator with new methods.
- Modified tests for PacketReceivedTranslator to work with new methods.

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

Do not deploy the karaf artifact

The karaf artifact is only used for testing. It does not need to be
released to the world.

Change-Id: Ieca324babbf2cf8ac710808fdf40134981ed38e5
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Bug-5534:RoleService does not take care of rpc errors

Change-Id: Id65e597f5952b689dec176c29ae9e78284123ce2
Signed-off-by: Kamal Rameshan <kramesha@cisco.com>