Merge "Addressing a flow programming issues on controler fail-over scenario in a Clustered setup."

Merge "Fix run.sh to work from any directory."

Fix run.sh to work from any directory.

Add cd command just before executing java process to work around the issue that osgi container does not correctly start
from any folder except where run.sh is located.

Change-Id: I42562cbb88922cefe2347b8e0d293067fbd10d3d
Signed-off-by: tomas <tolvecky@cisco.com>

Merge "Remove redundant expression true:false"

Merge "Declare a urlPrefix for reuse in NorthboundIT."

Merge "Unitests: remove conditions that are always true/false."

Merge "Use InetAddress.getLoopbackAddress in ClusterManagerCommon."

Merge "Use Java 7 new method getLoopbackAddress in ClusterManager"

Merge "private key configurable in config.ini"

Merge "Improve error logging in yang-store-impl"

Merge "Update maven-bundle-plugin version in config and netconf."

Merge changes If79fa865,Ia6652643

* changes:
  Fixed some TODOs in netconf.
  Update service reference persistance according to new ServiceRegistry API in config subsystem.

Addressing a flow programming issues on controler fail-over scenario in a Clustered setup.

FRM : Handling the uninstallAllFlowEntries() only on containerDelete cases
Simple Forwarding : Not to fail for false-reasons when Routing fails to get route when src and dst-node are the same

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

Merge "Fix testing support for OSGi services registration under String name(s)"

Merge changes I994c19ee,I245e9e7a

* changes:
  sal-rest-connector test refactoring
  More test for improving of code coverage + test refactoring

Bug 267: Upgraded maven-bundle-plugin to 2.4.0
While trying to fetch augmentation code, ClassNotFoundException is thrown for certain classes due to missing imports in Manifest.mf of model bundles. These classes were used as type parameters in Augmentation<T>.
maven-bundle-plugin 2.4.0 fixes the issue of missing imports for generics.

Signed-off-by: Deepthi V V <deepthi.v.v@ericsson.com>
Change-Id: If917e687a3815ad730a26bd46efbd057bd6e2243

private key configurable in config.ini

Netconf SSH bridge has private key stored in distribution.
Path to this file is configured in config.ini.

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

Update maven-bundle-plugin version in config and netconf.

Update from 2.3.7 to 2.4.0

Change-Id: Ie121db016d5532df58da05244d7883520cdf8b11
Signed-off-by: tomas <tolvecky@cisco.com>

Improve error logging in yang-store-impl

Resulting exception should contain getMessage information from each nested exception.

Change-Id: Idcb124a6e15f2be952d6b64028924929011e7156
Signed-off-by: tomas <tolvecky@cisco.com>

sal-rest-connector test refactoring

Mainly deleting unnecessary methods.

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

More test for improving of code coverage + test refactoring

Added tests to sal-rest-connector to improve code coverage.
Added test for leafref.
Some methods in TestUtils were refactored.

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

Merge changes I7e42fc60,I6a7335b4,Ie11ee943,Ic48123a3,I89b6349e,I23a09513,Ibe5395dd,If05e0412,I19f17f97,I6f4beab1,I678c46d9

* changes:
  Added WADL generation for Models
  Fix for Bug 271
  Add a flow by adding the flow data into the datastore
  Set Node on the AddFlowInput and RemoveFlowInput
  Fixed Topology Adapter to properly add Edge Properties
  Removed dead code in LLDPDiscoveryUtil
  Fixed two small FlowAdapter bugs.
  Fixed implementation of getNodeConnectorProps and getNodeProps
  Check for null srcNodeId and srcNodeConnectorId
  Fixed Node Properties in Adapter - with exceptions noted below
  Fixed adapter handling of Special Node Connectors

Merge "API for accessing the controller version"

Bug 267: Fixed augmentation read from configuration store

  - Fixed missing support of direct read of augmentation
    from configuration store.
  - Added regresion unit test, which test storing Node with
    FlowCapableNode augmentation and reading it directly
    via InstanceIdentifier pointing to the augmentation.

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

Fix testing support for OSGi services registration under String name(s)

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

Fixed advanced Netconf client functionality.

  - Added two-stage model for remote device
     - Schema-unaware - allows only for baseline netconf functionality
     - Schema-aware - allows for deeper get operations
  - Performance fixes for message sending and retrieval
  - Fixed received notification upstreaming by proper reimplementation
    of NetconfClientSessionListener

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

Merge "Support for mount points in Restconf"

Merge "Updated ContainerFlow to take multiple vlans"

Support for mount points in Restconf

Added support for mount points in Restconf paths, which allows for
deeper queries inside remote nodes such as Netconf.

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

API for accessing the controller version

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

Merge "Added constructor which allows for custom NetconfClientSessionListener"

Added constructor which allows for custom NetconfClientSessionListener

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

Added WADL generation for Models

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

Fixed Adapter to correctly pass NodeConnector Properties.

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

Fix for Bug 271

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

Fix to Topology Adapter to properly handle links coming and going

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

Add a flow by adding the flow data into the datastore

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

Set Node on the AddFlowInput and RemoveFlowInput

This avoids the NPE reported in bug #259

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

Fixed Topology Adapter to properly add Edge Properties

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

Removed dead code in LLDPDiscoveryUtil

Per aboch, lldp.getOptionalTLVList() never returns null
and thus the code checking for it is dead code.

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

Fixed two small FlowAdapter bugs.

1)  Handle NW Addresses correctly
2)  Add Order to Instructions

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

Fixed implementation of getNodeConnectorProps and getNodeProps

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

Check for null srcNodeId and srcNodeConnectorId

In response to comment here:

https://git.opendaylight.org/gerrit/#/c/3926/2/opendaylight/md-sal/topology-lldp-discovery/src/main/java/org/opendaylight/md/controller/topology/lldp/utils/LLDPDiscoveryUtils.java

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

Fixed Node Properties in Adapter - with exceptions noted below

Still need to fix Supported Actions -

Supported Actions present a bit of an issue.  In OF 1.0,
they come in with the GetFeatures message on switch connection.
In OF 1.3, they are per table, and come in with the response to
an empty bodied TableFeatures Multipart request.  So far,
no switch has been observed in the wild supporting the
empty bodied TableFeature message. OVS 2.0 doesn't, CPQD doesn't.

The net-net is that supported Actions are effectively unreportable
for OF 1.3 switches.

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

Fixed adapter handling of Special Node Connectors

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

Merge "Upgrade jackson library to version 2.3.0"

Merge changes Ib1fb74f6,I532fff58

* changes:
  Remove links from topology when port goes down
  Fix for port-state Bug 262.

Merge changes Idf3f69db,I4499d7c7,I512af911

* changes:
  Initial fix to topology-adapter.
  Fixed topology-manager now that we have topology-lldp-discovery
  Added topology-lldp-discovery module

Merge "Fixed northbound references to SAL"

Updated ContainerFlow to take multiple vlans

-In ContainerFlowConfig allowed for multiple vlan specified in list/range to be accepted
-getMatches now returns 1/2 match(es) for each vlan specified depending on bidirectional setting

Change-Id: Ib994f32b5540ea9fd4035c95e24c1ced3cbc06d6
Signed-off-by: Katelyn Kasperowicz <kkaspero@cisco.com>

Remove links from topology when port goes down

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

Initial fix to topology-adapter.

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

Fix for port-state Bug 262.

Changed from modeling port-state from an enum to a
group of boolean leafs in a container.

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

Fixed topology-manager now that we have topology-lldp-discovery

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

Added topology-lldp-discovery module

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

Adding support for the git-review command line tool

The OpenStack doc workflow (and entire gerrit workflow) has the
option of using a tool called git-review that automates ChangeID:
creation and management,automates rebasing of local commits
as well as simplifies submiting to gerrit and pulling and
pushing reviews.

Usage of the tool is entirely optional, however it does require
a pointer file named .gitreview in the root of the repository
that is referenced.

I am totally new to this project, if you guys don't want to merge
this then no big deal. However I think this will help with dev's
coming over from OpenStack as it will provide a optional workflow
that is closer to the OpenStack workflow.

Necessary to run git review -s to append the change-id into git-hooks

Change-Id: I188d1b67c84b7ac2815b5bf74885100bbe4f7c13
Signed-off-by: Colin McNamara <colin@2cups.com>

Fixed the API call to operationDataStore from RuntimeDatastore

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

Remove redundant expression true:false

The expression that is evaluated already returns boolean,
no need for additional 'true:false' decision, the correct value
is already returned.

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

Declare a urlPrefix for reuse in NorthboundIT.

All test cases use the same url prefix, but prior to this patch
it was copy pasted in all test methods.
For easier code maintenance, declared it once in the
class and reused it in all test cases.

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

Unitests: remove conditions that are always true/false.

Some conditions are unecessary as they are always true
or always false. They test Java and not
the actual logic, hence irrelevant and should be removed.

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

Use InetAddress.getLoopbackAddress in ClusterManagerCommon.

Replace hardcoded 127.0.0.1 with a standard Java method call.

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

MD-SAL Statistics-Manager - Minor fix in augmenting flow statistics

Change-Id: Ic5c44d0b9d29afbcf37d95537ac67b21d8fc57ac
Signed-off-by: Anil Vishnoi <avishnoi@in.ibm.com>

Use Java 7 new method getLoopbackAddress in ClusterManager

Replaces a hardcoded string 127.0.0.1

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

Merge "Update SubnetConfig Validation"

Merge "Add some log messages in case controller failed to add connected node."

Add some log messages in case controller failed to add connected node.

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

Upgrade jackson library to version 2.3.0

This change purpose is to use a newer version of the library,
as well as align it with the jackson version used by ovsdb project repo.

Main change areas:
1.Upgrade the jackson from codehaus.jackson version 1.9.x to
fasterxml.jackson version 2.3.0.
change all usages in pom files
and fix the imports in Java classes.
2.Remove the usage of jersey-json library.

Main resource for upgrade information:
http://wiki.fasterxml.com/JacksonUpgradeFrom19To20

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

Update SubnetConfig Validation

-SubnetConfig now validates that the address does not begin with 0 or 255
-Validates that host address is not set to all zeros or ones
-Caught exception when subnet not set to number

Change-Id: I825e2d54b48b7f74e35cfde8dc7e5967dac11684
Signed-off-by: Katelyn Kasperowicz <kkaspero@cisco.com>

Merge "Simplify method isMutualExclusive in Subnet. Remove redundant 'if' statements."

Merge "Refactor Subnet.isSubnetOf - reduce number of 'if' statements. Added unitests."

modified the FRM for group and meter to send ref to plugin for events

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

This commit contains following changes
1) Implemented statistics adaptor (IPluginOutReadService, IPluginInReadService ) interfaces in sal-compatibility layer
2) Minor change in Port-Statistis
3) Fixed FlowStatisticsAdaptor.java - Flow Statistics rpc/notifications were changed and that effected the class

Change-Id: Ie0a2170efa5338389f80cb0a586500297320b387
Signed-off-by: Anil Vishnoi <avishnoi@in.ibm.com>

Added notification for Flow add/delete/update.
Flow ref added to all Flow notification RPC messages.

Flow ref needed to for NSF to co-relate XID with the operation.

FlowTransaction.xtend modified to put flow reference.

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

Fixed as per comments by changed leaf name

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

Merge "Take care of a couple of exceptions in the process of handling an add flow on the Flow Programmer"

Merge "Use a String Id for both Node and NodeConnector"

Take care of a couple of exceptions in the process of handling an add flow on the Flow Programmer

There was an UnSupportedOperationException in MDFlowMapping and an NPE in FromSalConversionUtils which I
have taken care of.

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

Use a String Id for both Node and NodeConnector

Previously Node and NodeConnector were using NodeKey and NodeConnectorKey respectively as the Id. This causes problems
when you try to use some of the old north bound APIs as they expect to match a node or nodeconnector based on the string id
that is passed to it by the NB APIs

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

Merge "Provide an MDSalNodeFactory to construct an MD-SAL node for NB interfaces like the Forwarding Rules Manager"

Provide an MDSalNodeFactory to construct an MD-SAL node for NB interfaces like the Forwarding Rules Manager

When you try to get flows using the FR NB API for a given node you have to provide both a node type and node id. Since MD_SAL nodes
have a type MD_SAL which is not one of the standard types you need to provide and register a NodeFactory which can construct a Node
for an MD_SAL Node type.

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

Merge "On openflow plugin restart, NPE in tx poller"

Fixed northbound references to SAL

Change-Id: I1150ee6b4dda79f1648c8ad66799de8c69d792e2
Signed-off-by: mrajvaid <mrajvaid@cisco.com>

Merge "Add testing support for registering OSGi services to AbstractConfigTest"

Merge "Fix HashedWheelTimer exception for short living netconf clients."

changed models to include XID in add/update/delete service notification

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

Fix HashedWheelTimer exception for short living netconf clients.

An exception occurred in logs for short living netconf clients created by config persister.
These clients were closed before their initial check timer ran out.

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

Add testing support for registering OSGi services to AbstractConfigTest

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

Merge "Fixed bug in discovering JVM loaded case classes during code generation"

Fixed bug in discovering JVM loaded case classes during code generation

  - Added debug logging
       - routing table updates
       - rpc router creation, non-routed service registration
         proxy instantiation.

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

Fixed some TODOs in netconf.

Remove duplicate code
Remove commented modules in get-config response
Add test for netconf - persister integration
Remove already fixed TODOs

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

Update service reference persistance according to new ServiceRegistry API in config subsystem.

Utilize new proxy service mbeans that store reference name for dependency attributes.

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

Generate ADDED events for Node and NodeConnector

Components like switch manager need to see an ADDED event come in for Node and NodeConnector before
they accept changes for those elements. This commit first checks whether a Node/NodeConnector are
already known to the DataBrokerService. If they are not known then it notifies the listeners of an ADDED
event otherwise it notifies it of a CHANGED event

Also implemented node removal. NOTE: NodeConnector removed events do not seem to be coming through.

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

Merge "DataPacketAdapter fix"

Merge "Lazily initialize jaxb context in NB application"

DataPacketAdapter fix

added incomingNodeConnector for RawPacket from incoming PacketReceived so DataPacketService dont discard it

Change-Id: I7448296ae06b6170edf24289dfd12fc7fd185312
Signed-off-by: Juraj Sebin <jsebin@cisco.com>

Lazily initialize jaxb context in NB application

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

Fix method isSame in generated Abstract Modules.

When dealing with list of dependencies, == cannot be used. Instead,
both lists must be equal, and each dependency must be == to the old one.

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

Merge "Fixed Bug 250 - bits serialization between Binding Broker and DOM broker"

Fixed Bug 250 - bits serialization between Binding Broker and DOM broker

  - fixed serialization bug for YANG bits type when crossing brokers
  - Added regression test for bug, based on provided bug report
  - Decreased log level to debug and trace for some code generation
    functionality.

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

MD-SAL Statistics Manager - Changed Group/Meter Augmentataion location in operational data tree
Also added check, to only entertain multipart responses associated to the multipart request sent
by statistics-manager.

Change-Id: I51b1f4c9245ca30914293add0ea5fbefb3f1945b
Signed-off-by: Anil Vishnoi <avishnoi@in.ibm.com>