Added DELETE on cors.allowed.methods for a few missing nb bundles.
Also added toString() method to a few Neutron backend classes.

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

Merge "Fixed invalid value in TestFromSalConversionsUtils."

Fixed invalid value in TestFromSalConversionsUtils.

Casting 0x84 to byte created invalid input (-124) for uint8 field.

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

Added transactions statistics support.

Added implementation of DataBrokerImplRuntimeMXBean as DataBrokerRuntimeMXBeanImpl;
Modified DataBrokerImplModule with DataBrokerRuntimeMXBeanImpl;
Added "submitted" leaf into opendaylight-md-sal-common.yang;

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

Merge "Fix static state in PersisterAggregatorTest causing test failures depending on order in which tests are run."

Fix static state in PersisterAggregatorTest causing test failures depending on order in which tests are run.

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

Merge "Revert "Replace tabs with spaces in config yang files""

Revert "Replace tabs with spaces in config yang files"

This reverts commit d8559be3bef3cad55c9c08cbb7718df32744f44a.

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

Merge "Replace tabs with spaces in config yang files"

MD-SAL Statistics Manager - Minor change in group/meter type/capabilities
Group/Meter feature request is now moved to plugin. Plugin will send request
whenever switch will connect to the plugin.

Change-Id: I8821fcb6c1c3d0ebcf9df11611103e846e8c6ccf
Signed-off-by: Anil Vishnoi <avishnoi@in.ibm.com>

Merge "Wired ClusteredDataStore with Configuration Subsytem"

Wired ClusteredDataStore with Configuration Subsytem

MD-SAL clustered Data Store is available as separete
configurable modules.

Change-Id: I15a98f19dd2eb851e8dfb73a9c4df461e6860db5
Signed-off-by: Moiz Raja <moraja@cisco.com>
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Replace tabs with spaces in config yang files

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

Merge "Fixed major sonar warnings in Binding Aware Broker"

Fixed major sonar warnings in Binding Aware Broker

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

Merge "Fixed bug in Data store where multiple readers could overwrite other augmentations"

Fixed bug in Data store where multiple readers could overwrite
other augmentations

  - Added configuration / operational data validation - Commit
    with operational data stored in configuration store will fail.
  - Added initial data change event.

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

Merge "Fixed PersisterAggregatorTest."

Merge "Fix equals and add hashcode to generated abstract module."

Merge "modified meter and group enum from leaf to typedef."

Merge "Make sure config.yang is MD-SAL enabled"

modified meter and group enum from leaf to typedef.

Signed-off-by: Prasanna Huddar <prasanna.huddar@ericsson.com>
Change-Id: I5cb3d27519818979d9c5559bd834480b831d8049

Fix persister bug introduced in last commit.

Registering to JMX failed due to previous bugfix.

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

Merge "Fix bug in persister causing to persist last configuration in a loop."

Merge "Finding data nodes in choices and cases."

Fix bug in persister causing to persist last configuration in a loop.

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

Fix equals and add hashcode to generated abstract module.

Equals and hashcode should evaluate just identifier. For change detection between commits, new method
isSame has been added. Fields that are arrays now use Arrays.equals instead of wrong equals.

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

Fixed PersisterAggregatorTest.

Change-Id: Iefc264ddc026df33f5a8124c17835f177627cd50
Signed-off-by: Milos Fabian <milfabia@cisco.com>

Merge "Fix race conditions between config-manager and persister."

Make sure config.yang is MD-SAL enabled

In the controller-controlling-controllers scenario we need to be able to
generate MD-SAL bindings against config subsystem models. In order to do
that, we need the base definitions provided by config.yang.

This patch subjects all config components to MD-SAL binding generation
and introduces a new parent, which adds another execution to generate
JMX bindings. Infrastructure components continue to use config-subsystem
parent while the various config plugins (netty et al.) switch to the new
config-plugin-parent.

Change-Id: Id243d658a95ccfee0dbfe70044867f8de09cc20e
Signed-off-by: Robert Varga <robert.varga@pantheon.sk>

Merge "Resolved namespace from input data"

Fix race conditions between config-manager and persister.

Config-manager commits a blank transaction each time new ModuleFactory is discovered. At the same time
persister might try to push configuration. Add retries to both routines to overcome optimistic lock failure.

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

Finding data nodes in choices and cases.

Feature was rewritten. It is no more taken into account if schemas for data
nodes at the same level are in the same case for concrete choice. This check should be done in system which takes Json as input

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

Resolved namespace from input data

Data of POST and PUT operations can contain namespace/module name:
- xml element can contain yang namespace
- json element can contain yang module name

Change-Id: I503c36de5cf76d6e7ca852d4b4e54e93d171920c
Signed-off-by: Martin Sunal <msunal@cisco.com>

Merge "removed config-util from pax-exam minimum bundles as this is not a bundle"

Merge "Add support for enums as configuration attributes in config and netconf subsystem."

Merge changes Ia268965a,Iefa79f99

* changes:
  Add running configuration data to get netconf operation response
  Add support for configuration attributes based on yang leaves with custom types (typedef) to netconf.

Merge "Choice and case resolving in JSON output"

Merge "Add config-persister-directory-adapter. This implementation reads initial configuration from a directory. It can combine multiple files into one snapshot to be sent to netconf. It does not support writing when configuration changes."

Merge "Refactor persister to handle multiple storage engines. Allow persister to load and save configuration snapshots from/to multiple storage instances. Loading works by iterating list of storages backwards and pushing first non-empty response to netconf. This allows having a default (initial) configuration for the controller that will never be overwrittern. Saving configuration will be propagated to all storage engines except those configured as read only."

Merge "Refactor persister: require only capabilities referenced by the xml snapshot. Also repair logback bug preventing it from taking snapshot configuration."

Add config-persister-directory-adapter.
This implementation reads initial configuration from a directory. It can combine multiple files into one snapshot to be
sent to netconf. It does not support writing when configuration changes.

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

Refactor persister to handle multiple storage engines.
Allow persister to load and save configuration snapshots from/to multiple storage instances.
Loading works by iterating list of storages backwards and pushing first non-empty response to netconf.
This allows having a default (initial) configuration for the controller that will never be overwrittern.
Saving configuration will be propagated to all storage engines except those configured as read only.

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

Merge "Revert "Fix bug 171. EchoReply payload must be the same as the correspoding EchoRequest payload" because is causing the switches to periodically disconnect."

Revert "Fix bug 171. EchoReply payload must be the same as the correspoding EchoRequest payload" because is causing the switches to periodically disconnect.

This reverts commit e4f47d259077775ceeb1554c807eed8841263865.

Change-Id: I0aa07d67ca6e6651e4d29a03c4282d5d6d2168da
Signed-off-by: Giovanni Meo <gmeo@cisco.com>

Add running configuration data to get netconf operation response

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

Choice and case resolving in JSON output

Searching via choices and cases which are defined in YANG when resolving
composite node to find elements with corresponding name.

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

Merge "Re-Enable the pax-exam execution in Eclipse"

Refactor persister: require only capabilities referenced by the xml snapshot.
Also repair logback bug preventing it from taking snapshot configuration.

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

Merge "Refactor persister. Remove obsolete TLS netconf configuration. Prefix all ServiceAdaptor configurations with common namespace."

Merge "Fix bug 171. EchoReply payload must be the same as the correspoding EchoRequest payload"

Refactor persister.
Remove obsolete TLS netconf configuration. Prefix all ServiceAdaptor configurations with common namespace.

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

Merge "Fixed import of annotations for models."

Fixed import of annotations for models.

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

Fix bug 171. EchoReply payload must be the same as the correspoding EchoRequest payload

Change-Id: I096f8389068948faad734ae1fef4b9b40b0f3cb9
Signed-off-by: Artur Downar <adownar@advaoptical.com>

Add support for enums as configuration attributes in config and netconf subsystem.

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

Add support for configuration attributes based on yang leaves with custom types (typedef) to netconf.

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

Merge "Fixed handshake issues with SSH Netconf client"

Fixed handshake issues with SSH Netconf client

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

Merge "Modified Group, Meter and Table. Xpath pointing to Flows,Groups and Tables."

removed config-util from pax-exam minimum bundles as this is not a bundle

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

Merge "MD-SAL StatisticsManager- Stopping statistics thread until i fix all the relevant issue from openflowplugin"

Modified Group, Meter and Table. Xpath pointing to Flows,Groups and Tables.

Signed-off-by: Prasanna Huddar <prasanna.huddar@ericsson.com>
Change-Id: I6c94d92ba2bfde3f4503a4da589c4a3d0b5f870b

Merge "- netconf SSH bridge bundle - Implement bridge using socket - standard netconf configuration added to netconf-ssh - NetconfClietnSession no longer needs to propagate channel - introducet IOThread class for stream copy - serversession closed when -1 received - NetconfSSHActivator throws Exception where no valid configuration   is found - ganymed bundle fixed to propagate whole package ch.ethz.ssh2.signature - netconf-ssh bundle added to distribution pom - netconf ssh server starting in separate thread in order to finis   activator's start method"

Merge "- introduced netconf ssh wrapper bundle - incorporation of Tomas's notes - CRLF removed - introduced KeyStoreHandler for providing server keys - netconf server connection is created when subsystem 'netconf' received - data forwarding to netconf server on requestShell - netconfClientSession exposes channel - removed star imports in SocketThread - integration tests for netconf SSH - introduced SSHChannelInboudnHandler which forwards data from pipeline to SSH serer session - resolved dependencies - tests changed due netconf client sending hello message on session create, added netconf-it to main pom.xml - added logging for integration tests"

MD-SAL StatisticsManager- Stopping statistics thread until i fix all the relevant
issue from openflowplugin

Change-Id: I477d0bdd00f2e6c188cac4ccc65a69a8a18e18dc
Signed-off-by: Anil Vishnoi <avishnoi@in.ibm.com>

Merge "Fixed Group and Meter RPC model to use MD-SAL routing correctly"

Merge changes I178e467a,Ia8720c2c

* changes:
  Mark deprecated methods in DataModification as such
  Remove executable permissions from pom.xml

Merge "Removed POST, PUT for /restconf/operational"

Re-Enable the pax-exam execution in Eclipse

- Make sure the pax-exam plugin is executed in the IDE so IT can be
executed via Junit runner, ability that got lost during the change to
use target-ide

Change-Id: Id270027fe54d8bb4391dd2c60f11a299defe4efb
Signed-off-by: Giovanni Meo <gmeo@cisco.com>

- netconf SSH bridge bundle
- Implement bridge using socket
- standard netconf configuration added to netconf-ssh
- NetconfClietnSession no longer needs to propagate channel
- introducet IOThread class for stream copy
- serversession closed when -1 received
- NetconfSSHActivator throws Exception where no valid configuration
  is found
- ganymed bundle fixed to propagate whole package ch.ethz.ssh2.signature
- netconf-ssh bundle added to distribution pom
- netconf ssh server starting in separate thread in order to finis
  activator's start method

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

- introduced netconf ssh wrapper bundle
- incorporation of Tomas's notes
- CRLF removed
- introduced KeyStoreHandler for providing server keys
- netconf server connection is created when subsystem 'netconf' received
- data forwarding to netconf server on requestShell
- netconfClientSession exposes channel
- removed star imports in SocketThread
- integration tests for netconf SSH
- introduced SSHChannelInboudnHandler which forwards data from pipeline to SSH serer session
- resolved dependencies
- tests changed due netconf client sending hello message on session create, added netconf-it to main pom.xml
- added logging for integration tests

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

Removed POST, PUT for /restconf/operational

- regarding to RFC the operations POST and PUT do not have to be allowed for client on /restconf/operational

Change-Id: Ie0e8cb07154e4a1a016090fef9645aa38b790b41
Signed-off-by: Martin Sunal <msunal@cisco.com>

Merge "Table features : modified yang model. Patch set 2: Modified match types as identity for extensibility."

Fixed Group and Meter RPC model to use MD-SAL routing correctly

  - Changed RPCs in Group and Meter to use node-context-ref
    instead of node for routing information (same pattern as
    in flow-services).

  - Updated statistics manager to use updated code.

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

Mark deprecated methods in DataModification as such

Change-Id: I178e467a3885841175755e3e9ff8f977a4276e85
Signed-off-by: Robert Varga <robert.varga@pantheon.sk>

Remove executable permissions from pom.xml

Change-Id: Ia8720c2c29e4cc1f2ac3c0d69827fe4010c5870a
Signed-off-by: Robert Varga <robert.varga@pantheon.sk>

Merge "Fixes - config-registry registered to OSGi, used modules's factory bundle context for module instance registration."

Merge "Fixed GlobalEventExecutorCloseable - override "unsupported opearaion" methods."

Merge "Fixes for MD-FRM"

Merge "Flow Topology Exporter"

Merge "Moved Schema Aware logic from DataStore to SchemaAwareDataStore adapter"

Merge "Fixed typo in group service. Modified Group and Meter flow Impl"

Merge "Enabled documentation generator for models."

Enabled documentation generator for models.

Generated docs can be found in
opendaylight/md-sal/models/*/target/site/restconf/*.html

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

Fixes - config-registry registered to OSGi, used modules's factory bundle context for module instance registration.

Change-Id: Iaae34d47c7a8515ea68f2d8d3629edd0b073152e
Signed-off-by: Milos Fabian <milfabia@cisco.com>

Fixed typo in group service. Modified Group and Meter flow Impl

Signed-off-by: Prasanna Huddar <prasanna.huddar@ericsson.com>
Change-Id: Ie77e9f6ac4dead4a337dd16cf5cd5c891b5981bd

Moved Schema Aware logic from DataStore to SchemaAwareDataStore adapter

    All YANG schema logic necessary for proper function of DataStore
    for all components was moved to separate component which
    provides this functionality unified from one place for multiple
    datastores.

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

Merge changes I6ff6c55c,I92b7e285

* changes:
  Bug 165 and changed operations
  Json with prefix for augmented elements

Bug 165 and changed operations

Main changes according to corresponding RFC draft 01 and 02:
- corrected media types (used application/yang.data and application/vnd.yang.data)
- PUT operation was changed to POST
- POST operation was changed to PUT

There was also added media types due to Bug 165 :
- application/json
- application/xml
- text/xml

Change-Id: I6ff6c55c272cef0b63d5795d5b9842db83f1b627
Signed-off-by: Martin Sunal <msunal@cisco.com>

Fixed GlobalEventExecutorCloseable - override "unsupported opearaion" methods.

Change-Id: Ic0c65cc596bdf6ba1bbbada576052f11805a202e
Signed-off-by: Milos Fabian <milfabia@cisco.com>

Merge "Switched SAL Netconf Connector to config subsystem"

Merge "Fixed bug with multiple DataReaders participating in one read operation"

Fixes for MD-FRM

Patch 1:  Fix MD-FRM to listen to the correct portion of the data tree.
Currently we listen to /flows/flow/ for which we would need to specify a
key to indicate which flow in the tree we wanted to listen to.  Changing to
listen to /flows/ and thus hear commits involving all flows.

Patch 2:  Unpack Entry... we need the value of the entry which is a DataObject.
Otherwise, the cast fails.

Patch 3: NodeFlow is only used as an input to rpcs towards the
openflowplugin, it does not get filed in the data tree.  Flow gets
filed in the data tree.  Addtionally, commits can come in with multiple
objects, so if we get a commit with other things, we either need to ignore
them or handle them... but blindly casting them will not work.

Patch 4: Change to cope with moving from NodeFlow to Flow.  Fortunately,
NodeFlow is just Flow plus a NodeRef plus a TableRef, so it pretty much just
worked (at least at the typing level... not to running yet).

Patch 5: Fixed an NPE by providing a Collections.<RpcError>emptySet() for
RpcErrors instead of null.

Patch 6: Fixed a failure due to attempt to check with the old AD-SAL
infra for the Node.  This will not work correctly.  Added TODO for
validation of the node.  Also cleaned up substantially handling of
error conditions while I was in the neighborhood.

Patch 7: Something in the current use of ClusteringServices for
validation is breaking things.  At first it looked like
we were just trying to allocate caches before we initialized
out clustering services variable, and thus failing... but at
the end of the day there is more wrong here than make sense
to debug at this time... so commented out.

Patch 8: Get the modifications from the transaction and
deal with the create, updated, removed.  Also handled the
cast to Flow.  Put a small null check around handling of
instructions in FRMUtil.

Patch 9: Something in the current use of ClusteringServices
is breaking things. Commenting out in order to get things working.

Patch 10: Correctly sorted out the creates from the removes
from the *true* updates.  Note, both creates and updates appear
as updates... so you have to do a little setwise subtraction.
Also note, the Collections you get back are immutable, so
you have to do that subtraction with care.

Victory!  As of Patch 10 I can confirm using
https://git.opendaylight.org/gerrit/#/c/3118/

That flows being added, removed, or updated are being
correctly wired to an RPC that is registered for them.

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

MD-SAL Statistics Manager - Did temporary fix for avoiding CSIT test failure.
This code has dependency on openflowplugin code that is not yet merged.
http://git.opendaylight.org/gerrit/3037

Change-Id: Ia54a911d12f7a484a3475d93dc1b344478121006
Signed-off-by: Anil Vishnoi <avishnoi@in.ibm.com>

Switched SAL Netconf Connector to config subsystem

    Switched Programmatic APIs to configuration APIs
    for mounting Netconf southbound devices as a providers
    inside MD-SAL.

    Netconf Southbound Device is not exposed as top-level
    provider, but rather as an mountpoint with explicit
    namespace defined by it's location in overal data tree.

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

Fixed bug with multiple DataReaders participating in one read operation

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

Json with prefix for augmented elements

Json elements which are created via augmentation are written with
prefix which is equal to module name from where augmentation was defined.

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

Table features : modified yang model.
Patch set 2: Modified match types as identity for extensibility.

Signed-off-by: Deepthi V V <deepthi.v.v@ericsson.com>
Change-Id: I12f9f99ccc40162951889acc5515940e56e9f211

MD-SAL Statistics Manager - Minor bug fixing

Change-Id: I038195b03ac9f5abea9d23f793f69c2f307629c1
Signed-off-by: Anil Vishnoi <avishnoi@in.ibm.com>