Merge "Having InMemoryDOMDatastore in its own bundle.To make it configurable using the config subystem. patch 5 changes: md-sal/pom.xml correction"

Merge "Bug:1238 - Revert changes to SshClientAdapter."

Merge "BUG 1179 - rpcResult = false but response 200"

Having InMemoryDOMDatastore in its own bundle.To make it configurable
using the config subystem.
patch 5 changes:
md-sal/pom.xml correction

patch 4 changes:
Moved data tree api and impl to yang-tools based on earlier comment

Patch 3 changes:
1. Included sal-inmemory-datastore dependency in TestHelper.java for PaxExam
integration test cases [Weird that it was working earlier and my local controller full build were passing]

Patch 2 changes:
1. Keeping schema service as part of opendaylight-dom-broker-impl for backward comptability scenario
2. Initializing InMemoryDOMDataStore by default if config-dom-datastore and
operation-dom-datastore are not found
3. Updated 01-md-sal.xml based on 1.
4. Regarding the comment of moving data tree in yangtools. It shall be done in
separate patch.

This is done to replace InMemoryDOMDatastore with distributed-datastore

The changes done are:
1. Created a new sal-inmemory-datastore bundle and moved the
store/impl source files from sal-dom-broker and the corresponding
InMemoryDOMDatastore test cases.

2. Created two yang config models in sal-dom-spi that represent
the configurable config-dom-datastore and operational-dom-datastore, that
are of service type DOMStore

3. In opendaylight-dom-broker-impl.yang of sal-dom-broker, in the augment of dom-inmemory-data-broker added two container config-data-store and operational-data-store that refer the service implementations of service defined in sal-dom-spi

4.In sal-inmemory-datastore bundle, created a yang model opendaylight-inmemory-datastore that provide the implementation of the sal-dom-spi service definitions mentioned in 2 -- The implementaions are augmented to have schema-service container required by InMemoryDOMDatastore for registering itself for scheam changes

5. Updated the 01-md-sal.xml with the above changes

6. Update the distribution pom.xml to include sal-inmemory-datastore.

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

Merge "Helium: improve contract with ListenerRegistration"

BUG 1179 - rpcResult = false but response 200

Case for case when collection of errors is returned was added to
RestPostOperationTest.

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

Merge "Introducing Protocol Buffers Encoding Required for Distributed Datastore Messages Serialization"

Introducing Protocol Buffers Encoding
Required for Distributed Datastore Messages Serialization

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

Merge "Bug 1212: Values of VLAN match fields were wrong."

Bug 1129 - POM Versioning changes for automated weekly releases -  Moved all ommon third party dependencies to odlparent

${nexusproxy} variable does not get replaced after introducing odlparent. This is a known bug in maven 2 & 3 where property replacement does not work reliably.

http://stackoverflow.com/questions/1363514/maven-does-not-replace-a-variable-in-settings-xml-when-it-is-invoked

As a workaround, I had to use complete url for nexus in repository and distribution management sections.

Change-Id: I78f31e0446cebec6f84b9d3959645ad7e440989e
Signed-off-by: Surekha Bejgam <sbejgam@cisco.com>

Remove remoterpc dead code.

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

Merge "BUG 484 Anyxml normalized node"

Bug:1238 - Revert changes to SshClientAdapter.

Fix bug introduced in 412e6b4 by reverting back to previous state.
Also fix a resource leak in writeImpl.

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

Merge "Bug 1176: Lower log level when netconf ssh connection fails"

BUG 484 Anyxml normalized node

- DataNormalizationOperation: added AnyXmlNormalization for
  AnyXmlSchemaNode

- Moved DataNormalizerTest to sal-common-impl where the source code
  resides and added more tests including coverage for anyxml.

Originally commited under patch set 9
https://git.opendaylight.org/gerrit/#/c/7337/9 by Tom Pantelis

This commit depends (therefore is commited as draft) on commits:
- Add Anyxml normalized node to yang-data-api -
https://git.opendaylight.org/gerrit/#/c/7771/1
- Implementation of Anyxml builder -
  https://git.opendaylight.org/gerrit/#/c/8061/

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

Merge "BUG 484 - XmlReader improve readability"

Fixed concurrency bug in DOMTransactionChainTest

Delete transaction was commited before read was verified
since transaction commit was done in other thread
there was race-condition between allocating transactions
to verify datastore state.

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

BUG-944 Remove backwards compatibility config from sal-netconf-
connector.

BREAKS BACKWARDS COMPATIBILITY FOR CONNECTOR CONFIGURATION.
SEND EMAIL TO MAILING LIST ABOUT BREAKING COMPATIBILITY FOR NETCONF CONNECTOR BEFORE MERGE!
ALL DISTRIBUTIONS THAT USE NETCONF-CONNECTOR NEED TO UPDATE CONFIGURATION OF CONNECTORS ACCORDING TO 99-netconf-connector INITIAL CONFIG FILE

Deprecated config attributes and code removed.

Extracted common connector modules(threadfactory, threadpool and client dispatcher) from config file 99-netconf-connector into 01-netconf

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

Merge "Fix netconf ssh client"

But 1129: Removing hard coded versions from MD-SAL

Change-Id: Ic4ea06f1fd644808946dd5111d38152d19b9bf35
Signed-off-by: Vaishali Mithbaokar <vmithbao@cisco.com>

Merge "Bug 1129: Removing hard coded versions from config subsystem"

Merge "BUG 1129:Removed hard coded version from netconf project as part of the automated release changes for controller"

Merge "Cleanup: Removed unused inventory-config model."

Merge "API Clarity: Documented Async Data Broker APIs."

Merge "BUG-961 Fix Resource leak error from netty."

Cleanup: Removed unused inventory-config model.

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

Merge "Fixed yangtools feature repo, bug with base and lang3 property version"

Fixed yangtools feature repo, bug with base and lang3 property version

Change-Id: Ibd5095501b0f9fbe585ad1c6389e0385448f9321
Signed-off-by: Mathieu Lemay <mlemay@inocybe.com>
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Merge "Inventory manager: get rid of synchronized blocks"

Merge "BUG-1138 - wrong value in OFPXMT_OFB_VLAN_VID in FLOW_MOD"

Merge "Bug 1112: Updated toaster-it configuration to use InMemory Data Store."

Merge "Bug 1144: Fixed mising DataChangeEvents for leaf listeners."

BUG-1138 - wrong value in OFPXMT_OFB_VLAN_VID in FLOW_MOD

- vlanPresent set to true if setting vlanId value
- checkstyle whitespace fix

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

Merge changes I0df1f18f,I4f635329

* changes:
  Introduce ShardStrategy and related code
  Initial implementation of ListenerRegistration actor and all related messages

Merge "Initial implementation of ShardTransaction and all the messages it handles"

Merge "Implement basic ShardTransactionChain#CloseTransactionChain"

Merge "Initial implementation of ShardTransactionChain"

Merge "Add message to update the schema context of the InMemoryDOMDataStore"

Merge "Initial implementation of the Shard Actor"

API Clarity: Documented Async Data Broker APIs.

Documented AsyncDataBroker APIs, which are
base abstract APIs from which Binding and DOM
data broker APIs are derived.

Introduced code-examples illustrating
transaction isollation and conflict detection.

Change-Id: I3c881b1fceb0b68c0cc28a1caa7780b9b6c2af3f
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>
Signed-off-by: Devin Avery <devin.avery@brocade.com>

BUG 484 - XmlReader improve readability

- XmlReader - improve readibility of methods isSimple(Composite) node by
  extracting common logic to method skipCommentsAndWhitespace.

The change comes from patch set
https://git.opendaylight.org/gerrit/#/c/7337/9
which was originally commited by Tom Pantelis. In this commit is selected
only one change which doesn't relate with BUG 484

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

Helium: improve contract with ListenerRegistration

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

Inventory manager: get rid of synchronized blocks

This patch reworks the inventory manager to operate on batches, such
that it does not block notification threads and lowers the pressure on
the datastore.

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

Bug 1112: Updated toaster-it configuration to use InMemory Data Store.

As it turned out toaster-it used configuration with legacy data store
and not new one.

Updated controller config for toaster-it to have latest configuration
for MD-SAL.

controller.xml was restructured in such way, it is
easy to compare 01-mdsal.xml with controller.xml using diff
or meld tool.

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

Bug 1231 - fix of broken tests in sal-rest-connector

Dummy values were added to places where null values as SchemaPath were
presented.

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

Bug 1245: Dropped Binding prefix from Binding Data APIs.

Dropped Prefix Binding from Data API of Binding-Aware part,
to make it easier for incomming users to find them as entry
point to the system (eg. DataBroker).

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

BUG-961 Fix Resource leak error from netty.

Release ByteBuf instance in ChannelInputStream.

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

BUG-944 Fix nested union types in config subsystem module attributes

Before this patch only unions of non-union types are supported.

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

Merge "Fix checkstyle errors in sal-compatibility"

Merge "Bug 629: Make BindingDataBroker to be visible."

Merge "Bug 629: Make SchemaService required for Data Brokers"

Fix checkstyle errors in sal-compatibility

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

Merge "Add copyright messages for java files"

Merge "Introducing ShardManager"

Bug 1212: Values of VLAN match fields were wrong.

Values of match fields for VLAN tag were wrong when used AD-SAL OF
plugin and tried to create a flow entry which matched packets without
VLAN tags. V6Match class of AD-SAL OF plugin didn't have code for
handling packets without VLAN tags.

Changes:
 * Added code for handling packets without VLAN tags to V6Match class.
 * Added test cases related to the change of V6Match class.

Change-Id: Ic9c3a6c47c7e99d5273e48ec2ed8d0399289cddb
Signed-off-by: Hideyuki Tai <h-tai@cd.jp.nec.com>

Merge "Swagger UI for AD-SAL"

Bug 629: Make BindingDataBroker to be visible.

Inroduced support for injecting and retrieving
new BindingDataBroker via config-subsystem,
OSGI.

Note: This patch does not provide access to
BindingDataBroker is still not available via
Consumer/Provider context.

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

Bug 629: Make SchemaService required for Data Brokers

Binding Aware versions of Data Brokers did not
required SchemaService when constructed.
Refactored their constructor to make it
obvious that Schema Service is required.

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

Merge "BUG-1167: missing Notification of FLOW_REMOVED"

Bug 977: Return RpcError result on neconf failure

NetconfDocumentedException:
   - Added methods to convert to and from XML.

NetconfMessageTransformUtil:
   - Added code to convert a NetconfDocumentedException to an RpcError.
   - Modified checkSuccessReply and checkValidReply to throw
     NetconfDocumentedException instead of IllegalStateException.

NetconfDeviceCommunicator:
   - Modified processMessage to return an RpcError result for invalid
     message and netconf error reply.
   - Modified other places that previously returned an exception in the
     Future to intsead return an RpcError result.
   - Improved  synchronization by removing synchronized from method sigs
     and locking only the code that needs it.

SendErrorExceptionUtil:
   - Modified createDocument to call
     NetconfDocumentedException.toXMLDocument

XmlNetconfConstants:
   - Moved to netconf-api so NetconfDocumentedException could access it.

The rest of the changed files were to change the import for
XmlNetconfConstants.

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

Bug 1144: Fixed mising DataChangeEvents for leaf listeners.

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

Bug-835:Reserve ports should be logical ports

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

Bug 1129: Removing hard coded versions from config subsystem

Change-Id: I3342267ccc354e0979db754812813171c42af772
Signed-off-by: Vaishali Mithbaokar <vmithbao@cisco.com>

Introduce ShardStrategy and related code

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

Merge "Prepare for security-related catalina/tomcat version updates"

Prepare for security-related catalina/tomcat version updates

In preparation for updating tomcat/catalina versions to address
security issues, move catalina/tomcat version numbers to
properties in pom.xml

Change-Id: I619cd7b0cc01dd2437d3d0ff1286e14137dbb3ec
Signed-off-by: Ryan Moats <rmoats@us.ibm.com>

Merge "Bug 931, Bug 910 - Enhance Restconf Swagger Documentation"

Merge "BUG-1140: inPort disappeared from match"

Merge "Bug 225 - added support for depth RESTCONF parameter in URI"

Merge "BUG-1105: Implement subtree filtering in netconf"

Initial implementation of ListenerRegistration actor and all related messages

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

Initial implementation of ShardTransaction and all the messages it handles

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

Implement basic ShardTransactionChain#CloseTransactionChain

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

Initial implementation of ShardTransactionChain

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

Incorrect connect API is called in ConnectionMgr when NodeType is provided

Change-Id: I73d7539cfbd97cfd2f3da570c8e29ff0787d8f18
Signed-off-by: Pramila Singh <pramisin@cisco.com>

Add message to update the schema context of the InMemoryDOMDataStore

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

Swagger UI for AD-SAL

 -Use 'mvn clean install -P docs' to build swagger ui bundle
 -UI can be accessed at  http://<controller-ip:contoller-web-port>/swagger

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

Initial implementation of the Shard Actor

Things to note,
- Added a temporary dependency on sal-broker-impl. This is so that I could use the InMemoryDOMDataStore.
  Once InMemoryDOMDataStore is moved to it's own bundle then I will simply switch the dependency
- Shard has been only implemented to the point where it is using an InMemoryDOMDataStore and handling the CreateTransaction and RegisterChangeListener messages
  This commit is intended to give a feel for what kind of coding patterns will be used to implement Shard and related actors and their tests

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

Add copyright messages for java files

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

Introducing ShardManager

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

BUG-1140: inPort disappeared from match

- added inPort into ToSalConversionsUtils.toMatch
- added utility method for creating NodeId out of NodeConnectorId and unit test

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

Bug 931, Bug 910 - Enhance Restconf Swagger Documentation

    1) We now generate different data models for config and operation data stores
    2) We no longer generate URLs for leaf nodes since RESTConf doesn't support those
    3) We now expose our mount points in our system in a list, which a user can select and view the list of mounted resources
    4) Cleaned up non-ODL branding, adding in ODL branding.

    To test
    A) Add a device which has mounts in your system (consider ping 2.0 https://wiki.opendaylight.org/view/Ping_2.0_-_Discover_and_Model_Plugin or a netconf device)
    B) Navigate to http://localhost:8080/apidoc/explorer)

Patch Set 2: Original Change
Patch Set 3: Rebased, remformmated existing code which used 2 spaces instead of 4
Patch Set 4: Kicking off a second build, since I got mixed build messages.

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

Merge "Bug 629: Introduction of Configuration Commit Handler"

Merge "Added container wrapper against action to support augmentation"

BUG-272: Fixed last checkstyle offenders & activate checkstyle enforcement

This patch adds xtend-gen to excluded directories and finally removes
the enforcer plugin override -- thus making md-sal components subject to
controller-wide checkstyle rules.

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

BUG 1129:Removed hard coded version from netconf project as part of the automated release changes for controller

Change-Id: If4884987d8a0683a7a7817bc724fa707a93e3628
Signed-off-by: Vaishali Mithbaokar <vmithbao@cisco.com>

Merge "added containermanager shell feature"

Merge "Add some unimplemented proxy classes related to DOMStore"

Merge "Add more yang/sal dependencies"

Merge "Hook the Distributed Data store into the config sub-system"

Merge "Add akka dependencies"

Add some unimplemented proxy classes related to DOMStore

Added basic implementation of DistributedDataStore which does not do much more than fill
in the interface and put tests in place.

All the Proxy classes are still unimplemented.

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

Add more yang/sal dependencies

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

Hook the Distributed Data store into the config sub-system

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

Add akka dependencies

Just add the maven dependencies

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

Merge "Initial commit of the akka based distributed-datastore"

added containermanager shell feature

PatchSet5: Worked containermanager.shell into distro
           and pom files so that it both builds and
           is pulled into the old distribution/opendaylight.
           Also checked to make sure that it worked there.

PatchSet6: Fixed adding of new shell bundle to old
   distro

Change-Id: I7f4d86c9902a7f4a8f6bd4bae74ebe4a9634e3bc
Signed-off-by: Jonathan Fok kan <jfokkan@inocybe.com>
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Bug 629: Introduction of Configuration Commit Handler

One piece of functionality which was not-well defined
in original APIs and was missing in new APIs is
user-submitted commit handlers.

Contract of Commit Handlers was updated to reflect
more existing concept of three-phase commit and
to allow commit handlers only for configuration
subtree.

Access to proposed data is done via read-only
transaction instead of data change event,
since commit handlers may want to cross-reference
data with other and we need to capture this reads
as preconditions for later commit phases.

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

Bug 225 - added support for depth RESTCONF parameter in URI

Modified RestconfImpl to prune the resulting ComositeNode tree
based on the depth parameter, if specified.

A depth of 1 retrieves just the target resource and no children. This is
a way to determine if the resource exists. A depth of 2 retrieves the
target resource and just its direct children - any child container would
be empty.

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

Merge changes Ia119c9ae,Ic2290d7f

* changes:
  Bug 1106: Introduced OptimisticLockFailedException
  Bug 1073: Added support to DOMBrokerImpl for Transaction Chaining