Apply style rules on whole sal-rest-connector

No changes in function of code, just formatting.

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

Merge "BUG 1330 - list key counts|values diff in payload and URI"

Merge "Introducing simple merge strategy for config subsystem"

Bug 1225: Added DataBroker and DataChangeListener test suite.

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

Bug 1284: Fixed serialization of Augment Instance Identifier

Instance Identifier pointing to Augmentation, which pointed
to augmentation with leafs only, was not properly
serialized to DOM Instance Identifier and this caused
subscription for parent node.

Instance Identifier Codec was updated to detect
this case and use different algorithm to serialize
last argument.

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

Merge "Bug 1223: Cleaned up BindingTestContext from unsupported components."

Merge "IEEE802.1q Packet returns incorrect header size"

Merge "Bug 1139 Modify cors-config according to request"

Merge "Clarified the AsyncDataChangeEvent message - stated you must not block the thread."

IEEE802.1q Packet returns incorrect header size

IEEE8021Q class uses the Packet.getHeaderSize() which iterates through
header fields to determine the size. .1Q header is fixed in size and
should return the correct value

Change-Id: I4b0123dd943f6e833f14782f2ed7cc0e6e184e87
Signed-off-by: Yevgeny Khodorkovsky <ykhodork@cisco.com>

Merge "Add generic exception mapper"

Clarified the AsyncDataChangeEvent message - stated you must not block the thread.

Change-Id: I311b7c37f5848beb351d59fc830798d10302a3bb
Signed-off-by: Devin Avery <devin.avery@brocade.com>

Bug 1139 Modify cors-config according to request

Change-Id: I6276e6a41c43ffb430f520911cd8366a55dc8930
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>

Introducing simple merge strategy for config subsystem

MergeEditStrategy wasn't really merging.
Now it is, but only for ObjectName[].
However, since ObjectName[] is 90% of the interesting
cases, this should get us a long way.

This is needed to be able to handle reasonably
extension, with the ability to wire in multiple
extension modules independently, as we need to do
for things like OF extension.

Change-Id: Ieca579cbc6781d77f489f5f754490c8da8f65188
Signed-off-by: Ed Warnicke <eaw@cisco.com>
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>

Bug 1223: Cleaned up BindingTestContext from unsupported components.

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

Merge "Add BGPCEP logging configuration"

Merge "Bug 1341 - Make RESTConf websocket port configurable"

BUG 1330 - list key counts|values diff in payload and URI

Adds validation whether list key counts or list key values are different
between payload and URI. It difference occures then
RestconfDocumentedException is raised.

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

Merge "Deprecate model-flow-management as it was never used and is confusing people."

Extra Features for Controller (Akka, Spray and Scala)

PatchSet14: Cliped out spray as its not used in controller.

Change-Id: I20a9d9efb5e26686d29bed88c732c397258be7a0
Signed-off-by: Jonathan Fok kan <jfokkan@inocybe.ca>
Signed-off-by: Gabriel Robitaille-Montpetit <grmontpetit@inocybe.com>
Signed-off-by: Mathieu Lemay <mlemay@inocybe.com>

Add generic exception mapper

Change-Id: Ia3fbf4bcea6095e5c0eb6e56e8c886b38e66cd65
Signed-off-by: mrajvaid <mrajvaid@cisco.com>

Bug 1341 - Make RESTConf websocket port configurable

Change-Id: I54f4bcadd864eb264b92f00b82b81489e9ebecf0
Signed-off-by: Richard Kosegi <rkosegi@cisco.com>

Merge "Improve documentation of BindingAware{Provider,Consumer}"

Merge "Bug 1293 - Switch packaging of config-util to bundle"

Merge "Re-added config.version to config-module-archetype."

Merge "Bug 1112: Update toaster to use async best practices"

Merge "Implement PortNumberBuilder"

Merge "Bug 1292: Fix for NPE after notification get."

Bug 1292: Fix for NPE after notification get.

Fixed incorrect pass of SchemaContext in toNotification method in NetconfMessageTransformer class.
Added test for proper testing of resolving of notficiation message.
Added test models and test notification xml payload for testing purposes.

Change-Id: I4ada2fdf976003f4a2809d6cef53816ec39be364
Signed-off-by: Lukas Sedlak <lsedlak@cisco.com>

Bug 1339: Added doublecheck for RPC registration

Change-Id: I786934b6cc690817688e60fd3d422293f37056be
Signed-off-by: Lukas Sedlak <lsedlak@cisco.com>

Deprecate model-flow-management as it was never used and is
confusing people.

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

Merge "Bug 1224, Bug 1221: Added Test case scenarios for Data Change Events."

Merge "BUG 1204 - exception if multiple container node occures"

Merge "Bug 1328: Improved argument checks in generated RPC Router"

Bug 1328: Improved argument checks in generated RPC Router

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

Bug 1224, Bug 1221: Added Test case scenarios for Data Change Events.

Added suite of test-cases which tests Data Change Events
and their contents for various combination of
  - initial state of data store
  - performed operation (e.g. write sibling, child, delete child)
  - various level of data change listener
    eg. (/top or /top/top-level-list/top-level-list[*])
  - various data change scope (BASE, ONE, SUBTREE)

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

Fixed duplicate 'uses' statement in node-errors.yang

Change-Id: I2cc1116416cdfd559fdd3ef3c243040750e0df9b
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Improve documentation of BindingAware{Provider,Consumer}

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

Merge "Add netconf-cli temporary files to .gitignore"

Merge "Exclude protocol buffer generated source from Sonar renamed package to exclude from sonar included warning and note in readme_first for the same Rebased with master Change-Id: I065cb38eba7b84c059a351e0145ce23247e57915 Signed-off-by: Basheeruddin Ahmed <syedbahm@cisco.com>"

Merge changes I6e8805e1,I4df698a0,I41e0e30d

* changes:
  Minor cleanup of test suites in MD-SAL
  Moved data test model to sal-test-model
  Bug 1280: Ensure list parent if written item is list item

Merge "Bug 1283: Made return types of AsyncDataChangeEvent consistent"

BUG 1204 - exception if multiple container node occures

Container schema node + multiple data nodes ->
 exception RestconfDocumentedException is raised.
List schema node + one data node (e. g. in json is specified as container
"list-name":{} and not "list-name":[{},{}...,{}] -> this is acceptable case because
list can have even only one entry.
Leaf schema node + multiple data nodes -> special check whether at
equal objects is everytime uniqe key. If it isn't so then
RestconfDocumentedException is raised. For Json specific JsonParser was
implemented which is catch duplicity at object level an time of parsing.
For XML this duplicity name check is done during normalization phase.

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

Bug 1293 - Switch packaging of config-util to bundle

Add new bundle to the distribution - config-util, that was previously
packaged as regular jar and copied into where it was required.

Change-Id: I8a28b4a0b19e4dc6cbc2d71e60cf714cca188a84
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>

Implement PortNumberBuilder

Change-Id: I9f4f662a7a0f595046c213ea64332ce70b280a78
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>

Add netconf-cli temporary files to .gitignore

Change-Id: Ic5eeab930ad0508a7e373b227d9a814ab4ae0010
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>

Exclude protocol buffer generated source from Sonar
renamed package to exclude from sonar
included warning and note in readme_first for the same
Rebased with master
Change-Id: I065cb38eba7b84c059a351e0145ce23247e57915
Signed-off-by: Basheeruddin Ahmed <syedbahm@cisco.com>

Bug 1112: Update toaster to use async best practices

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

Changing the ovsdb.of.version to 1.3 by default in config.ini

This gerrit must be merged first prior to
https://git.opendaylight.org/gerrit/#/c/7842
in order to keep the ovsdb project dependency assumptions intact.

Change-Id: I47480b5255b4a595aabcac68900726e45319b178
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Merge "Bug 1235: Documented Notification services of MD-SAL"

Minor cleanup of test suites in MD-SAL

- Added SuppressWarning("deprecation") to test suites
  which test backwards compatibility with Hydrogen APIs
- Marked test which needs to be rewritten for new DataBroker API
- Removed superflous version for test dependencies

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

Merge "Bug 1316: Fixed loop in global RPC forwarding."

Bug 1316: Fixed loop in global RPC forwarding.

Change-Id: Ic133471d4883e367230901c8596ccd46f2e9dc08
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>
Signed-off-by: Lukas Sedlak <lsedlak@cisco.com>

Merge "BUG 1016 - JSON|XML output with|without white chars"

Moved data test model to sal-test-model

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

Merge "Cleanup: Removed unused code"

Bug 1280: Ensure list parent if written item is list item

One of properties of Java Binding is that it is imposible
to represent list as a whole and thus it was possible
to create put operation, which was correct by Binding Spec
but failed.

This fixes behaviour, if user is writing list item (MapEntryNode),
and whole list (MapNode) does not exists, list is automatically
added.

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

Bug 1283: Made return types of AsyncDataChangeEvent consistent

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

Merge "Bug 560: Fixed Mount Point RPC registration service"

Merge "Created Proper Feature File for AD-SAL"

Merge "Introduce SimpleNormalizedNodeMessage for serialization of NormalizedNode using protobuff Introduce EncoderDecoderUtil that helps in encode/decode of NormalizedNode usig protobuff Created Distributed Datastore messages in protobuff Included command to remove trailing spaces in the generated java sources of protobuf"

Merge "Initial MD-SAL XML for Distributed Data Store"

Merge "Enhancements to actor naming, logging and monitoring"

Merge "Implement DataChangeListener"

Merge "Fix NPE on reading data"

Introduce SimpleNormalizedNodeMessage for serialization of NormalizedNode using protobuff
Introduce EncoderDecoderUtil that helps in encode/decode of NormalizedNode usig protobuff
Created Distributed Datastore messages in protobuff
Included command to remove trailing spaces in the generated java sources of protobuf

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

Merge "A bunch of miscellaneous services to get Distributed Data Store ready for deployment"

Merge "BUG-1274 OF issue with datapath-id"

Merge "Registering Distributed Datastore for SchemaContext changes"

Merge "Bug 1306: Fixed incorrect AD/MD-SAL flow conversion."

Initial MD-SAL XML for Distributed Data Store

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

Enhancements to actor naming, logging and monitoring

- Actor names have now been changed to be more meaningful. This will be helpful when trying to follow the logging.
- Added logging for when the actor is created and when it is terminated

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

Implement DataChangeListener

- Add a DataChangeReply to confirm receipt of a DataChange
- Created an AbstractUntypedActor class from which all actors extend. Currently the only thing this base
  class does is uniformly log incoming messages. Going forward will automatically register actors for DeathWatch and such.

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

Fix NPE on reading data

When reading data if a null value is returned return Optional.absent();

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

A bunch of miscellaneous services to get Distributed Data Store ready for deployment

- Shard can work with both Persistent and NonPersistent messages which change the state of the InMemoryDataStore
- OSGi related changes so that the DistributedDataStore can load properly in the controller
- Other changes so that there are no startup errors (like creating a single actor system)

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

Registering Distributed Datastore for SchemaContext changes

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

Merge "Kill Dynamic Actors when we're done with them"

Merge "Distributed Datastore integration with config subsystem Updated with the usage of sal-dom-spi exposed Config Datastore and Operation Datastore services"

Merge "Basic DistributedDataStoreIntegrationTest added"

Merge "Complete implementation of ThreePhaseCommitCohortProxy"

BUG-1274 OF issue with datapath-id

- replaced Long with BigInteger in order to capture value greater then maxLong
- added unit test for that

Change-Id: I17d8b0c54c5c084d428883f33aa0972e3dd986ed
Signed-off-by: Michal Rehak <mirehak@cisco.com>

Merge "BUG-1200 changed help for agentpath && added comment describing functions.sh"

Merge "BUG-1200 added functions for help depends on argument (-help arg)"

Merge "Remove commons codec version from commons.opendaylight"

Merge "Bug 1007 - Statistics manager loses threads to uncaught exceptions"

Remove commons codec version from commons.opendaylight

This (1.7) was overriding odlparent's version (1.8)

Change-Id: I86612435ad858a8c1dc4b945d6eb42947b3c6c37
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>

Bug 1306: Fixed incorrect AD/MD-SAL flow conversion.

  * Convert NodeConnector in IN_PORT match field into MD-SAL
    NodeConnectorId.
  * Set false to vlan-id-present if MatchType.DL_VLAN_NONE is set in
    DL_VLAN match field.

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

Bug 560: Fixed Mount Point RPC registration service

The RPC Services can now be correctly registered for Mount Points. The RPCs are able to
correctly propagate between Binding Independent Broker and Binding Aware Broker.

This commit is dependant on commit: https://git.opendaylight.org/gerrit/#/c/8654/

Change-Id: I6c3c3e1f648d1faa8b4d43a24eacaf014123b880
Signed-off-by: Lukas Sedlak <lsedlak@cisco.com>

Merge "BUG-692 Add log to config persister test."

Merge "Adding a method to process Identities in a YANG Model"

BUG-692 Add log to config persister test.

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

Merge "BUG-1041 cli proposal #1"

Adding a method to process Identities in a YANG Model

Change-Id: Ic07025c3c86d847d2882d80fa360e9a5978275df
Signed-off-by: Mayank Agarwal <mayagarw@cisco.com>
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>

BUG 1016 - JSON|XML output with|without white chars

Pretty print formating can be turned on for XML | JSON output via URI parameter
prettyPrint=true.
In other cases (prettyPrint=false, other value, missing
URI parameter...) XML | JSON output are returned as stream of characters
without white characters.

Example of URI:
http://localhost:8080/restconf/config/opendaylight-inventory:nodes?prettyPrint=true

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

Kill Dynamic Actors when we're done with them

Kill ShardTransaction on close and on ThreePhaseCommitCohort#commit
Kill ThreePhaseCommitCohort on commit

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

Distributed Datastore integration with config subsystem
Updated with the usage of sal-dom-spi exposed
Config Datastore and Operation Datastore services

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

Basic DistributedDataStoreIntegrationTest added

Also added missing CreateTransaction handling in Shard and some logging

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

Complete implementation of ThreePhaseCommitCohortProxy

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

Merge "Complete implementation of DataChangeListenerRegistration and related classes"

Merge "Complete implementation of DataChangeListenerProxy"