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>

Removed import related warnings from most of YANG Models

  - Added revision-date statement to YANG models,
  - Added reusable grouping to inventory model, used this grouping
    in service models.

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

Enabling vlan in flow spec in backend

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

Added reuse support to MD-SAL for config subsystem

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

Fixed typo in node-inventory model.

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

Switched to using the single group repo from nexus.opendaylight.org

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

Refactor if/else with dead branch to explain corner case better

Change allocating address from subnet pool path to split the pool
in the main branch rather than the else and explain what happens
if the pool as a single address with a comment.

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

Add missing options to run.bat

  - Enhance run.bat to support same options as run.sh

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

Bug 73: Thread safety in FRM

- Have nodeFlows and groupFlows non cluster caches
- Update the two above indirectly on every installedSwView cache update (local or remote)
- Have the update be performed by the FRM event thread only so that all the updates are streamlined

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

Cannot delete static flow through northbound

- Removed wrong annotation in flowprogrammer.northbound

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

Added field getters to class ICMP

Change-Id: I436082be38ae00a1825b2cf83a607786f69cfb72
Signed-off-by: Noam Mor <noam.mor@contextream.com>

Fix star import and enable checkstyle rule to prevent it.

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

Make topomanager members protected instead of package protected

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

Handle conflicting types in BundleScanner

  - Change precedence order for JAXB types
  - Add a validation check to report conflicting types
  - Add exclude capability to explicitly exclude types

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

Edit Static Flow from NB APIs

Change-Id: I837a288755b7800abd8ccd76a35895eed82a4643
Signed-off-by: Sapan Shah <sapshah@cisco.com>

Miscelanous fix for Eclipse users

- Make sure the jacoco-maven-plugin is not contemplated by m2e in md-sal.
- Make sure the assembly target runs from the release pom project as
opposed to the distribution as it was done before.
- Silent warning from the module
org.opendaylight.yangtools.yang.parser.util.ModuleDependencySort they
don't seem at all warning rather simple advices

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

Base integration of Binding Aware Broker with Config Subsystem

Binding Aware Broker is now initialized via config subsystem
as default module, not bundle activator, this allows for
configurable modules declare BindingAwareBroker and MD-SAL as
one of their dependencies.

MD-SAL is still also available via OSGI registry and provided
Activators.

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

Enable enunciate to produce Neutron Service NB API documentation.

POM lacked full enunciate config to produce documentation

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

Refactor ClusterManagerTest to use correctly junit

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

Added support for the maven enforcer plugin to check java version is 1.7

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

Remove superfulous class from neutron NB bundle.

Remove superfulous NeutronNBInterfaces class from neutron NB bundle
and refactor other classes to use NeutronCRUDInterfaces methods

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

Stop using deprecated pax Configuration annotation. Move the code to use
not deprecated annotation.

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

Make existCache more efficient in ClusterManager

Allocate cacheName only when it's needed, when manager
is null the concatenation of strings is redundant.

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

Fixing warnings in the sal sub-project.

This fixes all of the warnings in this module mostly by (a) adding
full types to raw types when possible, (b) removing unused imports,
and (c) adding @SuppressWarning statements in tests where variables
are intentionally unused.

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

Refactored unreadable test in JMXGeneratorTest.

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

The class DataPacketServiceAdapter was added

It is mirror for DataPacketAdapter class.

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

Clean all unused and redundant imports in controller.
Enable verification of unused and redundant imports via checkstyle.
Rename checkstyle configuration file because it no longer checks only
for tabs.

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

Refactor allocateCaches in TopologyManagerImpl

Extract common code to a separate method and reuse it.

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

Added BundleContext reference to generated factories for config subsystem

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

FlowStatisticsAdapter was added.

It is mirror for InventoryAndReadAdapter.

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

Added case with Choice-case to instance identifier building

- Choice-case is resolved when instance identifier is builded
- added tests for creating instance identifier from uri

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

Class FlowServiceAdapter was created as mirror class to
FlowProgrammerAdapter

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