Commented out Sys.outs in runtime generated code.

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

Merge "Fix for Bugs 324"

Merge "run.sh: fix -status command exit codes"

Merge "Updated port-types yang file to adhere to pyang validation. OF Plugin review: https://git.opendaylight.org/gerrit/#/c/4434/ Signed-off-by: Deepthi V V <deepthi.v.v@ericsson.com> Change-Id: I4e1d0dd355864d7ed3f6f9ce4251d20db40a5a05"

Merge "pyang fixes for table/meter types"

pyang fixes for table/meter types

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

Updated port-types yang file to adhere to pyang validation.
OF Plugin review: https://git.opendaylight.org/gerrit/#/c/4434/
Signed-off-by: Deepthi V V <deepthi.v.v@ericsson.com>
Change-Id: I4e1d0dd355864d7ed3f6f9ce4251d20db40a5a05

Fix for  bug 288
https://bugs.opendaylight.org/show_bug.cgi?id=288

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

Fix for Bugs 324

Small fix to handle an NPE in TopologyMapping

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

Use version 0.6.0-SNAPSHOT of yangtools everywhere in controller.

As of yangtools commit f5e9689b95d945c22c7a38870681cab899326de9, all projects
in yangtools repo have version 0.6.0-SNAPSHOT. This commit bumps all
yangtools dependencies to this version.

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

pyang fixes for base model type. changes do not affect plugin build

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

pyang fixes for inventory and topology models

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

pyang fixes for service models

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

pyang fixes for statistics module

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

pyang error fixes for management models.
Error fixed:
   name should be same as module name.
   all warning removed.

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

Do not wait for non-existing modules

The two capabilities removed are exposed from bgpcep concepts, so md-sal
configuration should not depend on them.

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

run.sh: fix -status command exit codes

Currently -status command exits with exit code 0 to represent both
'running' and 'not running' state when pidfile is found and -1 when
pidfile is not found (likely not running).  Overloading exit code 0 makes
wrapper scripts incapable of differentiating 'running' from 'not running.'
Make 0 == running, 1 == not running, and -1 likely not running.

Change-Id: Ic258ab34e03ae1777b00bab75d1cfe30bc6c90da
Signed-off-by: Chris Wright <chrisw@sous-sol.org>

Merge "Move protocol framework from BGPCEP project"

Merge "Fix thread safety issue with StartExi operation"

Move protocol framework from BGPCEP project

This moves the current parts of BGPCEP protocol framework into the
controller project, eliminating the circular dependency forced by
netconf being resident in this project.

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

Remove dependecies on bgpcep concepts/util

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

Merge "Do not instantiate constant lists on each message"

Merge "Switch to using yangtools version of mockito-configuration"

Merge "Fixed NetconfDevice to store capability info."

Merge "increased test coverage"

Merge "Switch testing configuration for md-sal tests to xml."

Merge "Add deprecated warning for plaintext configuration files in config persister"

Merge "Update ietf yang dependencies from rev130712 to rev131021."

Merge "Rename initial config files from *.xml.conf to *.xml"

Merge "removing the old FRM code"

Merge "Refactor shutdown-impl: add parameter to RPC, remove secret validation and masking."

Merge "Fix for Bug 3"

Merge "Config save support for networkconfiguration.neutron"

Fixed NetconfDevice to store capability info.

Implemented functionality into sal-netconf-connector to support of storing netconf device capabilities.
Added "connected" state into netconf-node-inventory.yang.

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

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 NB capability to save configuration"

Merge "Move connection manager dashlet to left bottom"

Add NB capability to save configuration

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

Move connection manager dashlet to left bottom

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

Merge "Changed mount point URI decoding in restconf"

Fix thread safety issue with StartExi operation

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

Do not instantiate constant lists on each message

These really are constats, so treat them as such

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

Switch to using yangtools version of mockito-configuration

This commit switches to use the artifact from yangtools repository
introduced in https://git.opendaylight.org/gerrit/#/c/4316/, completing
purge of bgpcep dependencies.

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

Changed mount point URI decoding in restconf

- string "yang-ext:mount" in URI means mount point.
  After this string must be string in format moduleName:dataNode
  example:
  /rectconf/config/moduleName1:cont1/cont2/yang-ext:mount/moduleName2:cont3/cont4
     mount point path - moduleName1:cont1/cont2/
     path behind mount point - moduleName2:cont3/cont4

- refactoring of POST, PUT, GET, DELETE tests for status codes
- test for output of several modules to one xml and json file were added

Change-Id: I79bfaec1fab6ffd89295fb8461c167c0c85638ae
Signed-off-by: Martin Sunal <msunal@cisco.com>
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>

increased test coverage

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

Fixed bug : 205
https://bugs.opendaylight.org/show_bug.cgi?id=205

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

Switch testing configuration for md-sal tests to xml.

+ Fix bug regarding leading/tailing whitespace in capability string unmarshalled from xml.

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

Merge "Fix flow installation state in UI"

Merge "In the method to get flow stats for a particular match, the match from the stats can be null. Taking care of that in this commit."

Add deprecated warning for plaintext configuration files in config persister

Plaintext config files will no longer be supported in future versions.
Only xml format for config files is fully supported.

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

Fix flow installation state in UI

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

In the method to get flow stats for a particular match, the match from the stats can be null.
Taking care of that in this commit.

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

BugFix: Rename Input Port to just Port in Add Span Port Modal.

This fix just renames the input port dropdown on the Span Port Modal
to Port. Bug description says that "The port field for adding a span
port says "Input Port"  it should only say "Port" since a span port is
by definition an output port."

Change-Id: I2f123c5f81e8192bed7ec5bb9e74e2c262223938
Signed-off-by: lakshya <lakshya@cisco.com>

Added cache for YANG models downloaded from Netconf devices

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

Merge "Added extended ietf-netconf-monitoring detection for Netconf devices"

Update ietf yang dependencies from rev130712 to rev131021.

Update yang and xtend files to use october's version of ietf yang files.

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

Changed sal-compatibility from Consumer to Provider

sal-compatibility project uses DataProviderService
and registers commit handler thus it is
Provider contracts.

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

removing the old FRM code

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

Added extended ietf-netconf-monitoring detection for Netconf devices

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

Revert commit bedd505.

Revert adding ietf-topology bundle into yangstore's blacklist as this breaks bgpcep's distribution.

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

Merge "FRM should not remove a static flow config on async error report"

Merge "Add missing headers to config, netconf subsystems"

Merge "Add ietf-topology to yangstore blacklist."

Merge "Remove System.out from netconf tests"

Merge "increased test coverage"

Merge "Update to new version of ADSAL components and its dependencies proposed by jenkins-controller-version-changes-8"

Update to new version of ADSAL components and its dependencies proposed by jenkins-controller-version-changes-8

By splitting the effort into 2 phases, it is more managable and easy to test.
After this commit, the ADSAL components and all the NSFs are uptodate with the dependency convergence.

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

FRM should not remove a static flow config on async error report

- Currently FRM is removing the affected static flow config on
  receiving a flow error event from the switch.
  The correct action is to modify the config status.
  The config removal was a temporary fix for the flow configuration across
  cluster (request served by another controller node) when this request was
  always sent asynchronously. Now it is no longer needed as FRM implements
  both the synch and async flow rewust across the cluster.

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

Merge "BugFix : Changes to get ping between two hosts to work"

Fixed deployment of mount points (dynamic instances of MD-SAL)

  - Converted some xtend sources into Java classes
  - Added mount point deployer - service responsible for
    deploying dynamic instances of MD-SAL, which have
    separate context and could be created and retrieved via
    MountService.

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

Updated implementation of Netconf, fixed DOM Mountpoint

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

Refactor shutdown-impl: add parameter to RPC, remove secret validation and masking.

Remove old secret as it does not work well with config-persister.
Add parameter maxWaitTime to shutdown RPC that specifies amount of time before
server is forcibly shut down.
Work around buggy yuma message sending xml element without inner text
node if blank value is specified.

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

Allow initial config xml files to start with xml header.

E.g.

<?xml version="1.0" encoding="UTF-8"?>
<!-- vi: set et smarttab sw=4 tabstop=4: -->

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

increased test coverage

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

Add missing headers to config, netconf subsystems

Only java files

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

Add ietf-topology to yangstore blacklist.

Yang file of ietf-topology cannot be parsed by yuma's yangcli.

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

Rename initial config files from *.xml.conf to *.xml

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

Remove System.out from netconf tests

Change-Id: Iaf5b367649772ae0d52db109fd9d373e75107954
Signed-off-by: Maros Marsalek <mmarsale@cisco.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"

BugFix : Changes to get ping between two hosts to work

1. InventoryUpdates were not getting to ConnectionManager because the InventoryAndReadAdapter was not tracking all listeners
2. DataPacketServiceAdapter was not setup as an IPluginInDataPacketService
3. DataPacketServiceAdapters delegate was not set to the PacketProcessingService
4. Node was not set on the TransmitPacketInput causing an NPE somewhere in the MD-SAL internals (this should probably be better addressed as I have hit this twice already)
5. DataPacketServiceAdapter now handles situation where the NodeConnectors is null

Current state
An IllegalArgumentException is being thrown from the openflowplugin "Session cookie is invalid" - Need to look at that further

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

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"