Merge changes Ie84ea1ff,Ifa0a97da

* changes:
  BUG-848 Make base capabilities from netconf server configurable
  BUG-848 Fix netconf communication while using CHUNK encoding

Default auth parameters in config.ini.

    Hardcoded default username and password for ssh authenticaton moved from AuthProvider.java to config.ini
    IUSerManager in AuthProvider.java is not static anymore.

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

BUG-848 Make base capabilities from netconf server configurable

Refactor ConcurrentClientsTest, add cases for different netconf server configuration (EXI/CHUNK)

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

BUG-848 Fix netconf communication while using CHUNK encoding

Cause:
When using Chunk encoding, the negotiation process lasts longer due to pipeline manipulation.
Messages received while still in negotiation were not processed correctly (This is incorrect, messages should not be processed while still in negotiation).

Fix:
Introduce a message barrier that will forward only hello message until negotiation is completed (implemented in NetconfXmlToHelloMessageDecoder).
Modify SessionNegotiators to replace message barier/hello decoder when negotiation is finished and to process non-hello messages stopped by the barier.

* Exclude pax-url-aether dependency in netconf-it

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

Merge "Resolve Bug:748 - Remove test resources from netconf-impl."

Merge changes I8e90a7d5,I25249fe8

* changes:
  Resolve Bug:807 - Keep ModuleFactory references shutdown purposes.
  Resolve Bug:681 - Fix config module registration to Service Registry.

Merge "BUG 652 leafref CCE & BUG 720 colons problem"

Merge "Bug 460 - Fix warning throughout netconf subsystem"

Merge "BUG-732 Fix sal-netconf-connector unable to download schemas"

BUG-732 Fix sal-netconf-connector unable to download schemas

BUG#1 NetconfDeviceListener used wrong method to parse rpc results
BUG#2 Parsing rpc result in NetconfMapping was looking for schema to response even if it was not needed
BUG#3 Netconf client sent base netconf capability in wrong format

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

Bug 460 - Fix warning throughout netconf subsystem

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

Fix Operation not permitted warning.

This fix removes warning
WARN  o.o.c.n.c.m.c.ModuleElementDefinition - Operation not permitted on
current strategy  while default strategy is merge. Element definition
strategy set to default

that occured on osgi startup.

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

Merge "Bug 451 - Fix netconf exception handling"

Merge "BUG-832 Add initial configuration for controller self mount"

Bug 451 - Fix netconf exception handling

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

BUG-832 Add initial configuration for controller self mount

Netconf server for config subsystem is mounted to restconf under id "controller-config" by default

This illustrates the usage of sal-netconf-connector

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

BUG 652 leafref CCE & BUG 720 colons problem

BUG 652 leafref Class cast exception
By default if input data were in format which was possible to parse as
XPath if was done like this. It caused that for leafref which indexed to
instance-identifier the value was parsed as XPath and not as string.

BUG 720 colons problem in JSON
If input string contained colon (:) then it was dealed as namespace:value
and object of type IdentityValuesDTO was created (it is probable that
data could be of type identityref). If data wasn't of type leafref then it
was incorrectly processed as leafref.
The RestCodec was updated to correctly translate data (cases when data
contains ':' but aren't of type leafref)

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

Bug 722 - nagasena version bumped to 0000.0002.0038

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

Resolve Bug:681 - Fix config module registration to Service Registry.

Rework OSGi service registration during second phase commit: only service references
will be registered into SR, instead of old behaviour when each module's service
interfaces were used. Only one key-value entry is persisted in registration:
{name: name of service reference} .

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

Resolve Bug:748 - Remove test resources from netconf-impl.

Remove old testing resources that are not required in tests.

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

Remove unnecessary variable in netconf-it

- Another roadblock to release of artifact mdsal.version is used as
variable in netconf-it, remove it

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

Merge " Remove version mismatch in config.version"

 Remove version mismatch in config.version

 - End goal was to remove several config.version present in the pom, in
 the making i cleaned some of the repetition in config parent pom and
 aligned with the commons.opendaylight
- Removed some more warning on the pom files

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

Adding auto-sorting plugin to all Controller via parent/pom.xml

leveraging google's sortpom plugin, all controller poms are sorted
at build time as per the opendaylight/commons/parent/pom.xml configuration.

Sorted lists of properties, dependencies, and plugins will increase pom readability
and help prevent duplicate entry errors.

This first submission is huge as I changed one pom.xml and all are then sorted.
Only if the sortpom plugin is reconfigured will all pom's again be changed.

Note that if any pom.xml is pushed without having been through a build ... folks are
going to see pom.xml files showup as modified unintentionally.

Change-Id: I8553f38b9f0e5f6d27177ad6b9b9c35a4b43d55a
Signed-off-by: jameshall03885 <ghall@brocade.com>

Merge "BUG-731: do not do String.toString()"

Merge "BUG-731: do not catch Throwable"

BUG-472 Initial EXI encoder/decoder implementation in Netconf

according to http://tools.ietf.org/html/draft-varga-netconf-exi-capability-02

Change-Id: I861c126a8df1b26f477aa354ed1f8dba59d651cd
Signed-off-by: Martin Bobak <mbobak@cisco.com>
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

BUG-472: Add EXI codecs without wiring them

This patch implements the EXI codecs to be used in the NETCONF pipeline.
No wiring into the protocol stack is done in this patch.

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

BUG-472: Import OpenEXI nagasena-0000.0002.0035.0

This imports the OpenEXI nagasena library, on top of which the
implementation will be built.

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

BUG-731: do not do String.toString()

Fixes a sonar warning. Also adds missing copyright headers.

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

BUG-731: do not catch Throwable

This fixes the bad practice of catching Throwable, and uses logger
instead of e.printStackTrace().

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

Merge "Resolve Bug:745 - remove windows line endings from config and netconf."

Bug 491 - Use XmlUnit for xml comparing in junit tests.

-comparing XMLs as DOM Documents with XmlUnit
-added XmlUnit util class - contains common assertion tests

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

Resolve Bug:745 - remove windows line endings from config and netconf.

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

Merge "Bug 714 - Fixed creating DOM Document's element with namespace"

Merge "BUG 718: changed codec for URI parsing"

Merge "Resolve Bug:713 - Open snapshot only once per session."

Merge "Resolve Bug:707 - ConfigPusher should wait for netconf-impl to register JMX bean."

Resolve Bug:713 - Open snapshot only once per session.

Rework NetconfOperationServiceSnapshot lifecycle: Open it when
session negotiator is requested, thus at entry point where both
hello message and session listener is created. Close the snapshot
when NetconfSessionListener receives down/terminated event.

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

Resolve Bug:707 - ConfigPusher should wait for netconf-impl to register JMX bean.

Fixes race condition that was observed in ITs, where config pusher started pushing
empty configuration and immediately registered itself as notification listener of
a JMX bean that was not yet present. Fix waits both for netconf-impl and
config-netconf-connector services so that both dependencies are activated before
pushing of configuration can occur.

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

Merge "BUG-459 Remove deprecated NetconfClient class."

Merge "BUG-372: Rework sal-netconf-connector"

BUG-459 Remove deprecated NetconfClient class.

NetconfClient moved to test sources, now will be used only by tests.

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

Bug 651 - Added back IDE profile and changed target for yang sources

- The IDE maven profile was added back based on feedback from Giovanni Meo

- To avoid devs from having to run the IDE profile so eclipse can see the yang
generated source files, the yang source files are now generated under src/main/yang-gen-sals
or src/main/yang-gen-config instead of target/target-ide.

- The maven-clean-plugin was moved to the commons base pom and removed from
all child poms where it was specified. Thus, it will be automatically run for all
projects. The plugin's execution will clean all xtend and yang generated source dirs.

- The build-helper-maven-plugin was also moved to the commons base pom and removed from
all child poms. It will automatically run for all projects and no longer needs to be
specified in child poms. The plugin's execution will created eclipse source folders
for all xtend and yang generated source.

Tom P and Devin A really did all this work, Greg just gets credit unfairly.

Conflicts:
        opendaylight/config/config-module-archetype/src/main/resources/archetype-resources/pom.xml

Change-Id: I8d9e403629cab2e48d03d5c7a417a02f1d878ca7
Signed-off-by: tpantelis <tpanteli@brocade.com>
Signed-off-by: jameshall03885 <ghall@brocade.com>
Signed-off-by: dbavery <devin.avery@brocade.com>
Signed-off-by: jameshall03885 <ghall@brocade.com>

Bug 714 - Fixed creating DOM Document's element with namespace

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

BUG-372: Rework sal-netconf-connector

This makes the connector truly asynchronous and able to work with
backing datastore going away. It will retry connecting infintely to the
backed device, too.

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

BUG 718: changed codec for URI parsing

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

Merge "BUG-704 Fix failing integration test in netconf-it"

BUG-704 Fix failing integration test in netconf-it

Ignore for now, proper fix will follow

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

Merge "Resolve Bug:522"

Merge "Resolve Bug:623 : Generate private key using bouncycastle."

Merge "524 bug - %2F slashes translation behind mount point (unit test)"

Merge changes I6029bd48,Ie8eb4152

* changes:
  BUG-672 Add includeExtensions property to Xml directory persister adapter.
  BUG-672 Remove deprecated plaintext/autodetect persister adapters.

Merge "bug - missing excaping of % character in String.format() method"

BUG-672 Remove deprecated plaintext/autodetect persister adapters.

Xml adapters are the default adapters now for configuration persister.

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

Resolve Bug:623 : Generate private key using bouncycastle.

Add dependency on bouncycastle and use it to generate private key in PEM
format when key file is not present on the filesystem.

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

bug - missing excaping of % character in String.format() method

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

Merge "Resolve Bug:654 - Fix config-persister exception handling."

Resolve Bug:654 - Fix config-persister exception handling.

Modify NetconfUtil to fail on any <rpc-error>, not detecting transaction conflicts. Move
exception handling to ConfigPusher.
Also fix bug when config-netconf-connector detects discrepancy between yang store and
config manager factories: persister will retry as if some capabilities were not found.

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

524 bug - %2F slashes translation behind mount point (unit test)

New test getDataWithSlashesBehindMountPoint() tests whether URI identifier
which contains slashes encoded as %2F (e.g. in key part of URI) is translated to identifier with simple slash
characters (/).

It is checked whether InstanceIdentifier instance is created with simple
slashes. The instance is then used as input parameter of method
BrokerFacade.readOperationalDataBehindMountPoint() to get data from
datastore.

If %2F are correctly replaced with / test is succesful.

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

Merge "Bug 564 - add missing sal-remote dependency."

Merge "Resolve Bug:593. Persister should communicate via OSGi SR instead of TCP."

Bump netty.io to 4.0.17.Final

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

Resolve Bug:593. Persister should communicate via OSGi SR instead of TCP.

Config persister now pushes configuration to netconf service directly by
java calls instead of using TCP connection to localhost. This allows
removing netconf client dependency from config-netconf-connector and
removes confusion when running more than one ODL process with same
configuration.

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

Bug 564 - add missing sal-remote dependency.

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

Merge "Do not catch Throwables, but rather Exceptions"

Do not catch Throwables, but rather Exceptions

This turns catch Throwable into catch Exception, as doing the former
catches Errors, which are documented to be pretty much irrecoverable
conditions. Looking at
http://docs.oracle.com/javase/7/docs/api/java/lang/Error.html non of
these call sites can recover from an Error happening, so they should
never catch them.

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

Merge "Resolve Bug:448 - Remove yang-store api and impl."

Resolve Bug:522

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

Resolve Bug:448 - Remove yang-store api and impl.

Depends on yangtools commit https://git.opendaylight.org/gerrit/#/c/5579/ .
Instead of having multiple bundle trackers for config and netconf, keep only
one. Each bundle containing yang files should use sal code generator, which
generates YangModuleInfo objects. Using this instead of speculatively having
to parse all current yang files for each adding bundle event allows not having
to deal with broken yang dependencies when bundle events come in wrong order.
Put SchemaContextProvider to OSGi SR in config-manager, so that any bundle
working with yang can use it. This replaces too specific YangStoreService which
was coupled with yang-jmx-generator. Users can listen for changes directly
on OSGi SR - they receive service changed event when yang module is added or
removed.

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

Merge "Bug 453 - sal-rest-connector doesn't provide stream discovery feature"

Merge "Cleanup warnings in restconf-broker NotificationService"

Merge "Prune long-deprecated APIs"

Cleanup warnings in restconf-broker NotificationService

The registration map is not needed and the executor is not used, so
remove them. Also reuse AbstractListenerRegistration in place of a
custom-coded class.

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

Prune long-deprecated APIs

Prune APIs which have been deprecated well before the Hydrogen release.
No outside users should be relying on them at this point.

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

Remove unused import and raw type references

Fixes a couple of warnings just by removing imports and adding <?> to
Iterator.

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

Bug 453 - sal-rest-connector doesn't provide stream discovery feature

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

Merge "Remove duplicate netconf namespace constant"

Merge "Get rid of netconf operation filters"

Remove duplicate netconf namespace constant

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

Get rid of netconf operation filters

Merge NetconfOperationFilter interface into NetconfOperation
Execute all operations that can handle a certain request in chained execution (ordered by priority)
Reimplement all current filters as chained operations
Remove NetconfOperationRouter parameter from NetconfOperation
NetconfOperations handling the same request must have different priorities

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

Merge "Bug 383 - exception raised if key part of URI is incorrect"

Merge "Eradicate exificient from the repository"

Merge "Do not use currently-deprecated APIs"

Merge "Deprecate NetconfMessageHeader test class"

Merge "Cleanup dependencyManagement overrides"

Do not use currently-deprecated APIs

SchemaServiceListener has been renamed to SchemaContextListener
upstream, with the usual migration strategy. Let's use the latter where
possible.

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

Deprecate NetconfMessageHeader test class

This gets rid of the deprecation warnings, as the entire test will go
when NetconfMessageHeader goes.

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

Eradicate exificient from the repository

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

Merge "Exception for URI /restconf/operations/module_name:rpc ended with slash"

Cleanup dependencyManagement overrides

This patch gets rid of a few warnings coming from overriding version
specified in parent dependencyManagement section.

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

Bug 383 - exception raised if key part of URI is incorrect

Null pointer exception was caused for case when it wasn't possible to
decode key part of URI (null value was returned)

For case when key leaf type was identityref and wasn't specified as
module_name:identity_name NullPointerException was raised.

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

Merge "Refactor configuration for ConfigPersister."

Merge "Prune long-deprecated APIs"

Prune long-deprecated APIs

These APIs are from prototype days and have never properly worked. Prune
them to reduce the baggage we carry around.

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

Merge "Better exception handling when instantiating YangModelBindingProvider, fix confignetconfconnector activator."

Better exception handling when instantiating YangModelBindingProvider, fix confignetconfconnector activator.

Do not just log exception if it occurs while calling getModuleInfo, but add it as nested exception.
Fix confignetconfconnector activator: if activator fails to find ConfigRegistry, stop method should interrupt searching for it.

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

Merge "Integration test added to netconf-it to test identity (de)serialization."

SalRemoteClientDeployer - instantiate SalRemoteClient for given URL.

SalRemoteClient:
-Creates schema context.
-Initialize mapping service
-Creates restconf client
-Starts sal remote service broker
-Provides consumer registaration

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

Merge "Reuse AbstractListenerRegistration"