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>

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

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>

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."

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>

Added MD-SAL statistics-manager to dependendy in pom.xml

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

Merge "Fixed quote issue in flow-types.yang and table-types.yang"

Fixed quote issue in flow-types.yang and table-types.yang

This should resolve the errors on startup:

2013-11-26 07:49:17.846 PST [Start Level Event Dispatcher] ERROR o.o.y.y.p.util.ParserListenerUtils - Syntax error in module opendaylight-table-types at line 7: missing '"'.
2013-11-26 07:49:17.856 PST [Start Level Event Dispatcher] ERROR o.o.y.y.p.util.ParserListenerUtils - Syntax error in module opendaylight-flow-types at line 7: missing '"'

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

Merge "Added transaction parameter in Port RPC"

Merge "Fix bug 153: change the key container-config to ContainerConfig in the container manager module"

Merge changes I0f752636,Idd154499,Ic35fa3e8

* changes:
  Updated xml generated by get netconf operation to conform to yang schema in config-api
  Add logback as a default config module.
  Fix logging of yang parsing problems in yang-parser-impl

Merge "Queue yang model"

Merge "Better exception message in persister. Split information about missing capabilities into log message, and exception that only contains missing capabilities."

Merge "Fixed default configuration to not reference unused capabilities"

Merge "Update context-instance xpath format according to latest changes in config-api yang."

Merge "Null data from XML"

Merge changes I50218f21,I6bc2631b

* changes:
  Serialization to Json improvement
  Union type processing added to JsonMapper

Merge "Bug 164"

Fixed default configuration to not reference unused capabilities

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

Better exception message in persister.
Split information about missing capabilities into log message, and exception that only contains missing capabilities.

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

Added transaction parameter in Port RPC

Signed-off-by: HemaTG <hema.gopalkrishnan@ericsson.com>
Change-Id: Ia3e36463955690d338436c15c6ff9410b4e4c94a

Serialization to Json improvement

- During serialization of List and LeafList is not get parent from composite node
but he is added as a parameter

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

Merge "HostTracker hosts DB key scheme implementation"

Updated xml generated by get netconf operation to conform to yang schema in config-api

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

Flow Topology Exporter

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

Fixed two-phase commit bug when change originated in DOM Broker

  - DataServiceConnector is now tracking all opened transaction
    in order to prevent creating loop if transaction was
    already forwarded from other broker (DataServiceConnector
    is not publishing it back).
  - Added logging to TwoPhase Commit and Transaction
    forwarding
  - Fixed some warnings regarding wildcards in generics.

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

Added configuration options to MD-SAL

   Added default configuration to distribution
   which wires MD-SAL components together.

   Support for configuring wiring of following services:
     - Binding Aware Broker
        - Binding Aware Notification Service
        - Binding Aware Data Broker Service
        - Mapping service
     - Binding Independent Broker
     - Data Store

   Default configuration wires BA Broker, BA Notification service
   BA Data Broker and BI Broker together.

   BA Broker requires now BI Broker to be loaded in order to
   provide datastore functionality right from the beginning.

   Updated integration tests, added new test cases for BA to BI
   integration

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

Merge changes I4ba7cf7d,Ia3e319ee

* changes:
  Allow multiple services provided from same namespace parsed from netconf message
  Fix no response from netconf server if no operation is available to handle message.

Update context-instance xpath format according to latest changes in config-api yang.

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

Queue yang model

Signed-off-by: usha <usha.m.s@ericsson.com>
Change-Id: If80f902ced0b053b197263462cce57c753c78608

Bug 164

- if URI has bad format then error status 400 is returned
- corrected xtend conventions: triple equals for null

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

HostTracker hosts DB key scheme implementation

Requirements :-

i) IP only scheme for hosttracker hosts db key is by default supported.
ii) IP + MAC scheme for hosttracker hosts db key addition.
iii) Only one scheme at a time will be active. The configuration is
controller by the setting hosttracker.keyscheme in config.ini. By default
it has a value of zero which indicates IP only scheme.
The value 1 can be set for choosing IP+MAC scheme.

Changes done :-

i) Marker interface named IHostId is used for the implementation.
ii) IPHostId and IPMacHostId are the classes which implement the marker
interface.
iii) All the hostttracker apis have been aligned to the new key using the
marker interface IHostId as the key.
iv) Version of the hosttracker and it's dependent bundles like
arphandler,loadbalancer,static routing etc.. have been bumped up.
v) HostIdFactory has been provided to create the keys based on the scheme
the user is using.

Tests done :-

i) pingall has worked.
ii) NB APIs of hosttracker.

Made distribution changes which were needed and other pom changes as well.
Integration test for hosttracker has been fixed with the right version.
Other module integration test failures and review comments have been taken
care off.
Host DB Key classes have been serialized.

Change-Id: I328a1b6d3702d5268832f0e3ea6ee47122f20205
Signed-off-by: Deepak <dudapudi@in.ibm.com>

Null data from XML

In case of reading data from XMl to Composite node following rule was
added:
- XML with opening and closing tag without data is mapped to single node
  with empty string
  (e. g. <foo></foo> -> single node with empty string)
- XML with only closing tag is mapped to single node with null value (e.
  g. <foo /> -> single node with null value)

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

Union type processing added to JsonMapper

if data type according to schema is union then data from single node are validated as follows:
- if simple node value is null -> [null] is written to JSON
- if simple node value is number and union yang contains one of number types ->
  number is written to JSON
- if simple node value si true or false and union yang contains BooleanType ->
  true or false is written to JSON
- other cases are written to JSON as string (inside "")

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

Merge "Add prefixes to type attribute in generated xml from netconf Add support for parsing prefixed type attributes from incomming xml to netconf"

Merge changes Ia09a1107,I2c30f3b6

* changes:
  Change type of service-ref grouping to service-type-ref from leafref.
  Fix config-netconf-connector needed for interacting with yuma's yangcli.

Merge "Fixed YangStore exception thrown at startup"

Merge "TableFeatures Update,correcting InstanceIdentifier"

MD-SAL Statistics Manager- types and capabilities in group-features model is bitmap, so changed to bitmap from enum

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

Merge "Fix NPE in FlowProgrammerService in openflow plugin"

Merge "Adding another constructor to ContainerFlowConfig to take dlVlan. Being an immutable object, there is no other way to set the Vlan (other than via NB Serialization)"

Adding another constructor to ContainerFlowConfig to take dlVlan.
Being an immutable object, there is no other way to set the Vlan (other than via NB Serialization)

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

Add logback as a default config module.

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

Fix logging of yang parsing problems in yang-parser-impl

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

Allow multiple services provided from same namespace parsed from netconf message

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

Merge "1. Fixed not sending of events to the node that created the routing table entry 2. Deprecated listener pattern and introduced the WB pattern based on code review 3. Included some documentation and logging package 4. Modified/Added unit test case"

Fix no response from netconf server if no operation is available to handle message.

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

TableFeatures Update,correcting InstanceIdentifier

Signed-off-by: Gaurav Bhagwani <gaurav.bhagwani@ericsson.com>
Change-Id: I3722482a52a8b26cd48474838d21bf3ecc22a580

Merge "MD-SAL Statistics Manager -Added group-id to group-statistics API"

Add prefixes to type attribute in generated xml from netconf
Add support for parsing prefixed type attributes from incomming xml to netconf

Prefixed type attribute conforms to config api defined in yang

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

1. Fixed not sending of events to the node that created the routing table entry
2. Deprecated listener pattern and introduced the WB pattern based on code review
3. Included some documentation and logging package
4. Modified/Added unit test case

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

MD-SAL Statistics Manager -Added group-id to group-statistics API

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

Change type of service-ref grouping to service-type-ref from leafref.

Also fix bug in config-netconf-connector where edit strategy was in global scope.

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

Fix config-netconf-connector needed for interacting with yuma's yangcli.

Mitigate issue when yuma is sending services/service/type with a prefix that is not referenced anywhere - it will be ignored.
When issuing edit-config with merge strategy, populate services as they appear in get-config.

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