Merge "Updated sal-netconf-connector mountpoint integration"

Merge "Unified Two Phase Commit implementation, fixed BA to BI connection"

Merge "Added inventory-manager to the distribution."

Merge "Clear the receive buffer when message parser throws exceptions. Don't terminate the connection when message parser throws exceptions. Send Features Request msg if not done so."

Merge "creating a default subnet"

Updated sal-netconf-connector mountpoint integration

  - Updated sal-netconf-connector mountpoint integration, added
    end-to-end test container from restconf to netconf to config
    subsystem.

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

Unified Two Phase Commit implementation, fixed BA to BI connection

  - Unified implementation of Two Phase Commit, Data Reading by abstract
    sal-common-impl.

  - Updated implementation of BA to BI connector to do schema-aware
    serialization / deserialization.

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

Added inventory-manager to the distribution.

Change-Id: Ibc92bac57690d5c5bf3ec2682e98576ff95920f3
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Updated MountPoint implementation

  - Fixed bug in AbstractDataReadRouter

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

Removing sysouts and adding log statments

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

Adding copyright notice

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

Clear the receive buffer when message parser throws exceptions. Don't terminate the connection when message parser throws exceptions. Send Features Request msg if not done so.

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

Merge "Bump versions of config-subsystem and netconf-subsystem to 0.2.3-SNAPSHOT. Remove ConfigProvider."

Merge "Updating ZeroMQ connector implementation. Its a work inprogress. The current implementation only handles announcement and rpc routing for global servies. Notification and Identity based routing will be added. Details about the implementation would be available at https://wiki.opendaylight.org/view/Zeromq_connector"

ClusteredDataStoreManager unit tests

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

Updating ZeroMQ connector implementation. Its a work inprogress. The current implementation only handles announcement and rpc routing for global servies. Notification and Identity based routing will be added. Details about the implementation would be available at https://wiki.opendaylight.org/view/Zeromq_connector

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

Merge "Initial implementation of the ClusteredDataStore"

Convert neutron northbound classes to unix line delimiters

Replace original Windows line delimiters.

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

Convert neutron implementation classes to unix line delimiters

Replace original windows line delimiters.

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

Convert neutron service base classes to unix line delimiters.

Replace original windows line delimiters in base classes.

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

Merge "Added threadpool config tests."

Merge "Improved translation from XML to Json"

Merge "Fixed RESTCONF integration, added IT skeleton"

Improved translation from XML to Json

- Fixed bugs in translation

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

Bump versions of config-subsystem and netconf-subsystem to 0.2.3-SNAPSHOT.
Remove ConfigProvider.

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

-SNAPSHOT added to exificent version

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

Added threadpool config tests.

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

Fixed RESTCONF integration, added IT skeleton

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

Merge "Added config wrapper for netty's NioEventLoopGroup."

Merge "Fix for incorrect request assumption in Update Neutron Port op"

Merge "Fix for bug 136 and bug 137."

Merge "- Added exi capability utilities, handlers and necessary modifications - Exi parameters decomposition - Introduced HandlerManipulator class in netconf-api to be used for   handlers management on channel pipeline - Exi encoding/decoding test added - exi returns NetconfMessage - added DOM attributes - exi encode decode test to expect NetconfMessage instead of Document - XmlElement.getName() returns localName first and tagName if localName not present, incorporated notes from Robert - NetconfSession implements all common methods - added copyright where needed - ProtocolMessageEncoder/Decoder stored in session and replaced with Exi   enc/dec - exi encoder/decoder interface become comaptible with   ProtocoleMessageEncoder/Decoder and no longer needs to be taken care of   this handlers - created thirdParty bundle for exi - incorporation of Maros's notes"

Merge "Test of conversing of yang + xml files to json 2."

Merge "Unified implementations of BrokerService Proxies which are provided to Consumers/Producers."

Merge "Change fix for default module's dependency resolver creation into two steps."

Merge "Added trasaction capabilities."

- Added exi capability utilities, handlers and necessary modifications
- Exi parameters decomposition
- Introduced HandlerManipulator class in netconf-api to be used for
  handlers management on channel pipeline
- Exi encoding/decoding test added
- exi returns NetconfMessage
- added DOM attributes
- exi encode decode test to expect NetconfMessage instead of Document
- XmlElement.getName() returns localName first and tagName if localName not present, incorporated notes from Robert
- NetconfSession implements all common methods
- added copyright where needed
- ProtocolMessageEncoder/Decoder stored in session and replaced with Exi
  enc/dec
- exi encoder/decoder interface become comaptible with
  ProtocoleMessageEncoder/Decoder and no longer needs to be taken care of
  this handlers
- created thirdParty bundle for exi
- incorporation of Maros's notes

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

Added config wrapper for netty's NioEventLoopGroup.

NioEventLoopGroup should be reused by dispatchers based on netty e.g. netconf-impl's dispatcher.

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

Test of conversing of yang + xml files to json 2.

Test was broken to two test. First one uses regular expression. Second one
uses JsonReader class.

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

Fix for incorrect request assumption in Update Neutron Port op

Neutron Port update code incorrectly assumed that request body
would always include fixed_ip information.  Bracket fixed_ip
adjustment code with proper test.

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

Fix for bug 136 and bug 137.

Change-Id: I3b8aa0006f2cd342e2ca87abf67cd964e8390f52
Signed-off-by: Baohua Yang <baohyang@cn.ibm.com>

Unified implementations of BrokerService Proxies which are provided to Consumers/Producers.

    - Added AbstractBrokerServiceProxy which is Autocloseable,
      and specific for each provider.

      This proxy is convenience wrapper, which keeps track of all
      object listeners retrieved from proxied service,
      so when proxy is closed (deregistered) all registrations
      are also removed.

    - Added concrete implementations of AbstractBrokerServiceProxy
      for Data*Services, Notification*Services and SchemaService
    - Updated implementation of SchemaService to provide notifications
      when new bundle with model is loaded.

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

Fixing a couple of NPE in the newly added Connection Manager UI.

Change-Id: Ib0904df810b5dbab8e8642d2650d58b63d28bec1
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

creating a default subnet

This changes how SwitchManager handles subnets so that when there are
no configured subents, it returns a default subnet that spans all ports
in the network and spans all IP addresses, i.e., 0.0.0.0/0.

This makes it so that HostTracker will discover all hosts by default
instead of only discovering hosts in configured subents. This, in turn,
makes SimpleForwarding forward traffic correctly without any
configuration.

Change-Id: Ice31b6035b970dd69fc69df3d9560b4aac41630d
Signed-off-by: Colin Dixon <ckd@us.ibm.com>

Initial implementation of the ClusteredDataStore

Please review and see if this implementation is headed in the right direction.
At this point this is mostly unit testing. This should give an idea of what the code intends to do.

The ClusteredDataStore is dependent on IClusterGlobalServices and provides a simple pass through mechanism.

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

Refactor to remove empty catch exception.

Initializing a subnet object included an empty catch exception.
Refactor to return a boolean from this method so that exception
is reflected to calling methods, which can then take appropriate
action.

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

Merge "Add checkstyle rule EqualHashCode that ensures that those methods are implemented together, and not just one of them."

Merge "Added DELETE support for Bridge and Port resources"

Change fix for default module's dependency resolver creation into two steps.

Call dependencyResolverManager.getOrCreate in scope of creating default modules. Add check to
putConfigBeanToJMXAndInternalMaps so that identifier of module and dependency resolver must match.
Remove deprecated method from DependencyResolverImpl. DependencyResolver interface now extends
Identifiable<ModuleIdentifier>

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

Add checkstyle rule EqualHashCode that ensures that those methods are
implemented together, and not just one of them.

Change-Id: Ie565d97208200487d2c9e4d43ec2c94df730d097
Signed-off-by: Alissa Bonas <abonas@redhat.com>

Merge "Updated AbstractDataModification to also implement DataChange"

Merge "SSH netty handler"

Added DELETE support for Bridge and Port resources

Change-Id: Ica167c669115706c5b5b916134c7ec4c168c9253
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Added trasaction capabilities.

Signed-off-by: Prasanna Huddar <prasanna.huddar@ericsson.com>
Change-Id: I3d176d25ec1478d5c6f1ed546bfab511f204dda9

Updated AbstractDataModification to also implement DataChange

  - AbstractDataModification now creates DataChange in the background
    during the creation of diff.
    This DataChange is easier for Commit Handlers to read
    and do processing based on more granular changes.

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

SSH netty handler

Added Netty SSH handler and
underlying virtual socket infrasctructure

Change-Id: I119494835e61820d23789125b70bb70c1299ae31
Signed-off-by: Robert Gallas <rgallas@cisco.com>

Merge "Bug:129 Connection Manager Dashlet"

Merge "Added Inventory Reader for SwitchManager"

Merge "Added configuration modules and closeable wrappers for FixedThreadPool, FlexibleThreadPool, NamingThreadPool, ScheduledThreadPool and NamingThreadFactory."

Merge "Unification of broker concepts implementations"

Added Inventory Reader for SwitchManager

  - Inventory Reader is runtime data reader, which delegates
    all state retrievals to the SwitchManager.

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

Added configuration modules and closeable wrappers for FixedThreadPool, FlexibleThreadPool, NamingThreadPool, ScheduledThreadPool and NamingThreadFactory.

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

Merge "Ganymed patch fix"

Merge "Added AsyncEvenBus and EventBus configuration modules implementations. Added EventBus and AsyncEventBus closeable wrappers."

Unification of broker concepts implementations

  - Introduced AbstractDataReadRouter into sal-common-impl
    which deals with data read routing.

  - AbstractDataReadRouter is then used inside sal-binding-broker
    to route reads between binding aware components

    and inside sal-dom-broker to route reads inside a mount point
    or between binding independent components.

    Extracted Rpc Routing logic from BrokerImpl.xtend and moved it to
    separate class, which is used in the broker (global context)
    and also in the mount-points (nested subsystems).

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

Merge "Added MD-SAL Topology Reader for reading and accessing AD-SAL topology."

Added AsyncEvenBus and EventBus configuration modules implementations. Added EventBus and AsyncEventBus closeable wrappers.

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

Added MD-SAL Topology Reader for reading and accessing AD-SAL topology.

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

Subnet gw can be configured w/o name

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

Threadpool APIs and thread-related configuration service interfaces.

EventBusServiceInterface - Service representing an event bus. The service acts as message router between event producers and event consumers.
ScheduledThredPoolServiceInterface - An extension of the simple pool of threads able to schedule work to be executed at some point in time.
ThreadFactoryServiceInterface - Service representing a ThreadFactory instance. It is directly useful in Java world,
where various library pieces need to create threads and you may want to inject a customized thread implementation.
ThreadPoolServiceInterface - Representing a simple pool of threads able to execute work.

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

Test of conversion of yang + xml file to json

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

Make default strategy for edit-config netconf operation MERGE in config-persister-impl

Replace strategy, deletes default modules

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

Convert to SAL Property instances.

Converts from FlowCapableNodeConnector to instances of SAL Property class.
Conversion for Bandwidth, Name, MacAddress are included.

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

Ganymed patch fix

Fixed missing java.net.socket import in ch.ethz.ssh2.Connection.java

Change-Id: Idc748894cd7bf30675e35083da1a35e5ff435b31
Signed-off-by: Robert Gallas <rgallas@cisco.com>

Added Compatibility for Switch Manager and Topology Manager

  - Created model-topology under controller for easier
    retrieval of topology models (baseline model is ietf draft
    and is present in YANGTools git).
  - Added aggregate topology view
  - Added mapping to inventory

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

Add checkstyle rule to prevent empty statements (;)

Change-Id: Id8b34fde64988326cf6cab494019f99d2d61cbc5
Signed-off-by: Alissa Bonas <abonas@redhat.com>

Ganymed library patch

SSH client patch of Ganymed library to accept
to accept precreated socket.

Change-Id: Icbccba360e9a86be41cc65a04f1e7aca8c4bffca
Signed-off-by: Robert Gallas <rgallas@cisco.com>

Ganymed SSH library provisioning

Creation of Ganymed OSGI wrapper bundle
Updated pom.xml in relevant projects
Enforcing Ganymed version 261

Change-Id: I2615f074aa463e49e18a30ef4824b7d09dc87876
Signed-off-by: Robert Gallas <rgallas@cisco.com>

Fix {} vs. %s mixup

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

Create mvn profile for integrationtests

Change-Id: I119cb15f184523201cb190f2dfe8b0639995f704
Signed-off-by: Alissa Bonas <abonas@redhat.com>

Updated ZeroMQ implementation

ZeroMQ implementation listens for rpc service implementations
on one controller node and when new implementation is registered
on controller it announces it via message bus.

The receivers of announcement registers itself as the implementation
of the same service in that node, and delegates actual processing
via message bus to originating controller instance.

This allows for deployment where one controller has a bundle
which provides service A, and other which has bundle which
only consumes service A to cooperate.

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

Bug:129 Connection Manager Dashlet

First iteration of the Connection Manager dashlet

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

Modify HostTracker to be able to interact with multiple host finders

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

Fixed bug with groupRef and meterRef in group and meter models.

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

Added listener for rpc registrations.

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

Added skeletons for ZeroMQ APIs for Binding Aware ZeroMQ Connector

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

Fix configuration for default modules

Automatically add default modules to jmx, even if they do not use dependencyResolverFactory

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

Add UM API to modify local user

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

Added Model-Driven Inventory Provider for Flow-Capable Node.

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

Fixed dependencies for yang-jmx-generator-plugin

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

Initial commit for the ClusteredDataStore

The ClusteredDataStore provides a mechanism for the MD-SAL Broker to share data across a cluster. It can be used by other components/services as well to share data.

The ClusteredDataStore will use the ClusteringServices interface to create a cache and read/write data to it.

This only includes the structure of the project.

Moved clustered data store within md-sal

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

Completed the missing configuration parameters in NetworkConfiguration Northbound APIs for addBridge and addPort

Change-Id: Ie92c90f61aa04fa2a18c7f4b68a8daea1819b1f0
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Altering ARPHandler and SimpleBroadcastHandler to forward traffic immediately.

This change modifies how ARPHandler handles punted IPv4 packets so that
they are immediately forwarded if the destination is known. This allows
for the first ARP response to come back to the requestor.

Change-Id: I278762c4048cb169bb9513e7237cac562bdc40ca
Signed-off-by: Colin Dixon <ckd@us.ibm.com>

Added an extra validation check so that
the gateway mask cannot be 0 or 32

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

Disabled yang-jmx-generator-plugin until we can figure out the Nexus issue with it.

yang-jmx-generator-plugin suddenly, and with no code changes whatsoever, began
having build issues.  It turns out that there is some slippery nexus issue
that is causing difficulties.  Debugging currently, this should unblock
us until a solution can be found.

Change-Id: I842152c1e0d5d1c6922722fc3c2dc41d9e19990f
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Add checkstyle rule UpperEll and fix all code accordingly.

UpperEll rule ensures that numbers of type long
are defined with a capital "L" in the end instead of "l"
for a better readability - per java language specification
recommendation.

Change-Id: I068c32d4aaed0980bee1d93cfeec511e72bad2f1
Signed-off-by: Alissa Bonas <abonas@redhat.com>

Improved exception handling thrown by client threads in ConcurrentClientsTest

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

NetconfClientDispatcher moved to constructor arguments in NetconfClient
This way NioEventLoopGroup instance can be reused by multiple clients.

Dispatcher created in NetconfCLient's constructor spawned an instance of NioEventLoopGroup (in AbstractDispatcher in bgpcep repo).
Instance of NioEventLoopGroup spawns new threads and with multiple concurrent clients system could get out of resources e.g. Too many open files, Unable to create new native thread.

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

Added serializer/deserializer for JSON/XML

- exceptions are translated to rest call response with state code 400 or 500
- imporoved singleton pattern
- tests
- added providers for serialization/deserialization of JSON/XML
- supported rest methods: readData, invokeRpc

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

Changing the file names from run.sh/run.bat to runsanity.sh/runsanity.bat to make it explicit.
A lot of human errors were made due to the identical name to the run.sh under target.

Change-Id: Ic8289d22ba3dd7fec004a9421eac3fbd978fc5fb
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>