Config save support for networkconfiguration.neutron

With the ODL ML2 Integration, the REST APIs are uni-directional, hence a simple controller restart
results in synchronization issues between Openstack/CloudStack ML2 and ODL.

The solution that was agreed with Kyle, Ryan, Hugo & others is to add the config save support on the
ODL side which will effectively remove the inconsistency issues between Openstack / Cloudstack and ODL.

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

Merge "Add shutdown hook."

Merge "Switch initial config files format to xml and add autodetect adapter for config persister."

Merge "Better logging for netconf-ssh."

Merge "Restconf operation GET, PUT, POST tests refactoring"

Merge "* fix : yang-generator can't find SIE if its base is another SIE"

Merge "Invalid cast results in HTTP 500 error returned by the Neutron REST APIs."

Invalid cast results in HTTP 500 error returned by the Neutron REST APIs.

I think this is a copy+paste error where a INeutronSubnetAware is incorrectly casted to INeutronPortAware.

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

Restconf operation GET, PUT, POST tests refactoring

Tests for GET, PUT, POST were refactored. For every operation there is one
class with name Rest<OPERATION_NAME>OperationTest.java and special class
RestOperationUtils for common subrutines.

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

Better logging for netconf-ssh.

Log ServiceReference object in ServiceTrackerCustomizer methods.

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

Switch initial config files format to xml and add autodetect adapter for config persister.

Autodetect adapter allows to read initial configuration from xml as well as plaintext files.
The detection of file type is based on file content.

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

Merge "Added flow-node-inventory import to group-statistics.yang"

Added flow-node-inventory import to group-statistics.yang

Change-Id: I0382e8229a448e48de9b382f0d9a64dcd0bc0250
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Merge "Modify uncaught exception handler to call System.exit(1)."

missing dependecies reviewed and fixed

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

* fix : yang-generator can't find SIE if its base is another SIE

JMXGenerator processes modules in dependency sorted order. Known SIE
tracker introduced for use in ServiceInterfaceEntry#create.

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

Merge "ArpHandler to ignore ip packet sent to default GW"

Merge "incorrect dependencies and unnecesary export-package removed from pom.xml"

incorrect dependencies and unnecesary export-package removed from pom.xml

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

Merge "increased test coverage of config-persister"

Merge "OF 1.3: Remove range check for max-length attribute for Output action."

Merge "Added tests for DELETE operation"

Merge "Unify topology versions used in controller"

Merge "Fix a few POM versioning/dependency warnings"

Merge "Updated review comments.Bulk transaction support added trasaction attribute with RPC inputs"

Merge "Model PacketInReason on yang model from openflow-types.yang Signed-off-by: Srikar Rajamani <srikar.rajamani@ericsson.com> Change-Id: I2103ce75ea3506cf9864b22ce18899f37a4f7062"

increased test coverage of config-persister

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

Modify uncaught exception handler to call System.exit(1).

This introduces the ability to specify how uncaught exceptions are
handled via setting controller.uncaughtExceptionPolicy property.

There are two policies: IGNORE and ABORT. The former logs which thread
has died and does nothing else. The latter forces a virtual machine
shutdown when a thread dies.

The default policy is IGNORE, as that has been the historic setting. For
Helium release, this should probably get discussed, as uncontrolled
thread death may leave the controller in an inconsistent/invalid state.

Change-Id: Ib7c64b23e1cd3c7c45cfa31401410bba22acd431
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>

OF 1.3: Remove range check for max-length attribute for Output action.

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

Model PacketInReason on yang model from openflow-types.yang
Signed-off-by: Srikar Rajamani <srikar.rajamani@ericsson.com>
Change-Id: I2103ce75ea3506cf9864b22ce18899f37a4f7062

Add shutdown hook.

Implement shutdown bundle that stops system bundle on JMX, netconf RPC, or when invoked on
shutdown service taken from OSGi service registry. User must provide a shutdown secret
set during initial module configuration and may provide a reason of shutdown.
Currently this bundle is not instanciated during server startup.

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

Added tests for DELETE operation

Change-Id: Iba7874231d8662a5626a262228dd1baafead897f
Signed-off-by: Ladislav Borak <lborak@cisco.com>

Updated review comments.Bulk transaction support added trasaction attribute with RPC inputs

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

Unify topology versions used in controller

This change is required to prevent a split-package from occuring in the
SP editions, as MD-SAL pulls in an old topology artifact, contents of
which are also provided by an updated version.

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

Fix a few POM versioning/dependency warnings

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

Switch from using a Long as FlowId to a Uri

Couple of things you may want to pay attention to,

a. FlowProgrammerAdapter maintains a mapping between a Flow and it's id in a map in infinispan
b. Look at internalAddFlowAsync. If it finds that a flow is in the flow to id mapping cache. It will remove the flow and add it back in. This is what makes the switch add back a flow when it a switch comes back alive.
c. The sync vs async behavior should now be properly done

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

Merge changes I85b49247,Icca28a4a

* changes:
  Added DELETE operation
  Changed POST operation

Merge "Added functionality to store capability info."

Merge "Fixed bug when converting augmentation in Instance Identifier"

Merge "Add test for generated code checking list of dependencies."

Added DELETE operation

- fix of bug 266
- corrected @Produces and @Consumes annotations in RestconfService and RestconfServiceLegacy

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

Added functionality to store capability info.

Implemented functionality into sal-netconf-connector to support of storing netconf device capabilities.
Inventory model extended for augmentation of netconf device capability information.

Signed-off-by: Lukas Sedlak <lsedlak@cisco.com>
Change-Id: I205fb2b1505cf823f436c6a9e7cd47e88b632c0a

Changed POST operation

POST operation for creating resource was changed according to RFC
http://tools.ietf.org/html/draft-bierman-netconf-restconf-02#section-3.4
- test fix
- ResponseException tests

Change-Id: Icca28a4a57499be5f29a1b618a432dd61dde2816
Signed-off-by: Martin Sunal <msunal@cisco.com>
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>
Signed-off-by: Ladislav Borak <lborak@cisco.com>

Fixed bug when converting augmentation in Instance Identifier

QName for augmentation s not present in compile-time generated
classes, but Instance Identifier codec did not explicitly checked
that.

Changed implementation in a way, that now it does not uses
reflection, but runtime generated Augmentation codec to
provide valid QName.

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

ArpHandler to ignore ip packet sent to default GW

- If the host pings the default gw, when ArpHandler receives the
  ICMP packet it does not realize it is sent to the controller,
  and it triggers the discovery for an host with that IP through arp request
- This patch is for ArpHandler to detect the if IP packet is sent
  to the controller and stop the processing if that is the case.
  We may consider to have ArpHandler reply if packet is ICMP echo req.

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

Merge "Always set interrupted flag when catching InterruptedException."

Merge "bulk transaction and co-relation support"

Merge "Minor changes in md-sal tests."

Merge "MD-SAL Statistics Manager-Fixed issue related to aggregate flow stats Also fixed minor bug in individual flow statistics augmentation."

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

bulk transaction and co-relation support

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

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

Minor changes in md-sal tests.

Temporary ignored test, removed @Override annotation.

Change-Id: I4d483770a96bf57973164f82c27f20f7d29a1e1f
Signed-off-by: Martin Vitez <mvitez@cisco.com>

MD-SAL Statistics Manager-Fixed issue related to aggregate flow stats
Also fixed minor bug in individual flow statistics augmentation.

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

Always set interrupted flag when catching InterruptedException.

Fixed catching InterruptedException in config-subsystem and netconf.
In netconf persister propagate interrupted exception as it indicates
that netconf connection was not established and thus config pushing
does not work.

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

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.