When node disconnects from master controller, no node takes over

- Node connections caches in connection manager were allocated after
  configureInstance call, thus failing CacheUpdateAware registration.

- Fix couple of NPEs in devices.web

Change-Id: I51975a4a984606ecfad8db0941315164eedc83cb
Signed-off-by: Yevgeny Khodorkovsky <ykhodork@cisco.com>

Topology Manager to avoid redundant edge updates

- Redundant edge updates (such as in the case of
  cluster switchover) should not be notified to apps/listeners, to avoid
  expensive tasks

Change-Id: I44845d99085771e1dfebd63d12685fd2c2d8260c
Signed-off-by: Yevgeny Khodorkovsky <ykhodork@cisco.com>

Add northbound archetype

Change-Id: If7b42ddc1e6053450f8a7b17f3944c7ff026c5c4
Signed-off-by: Andrew Kim <andrekim@cisco.com>

Fix clustering versions

This fixes the versions referenced in clustered-data-store and warnings
for overridden versions in sal-remoterpc-connector.

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

Use ObjectRegistration instead of Registration

Registration is becoming a generic concept, not tied to a particular
object. ObjectRegistration succeeds it for uses where you need to a
getInstance() object to get the associated object back. Migrate to using
that to get rid of deprecation warnings.

Drive-by:
- fix raw type use of ServiceRegistration

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

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>

Cleanup AbstractDataTransaction state changes

Centralize state to be internal and add time tracking, such that it can
be used in logs and statistics.

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

Remove SupressWarnings("all")

It turns out to be unnecessary except for a raw type usage -- which is
fixed.

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

Add interface documentation

Change-Id: I3573d1153a62bfe36a01504198a3d1182d1013c1
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>

Added explicit revision date import of ietf-inet-types to opendaylight-statistics-type.

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

added generated files to .gitignore

- files and directories generated by maven-eclipse-plugin

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

Resolve Bug:419. Bump logging bridge reference to 0.4.2-SNAPSHOT.

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

Merge "Remove duplicate netconf namespace constant"

Merge "Get rid of netconf operation filters"

Take advantage of MultipartTransactionAware

Now that we have MultipartTransactionAware as a modeling concept, we can
simplify the message tracking code by not having to pass around another
boolean.

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

Merge "Enhanced message handling"

Merge "Overrode equals and hashcode in RouteIdentifierImpl Updated pom to reflect correct dependency version"

Merge "Adding import for google.common.collect in pom"

Merge "Enabling Remote RPC Router module in ODL distribution."

null check for the node connector in the container's edgePropsMap

Change-Id: I448803eb09a39d8d9ddb598231bad016eaaa1590
Signed-off-by: Asad Ahmed <asaahmed@cisco.com>

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>

Enhanced message handling

-Added support for ping/pong messages
-Improved logging
-Refactored code, added MessageHandler for better handling of message
-Updated dependency version in pom

Change-Id: I04ff607c287afbe8a2ea156a0f8cd9029c1504ab
Signed-off-by: Abhishek Kumar <abhishk2@cisco.com>

Overrode equals and hashcode in RouteIdentifierImpl
Updated pom to reflect correct dependency version

Change-Id: Ie88cecec686a5c542f8d3edcc99e895ecbf39c3b
Signed-off-by: Abhishek Kumar <abhishk2@cisco.com>

Adding import for google.common.collect in pom

Change-Id: Iabddd5cb310a6bf0ff40b38c4decc1a5ae915ffb
Signed-off-by: Abhishek Kumar <abhishk2@cisco.com>

Enabling Remote RPC Router module in ODL distribution.

- Added initial Route Change event
- Updated pom
- Fix rebase conflicts
- Added 02-clustering to initial configuration

Change-Id: I64dc6550171755af52a403f576c086ef2257b4a2
Signed-off-by: Abhishek Kumar <abhishk2@cisco.com>

Merge "The code through out hosttracker assumes the switch manager service reference is not null. Hence making switch manager as a required dependency to satisfy the above assumption."

Merge "Remove static state and default beans from RuntimeMappingModuleFactory,SchemaServiceImplSingletonModuleFactory."

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"

The code through out hosttracker assumes the switch manager service reference is not null.
Hence making switch manager as a required dependency to satisfy the above assumption.

Change-Id: I067e35de9910a6d5ba057f94ec161468d6b37ad9
Signed-off-by: Asad Ahmed <asaahmed@cisco.com>

Merge "included the nodetype in the congruence check. also handled the condition where the node id is null."

included the nodetype in the congruence check.
also handled the condition where the node id is null.

Change-Id: I536312801719f7596381db3294c091560d3b5e49
Signed-off-by: Asad Ahmed <asaahmed@cisco.com>

Merge "Eliminate superfluous pluginManagement section"

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>

Remove static state and default beans from RuntimeMappingModuleFactory,SchemaServiceImplSingletonModuleFactory.

Remove static reference from factories as this breaks tests: it is useful to tear down config subsystem
between tests, but there is no way to clean up static fields.
Since default beans are discouraged, rework those factories to enforce instance naming so that only a single
instance can be created.
01-md-sal.xml already creates both modules.

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

Merge "Add toaster sample consumer/provider modules to initial configuration."

Eradicate exificient from the repository

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

Path class exposes internal field

- Currently a caller of Path.getEdges() can mutate the Path object's edge list

Change-Id: I35659f73b88e1426df05ea3ac4157f9ef56f7f77
Signed-off-by: Alessandro Boch <aboch@cisco.com>

Merge "method unsetFlowProgrammerNotifier was mis-spelled in activator causing the corresponding unset method in flow programmer service never to get called"

Merge "Use same memory settings for surefire and failsafe plugins"

method unsetFlowProgrammerNotifier was mis-spelled in activator
causing the corresponding unset method in flow programmer service
never to get called

Change-Id: I33ee35b3b75d7c389189cc1609e1991bc80223c0
Signed-off-by: Asad Ahmed <asaahmed@cisco.com>

Merge "FlowConfig to only run syntactic validation"

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

Merge "We already have a merged and validated flow entry. There is no need for extracting the original flow entry and then validating and merging it again with the containerflow."

Merge "Centralize yang-maven-plugin version"

Merge "Deprecate org.opendaylight.controller.sal.common.util.Futures"

Use same memory settings for surefire and failsafe plugins

  - This addresses the OOME issue seen during the tail end
    of the build when running unit tests.

Change-Id: I054e1dfa502bca19a62521a3b714f1a072b59ded
Signed-off-by: Prasanth Pallamreddy <ppallamr@cisco.com>

Fixed order of applying changes when forwarding transactions

Commit 5659f588f80522a5be5c80d0e5ff34068b5f4786 changed the semantics of
remove/update operations such that the order of application became
significant: remove, then put.

Unfortunately we failed to notice that there are two places which apply
the operations in wrong order, thus breaking clients who perform a
remove/put in a single transaction -- e.g. replacing data without
merging it.

Fix DataStore and Binding->DOM connector.

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

Eliminate superfluous pluginManagement section

This structure is a recipe for a warning: jacoco.version is specified in
a parent pom, so there is no use specifying it in the pluginManagement
section here. Rather move it to the parent -- where it belongs.

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

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>

Centralize yang-maven-plugin version

There is no reason for individual projects to define the version over
and over again -- let's keep it in one place.

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

Deprecate org.opendaylight.controller.sal.common.util.Futures

We use guava throughout -- and those are better.

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

Add toaster sample consumer/provider modules to initial configuration.

Toaster modules are now part of configuration subsystem, but were not started by default so far.
Toaster consumer/provider are now started by default and behave just like before.

In addition, now you can access toaster modules via netconf endpoint started by the controller (port 8383).
Toasts-made count for toaster-consumer is now reported as operational data and can be accessed using get rpc.

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

We already have a merged and validated flow entry. There is no need
for extracting the original flow entry and then validating and merging it again
with the containerflow.

Change-Id: I131abdefc3bde14b148af0264cd220ea0c61c528
Signed-off-by: Asad Ahmed <asaahmed@cisco.com>

Adding routed RPC support in Remote RPC Router

 - Updated RoutingTable to add a new routed RPC cache
 - Updated sal-core-api to add Remote RPC Router as default delegate in SchemaAwareRpcBroker
 - Refactored RemoteRpcProvider code.
 - Added unit tests for routing table
 - Removed RouteIdentifier <-> JSON conversion. Its not used anymore
 - Rebased with master

Change-Id: Id13a0a6a9ae6ded7ea9068b7a613a9e196c89a7b
Signed-off-by: Abhishek Kumar <abhishk2@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 "Migrate toaster samples to config-subsystem"

Merge "Add reverse() method in Edge and Path classes"

FlowConfig to only run syntactic validation

- The validate function of FlowConfig class should only run syntactic validation
  The opearational validation like input/output/enqueue port presence, flood
  action on non default container check, etc. should be run only at installation
  time. Also this validation should be run for all the flow installation requests
  either they come from static flow config or from Flow Entry install/modify
  request coming from other bundles.
- With the above change at controller restart, static flow configurations which
  were valid at controller stop but no longer after reboot (for ex. switch sdn
  enabled ports have changed) will still show on the Flow GUI as non installed,
  and their toggling with fail with proper message.

Change-Id: I49d97af49d58102e51a3c8c12db915b60bcb6f4b
Signed-off-by: Alessandro Boch <aboch@cisco.com>

Merge "Refactor configuration for ConfigPersister."

Merge "Fix for bugs 446 and 456"

Migrate toaster samples to config-subsystem

 - Fix integration test for toaster samples.
 - The toaster is now disabled by default in distribution
   in order to enable it, you must change configure toaster module
   as in opendaylight/md-sal/samples/toaster-it/src/test/resources/controller.xml

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

Merge "Suppress unchecked cast warnings"

Merge "Prune long-deprecated APIs"

Suppress unchecked cast warnings

These casts are either safe, or we need to throw an exception --
ClassCast is as good an exception as any.

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

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>

Fix logging arguments in two-phase commit

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

Fix comments and remove use of deprecated class

This primarily removes SchemaServiceListener, as that is deprecated and
implied by SchemaContextListener, which is replacing it.

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

Add reverse() method in Edge and Path classes

Change-Id: I27ada22079999facf6e9f00e05b60aa480e4f2b9
Signed-off-by: Chi-Vien Ly <chivly@cisco.com>

Merge "Close Bug:443 - Implement ImmediateEventExecutor wrapper to config-subsystem."

Merge "Add watchdog thread to detect illegal blocking calls during second phase commit."

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

Close Bug:443 - Implement ImmediateEventExecutor wrapper to config-subsystem.

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

Add watchdog thread to detect illegal blocking calls during second phase commit.

Add a thread that starts when second phase commit starts, that will monitor if
a module instance occupies the commit thread for more than 5 seconds. If so, log
a warning every second informing about possible misuse of config subsystem to
block on external resources.
Added a proxy module that caches Module#getInstance() call as this method is
idempotent.

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

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 "fix bug 431"

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

Merge "Accept agentpath as a parameter to the run script"

Merge changes I1b834def,Iece78be2

* changes:
  Removed duplicated dependency declaration: ietf-netconf-monitoring,ietf-netconf-monitoring-extension from commons.opendaylight.
  Move xtend dependency definition from md-sal pom to commons.opendaylight

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"

Removed duplicated dependency declaration: ietf-netconf-monitoring,ietf-netconf-monitoring-extension from commons.opendaylight.

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

Move xtend dependency definition from md-sal pom to commons.opendaylight

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

Merge "Fix some sonar bugs in config-api and config-manager."

RpcRegistration.close() should not throw

Closing an RPC registration is an atomic act, e.g. it should never be
allowed to throw exceptions.

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

Reuse AbstractListenerRegistration

No sense in defining a new class -- just reuse what's already there.
Also uncovers the fact that we're losing an underlying registration --
but let's deal with that once the ListenerRegistration API changes not
to throw exceptions on close().

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

fix bug 431

Change-Id: Ifd972d7e8acfbc8ed2b9d64391f156b09617d304
Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>

Refactor configuration for ConfigPersister.

Pulled Configuration class from ConfigPusher that contains all the delays, timeouts and attempts numbers.

Add tests for config perister.

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

Merge "Modify config-api exceptions, bump config and netconf to 0.2.5-SNAPSHOT."

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

If URI of type /restconf/operations/module_name:rpc ends with slash
character (/) then exception with HTTP return code 404 and text
"Identifier IDENTIFIER_NAME can't contain slash character (/). If slash is
part of identifier name then use %2F placeholder."

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

Fix for bugs 446 and 456

Fix for "Null pointer exception on remove_router_interface" (Bug 446)
and "Return better error messages from NeutronAPIService" (Bug 456)

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

Added support to register of RPC Implementation.

NetconfDevice has now has capability to support of rpc implementation which adds support for ODL to send rpcs towards netconf server.
Added fix for commit handler registration. Now the commit handler is registered only and only when mount instance is availible.

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