Bug 500: DOMStore SPI

This is the API which needs to be implemented by a single logical tree
store, such that it can be plugged into the in-memory datastore
prototype.

The prototype keeps one logical data store for each of operational and
configuration subtrees. A front-end (client-visible) transaction results
in two back-end (data store) transactions. State transitions between the
three are coordinated using three-phase-commit protocol (3PC), with the
frontend transaction acting as the coordinator and the backend
transactions acting as cohorts.

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

Bug 500: Type capture for DOM and Binding APIs

This commit introduces the type captures of new interfaces introduced in
the preceding commit.

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

Bug 500: Updated data APIs with new concepts.

  - Logical Datastore Type idenfier
  - Asynchronous Data Broker contracts - some data broker contracts
    were synchronous which could misslead consumers of APIs

  - New Transaction subtypes with support for Asynchronous reads
    - read-only transaction
    - write-only transaction
    - read-write transaction
  - Scoped Data Change Publisher - client code is able
    to specify scope of changes and logical datastore
    type on which it is listening.

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

Merge "1. Delete TLS KeyStore password from system property so that it won't be shown by getprop command. 2. Add a default location for TLS KeyStore files when the arguments are not provided."

Add Flow actions to UI

Flow UI now exposes only supported flow actions for particular network element

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

1. Delete TLS KeyStore password from system property so that it won't be shown by getprop command.
2. Add a default location for TLS KeyStore files when the arguments are not provided.

Change-Id: I017d20d460f755ddb94a18c8a30c037aec76de54
Signed-off-by: Jason Ye <yisye@cisco.com>

Update to use yang concepts

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

Fix sal-binding-api dependencies

Remove unused dependencies on sal-common and add used dependencies on
yang concepts and guava.

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

Merge "Add MountInstance client documentation and promote to ListenableFuture"

Merge "Fix sal-common-api dependencies"

Merge "Do not catch Throwables, but rather Exceptions"

Fix sal-common-api dependencies

sal-common-api should not depend on org.osgi.core nor sal-common. Remove
these dependencies and fix users which relied on org.osgi.core being
brought into compilation transitively.

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

Merge "Moving simple fwding logic from ARPHandler into SimpleForwarding"

Add FLOOD_ALL action to FlowConfig

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

Moving simple fwding logic from ARPHandler into SimpleForwarding

- Per Colin's idea, moving reactive fwding logic into SimpleForwarding
- SimpleForwarding listens to packet-in. Forwards IP packets
  between known hosts to dest (assuming a simple path will be installed).
- Some cleanup and debug stmts

Change-Id: Ib0f29af34d6dead26f7393b4238d62fe1935ecdf
Signed-off-by: Colin Dixon <ckd@us.ibm.com>
Signed-off-by: Yevgeny Khodorkovsky <ykhodork@cisco.com>

Fix sal-netconf-connector's pom.xml

The offending pom was not configured to use sal generator, so there was no way
how to get required capability for netconf southbound plugin.

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

Fix sal-remoterpc-connector's Import-Packages.

BND version 2.4.0 produces illegal manifest with the old code - inserts
com.google.common.collect twice. This will propagate as persister failure.
Remove offending line, format distribution.opendaylight pom.xml,
move sal-remoterpc-connector version to common's distribution management.

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

Merge "Fixed the problem in which different containers could have different mac addresses. This caused problems such as a gateway appearing as host, host appearing on isl's etc."

Fix for Bug #533

Ensure start of Allocation Pool is set properly in all cases

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

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>

Fixed the problem in which different containers could have different mac addresses. This caused problems such as a gateway appearing as host, host appearing on isl's etc.

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

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

Merge "Take advantage of MultipartTransactionAware"

Add MountInstance client documentation and promote to ListenableFuture

This adds documentation to MountInstance interface. Also promote
Future<> to ListenableFuture<>, as there is only a single implementation
anyway.

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

update the pom files to enable the nexus repository path to be configured by properties such that if can be overriden via the command line to support deployment to alternative locations

Change-Id: Ie96244a1a2a3f6b2f3760f366899017a44aa381d
Signed-off-by: David K. Bainbridge <dbainbri@ciena.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 "Add a container for pass-through netconf access"

Merge "Fix clustering versions"

Merge "Fixed whitespace dependent tests in config-persister-directory-adapter"

Merge "Use ObjectRegistration instead of Registration"

Merge "added generated files to .gitignore"

Merge "Fix typos in documentation"

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

Fix typos in documentation

Change-Id: I0023e7256f02170c04cc00579ee791f5a3578b1d
Signed-off-by: Moiz Raja <moraja@cisco.com>

Merge "Adding some more traces for better debuggability"

Adding some more traces for better debuggability

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

Merge "Fixed bug in NetUtils.isMulticastMACAddr() caused by sign extension."

Fixed bug in NetUtils.isMulticastMACAddr() caused by sign extension.

This patch fixes a bug that returned false incorrectly when the MSB was set
in the MAC address.

Change-Id: I8d04534962b0a1a585cf0217969a6415f360140b
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Merge "When node disconnects from master controller, no node takes over"

Merge "Topology Manager to avoid redundant edge updates"

Add a container for pass-through netconf access

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

Merge "Cleanup warnings in restconf-broker NotificationService"

Merge "Prune long-deprecated APIs"

Fixed whitespace dependent tests in config-persister-directory-adapter

Change-Id: I487011b2628d1b229d72fce19a37425c6a109a64
Signed-off-by: Michal Polkorab <michal.polkorab@pantheon.sk>

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