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>

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>

Cleaning up code, comments, and documentation for SimpleForwarding

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

fixing spelling errors

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

Merge "toaster-it: add missing version for maven-paxexam-plugin"

Merge "yang-prototype: add missing version for maven-site-plugin"

Fix IP/MASK intersection in Match

- Match.getIntersection() picks the wrong IP/MASK
- Updated Junit to assert on expected behavior

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

Merge "Few additional fixes to enable Client script access to Northbound."

Few additional fixes to enable Client script access to Northbound.

After Tomcat 7, httpOnly is true by default. This causes the httponly flag set on the Cookie.
This prevents the Scripts to access the cookie. setting it to false by default.
Due to the recent fix on disabling authentication for OPTIONS, if a script uses OPTIONS for an
NB-API, the username is null causing null pointer exception. Hence adding null pointer safety
checks on all the northbound APIs.

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

Fixed .gitignore around bin/ directories

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

Fix to remove @ from id and type parameter of node and nodeconnector in Northbound javadocs

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

toaster-it: add missing version for maven-paxexam-plugin

This fixes the following warning:

[WARNING] Some problems were encountered while building the effective model for org.opendaylight.controller.samples:sample-toaster-it:jar:1.0-SNAPSHOT
[WARNING] 'build.plugins.plugin.version' for org.ops4j.pax.exam:maven-paxexam-plugin is missing. @ line 23, column 12

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

yang-prototype: add missing version for maven-site-plugin

This fixes the following warnings:

[WARNING] Some problems were encountered while building the effective model for org.opendaylight.controller:concepts-lang:jar:0.5-SNAPSHOT
[WARNING] 'build.plugins.plugin.version' for org.apache.maven.plugins:maven-site-plugin is missing. @ org.opendaylight.controller:yang-prototype:0.5-SNAPSHOT, /home/chrisw/git/opendaylight/controller/opendaylight/sal/yang-prototype/pom.xml, line 227, column 21
[WARNING]
[WARNING] Some problems were encountered while building the effective model for org.opendaylight.controller:yang-prototype:pom:0.5-SNAPSHOT
[WARNING] 'build.plugins.plugin.version' for org.apache.maven.plugins:maven-site-plugin is missing. @ line 227, column 21

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

Merge "When ports are added to a container, construct its topology immediately based on the existing global topology."

Merge "Gracefully stop HT threads when the bundle is being stopped (cache terminated exception)"

Merge "Fix undefined event, add check prior to delete"

Merge "FRM event sync developement"

Gracefully stop HT threads when the bundle is being stopped (cache terminated exception)

ISSUE:
When HT bundle is being stopped because container is being destroyed,
infinispan cache terminated exceptions are thrown because HT treads
during their last run are accessing terminated HT caches.
CHANGE:
- Have a stopping flag which gets set when dep mgr calls the HT stopping() method,
so that pending threads and timer tasks can be stopped before Cluster Mgr
terminates the HT caches.
- Fix other static analysis issues in HT code

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

Fix undefined event, add check prior to delete

Also changed remove button colors to btn-danger

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

When ports are added to a container, construct its topology immediately based on the existing global topology.

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

Fix for the Nodes Learnt dashlet error when there are no ports in the topology.
This fixes the issue of the dashlet not loading by introducing a null check.

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

FRM event sync developement

- Created necessary logic for distributing entry programming among the
FRM instances in the cluster.
- Made serializable Status so it can be distributed.
- Hooked FRM with connection manager, such that the FRM programming
can be distributed if the node is not local
- In case of non-local node someone picks it and process it, then the
status is returned back to caller.
- At the moment the whole logic to timeout entries that don't get an
answer is missing, will come along.
- HostTracker ages out the ARP entries in clustered environment
because when refreshed the timeout was not being refreshed
clusterwide, probably need a better way to deal with the timing out
-Added license in files missing it.
- Fixed integration test failing, now in fact we can no longer use
fake values else the connection manager will not report them as
attached to local controller and will stuck the test case

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

Northbound api cleanup for Subnets

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

Change 'Nodes Learnt' to 'Nodes Learned'

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

Merge "Added archetypes to releasepom"

HostTracker REST APIs compliance of new guidelines

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

Added archetypes to releasepom

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

Merge "Prepare for mvn release:prepare release:perform operation"

Prepare for mvn release:prepare release:perform operation

- Make sure every bundle include a proper scm section
- Provided a pom to be used for release operations in the root
directory, this is due to some implicit assumptions
maven-release-manager does when performing a release. The
maven-release-manager in fact calculate a tag name starting from the
developerConnection and removing from the URL layers based on how
deep the release pom is versus the assumed root. This is hardcoded in:
http://svn.apache.org/viewvc/maven/release/tags/maven-release-2.3.2/maven-release-manager/src/main/java/org/apache/maven/shared/release/util/ReleaseUtil.java?view=markup
Line 182 (this logic is not really specific to 2.3.2 version, but
pretty much in every version).
The tag calculation is fine if applied to SCM live SVN but not to GIT
hence this limitation of having to place the root pom in the root
itself.
- Removed uneeded distributions like sdk and parent, because are
combined in the main distribution.
- Corrected some wrong dependencies in clustering
- Added parent poms to distribution so to simplify the deploy of the
artifacts.
- Disable signing of openflowJ during release, it hangs
- Forcing maven-release-plugin to version 2.3.2, latest today (2.4.1)
has some issues.
- Updated README.OPENDAYLIGHT with process for releasing artifacts
- pom file in the root directory is merely a pointer to the main
distribution, this doesn't change any of the current workflow and
enable release support
- Create a profile triggered when the -DDOBUILDRELEASE is set which
exclude modules that are still depending on snapshots not under
controller project

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

Added distribution management to archetypes/ root pom

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

Merge "Added an archetype odl-model-project"

Added CorsFilter to enable secure cross site scripting

This is in addition to Ed's original Cors Filter changes. Default Cors Config
doesnt seem to work in certain scenarios. Added some custom configurations
and also added it per-bundle (started with Flow & i will add it to other bundles
once this is verified).

Also, by default AngularJS like frameworks uses HTTP OPTIONS method to check for
server options and that doesnt carry authentication headers. Hence in order for
the cors to work properly, we have to ignore authentication for OPTIONS method
alone. This is taken care in the web.xml configuration for all the northbound
bundle.

See: http://tomcat.apache.org/tomcat-7.0-doc/config/filter.html#CORS_Filter
And: http://en.wikipedia.org/wiki/Cross-origin_resource_sharing
This is done to allow a web page using javascript to be able to make calls
to our REST APIs even though it does not originate in our domain.

Added CorsFilter bundle in Third Party to bring in the class as a Fragment
on the org.apache.catalina bundle.

Added CorsFilter to the web/root web.xml file so it will be used for
all WebApps.

Fixed the northbound integration tests

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

Reverting because it caused instability to the northbound, wondering why verify didn't catch those.

Revert "Added CorsFilter to enable secure cross site scripting"

This reverts commit 8cbcc63bbb004b50c66ce3c65d0b8d7943c8ffac

Change-Id: I41e292b91dfff6c7ceefe33f92f63c081ca2e499

Added CorsFilter to enable secure cross site scripting

This is in addition to Ed's original Cors Filter changes. Default Cors Config
doesnt seem to work in certain scenarios. Added some custom configurations
and also added it per-bundle (started with Flow & i will add it to other bundles
once this is verified).

Also, by default AngularJS like frameworks uses HTTP OPTIONS method to check for
server options and that doesnt carry authentication headers. Hence in order for
the cors to work properly, we have to ignore authentication for OPTIONS method
alone. This is taken care in the web.xml configuration for all the northbound
bundle.

See: http://tomcat.apache.org/tomcat-7.0-doc/config/filter.html#CORS_Filter
And: http://en.wikipedia.org/wiki/Cross-origin_resource_sharing
This is done to allow a web page using javascript to be able to make calls
to our REST APIs even though it does not originate in our domain.

Added CorsFilter bundle in Third Party to bring in the class as a Fragment
on the org.apache.catalina bundle.

Added CorsFilter to the web/root web.xml file so it will be used for
all WebApps.

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

Added an archetype odl-model-project

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

Merge "Add Select all checkbox to all datagrids, remove out port and out vlan columns from troubleshooting, Change WEB to UI for UI generated requests in audit log Signed-off-by: Sapan Shah <sapshah@cisco.com>"

Merge "Misc fix for some error seen during IT and javadoc"

Merge "On ARPHandler restart Threads are left dangling, very visible in the IT tests"

Misc fix for some error seen during IT and javadoc

- Silent StaticRoutingImplementation when thread is interrupted, it can
happen.
- During IT logging bridge complain for a null exception because
unhandled.
- Fix some deprecation warning not really needed and some spellings in
javadocs

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

On ARPHandler restart Threads are left dangling, very visible in the IT tests

- Also added debug logs to trace how the ARP Handler is doing event syncs

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

Add Select all checkbox to all datagrids, remove out port and out vlan columns from troubleshooting, Change WEB to UI for UI generated requests in audit log
Signed-off-by: Sapan Shah <sapshah@cisco.com>
Change-Id: I5501698720177e51cfd91127fd9e867823575a44
Signed-off-by: Sapan Shah <sapshah@cisco.com>

Merge changes I5809ac06,I4362cd4e

* changes:
  Added Intellij files to gitignore
  Fixed typo and remove unecessary java.lang import

Added Intellij files to gitignore

Added IntelliJ IDE specific files extensions
to .gitignore.

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

Fixed typo and remove unecessary java.lang import

Fixed typo in comment and remove unecessary java.lang import
(as proposed by Sonar report)

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

Allow controller to be managed via JMX remotely

- In order to inspect the controller remotely JMX comes very handy
especially in clustered environment, this patch allow to start the
controller and to enable JMX remotely, this is still only the unsecure
version, for secure version a subsequent patch will be provided

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

Fixing a Northbound IT test breakage due to a recent NB change

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

Merge "Northbound cleanup for Flow programmer"

Merge "Northbound cleanup for static routing"

Northbound cleanup for Flow programmer

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

Merge "Cleaup Statistics northbound and add exmaples"

Northbound cleanup for static routing

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

Cleaup Statistics northbound and add exmaples

- Clean up stats northbound to align with common REST conventions,
  add more descriptive javadoc with examples

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

OpenFLow plugin & SAL API documentation

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

Merge "Fix to remove camel case for Northbound API"

Fix to remove camel case for Northbound API

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

Fix to show Node name if present when adding ports to Subnet Gateway on the devices tab.
A minor change from Mac to MAC in nodes learnt popout.

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

Fix for NorthboundIT test failure

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

Merge "HostTracker immediately ages out hosts on coordinator"

Merge "Integration of MDSAL into distribution"

Merge "Northbound cleanup for Switch Manager"

Integration of MDSAL into distribution

- Fixed Class Loader bug in runtime code generator

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

HostTracker immediately ages out hosts on coordinator

- Let say an Host 1.1.4.1 is learnt by CTRL2 and CTRL1 is doing
cleanup of the host tracker database, the CTRL1 was cleaning
immediately the hosts learnt on other controllers because the host
count was not serialized appropriately

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

Merge "Fixing a bug to show node name if present instead of node id while adding SPAN ports on the devices page."

Northbound cleanup for Switch Manager

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

Fixing a bug to show node name if present instead of node id while adding
SPAN ports on the devices page.

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

Fixing a minor bug in Cache replace code in Connection Manager.

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

Merge "Fix Connection manager to retrieve inventory when is up"

Merge "Fix Match intersection logic"

Fix Match intersection logic

    - Change in Match.java: Empty match represents all packets (universal set) while empty set is the null match.
                        So, in case of no intersection, return a null Match.
                            intersect() and getIntersection() optimized in case of universal and empty sets
    - Changes in MatchTest.java to junit the above changes
    - Change in FlowEntry.java: Match.mergewithFilter() is now returning a new Match instance

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

Northbound cleanup for topology manager

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

Merge "Fixed run.sh to autodetect JAVA_HOME on Mac OS X"

Merge "Add intersection methods to Match"

Merge "Delete Multiple flows in a single click"

Merge "Change nodeconnector string shown on the troubleshoot tab when clicked on ports to show node name if it is available and node id otherwise."

Delete Multiple flows in a single click

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

Enabled classes not in org.opendaylight.controller package to output log.

Change-Id: I85819272500c1f4ed3a5eb3f6580b6aa6aac75d9
Signed-off-by: Hideyuki <h-tai@cd.jp.nec.com>

Change nodeconnector string shown on the troubleshoot tab when clicked on ports
to show node name if it is available and node id otherwise.

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

Fixed run.sh to autodetect JAVA_HOME on Mac OS X

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

Add intersection methods to Match

- Add method to test and get intersection of two Match objects and correspondent Junit
- Fixed bug in NetUtils.inetAddressConflict()
- Made Match.toString() less verbose

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

Fix Connection manager to retrieve inventory when is up

When ConnectionManager comes up, it's very much possible that may have
missed some inventory updates, so it needs to retrieve them to make
sure holes are covered, this is seen during integration test with the
stub protocol plugin, hence also make sure stub protocol plugin
provides the necessary hooks.

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

Added startImpl to provide BundleContext to providers

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

Fixed reference to missing file.

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

Merge "Added distributionManagement section to md-sal parent pom."

Merge "1. Controller switchEvents queue should be priority based. The queue holds switch up, down, error and message events. Assign higher priority to switch up/down events. 2. When switch is going to disconnect, SwitchHandler can clean up some internal states to speed up the process in order to avoid unnecessary messages backing up in the queue. 3. In SwitchManager, don't send additional port down notifications after node down event is received. 4. Fixed an issue where links are not recovered when the switch is reconnected to the controller. 5. Fixed TLS exception@java.security.ProviderException."

Added distributionManagement section to md-sal parent pom.

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

Merge "Minor correction to UI element"

Merge "Enhancements to common.northbound"

Minor correction to UI element

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

Merge "Frm to install/uninstall flow when port is added/removed from slice"

Fix for JS error on Flows tab.
This fixes the JS error "TypeError: $span.data(...) is null" on the flows tab by introducing a null check.

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