fix bug 431

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

Merge "Modify config-api exceptions, bump config and netconf to 0.2.5-SNAPSHOT."

Modify config-api exceptions, bump config and netconf to 0.2.5-SNAPSHOT.

Change ConflictingVersionException and ValidationException to be checked instead of
runtime exceptions.
Fix ConfigPusher's ConflictVersionException handling that was introduced when IOException was
used to wrap RuntimeException.
Small fixes in netconf, more refactoring of exception handling will be needed there.

Maros Marsalek: Rebase commit + fix compliation issue in ShutdownTest

Change-Id: Iab89de6ef6ced0de0a267470f860912ae3c56892
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Add ModuleInfoBackedContext loading strategy to RuntimeGeneratedMapping service.

RuntimeGeneratedMappingServiceImpl is now started by the config-manager.
ModuleInfoBackedContext is filled by BundleTracker in config-manager.

This fix allows use of yang identities in netconf and restconf.

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

Add logging for HashMapDataStore

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

Merge "Activate enforcement of Path type"

Activate enforcement of Path type

This activates pre-prepared requirement of P to be of type Path<P>,
which had prerequisites in generated DTOs. Yangtools have been updated,
so it should be safe to activate this now.

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

Fix a few more warnings

- Remove unused string constants
- Add type arguments
- Removed unused field

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

Optimize empty transactions

This is a micro-optimization: if we know there are no affected paths,
we used to spin an entire transaction -- which ended up revalidating an
empty delta. Let's skip the entire circus and proceed to commit
directly.

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

Another round of warnings fixes

Removes one unused method
Fixes raw type usage where possible
Organize imports to get rid of unused/duplicate imports
Remove the use of deprecated InstanceIdentifier methods

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

Place an upper bound on notification concurrency

In Cbench tests we have seen as many as 2000 threads being created. This
is certainly an overkill and thus let's place an upper bound on the
number of threads we create.

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

Merge changes I393151af,I0a302cde

* changes:
  Fix unused field warnings
  Fix warnings about overriden versions

Fix warnings about overriden versions

With proper parent structure, version overrides should not be present.

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

Merge "Add a bit of documentation"

Add a bit of documentation

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

Merge "Bug fixes for netconf southbound plugin."

Bug fixes for netconf southbound plugin.

Fixed wrong operation type sending in edit-config for netconf device. Action type replaced for "operation" type.
Fixed conversion of toRpcMessage in NetconfMapping. Fixed wrong QName - changed from "action" to "operation".

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

Merge "Fixed bug when RPC was not propagared"

Fixed bug when RPC was not propagared

 - Fixed DOM Broker and BI Connector code
 - The refactoring was done so that DOM Broker does not get exposed to
 - BA Broker directly but rather through a proxy. This also fixes the
   issue where DOM Broker would not be initialized in BI Connector, therefore
   any messages from BA Broker would not flow to DOM Broker via BI Connector.

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

Merge "Fixed bug when Binding-Aware Data Change Listeners we're not triggered."

Fixed bug when Binding-Aware Data Change Listeners we're not triggered.

  - Added additional strategy to inspect changes based on registered listeners
  - Fixed normalization of data for changes where parent node was written
      - Extracts data for listener, so listener does not need to
        extract that data manually
  - Splitted AbstractDataBroker.xtend into several classes

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

Merge "sal-restconf-broker initial implementation  needs https://git.opendaylight.org/gerrit/#/c/5259/  and https://git.opendaylight.org/gerrit/#/c/5282/ to be merged for  successful build"

sal-restconf-broker initial implementation
 needs https://git.opendaylight.org/gerrit/#/c/5259/
 and https://git.opendaylight.org/gerrit/#/c/5282/ to be merged for
 successful build

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

Merge "Fixed removals from data store did not clean direct-match indexes to child nodes."

Merge "Fixed bug when publisher's thread was blocked until all notifications were delivered."

Fixed removals from data store did not clean direct-match indexes to child
nodes.

  - Added deepRemoveStrategy, which checks for all quick indexes
    if they are removed correctly.

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

Fixed bug when publisher's thread was blocked until all notifications were delivered.

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

Merge "Fix a few eclipse-reported warnings"

Merge "Cleanup imports/whitespace in MD-SAL"

Cleanup imports/whitespace in MD-SAL

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

Fix a few eclipse-reported warnings

- unused suppresswarnings
- missing serialVersionUuid
- missing generic arguments

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

Merge "Fixed publishDataChangeEvent in 2phase commit"

Fix unintentional breakage

Yangtools commit 6e7a42bd344f5928343ad67328bc8a8a734bea2e introduced a
new method which broke the controller-side proxy. Fix it up by simply
forwarning it through the delegate.

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

Merge "MD-SAL transaction chaining API"

Merge "Removed unused code from md-sal."

Merge "Restrict MD-SAL data operations to a single thread"

Merge "Clarify the semantics of put{Configuration,Operational}Data()"

Clarify the semantics of put{Configuration,Operational}Data()

This just adds documentation to how the put operations interact with
pre-existing data.

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

Make sure transaction applies whole removal list

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

Fixed bug when new childs were ommited during data store merge.

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

MD-SAL transaction chaining API

This API complements the normal MD-SAL transaction API. Applications can
use this API to maintain inter-transaction ordering and data
consistency.

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

Restrict MD-SAL data operations to a single thread

MD-SAL does not give us transaction ordering guarantees, which mean that
data may hit the data store than what the application intended. Change
the model to single-threaded until proper transaction ordering APIs are
defined.

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

Merge "Fix typo and add more descriptive error"

Fix typo and add more descriptive error

When comparing keys, also provide a hint to the node which violated the
key constraint.

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

Bump yangtools to 0.6.2-SNAPSHOT

This will pick up upcoming yangtools release.

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

Removed unused code from md-sal.

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

Merge "Added utility SynchronizedTransaction wrapper implementation."

Merge "Bug 387 : Fixed Binding Independent Connector to properly serialize output."

Merge "Fix of issue 377 for Notifications resolving."

Merge "Added explicit locking of the data change listener / commit handler registration"

Added utility SynchronizedTransaction wrapper implementation.

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

Bug 387 : Fixed Binding Independent Connector to properly serialize output.

Change-Id: Ibabb08983ca31182e087edbdf6821bd1a0216516
Signed-off-by: Robert Gallas <rgallas@cisco.com>
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Fix of issue 377 for Notifications resolving.

Fixed issue when Notification didn't pass from Netconf client to registered and configured ODL plugin application. After Fix, when notification is send to ODL,
notification is forwarded through md-sal directly into application that is registered for listening on mount point where notification arrive.

Change-Id: I4e0c644efb72ba0c26c75e9bd7452afbb0f4897c
Signed-off-by: Lukas Sedlak <lsedlak@cisco.com>
Signed-off-by: Ed Warnicke <eaw@cisco.com>
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Added explicit locking of the data change listener / commit handler registration

  - This locking fixes ConcurrentModificationException in cases
    when transaction processing is computing affected commit handlers
    and data change listeners.

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

[maven-release-plugin] prepare for next development iteration

[maven-release-plugin] prepare release master-tagforprepareonly-controller-bulk-release-2-1

Update to new version of the artifacts proposed by jenkins-controller-version-changes-16

Patch 2:  Added manual fixes not caught by mavne change versions plugin
Patch 3:  [chrisw] added two more manual fixes

Change-Id: I84e1e32de85ab817f1925a5a008088e3cfb91136
Signed-off-by: jenkins-controller <jenkins-controller@opendaylight.org>
Signed-off-by: Ed Warnicke <eaw@cisco.com>
Signed-off-by: Chris Wright <chrisw@sous-sol.org>

Fixed publishDataChangeEvent in 2phase commit

Modified publishDataChangeEvenet to work asynchronously in TwoPhaseCommit

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

Merge changes I8c23739a,Ia0e70828

* changes:
  Bulk-add copyright headers to java files
  Bulk-add copyright headers to .xtend files

Merge "Fix RPC forwarding related bugs in Binding Independent Connector"

Merge "Changed multimaps to synchronized multimaps"

Bulk-add copyright headers to java files

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

Bulk-add copyright headers to .xtend files

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

Changed multimaps to synchronized multimaps

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

Fix RPC forwarding related bugs in Binding Independent Connector

  - Fix for set of small bugs which prevented invocation of RPC via Restconf

Change-Id: I090516187007c5b1411fd6a9aecf25724465228a
Signed-off-by: Robert Gallas <rgallas@cisco.com>
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Added listener registration when not reusing global instance

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

Resolve the service reference before intantiating proxy for it

The original code assumed that if it has a ServiceReference, that
service does not disappear. Now it resolves it before deciding whether
to use a proxy.

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

Bug 319: Fixed two-phase commit verification of data.

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

Extract dom-serializer-api|impl from binding-broker-impl

Change-Id: I1b2844bfac5062fd15c1f011c3104b3dddb33b51
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Merge "Make MD-SAL use opendaylight parent"

Make MD-SAL use opendaylight parent

Since MD-SAL code does not comply with coding style enforced by the
checkstyle plugin, disable its execution for now -- BUG-272 tracks the
need to re-enable it.

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

Fixed incorrect null check.

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

Merge "Change target to ${project.build.target} in a bunch of pom file. Add some xtend-gen source dir to the build path. The commit purpose is to eliminate errors in eclipse."

Introduced advanced strategy for parsing of YANG schemas

  - This strategy increases convergence time, because it does not
    parse files with unsatisfied dependencies.

  - New strategy for schema parsing in runtime uses two separate parser
     Minimalistic - is used when YANG source is discovered and scans only
     for module name, revision and dependency information
     Full - parses set of YANG sources and creates model representation.

     When bundle is added / removed - tracker gets yang sources,
     registers it to SchemaResolver, which immediatelly uses
     Minimalistic parser to obtain base information.

     Then context of all known sources and their dependencies is sorted
     and filtered only to use sources which has their dependencies resolved.

     Full parser is then invoked only for models, which have dependencies
     resolved.

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

Merge "Fixed bug when global RPCs we're not forwarded"

Merge "Added DEBUG log statements for transaction translation."

Merge "Fixed inappropriate uses of log level INFO"

Fixed inappropriate uses of log level INFO

This is done per the recommendation here:

https://wiki.opendaylight.org/view/Draft_Syslog_Level_Settings

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

Change target to ${project.build.target} in a bunch of pom file.
Add some xtend-gen source dir to the build path.
The commit purpose is to eliminate errors in eclipse.

Change-Id: I90727a0096154d7633f4b6cc309dcc09c9a51678
Signed-off-by: Elek Jani <elekjani@gmail.com>

Fixed bug when global RPCs we're not forwarded

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

Added DEBUG log statements for transaction translation.

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

Merge "Default DataBroker does not read subtrees from Mountpoint"

Merge "Log transactions being allocated"

Default DataBroker does not read subtrees from Mountpoint

Changed default behaviour of DataBroker, which involved
also read from remote Netconf nodes to return only
data from local store. To access Netconf device
as a datastore one must get MountInstance for it -
MD-SAL instance exclusive for that device.

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

Log transactions being allocated

We do get logs when the transaction processing starts, unfortunately we
do not get the context in which the transaction was created. This fixes
the deficiency by logging when the transaction is first allocated. Also
adds explicit logs about transaction state transitions.

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

Merge "More defensive RPC handling in DOM Broker"

More defensive RPC handling in DOM Broker

  - RPCs registered to DOM broker now must comform to the schema
    - at the time of registration of RPC implementation,
      the YANG schema must be present for that RPC.
      This does not affect RPC registrations for Binding Aware Broker.

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

Merge "Commented out Sys.outs in runtime generated code."

Merge "Update to new version of the artifacts proposed by jenkins-controller-version-changes-11"

Merge "Enhancements to remote rpc server. Using zmq router-dealer bridge to make the server async. Enhancements to remote rpc client. Using zmq router-dealer bridge to make the client async. Added configuration for remote rpc in Configuration subsystem On Server startup reading of routingtable to populate remoterpcs was giving exception - fixed the same Client was not registered with Server and addRPCImplementation was failing - fixed the same ServerImpl was not getting registered as listener to RoutingTable hence announcement was not received on remote - fixed the same Patch 6: Some unit tests were intemittently hanging. ZMQ Test server was not gracefully shutting down."

Commented out Sys.outs in runtime generated code.

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

Update to new version of the artifacts proposed by jenkins-controller-version-changes-11

Change-Id: If1cdd56839fc0e15cb2c1fa946897b58d7bcfa37
Signed-off-by: jenkins-controller <jenkins-controller@opendaylight.org>

Enhancements to remote rpc server. Using zmq router-dealer bridge to make the server async.
Enhancements to remote rpc client. Using zmq router-dealer bridge to make the client async.
Added configuration for remote rpc in Configuration subsystem
On Server startup reading of routingtable to populate remoterpcs was giving exception - fixed the same
Client was not registered with Server and addRPCImplementation was failing - fixed the same
ServerImpl was not getting registered as listener to RoutingTable hence announcement was not received on remote - fixed the same
Patch 6: Some unit tests were intemittently hanging. ZMQ Test server was not gracefully shutting down.

Change-Id: I6054443b39394f82258522205ccd4be470f597f0
Signed-off-by: Basheeruddin Ahmed <syedbahm@cisco.com>
Signed-off-by: Abhishek Kumar <abhishk2@cisco.com>

Use version 0.6.0-SNAPSHOT of yangtools everywhere in controller.

As of yangtools commit f5e9689b95d945c22c7a38870681cab899326de9, all projects
in yangtools repo have version 0.6.0-SNAPSHOT. This commit bumps all
yangtools dependencies to this version.

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

Fixed deployment of mount points (dynamic instances of MD-SAL)

  - Converted some xtend sources into Java classes
  - Added mount point deployer - service responsible for
    deploying dynamic instances of MD-SAL, which have
    separate context and could be created and retrieved via
    MountService.

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

Updated implementation of Netconf, fixed DOM Mountpoint

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

Merge "Fix a few POM versioning/dependency warnings"

Fix a few POM versioning/dependency warnings

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

Switch from using a Long as FlowId to a Uri

Couple of things you may want to pay attention to,

a. FlowProgrammerAdapter maintains a mapping between a Flow and it's id in a map in infinispan
b. Look at internalAddFlowAsync. If it finds that a flow is in the flow to id mapping cache. It will remove the flow and add it back in. This is what makes the switch add back a flow when it a switch comes back alive.
c. The sync vs async behavior should now be properly done

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

Merge "Fixed bug when converting augmentation in Instance Identifier"