Add missing validation on ContainerFlowConfig objects

- Some basic validation is missing when modifying the container flow configs in a container

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

Cookie value is ignored while converting OF Flow statistics to SAL statistics

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

Remove declaring jacoco plugin twice in integrationtests pom.xml files -
because the pluginsManagement section is intended for use by child pom
files which do not exist in integration tests case.

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

Build fix: Fix bundle type for the bundlescanner.implementation sal dependency

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

Merge "UI support for multiple host per port"

Merge "Add password recovery procedure for default admin user"

Merge "Add missing null check in Container name validation"

Merge "Address @XmlSeeAlso limitation. Provide the ability to inject the JAXB types without having the need to explicitly define the inherited types with @XmlSeelAlso.  Introduced BundleScanner service which scans bundle classes for annotations dynamically and provides proper artifacts for NB applications to bootstrap. Eliminate the need for NB application extensions."

Merge "Reorder public/private modifiers as per JLS. (fixes sonar warnings)"

Add missing null check in Container name validation

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

Add password recovery procedure for default admin user

ISSUE:
  Need for a password recovery procedure for the controller default network administartor user (admin)
  which can be performed only by a user which has access to the host machine file system
PROCEDURE:
  1) System admin creates a flag file in startup directory
  2) Restarts User Manager implementation bundle (or the whole controller) through osgi console

  The above two actions will reset the default network administrator password to the factory one, "admin".
  Will log the change to console and remove the flag file.

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

UI support for multiple host per port

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

Fix for Span Port added to a Switch should show port name or port id.

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

Address @XmlSeeAlso limitation. Provide the ability to inject the JAXB types
without having the need to explicitly define the inherited types with
@XmlSeelAlso.  Introduced BundleScanner service which scans bundle classes for
annotations dynamically and provides proper artifacts for NB applications to
bootstrap. Eliminate the need for NB application extensions.

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

Merge "one.x.troubleshooting is undefined while updating node name"

one.x.troubleshooting is undefined while updating node name

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

Reorder public/private modifiers as per JLS. (fixes sonar warnings)

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

Remove heavy operation in the context of OSGi framework

The routine modeChangeNotify can be called in the context of the OSGi
framework via the chain:

<snip>
at org.opendaylight.controller.forwardingrulesmanager.internal.ForwardingRulesManager.modeChangeNotify(ForwardingRulesManager.java:2137)
at org.opendaylight.controller.switchmanager.internal.SwitchManager.switchManagerAwareNotify(SwitchManager.java:1797)
at org.opendaylight.controller.switchmanager.internal.SwitchManager.setSwitchManagerAware(SwitchManager.java:1677)
</snip>

now given the modeChangeNotify trigger an interaction with the network
element, it's better to keep that off a different thread to avoid to
stuck the OSGi framework for to long.

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

Merge "User Manager to hash users passwords"

Merge "Log TLS auth failure"

Merge "improved flow-type model"

User Manager to hash users passwords

- When a UserConfig object is created, hash the user password

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

Log TLS auth failure

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

Fix flows detail

- Fix flow with same names not correctly updating details table
  - Add extra check to differentiate between flows on different nodes
- Minor refactor in backend to consolidate Gson

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

Fix for Cannot delete static flow in a container via GUI.

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

Addressing gerrit 1182 Madhu's comment

- Making sure the executor service is available before the loadconfig
gets called, just in case of some race condition

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

Merge "Make FRM to finally distribute the work orders"

Merge "Fix for cache cleanup in protocol plugin on container deletion"

Make FRM to finally distribute the work orders

- Transformed frm.{workOrder|workStatus} caches to be transactional,
as a stop gap to an issue we have run into.
- The transformation of FRM caches in transactional opens an issue
where if used in a context of a transaction the FRM updates will not
travel in the cluster. We use the frm.{workOrder|workStatus} caches as
a way to synchronize operations hence this communications need to
happens also before the transaction commits. To avoid then this
chicken-and-egg problem we are forcing the distributeWorkOrder
business logic to execute off a different thread just to avoid to be
held by the transaction.
This is a stop gap solution till we caracterize and fix the
non-transactional cache issue seen

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

improved flow-type model

added priority, idletime, hardtime, cookies

Change-Id: I8550668d2317222cad10c61d5c3d0e6cec42b2e5
Signed-off-by: Michal Rehak <mirehak@cisco.com>

Fix modal session timeout

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

UI: Reset the scroll-bar to the top of modal

- Firefox retains scroll position of a modal window.
  Want to reset it so the user doesn't have to.

Change-Id: I4f80ebbd9938c3e8e957c2bf807d0a44e0362d80
Signed-off-by: Yevgeny Khodorkovsky <ykhodork@cisco.com>

Fix for cache cleanup in protocol plugin on container deletion

Change-Id: I5c16211ea12c384726a5a9d3f1cc23d40c1d0d3f
Signed-off-by: Pramila Singh <pramisin@cisco.com>

Merge "Supply from filesystem infinispan configuration"

Merge "FRMsync get stuck, miscelaneous fix"

Supply from filesystem infinispan configuration

- In order to speed up the prototyping time, allow to supply an
infinispan configuration from the filesystem along the one embedded in
the clustering.services-implementation. This can be specified by
supplying the property org.infinispan.config.file
- Align to 5.3 the XSD used in the infinispan config file, seems there
are different defaults between 5.1 XSD and 5.3

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

Merge "Refactor SubnetConfig"

Refactor SubnetConfig

- Change Subent Config to represent the switch ports with the NodeConnector notation
- Added validity check on zero subnet configuration
- Change device js to correctly set the value in the ports selects, JQuery changes from Andrew (andrekim@cisco.com)
- While uniform Subnet northbound to other NB modules, removed the node-ports endpoints. Have the subnet as end resource,
  we already have the POST for it.
- Added ISwitchManager.modifySubnet(SubnetConfig s)
- Added SubnetConfigTest.java for junit
- Updated subnet northbound junit

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

Added Configurable parameter for the Connection Manager scheme and Container profile.

Change-Id: I2b65f80b9d2d0d914ff34cdf18af2e652d04a8b1
Signed-off-by: Madhu Venugopal <vmadhu@cisco.com>

Merge "On container removal FRM does not uninstall flows"

Merge "Declare a property for commons-lang version in poms and use it."

Merge "Add helper class for RpcError interface"

FRMsync get stuck, miscelaneous fix

- The future FlowEntryDistributionOrderFutureTask was erroneously
thinking and entry was not expected simply because the equality check
was failing.
- Added more extensive logging to catch further bugs in the area.

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

Container Management and associated Northbound APIs.

Features :
1. Create, Modify and delete containers
2. ContainerFlow management
3. Northbound APIs
4. Application authorization support (contributed by Alessandro Boch)

Change-Id: I080d58c2869720c7e78fcdff673b51b142b1c2c2
Signed-off-by: Madhu Venugopal <vmadhu@cisco.com>

Fix NPEs on switch disconnect in cluster mode

- Fixed null pointer exceptions on switch disconnect that was
  connected to 2 or more controllers in "Any controller-One master" scheme
- Added node count to clustered controllers table in UI

Change-Id: I0c322a7a4105955a545b24a9ba1f36743800a802
Signed-off-by: Yevgeny Khodorkovsky <ykhodork@cisco.com>

Merge "Add page loading semaphore"

Merge "Bug #65 - Fix inconsistencies in the NB REST APIs"

On container removal FRM does not uninstall flows

ISSUE:
If a container had container flow specified, when it is removed FRM fails to uninstall
the flows that were installed in that container context

ROOT CAUSE:
On FRM stop(), the uninstallAllFlowEntries() function is called.
uninstallAllFlowEntries() does 2 things:
 1) it stores the original FlowEntry elements for later re-install irrespectively
it is called because FRM is being shutdown (container removal) or because controller
is moving to container mode (and this is the default container context) which is a bug.
 2) It uninstall the flows using the removeEntry(FlowEntry fe, ...) function which recomputes
the hw flows to be removed merging fe with the container flows for this container
returned by IContainer service, which is unefficient anyway.

Problem is that when FRM is being stopped because of container removal, IContainer service does no
longer have the container flows for the given container when queried by FRM.

FIX:
- Have uninstallAllFlowEntries() take a parameter to specify if the back up of the original flow entries is needed.
- Have uninstallAllFlowEntries() use the removeEntryInternal() to directly remove the FlowEntryInstall elements, because
  besides of the inefficinecy of recomputing the container flow merged entries there is really no need for it.
- Updated the java docs

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

Add page loading semaphore

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

Merge "Remove unused class member SAVE."

Merge "Remove public from interface methods. Cleans sonar warnings."

Declare a property for commons-lang version in poms and use it.

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

Remove unused class member SAVE.

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

Merge "Fix a ClassNotFoundException raised during demarshalling"

Add helper class for RpcError interface

Change-Id: I981d63d0c6ccc899a3f2db5985b16d31c8e2742e
Signed-off-by: Michal Rehak <mirehak@cisco.com>

Merge "Add curly braces to if/else statements. (sonar)"

Bug #65 - Fix inconsistencies in the NB REST APIs

1) PUT is used to create resources (when resource path is known to client and
the action is idempotent).
2) Use plural resource name when listing contents.
3) Rename service names (flow -> flowprogrammer, host -> hosttracker, subnet ->
subnetservice) as they conflict with resource names.

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

Remove public from interface methods. Cleans sonar warnings.

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

Add curly braces to if/else statements. (sonar)

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

Fix a ClassNotFoundException raised during demarshalling

Under certain conditions an error like:

Caused by: java.lang.ClassNotFoundException: org.opendaylight.controller.sal.reader.FlowOnNode
at org.eclipse.gemini.web.tomcat.internal.loading.BundleWebappClassLoader.loadClass(BundleWebappClassLoader.java:312) ~[na:na]
at java.lang.ClassLoader.loadClass(ClassLoader.java:356) ~[na:1.7.0_09]
at java.lang.Class.forName0(Native Method) ~[na:1.7.0_09]
at java.lang.Class.forName(Class.java:264) ~[na:1.7.0_09]
at org.jboss.marshalling.AbstractClassResolver.loadClass(AbstractClassResolver.java:135) ~[na:na]
at org.jboss.marshalling.AbstractClassResolver.resolveClass(AbstractClassResolver.java:116) ~[na:na]
at org.jboss.marshalling.river.RiverUnmarshaller.doReadClassDescriptor(RiverUnmarshaller.java:893) ~[na:na]
at org.jboss.marshalling.river.RiverUnmarshaller.doReadNewObject(RiverUnmarshaller.java:1205) ~[na:na]
at org.jboss.marshalling.river.RiverUnmarshaller.doReadObject(RiverUnmarshaller.java:272) ~[na:na]
at org.jboss.marshalling.river.RiverUnmarshaller.doReadObject(RiverUnmarshaller.java:209) ~[na:na]

this is due to the fact that by default the Jboss Marshaller used by
Infinispan uses the Thread Context Class Loader to retrieve the
classloader to be used in the demarshalling function. Equinox provides
a default one
(org.eclipse.core.runtime.internal.adaptor.ContextFinder) as TCCL,
however it heuristically try to find the classloader and in some cases
can fetch the wrong one. Now given that all the classes managed by
clustering services must go via OSGi it make sense to force the usage
of only the bundle class loader and we let OSGi framework to figure
out how to load the class.

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

Merge "Replace exception.printStacktrace with write to log."

Bug #60 : JSON list responses from the NB-APIs returns native object (instead of List) when the API returns a single element.

We use Jersey and JAXB frameworks for the NB-API implementation.
Jersey by default uses Jettison for the JSON marshalling/demarshalling. It seems like there is a bug in Jettison for this case.
And the recommended approach is to use Jackson instead.
Among various approaches to replace Jettison with Jackson (including package scanning, etc..), the only easy approach in an
OSGi environment is to directly refer to the JerseyJaxbJsonProvider class from the Northbound application.
JerseyJaxbJsonProvider has dependancies on jersey-xc which is also pulled in.

The Northbound Integration Test has inbuilt assumption that the NB-APIs use Jettison & hence a whole bunch of such bugs were
hidden and not visible to the tests as well. The correct approach is to use JAXB bindings in the IT so that immaterial of
Jettison or Jackson as the NB Marshalling infra, the IT need not change.
But, that is a bigger change and hence to restrict to the changes being addressed, this changeset just fixed these incorrect
assumptions and continued to use the Jettison library for demarshalling (while the actual Northbound API uses Jackson).

Also note that, such infra changes call for bumping both the NB-API version and the bundle version.
But, given the fact that the API freeze is scheduled later in October and more NB related changes are coming, the version bump
process can be scheduled later to ease the development work.

Change-Id: I47d8309dfbe11c251a30316bca37c7823cb0325a
Signed-off-by: Madhu Venugopal <vmadhu@cisco.com>

Merge "Remove redundant class members, turn others to local variables. (sonar)"

Merge "Remove java.lang redundant imports. Taken from sonar."

Merge "Added NOTICE file"

Updated models, modeling existing SAL and its functionality

model-flow-base
  - match-types.yang - Match types categorized by protocol / layer
  - flow-types.yang - Base flow types / actions
model-flow-service
  - flow-service.yang - Model of Flow Programming service
model-flow-statistics
  - flow-statistics.yang - Flow Statistics service
model-inventory
  - model-inventory.yang - Device inventory

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

Merge "Display Confirmation dialog before removing network configuration"

Merge "Fix for exceptions seen in FlowProgrammer NB, when get/put is done for invalid flow"

Merge "Add snapshots/releases setting to all maven repos."

Replace exception.printStacktrace with write to log.

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

Merge "- DiscoveryService yields to avoid hogging the cpu - SwitchManager getNodes() not to return null"

Fix cluster menu

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

Remove redundant class members, turn others to local variables. (sonar)

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

Remove java.lang redundant imports. Taken from sonar.

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

Add snapshots/releases setting to all maven repos.

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

Display Confirmation dialog before removing network configuration

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

Fix for exceptions seen in FlowProgrammer NB, when get/put is done for invalid flow

Change-Id: I6a247072bab9a21c82d7247a852c0d19766bcf13
Signed-off-by: Pramila Singh <pramisin@cisco.com>

- DiscoveryService yields to avoid hogging the cpu
- SwitchManager getNodes() not to return null

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

Add Cluster Menu

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

Added NOTICE file

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

Merge "Enhancement to switchmanager CLI commands to display all the properties of node and nodeconnector"

Merge "Bugfix: Exception when adding  static host"

Merge "Add sorting to js select.create()"

Enhancement to switchmanager CLI commands to display all the properties of node and nodeconnector

Change-Id: I87ee445168e64cc74a7538593503cb43f4e1f50a
Signed-off-by: Pramila Singh <pramisin@cisco.com>

Remove version hard coding in integration tests

- Version hardcoding on integration tests will cause problems when
releasing artifacts, hence removing it and relaying solely on the
version number present in the pom file.
- Enabled verify phase in integration tests else developer build will
happily succed even if there are integration test failures, which will
be only discovered during the controller-verify phase.

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

Bugfix: Exception when adding  static host

- Fix NumberFormatException when adding static host
  through REST API with no vlan configured.
- Add IT to verify this
- Clean up Host Tracker REST API

Change-Id: Iefc672a0a978640ca47903f20962f150298ba11c
Signed-off-by: Yevgeny Khodorkovsky <ykhodork@cisco.com>

Merge "SwitchManager NB - Added URL info in response on successful creation of Node property and NodeConnector Property"

Add sorting to js select.create()

- Add optional parameter to the select constructor to get the options array sorted

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

SwitchManager NB - Added URL info in response on successful creation of Node property and NodeConnector Property

Change-Id: Iae62b1208a6b6be0f00da71e402aa2f3b8ea9843
Signed-off-by: Diti Bhatia <dibhatia@cisco.com>

Correct modal ID

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

Merge "Checksum errors seen on some setup while downloading guava"

Merge "fixing spelling errors"

Merge "Cleaning up code, comments, and documentation for SimpleForwarding"

Updated SAL Flow Models

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

Checksum errors seen on some setup while downloading guava

- Due to some issues with the mirroring of {repo1|repo2}.maven.org of
some artifact, forcing all the repo1/repo2 artifacts to be downloaded
directly and not via ODP nexus proxy. This has been adviced also by
sonatype support folks, now they have a content delivery network in
place and can handle the load.

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

Merge "Updated toaster sample to match new Binding specification of YANG Tools."

Merge "Fix "unary operator expected" err in run.sh line47"

Updated toaster sample to match new Binding specification of YANG Tools.

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

Controller network interface enhancements:
- Provide a configuration parameter "of.address = " in config.ini which allows a specific IP address to be specified.
  If a valid IP address is specified, the controller will ONLY accept connections from switches with destination address
  matching the specified configured address. If an address it NOT specified (which is the default case) or is syntactically
  incorrect, the controller will use "any" (wildcard) address. The default case is what currently been implemented.
- If the IP address is specified, the controller will determine the networking interface (eg, eth0) on which the
  IP address is configured. If the interface is NOT up and is changed status between up and down, a log will be generated
  to alert the network operation to take corrective actions. If the address is not specified (or is specified with an invalid
  address syntax), the controller will use any interfaces and will NOT monitor the status of these interfaces.

Change-Id: Ie2a92ad99d35367a1d89dca390e7af54321bfe7c
Signed-off-by: Chi-Vien Ly <chivly@cisco.com>

Fix "unary operator expected" err in run.sh line47

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