Removed getClass comparison if FlowComparator.

Change-Id: I1acb4201afe56bd83ac8e8b3f16071981f95c186
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Merge "BUG 932 - Swagger HTTP POST contains incorrect object"

Merge "Fix exception when netconf device provides YIN schemas"

BUG 932 - Swagger HTTP POST contains incorrect object

POST link now contains specific JSON structure which ends with POST suffix
and contains all containers and lists which are direct subchildren of
current node (URI link). It is list of possible objects which can be used
in POST request. Concretelly are these objects specified below this *POST
JSON object and are also all specified in Parameters section of GUI POST
URI link with ** prefix. Two stars (**) mean that only one of parameters
with ** prefix should be specified.

Change-Id: I3acdfd1f181fbef7cca0060d534b706d934bb1e3
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>

Added a missing dependency to codahale's metrics
package in remoterpc bundle

Change-Id: Ieb06010fa56686d46d95625dde3249cee25354b2
Signed-off-by: Abhishek Kumar <abhishk2@cisco.com>

Merge "Bug 1534: Changed blocking calls to async in dist data store"

Merge "Custom mailbox that is bounded and instrumented."

Merge "Moving common code out of remote rpc to clustering common bundle"

Merge "Turn on feature testing for those features that pass"

Merge "Moved the resources to a separate plugin so they can be used by others."

Merge "Revert "Added the Buffer-Id to packet-in.""

Revert "Added the Buffer-Id to packet-in."

This reverts commit 41a6be9bc180cd81e24e5872ca0682e15b37a7f6.

This revert was because it *appears* this patch is breaking integration (not sure how yet).

Change-Id: I1daf9b240c8c31a17e8b4a3f7fc5d20186a6cd0e
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Fix exception when netconf device provides YIN schemas

Change-Id: I8397b709f2202e76f8b25915563482c8d05ee83b
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Merge "Added the Buffer-Id to packet-in."

Custom mailbox that is bounded and instrumented.

This is for use with Akka actors. It enables monitoring
of actor's mailbox size via JMX.

Change-Id: Ic1e478e4411f53ff0239d316a1ca02eaa80360ed
Signed-off-by: Abhishek Kumar <abhishk2@cisco.com>

Turn on feature testing for those features that pass

(includes a fix for features/config)

From this point onward, those features should
break the build if they are broken.

Change-Id: I9eecdabe2e7556a5d60a720f86d7a5e78421b6ef
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Moved the resources to a separate plugin so they can be
used by others.

The idea here would be to allow the projects to build
their own local runnable instances, as well as integration,
while keeping things in sync.

A couple of notes:
1)  Please be patient, this is a pull apart of work to
get features working in general, so there may be
some extra things in commons/opendaylight/pom.xml
for that are for that follow on patch.  Mea culpa.

2)  I created an opendaylight-karaf-empty distribution
that *just* pulls in the resources to be used
for feature testing (see next patch)

Change-Id: Id4816f7083cedde8fe1e1ff2aaa58d581d45c8ab
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Propogating cohort failure back to calling actor

Created new ThreePhaseCommitCohortFailureTest negative test cases

patch 2: updated based on Tom's code review comments
patch 3: reformatted Shard class
patch 4: minor comments taken care of.
patch 5: review comments taken care of
patch 6/7: getSender/getSelf issue fixed
Change-Id: I403d8e7cbf44739b16a21345c645b7b66f9f8ac6
Signed-off-by: Basheeruddin Ahmed <syedbahm@cisco.com>

Moving common code out of remote rpc to clustering common bundle

Change-Id: Iad469f809ac55b2d873ff4d48e912ad66184c68d
Signed-off-by: Harman Singh <harmasin@cisco.com>

Merge "Bug 1430: Obtain config params from config system"

Merge "Modifications to error handling in restconf"

Merge "BUG-1541 Netconf device simulating testtool"

Merge "BUG-1541 Extract AuthProvider interface"

Merge "BUG-1535 Ignore exception on toaster.close"

Merge "BUG-1558 Fix BindingAwareActivator ignoring removedService call."

Merge "Fix precondition formatting"

BUG-1541 Netconf device simulating testtool

Produces executable jar file.

The jar can simulate arbitrary number of netconf devices that listen on configured ports
Schemas are loaded from a provided folder
Can also generate ODL initial config files for started simulated devices to support testing with ODL distribution

Change-Id: I8fce73fa7c568a272c29073f26a4e8aafeebfd82
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

BUG-1541 Extract AuthProvider interface

Change-Id: I8df939b1c712fe85f3cd269df50b01ceee2d9538
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

BUG-1535 Ignore exception on toaster.close

It tries to delete data from store, but if the broker gets closed it cannot remove data since the broker throws exception

Change-Id: I49bbbe410b9f1cb67711974c27f0375dc15c17dc
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

BUG-1558 Fix BindingAwareActivator ignoring removedService call.

Change-Id: I0a062bb144af7d4d70bc3f708e0f919bcce3dc9b
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Bug 1534: Changed blocking calls to async in dist data store

Changed the read and commit methods to use async akka actor calls instead of
submitting tasks to a ListeningExecutorService that do blocking akka
calls. This obviates the need for the ListeningExecutorService.

Change-Id: I7a3f369917431067ad1817a3ab53a358bc21f123
Signed-off-by: tpantelis <tpanteli@brocade.com>

Merge "bug 537 - Node Reconciliation"

Merge "Bug 1503: RestconfError "data-missing" is error code 404."

Merge "BUG-1554: fill out ID/KEY"

bug 537 - Node Reconciliation

https://wiki.opendaylight.org/view/OpenDaylight_OpenFlow_Plugin:Backlog:Node_Status_Reconciliation

* FRM Reconciliation Notify Listener (DataChangeScope.BASE for FlowCapableNode in DataStore/OPERATIONAL)
* FlowCookieProducer - MapHolder for an unique relation flow_cookie to flow_id via FRM
* FlowStatTracker changes
        * search flow_id by flow_cookie for exist flow
* add flow_id - flow_cookie for new Flow
* FRM migration for new DataBroker API
* merge with new DataStore/OPERATIONAL hierarchy (Flow vs. FlowStatistics)
* fixed flow delete - generated cookie is bypassed with mask=0

In additional commit has fixed Bug 282 - Pre-configuration of flows causes exception
+ remove the transaction validators - not needed for now

Change-Id: Iadc2b111780524b9a8fd86cfc248e5fa194e6638
Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>
Signed-off-by: Michal Rehak <mirehak@cisco.com>

BUG-1554: fill out ID/KEY

When a list is being inserted it is required to fill out both its key
and its mandatory leaves.

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

Merge "Adding more unit tests for remote rpc connector and Integrating routing table"

Merge "Hotfix bug 1556 - FRM: NPE thrown in notification"

Bug 1503: RestconfError "data-missing" is error code 404.

The base specification listed the error code to be 409, which is in
direct violation on HTTP. The restconf specification will be fixed in
the upcoming draft, meanwhile we are going for the new, sane behavior
regardless.

Change-Id: I7a6cf67a794f8348d746ba94b7de9580d0472a61
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>

Hotfix bug 1556 - FRM: NPE thrown in notification

* FlowCapableTopologyExpoerter - flow:1 is not exist because it is not created correctly

Change-Id: Ifa5b984329796e5df7aa0f71b1932b73d0ff99a0
Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>

Merge "Revert "BUG-1425: Integrated new Binding to Normalized Node codec for write path""

Revert "BUG-1425: Integrated new Binding to Normalized Node codec for write path"

This reverts commit eb2bd48a9148a4fbeae292ffd03e30083c2ff024.

Change-Id: Ic8183f6620c51d8a4a87a38af1e9a554e7f9016a
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Merge "write /nodes/node/table/0 with ensure parents when node first appears"

Merge "Bug-1520 Config subsystem:config-impl tests"

write /nodes/node/table/0 with ensure parents when node first appears

Change-Id: Ic30ee64e7f1c53de7902aa322fc2de20bcf5b9b8
Signed-off-by: Martin Bobak <mbobak@cisco.com>

Merge "Bug-1520 Config subsystem:config-api tests"

Merge "Bug-1520 Config subsystem:config modules tests"

Added the Buffer-Id to packet-in.

Acordingly with OF-spec Packet-In have a Buffer-Id.

Change-Id: I93a1007a48d1d29c34d981ef6965b3062ad8fc4b
Signed-off-by: André Martins <aanm@ua.pt>

Modifications to error handling in restconf

Modified error handling for read and write Tx to transfer to transfer
RpcError info from ReadFailedException and
TransactionCommitFailedException to the RestconfDocumentedException.

Change-Id: Ifb47f0cdfda5a11d53add9d7ef8fbe50954c0206
Signed-off-by: tpantelis <tpanteli@brocade.com>

Adding more unit tests for remote rpc connector and Integrating routing table

Updated with code review comments

Change-Id: Ie88870e6cf6e9446f352e5b931e5063f5f0ec012
Signed-off-by: Harman Singh <harmasin@cisco.com>

Fix precondition formatting

Preconditions take String.format() arguments, not {} placeholders.

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

Removing additional plugin, not required for config files

Change-Id: I7da3ac25b4d19a91ed7bf198125c597cb0473c56
Signed-off-by: Harman Singh <harmasin@cisco.com>

Merge "Akka config change"

Merge "Move the CompositeModificatoinPayload message to proper package The package of protobuff generated code is excluded from sonar testing"

Bug 948 : Inventory and topology manager  shifted to new DataStore API

Change-Id: I1a3f2fb47742da0622a096bd6305e7a848a59a36
Signed-off-by: Martin Bobak <mbobak@cisco.com>

hotfix Bug 1499 - deleted flow still present

StatisticManager needs to delete the statistics and the flows too.
* fix a path to delete whole flow (no only statistics)

Note: bugfix could help for bug 1345 too.

Change-Id: I8a8bee995d42daa9f1832cc8e26cc2c02b2e64c1
Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>

Merge "Add additional fields to Neutron Router"

Merge changes I162c84f7,Iffa37d3f

* changes:
  BUG-1425: Integrated new Binding to Normalized Node codec for write path
  BUG-1425: package binding-data-codec

Add additional fields to Neutron Router

This adds additional fields to the Neutron Router object that can assist
with providing Distributed L3

Change-Id: I71dabe323a55a9028e54b1b46e923a7e4614f46d
Signed-off-by: Dave Tucker <djt@redhat.com>

Move the CompositeModificatoinPayload message to proper
package
The package of protobuff generated code is
excluded from sonar testing

Change-Id: I4eeae429febe32a8a347c319f6dec60cd0df5946
Signed-off-by: Basheeruddin Ahmed <syedbahm@cisco.com>

BUG-1425: Integrated new Binding to Normalized Node codec for write path

Integrated binding to normalized node codec for write path to datastore,
for deserialization legacy codecs are still used.

This patch requires https://git.opendaylight.org/gerrit/9795
to be merged in YANGTools before.

Removed private dead data broker code from MD-SAL.

Change-Id: I162c84f7da8330d94992122de3af19e76c1e41d2
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>

Akka config change

Renamed host name from localhost to 127.0.0.1
Some jenkins slaves are unable to resolve localhost

Change-Id: I4079dccc33616eea135a516f4cf6f8335f3df50d
Signed-off-by: Abhishek Kumar <abhishk2@cisco.com>

BUG-1425: package binding-data-codec

Preparatory patch, which includes the codec in the base package.

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

Fix comparison logic in neutron floating ip rest implementation

* Detect when external network has no subnets (in addition to more than one)
* Fix compare after looking up NeutronFloatingIP singleton

Change-Id: Id84360fc9d984e9815de56b90be4a1c753fdbcd8
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Bug 1430: Obtain config params from config system

This is a follow-up patch to obtain the various data store executor
config params from the config system intsead of system properties.

Change-Id: Ib7fa03f053d6165fdcb52300be9add8ebe80b2c2
Signed-off-by: tpantelis <tpanteli@brocade.com>

Bug-1520 Config subsystem:config-impl tests

Change-Id: I8c09cbf90585e179f84e3ac1e83e5946379be66f
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Bug-1520 Config subsystem:config-api tests

Change-Id: Iefb8b8ac1a5099c276ab200a3a6337cb8befefe7
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Bug-1520 Config subsystem:config modules tests

Change-Id: I2ad29ce1eb8ba0d5286c426ab3e61627d8e7a8dc
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Merge "Bug-1338: Create a grouping for order to help create generic OrderComparator in OFP"

Merge "BUG-997 Use shared schema context factory in netconf-connector"

Merge "Add exists method on DOMStoreReadTransaction and DOMDataReadTransaction"

Merge "Better handle remove interface from router for unknown port uuid"

Better handle remove interface from router for unknown port uuid

When doing "Remove interface from router" and only port uuid is
provided; the lookup in hash map may return null. If so, there
needs to be a check against NPE.

Change-Id: Ibaea49ac3fdfc08e92c95e818e5ec82628822f3f
Ref: http://docs.openstack.org/api/openstack-network/2.0/content/PUT_removeRouterInterface__v2.0_routers_remove_router_interface_router_ext.html
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Merge "Bug #1532 Neutron Port bindings missing Security Group List"

Merge "Optimizations, Monitoring and Logging"

Bug #1532 Neutron Port bindings missing Security Group List

Neutron Port calls are returning empty SecGroup lists. The
bindings are there, just not the return.

Change-Id: I882f0921af35e690672ee3a728cc2c029c4739fa
Signed-off-by: Brent Salisbury <brent.salisbury@gmail.com>

Merge "Fixed remove rpc api in rpc registry"

Add exists method on DOMStoreReadTransaction and DOMDataReadTransaction

Introducing this API does not impact clients as it is mostly internal.

One reason for adding this API is because of ensureParentsByMerge used in a couple of places
which keeps checking if a parent node exists by reading a node to figure out if it exists. This is
fine for the InMemoryDataStore but it can be terrible in a distributed data store where the shard
which contains the data is remote. All sorts of overhead is associated with a remote read including
serialization which can actually be pretty expensive.

Change-Id: Ib5be5f6dc60be683d7a04c81dad08c56cd5681f4
Signed-off-by: Moiz Raja <moraja@cisco.com>

Merge "Working md-sal features, including restconf, toaster, flow-services"

Merge "Fix to topology-lldp-discovery to allow for clean karaf feature"

Merge "config-persister-feature-adapter to push configs from karaf features"

Merge "Minor changes to config-persister to allow for karaf"

Fixed remove rpc api in rpc registry

Added tests

Change-Id: Ib5204f5226d11a1d8cdec7cf99809333bd1f1457
Signed-off-by: Abhishek Kumar <abhishk2@cisco.com>

Merge "Updated RpcRegistry to accept a list of route identifiers while adding or removing routes."

Optimizations, Monitoring and Logging

- Made identifiers type-safe
- Changed logging so that most of it is done at debug level instead of info
- Make cluster logging configurable via logback.xml
- Add Bean for looking at the local shards on shard manager
- Add Preconditions in a places where it matters

Change-Id: Ie2b17e89bd88edde1366d0a4d23abf9fb97e6e55
Signed-off-by: Moiz Raja <moraja@cisco.com>

Merge "Missing capability in 10-rest-connector.xml file"

BUG-997 Use shared schema context factory in netconf-connector

+ Fix disconnect by netconf-connector. Until now, the connection to remote device was still open even if initialization failed.

Change-Id: I23ee999c7898665498519abf19610deef3d410a3
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Handling ShardTransaction Failure Scenarios
and corresponding test cases

Note: Cohort failures handling is being worked on
and will be taken care in separate commit

Change-Id: I7ef165b9cb8c91088b4f20ae6c19418f92edb37c
Signed-off-by: Basheeruddin Ahmed <syedbahm@cisco.com>

Updated RpcRegistry to accept a list of route identifiers while
adding or removing routes.

Removed few redundant defensive null checks.

Change-Id: I24310d2a3871a4bff2afa5a06b3657f4d2709794
Signed-off-by: Abhishek Kumar <abhishk2@cisco.com>

Include lastCommittedTransactionTime in Shard JMX
MBean
Changed the maximum-frame-size to a larger value

patch 2: rebased
patch 3: merge with other changes
patch 4: commit not returning if cohort was null, included failedTransactionsCount
patch 5/patch6: updated with review comments
patch 7: reformatted based on comment

Change-Id: I85218131dc8edb51f4c2c9fcbb34be79b547789c
Signed-off-by: Basheeruddin Ahmed <syedbahm@cisco.com>

Working md-sal features, including restconf, toaster, flow-services

Note: DO NOT MERGE until
https://git.opendaylight.org/gerrit/9847 (yangtools)
is merged as it is needed for this change.

Change-Id: Iad975cfb63b62930465edcd9e415856873b5acde
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Fix to topology-lldp-discovery to allow for clean karaf feature

Change-Id: Ife3388610a2d816f7bb8e5214cc98d123c4b604b
Signed-off-by: Ed Warnicke <eaw@cisco.com>

config-persister-feature-adapter to push configs from karaf features

Change-Id: I66063a44210d1a6d7804903f17bc8fc0b2995af8
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Minor changes to config-persister to allow for karaf

These minor changes expose a ConfigPusher service for use
by a config-persister-feature-adapter (to follow) that
can push configs attached to features.

Change-Id: I382d2ab578d19ea9c24afa521a9a5a047360fc50
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Merge "Fix broken 05-clustering.xml"

Merge " Expose more information about a Shard via JMX"

Merge "Bug 1430: Off-load notifications from single commit thread"

Merge "BUG 907 distinguish augmented nodes"

Merge "BUG-1511: add FIXME markers"

BUG 907 distinguish augmented nodes

All nodes and their child contains also information about module from
which they comes from (module_name:node_name)

Change-Id: I7b146e820da1b5d3bdb62510a48e6845a78c80a2
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>