Move protocol framework from BGPCEP project

This moves the current parts of BGPCEP protocol framework into the
controller project, eliminating the circular dependency forced by
netconf being resident in this project.

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

Remove dependecies on bgpcep concepts/util

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

Merge "Do not instantiate constant lists on each message"

Merge "Switch to using yangtools version of mockito-configuration"

Merge "Switch testing configuration for md-sal tests to xml."

Merge "Refactor shutdown-impl: add parameter to RPC, remove secret validation and masking."

Merge "Changed mount point URI decoding in restconf"

Do not instantiate constant lists on each message

These really are constats, so treat them as such

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

Switch to using yangtools version of mockito-configuration

This commit switches to use the artifact from yangtools repository
introduced in https://git.opendaylight.org/gerrit/#/c/4316/, completing
purge of bgpcep dependencies.

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

Changed mount point URI decoding in restconf

- string "yang-ext:mount" in URI means mount point.
  After this string must be string in format moduleName:dataNode
  example:
  /rectconf/config/moduleName1:cont1/cont2/yang-ext:mount/moduleName2:cont3/cont4
     mount point path - moduleName1:cont1/cont2/
     path behind mount point - moduleName2:cont3/cont4

- refactoring of POST, PUT, GET, DELETE tests for status codes
- test for output of several modules to one xml and json file were added

Change-Id: I79bfaec1fab6ffd89295fb8461c167c0c85638ae
Signed-off-by: Martin Sunal <msunal@cisco.com>
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>

Switch testing configuration for md-sal tests to xml.

+ Fix bug regarding leading/tailing whitespace in capability string unmarshalled from xml.

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

Merge "Add missing headers to config, netconf subsystems"

Merge "Remove System.out from netconf tests"

Refactor shutdown-impl: add parameter to RPC, remove secret validation and masking.

Remove old secret as it does not work well with config-persister.
Add parameter maxWaitTime to shutdown RPC that specifies amount of time before
server is forcibly shut down.
Work around buggy yuma message sending xml element without inner text
node if blank value is specified.

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

Add missing headers to config, netconf subsystems

Only java files

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

Remove System.out from netconf tests

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

Merge "Better logging for netconf-ssh."

Merge "Restconf operation GET, PUT, POST tests refactoring"

Restconf operation GET, PUT, POST tests refactoring

Tests for GET, PUT, POST were refactored. For every operation there is one
class with name Rest<OPERATION_NAME>OperationTest.java and special class
RestOperationUtils for common subrutines.

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

Better logging for netconf-ssh.

Log ServiceReference object in ServiceTrackerCustomizer methods.

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

missing dependecies reviewed and fixed

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

Merge "incorrect dependencies and unnecesary export-package removed from pom.xml"

incorrect dependencies and unnecesary export-package removed from pom.xml

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

Merge "Added tests for DELETE operation"

Merge "Fix a few POM versioning/dependency warnings"

Added tests for DELETE operation

Change-Id: Iba7874231d8662a5626a262228dd1baafead897f
Signed-off-by: Ladislav Borak <lborak@cisco.com>

Fix a few POM versioning/dependency warnings

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

Merge changes I85b49247,Icca28a4a

* changes:
  Added DELETE operation
  Changed POST operation

Added DELETE operation

- fix of bug 266
- corrected @Produces and @Consumes annotations in RestconfService and RestconfServiceLegacy

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

Changed POST operation

POST operation for creating resource was changed according to RFC
http://tools.ietf.org/html/draft-bierman-netconf-restconf-02#section-3.4
- test fix
- ResponseException tests

Change-Id: Icca28a4a57499be5f29a1b618a432dd61dde2816
Signed-off-by: Martin Sunal <msunal@cisco.com>
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>
Signed-off-by: Ladislav Borak <lborak@cisco.com>

Merge "Always set interrupted flag when catching InterruptedException."

Merge "Minor changes in md-sal tests."

Minor changes in md-sal tests.

Temporary ignored test, removed @Override annotation.

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

Always set interrupted flag when catching InterruptedException.

Fixed catching InterruptedException in config-subsystem and netconf.
In netconf persister propagate interrupted exception as it indicates
that netconf connection was not established and thus config pushing
does not work.

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

Merge "private key configurable in config.ini"

Merge "Update maven-bundle-plugin version in config and netconf."

Merge changes If79fa865,Ia6652643

* changes:
  Fixed some TODOs in netconf.
  Update service reference persistance according to new ServiceRegistry API in config subsystem.

Merge changes I994c19ee,I245e9e7a

* changes:
  sal-rest-connector test refactoring
  More test for improving of code coverage + test refactoring

private key configurable in config.ini

Netconf SSH bridge has private key stored in distribution.
Path to this file is configured in config.ini.

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

Update maven-bundle-plugin version in config and netconf.

Update from 2.3.7 to 2.4.0

Change-Id: Ie121db016d5532df58da05244d7883520cdf8b11
Signed-off-by: tomas <tolvecky@cisco.com>

sal-rest-connector test refactoring

Mainly deleting unnecessary methods.

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

More test for improving of code coverage + test refactoring

Added tests to sal-rest-connector to improve code coverage.
Added test for leafref.
Some methods in TestUtils were refactored.

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

Merge "Support for mount points in Restconf"

Support for mount points in Restconf

Added support for mount points in Restconf paths, which allows for
deeper queries inside remote nodes such as Netconf.

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

Merge "Added constructor which allows for custom NetconfClientSessionListener"

Added constructor which allows for custom NetconfClientSessionListener

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

Merge "Fix HashedWheelTimer exception for short living netconf clients."

Fix HashedWheelTimer exception for short living netconf clients.

An exception occurred in logs for short living netconf clients created by config persister.
These clients were closed before their initial check timer ran out.

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

Fixed some TODOs in netconf.

Remove duplicate code
Remove commented modules in get-config response
Add test for netconf - persister integration
Remove already fixed TODOs

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

Update service reference persistance according to new ServiceRegistry API in config subsystem.

Utilize new proxy service mbeans that store reference name for dependency attributes.

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

Merge "Switched codecs infrastructure to use yang-data-impl codecs"

Merge "Fixed RESTConf support for identity-ref build-in datatype"

Merge "Verification of keys in list"

Switched codecs infrastructure to use yang-data-impl codecs

- Codecs are used for tranlation of JSON/XML values to SimpleNode values and vice versa
- Leafref is encoded as String

Change-Id: Ibe5b4a23d5dad6373a2dcf7d8723a7dff135cf2a
Signed-off-by: Martin Sunal <msunal@cisco.com>
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Fix date initialization in NetconfOperationServiceImplTest.

Use SimpleDateFormat instead of Date(0) as the later fixes the date to UTC rendering local date to
1969-12-31 for timezones > 0

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

Fixed RESTConf support for identity-ref build-in datatype

- Created implementation of codec for identityref
- Identityref is supported in XML and JSON
- Tests

Change-Id: I813f1f52e990317b6adf30f1d0d8a164c60a4901
Signed-off-by: Martin Sunal <msunal@cisco.com>
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Verification of keys in list

- verification of keys in list against list yang structure

Change-Id: Ieb46eea67a7d3173958184ced10cec1b853bb7fe
Signed-off-by: Martin Sunal <msunal@cisco.com>
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Merge "Netconf SSH user authentication using UserManager"

Netconf SSH user authentication using UserManager

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

Merge "Added tests for reading: - operational data - configuration data"

Merge "Add consistency check of visible yang modules to netconf."

Merge "Fixed netconf monitoring."

Added tests for reading:
- operational data
- configuration data

Change-Id: I8d771ac061d0d7cc5c4187133e2bf0a3ddb87252
Signed-off-by: Ladislav Borak <lborak@cisco.com>

Merge "Added support for union yang types to netconf."

Added support for union yang types to netconf.

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

Add consistency check of visible yang modules to netconf.

Netconf connector to config subsystem performs this check before a netconf session is established.
This check consists of comparing yang modules visible to config subsystem to modules visibile to yangstore.
If an inconsistency is detected, session negotiation is cancelled and the session is dropped.

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

Fixed netconf monitoring.

-added extension of ietf-netconf-monitoring
-fixed reporting of host address
-fixed reporting of transport
-added reporting of session identifier (client,persister)
-fixed send netconf message with additional header

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

Merge "Added xml storage adapter bundle fragments for config persister."

Added xml storage adapter bundle fragments for config persister.

File and Directory based xml adapters are now available.
Only File xml adapter is used so far to persist current configuration.

Directory based persister is still using plain text to keep backwards compatibility.

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

Merge "Improve logging of persister and netconf client."

Improve logging of persister and netconf client.

Add a debug message measuring time spent waiting for netconf server to reply in NetconfClient.
Log with error level failure to send/recieve netconf message in config-persister-impl.
Add TODOs to config-manager's dependency resolver for detecting cycles.

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

Merge "Added support for persistance of service references to netconf."

Added support for persistance of service references to netconf.

Created reference names under services/ subtree are now persisted and available for further edit-config operations.

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

Merge "Fixed deserialization of IdentityRefs in Restconf URI."

Fixed deserialization of IdentityRefs in Restconf URI.

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

Merge "Corrected response status codes from restconf"

Merge "Improved support for default yang statements in configuration subsystem"

Merge "Fix for bug 211, where direct write and read of augmentation was not processed correctly"

Fix for bug 211, where direct write and read of augmentation was not processed correctly

  - Fixed TypeNotAvailable issue with end-to-end Openflow

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

Corrected response status codes from restconf

- POST /restconf/config/* - 204 no msg body
- POST /restconf/operations/* - 200 msg body or 204 no msg body or 500 operation failed
- PUT /restconf/config/* - 200 no msg body

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

Improved support for default yang statements in configuration subsystem

Generated code for configuration contains default values with correct constructor if specified for leaves
Fixed binary type (de)serialization

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

Merge "Fix a typo in debug message"

Merge "Fixed sal-netconf-connector readConfiguration"

Fixed sal-netconf-connector readConfiguration

  - Promoted Netconf client to v1.1.
  - Added reconnect retry.
  - Fixed readConfiguration for mounted netconf device.
  - Fixed serialization issue in restconf related to mount points with unknown schema.

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

Fix a typo in debug message

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

Merge "Updated xpath pointing to a module instance under services subtree."

Merge "Refactor yang-jmx-generator and -plugin to support list of dependencies."

Refactor persister: Add ability to publish multiple snapshots while loading initial configuration.

Persister adaptors now return list of snapshots to be pushed to netconf at server startup. Persister maintains an ordered list of adaptor
instances, each with its configuration. During server startup it iterates the list backwards until non empty list of snapshot is returned.
Each snapshot can depend on different capabilities, once all capabilities for given snapshot are announced by server, persister pushes this
snapshot.

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

Merge "Initial implementation of netconf monitoring module according to tools.ietf.org/html/rfc6022"

Refactor yang-jmx-generator and -plugin to support list of dependencies.

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

Merge "Used log instead of exception"

Used log instead of exception

- logging was added instead of throwing exception in problematic part

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

Merge "netconf-ssh dependency cleanup"

Merge "Common yang and xml loader for tests"

Merge "RestconfImpl.invokeRpc method test"

Updated xpath pointing to a module instance under services subtree.

From /config/modules/module[name='hash-map-data-store']/instance[name='hash-map-data-store']
To   /modules/module[type=hash-map-data-store][name=hash-map-data-store']

Affected netconf operations: get-config, edit-config.
Backwards compatibility with previous xpath version is preserved.

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

Initial implementation of netconf monitoring module according to tools.ietf.org/html/rfc6022

Response from netconf GET operation now contains monitoring information about current netconf state.

Modified netconf-impl module: added monitoring capabilities to netconf-impl module exposed as OSGi service.

Added netconf-monitoring module that provides a filter for netconf GET operation
and adds monitoring information retrieved from netconf-impl to the response of the GET operation.
It implements netconf-mapping-api similar to config-netconf-connector and
currently provides information about current netconf SESSIONS and yang SCHEMAS
(these information is retrieved via the netconf monitoring OSGi service provided by netconf-impl).

Added ietf-netconf-monitoring module that contains sources generated from ietf-netconf-monitoring yang.
These sources serve as an interface between netconf-impl and netconf-monitoring.

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

Merge "Fixed serialization issues with Augments and java types"

Merge "client IP/port definition in configuration separation new netconf.client properties added to integration tests"

netconf-ssh dependency cleanup

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