fix for BUG 663 - causing derived IT to fail

- added version to netty-config-api test dependency

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

Add Flood and FloodAll supported actions

This patch will return Flood and FloodAll as supported actions when a switch is queried for its supported flow actions

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

Merge changes I6bc37e88,I0514b26e

* changes:
  Fixed missing keys in sal-compatibility flows.
  Bug 508: Fixed concurrent conflicting commits by InventoryManager.

Added a comment that briefly describes the ArpHander service and explains some design decisions.

Change-Id: I2dd3877e5a12ad59f4c5057de09ecab2b4e9f768
Signed-off-by: Michael Bredel <michael.bredel@cern.ch>

Merge "BUG-467: reconnect-strategy configuration moved into controller/commons/protocol-framework -changed reconnect-strategy service and modules to reconnect-strategy-factory - configuration of ReconnectStrategyFactory."

Fixed missing keys in sal-compatibility flows.

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

Bug 508: Fixed concurrent conflicting commits by InventoryManager.

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

Merge "Bug 499: Added support for old DOM Broker APIs."

Merge "Bug 499: Added support for change listeners."

Merge "Bug 499: Initial draft of in-memory datastore and data broker"

Merge "Updater toaster to use datastore"

Merge "Initial commit for new Match classes"

Merge "Bug 499: Initial implementation of data tree modifications"

Merge "Bug 499: Initial implementation of supporting tree structures"

Updater toaster to use datastore

This patch adds the ability to expose the toaster status into MD-SAL, as
expected by the module. It further tweaks a thinko: dark toasts take
longer to make!

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

BUG-467: reconnect-strategy configuration moved into controller/commons/protocol-framework
-changed reconnect-strategy service and modules to reconnect-strategy-factory - configuration of ReconnectStrategyFactory.

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

Merge "Fix small resource leak in config-manager."

Merge "Update toaster example to emit lifecycle logs"

Merge changes I28d517fe,Ia6f0b6ce

* changes:
  Remove getPort API from sal-port.yang
  Avoid nullpointer exception on starting up the TopologyProvider and the InventorReadAdapter

Merge "Do not use InstanceIdentifier.builder()"

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

Fix yang-parser-impl not being pulled

sal-netconf-connect depends on it, but relies on in transiently via
yang-data-impl. That transient dependency is going away, so introduce a
proper dependency.

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

Initial commit for new Match classes

- Existing match classes rely on a static enum of match types.
  This change is to allow extensibility for new matching fields.
- This commit introduces new classes without replacing old ones
  and without migrating any code to use the new ones yet.

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

Bug 499: Added support for old DOM Broker APIs.

Added implementation of BackwardsCompatibleBroker
which exposes sal.core.api.DataBrokerService
using new DOMDataBroker and Data Store.

Enabled configuration which allows for configuring
specificing which implementation of Data DOM Broker
should be used in the MD-SAL.

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

Bug 499: Added support for change listeners.

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

Bug 499: Initial draft of in-memory datastore and data broker

Initial implementation of InMemoryDOMDataStore with support
for commits to the store and conflict checking

  - DOMDataBrokerImpl - initial implementation of DOM Data Broker which
       uses InMemoryDOMDataStore instances as store for Operational
       and Configuration data

  - InMemoryDOMDataStore - implementation of in-memory DOM Data Store
      which uses MutableDataTree

  - Updated SchemaAwareApplyOperation to provide
    more granular operations as
       - verifyStructure - verify if correct tree node types are used
       - isApplicable - verifies if modification is applicable to other
           snapshot

  Added test suite which tests InMemoryDOMStore, performance
  of DOMDataBroker and DOM Data Broker.

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

Bug 499: Initial implementation of data tree modifications

Initial implementation of data tree modifications using
StoreMetadataNode and NodeModification.

  - DataAndMetadataSnapshot - helper class, which
      contains immutable snapshot of data and metadata

  - ModificationApplyOperation - interface definition
      of composite hierarchical operation, which
      is responsible for applying changes to provided
      subtree

  - SchemaAwareApplyOperation - implementations of
      ModificationApplyOperation based on parsed
      SchemaContext

  - MutableDataTree - class which integrates
     Data tree, Metadata tree, Node Modification tree
     and ModificationApplyOperations into mutable logical
     tree, which will serves as a basis for read-write
     transactions.

  - Unit test for MutableDataTree in following scenarios
      - Create empty MutableDataTree, write subtree, read nested nodes
      - Create empty MutableDataTree, write subtree, modify subtree
      - Create MutableDataTree with existing date, write changes and deletes

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

Bug 499: Initial implementation of supporting tree structures

  - StoreTreeNode - interface defining common interface to tree
    structures used by data store.

  - TreeNodeUtils - utility methods which uses StoreTreeNode
      as tree definitions and provides functionality such
      as retrieval of deep nodes.

  - StoreMetadataNode - immutable tree node (and tree), representing
        versioning metadata and data.

  - NodeModification - mutable tree node (and tree),
       representing modification (diff) which should be applied to
       data tree. Tree is lazily populated by invoking client
       operations to be maded.

This patchset requires https://git.opendaylight.org/gerrit/#/c/5646/
to be merged.

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

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

Update toaster example to emit lifecycle logs

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

Merge "Add Enqueue validation check in FlowConfig"

Add Enqueue validation check in FlowConfig

Backend validation to check enqueue queue is within range 0 - 2147483647

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

Merge changes Id6b8ba01,Ic4081b36

* changes:
  Bug:553 - RPC providers are not cleaned up on bundle stop, causing bundle restart to fail
  Bug:441-closing listener on stop of md-topologymaneger bundle

Merge "fixed IT after netty version bump"

fixed IT after netty version bump

- we need access to sun.nio.ch
- netty is needed before protocol-framework
- netty bundles internal order changed

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

Fix IPluginInBridgeDomainConfigService API

IPluginInBridgeDomainConfigService protomoted Throwable into a
checked-exception status. This also includes Errors, which is not a good
thing, as Errors are not something we can handle gracefully.

Introduce a dedicated exception which implementations can use to report
instantiation problems.

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

Merge "Bug 500: DOMStore SPI"

Merge "Bug 500: Type capture for DOM and Binding APIs"

Merge "Bug 500: Updated data APIs with new concepts."

Fix Enqueue Action in Flow UI

Enqueue flow action accepts port and queue is optional

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

Remove getPort API from sal-port.yang

This API is not in use and it has not been properly modeled.

An RPC needs to have an input to be properly routed otherwise an exception will be thrown when registering.

Also see,

 - Bug 596
 - Gerrit https://git.opendaylight.org/gerrit/#/c/5781/

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

Avoid nullpointer exception on starting up the TopologyProvider and the InventorReadAdapter

The TopologyProvider and InventoryAndReadAdapter are Felix services which get started when all it's
dependencies are resolved (that is their start method gets called). The MD-SAL DataProviderService is not
or cannot be injected into them and so when start is called and the dataProviderService is used it throws an NPE

To fix this I simply moved most of the startup code into a separate method which I then call from the ComponentActivator
instead of start

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

Merge "Bug 141: Connection Manager special characters"

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 "Added explicit revision date import of ietf-inet-types to opendaylight-statistics-type."

Bump netty.io to 4.0.17.Final

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

Do not use InstanceIdentifier.builder()

Stop using the type-usafe identifier builder. This eradicates the
obvious places. The leftover now are just the BI connector.

Change-Id: Iff1d1fc58e638d02e08951dbddead067d3e746e1
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."

Bug:553 - RPC providers are not cleaned up on bundle stop, causing bundle restart to fail

Change-Id: Id6b8ba01194d71d2eb7ea97ca70088dec8b29d6e
Signed-off-by: Kamal Rameshan <kramesha@cisco.com>

Bug:441-closing listener on stop of md-topologymaneger bundle

Change-Id: Ic4081b36a557110f6daeb7a6fb8a3b39ee1be8b2
Signed-off-by: Kamal Rameshan <kramesha@cisco.com>

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>

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>

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>

Bug 141: Connection Manager special characters

Change-Id: I6778e0f9e20ab2cd22d01d77b43e6a1e3bade5bd
Signed-off-by: Sapan Shah <sapshah@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>

Bug 564 - add missing sal-remote dependency.

Change-Id: Ic6968699f0214226dd9881d08684febeaf031525
Signed-off-by: Milos Fabian <milfabia@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 small resource leak in config-manager.

When transaction is aborted, its ObjectName was not unregistered from platform mbean server.

Change-Id: Ia9b70ca68622fc187ed3db734d9344bd0d7b4ce9
Signed-off-by: Tomas Olvecky <tolvecky@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>