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"

Merge "Protocol framework: use pooled ByteBuf allocator"

Add BGPCEP logging configuration

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

Fix MD5 channel client channel not working

As it turns out, the client code needs the same fix server-side
received, which is to allow customizeBootstrap() to set channel.

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

Bug 1235: Documented Notification services of MD-SAL

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

Merge "Bug 1290 - Swagger Documentation is failing to load - java script exception"

Merge "Refactor Neutron Pagination Code to support Network, Port and Subnet"

BUG-868: donot use deprecated call

WebSocketServerHandshakerFactory.sendUnsupportedVersionResponse() is the
replacement, so use it instead of
sendUnsupportedWebSocketVersionResponse().

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

BUG-1281: optimize JSON codec

Multiple optimizations made in one go:

- do not use string concatenation for LOG.debug()
- do not use string concatenation in Preconditions.checkState()
- methods which do not need state are made static
- use chars instead of strings where possible
- do not instantiate ArrayList needlessly to access first element in a set
- optimize namespace/localname parsing by not using String.split()
- use Charset constant instead of charset name

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

BUG-1281: Optimized CompositeNodeWrapper

When wrapping entries, we know how many we will need, so let the
ArrayList know that.

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

BUG-1281: optimize ListenerAdapter

TransformerFactory should be reused, so that's what we're doing here.
Here we do not cache the transformer, as we are called from datastore
thread, which has essentially no business in this. Same goes for
DocumentBuilderFactory.

Also, the transformation to RFC3339 timestamp was compiling the same
regex over and over, so compile it once and reuse it.

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

BUG-1281: cache TransformerFactory and Transformer

The code used to instantiate a TransformerFactory for each individual
request, where a single factory can be shared between threads as long as
it is not reconfigured.

As a futher optimization, we also create a thread-local cache for
transformers, as they can be use for multiple transformations.

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

Merge "BUG-1281: optimize ControllerContext"

Merge "BUG-1275: drop a marker for follow-up"

Merge "BUG-868: Fix a warning in EmptyNodeWrapper"

Merge "Add GlobalBundleScanningSchemaServiceImpl.destroyInstance for cleanup in tests"

Bug 1290 - Swagger Documentation is failing to load - java script
exception

Fixed how we generated the json model representing yang union types
Fixed json model naming for nested container types.

To validate:
1) deploy fix, and load http://localhost:8080/apidoc/explorer.
2) Verify the page loads and you can expand
opendaylight-inventory:nodes.
3) Search for "node-connector" and expand a node-connector definition.
Verify that the "port-number" type says "integer or string".

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

Add GlobalBundleScanningSchemaServiceImpl.destroyInstance for cleanup in tests

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

BUG-1281: optimize ControllerContext

- use a single instanceof of Splitter
- do not instantiate unneeded collections
- static methods should be static
- do not use string concat in LOG.error()
- do not use String.split()

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

BUG-1275: drop a marker for follow-up

This class duplicates a lot of code from yangtools, so a proper middle
ground needs to be found. Drop a FIXME: so we do not forget.

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

BUG-868: Fix a warning in EmptyNodeWrapper

Simple use of deprecated method to implement the same method. Add
@Deprecated marker to silence the warning.

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

Merge "BUG-692 Improve log message when negotiation fails"

Merge "BUG-868: remove InstanceIdentifier.getPath() users"

Merge changes I38344133,Ib209ceed,I3f643034,I2f4ed366

* changes:
  BUG-868: getChildren() has been superseded by getValue()
  BUG-868: mark inherited methods as deprecated
  BUG-868: remove deprecated use of QName constructors
  BUG-868: do not use deprecated InstanceIdentifier methods

Refactor Neutron Pagination Code to support Network, Port and Subnet

- Move pagination logic to PaginatedRequestFactory
- Add INeutronObject interface to bound Generics
- Add INeutronRequest interface to enable pagination to be generic

Patchset 4:
- Fix typos in @XMLElement annotation on the 'links' attributes

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

Cleanup: Removed unused code

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

BUG-1041 cli proposal #1

Generic model based cli for netconf servers.

Command are defined with yang language.

Change-Id: I9da0c764a92707d6ede3853bae021d9df01755d7
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Move ServiceReferenceMXBean to config-api"

BUG-868: remove InstanceIdentifier.getPath() users

getPath() has been deprecated. Remove the easily call sites to
getPathArguments().

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

BUG-868: getChildren() has been superseded by getValue()

Silences two eclipse warnings.

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

BUG-868: mark inherited methods as deprecated

These methods are deprecated by the specifying class and we use them
internally. Mark them as deprecated, silencing the warnings.

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

BUG-868: remove deprecated use of QName constructors

Direct instantiation of QName is deprecated, so remove references to
them.

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

BUG-868: do not use deprecated InstanceIdentifier methods

This removes the use of the deprecated constructor and some of the
getPath() call sites in favor of getPathArguments() and
InstanceIdentifier.create()/.node() methods.

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

Move ServiceReferenceMXBean to config-api

Move ServiceReferenceMXBean to api in order to use it in config-util.
Deprecate newMBeanProxy methods in config registry and transaction client.
Make methods newMXBeanProxy aware of service references, add transaction
name silently when using transaction client's newMXBeanProxy.

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

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

Change-Id: Ia2efe44f4eead3f816ec3effc2210ca9bd506c7f
Signed-off-by: Jakub <jatoth@cisco.com>

Merge changes I8598d0bd,I560c2072

* changes:
  Bug 488: Removed BundleContext usage for registering to DOM Components.
  Bug 488: Removed BundleContext from InMemoryDataBroker.

Merge "Bug 488: Removed requirement for DOM providers to supply BundleContext."

BUG-692 Improve log message when negotiation fails

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

Complete implementation of DataChangeListenerRegistration and related classes

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

Complete implementation of DataChangeListenerProxy

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

Bug 762: Fixed bug in CsSnToXmlTest java.

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

Merge "Bug 1081 - Deprecate non-asynchronous Data Broker API"

Merge "Make dependencyResolver protected in generated code"

Bug 1081 - Deprecate non-asynchronous Data Broker API

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

Merge "Bug 1236 - Documented Binding-aware RPC services of MD-SAL"

Bug 488: Removed BundleContext usage for registering to DOM Components.

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

Bug 488: Removed requirement for DOM providers to supply BundleContext.

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

Bug 488: Removed BundleContext from InMemoryDataBroker.

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

Make dependencyResolver protected in generated code

Make Abstract*Module's dependencyResolver protected instead of private,
so that methods like getAttribute can be used from concrete Modules.

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

Merge "Bug 1073: Added Transaction Chain support to Binding transactions."

Merge "Bug 1073: Extracted Implementation of Binding Transaction"

Merge "Bug 1027: Sort map of instance-identifiers and data by length."

Bug 1073: Added Transaction Chain support to Binding transactions.

Added Binding Aware version of Transaction Chain
which only forwards all calls to DOM Transction Chain
and uses BindingForwarded transactions as wrappers
on top of real DOM Forwarded transactions.

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