Make FRM cleanup only local entries

- FRM when shutting down should only clean local entries and avoid to
trigger remote deletion because that cleanup will also be initiated on
the remote controller
- On shutdown make sure to wake up who may still be sleeping

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

Merge "To SAL conversion test."

Porting Tristate logic to SAL Connection Service and make use of it in Inventory Service Shim

ConnectionManager's locality status was moved from a binary state to tristate to address timing issues.
With this push, it is ported to the SAL Connection Service so that the protocol-plugins can make use of it as well.

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

Fix wrong dependency

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

Merge "Silent false positive, and enhance real error message"

Merge "Stop overriding method just to call super."

Merge "Stop creating new Boolean, move to use valueOf method"

Merge "Add SCM section to all the poms"

To SAL conversion test.

The test was added

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

Declare version property for maven bundle plugin, reuse it in poms and
use version 2.3.7 instead of 2.3.6 and 2.4.0 used in parallel.

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

Stop overriding method just to call super.

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

Add SCM section to all the poms

- For the release process we need SCM section to all the poms, some
are still missing it so adding them.

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

Stop creating new Boolean, move to use valueOf method

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

Merge "Strong password check to consider underscore as a special character"

Silent false positive, and enhance real error message

- With gerrit https://git.opendaylight.org/gerrit/#/c/1366/ i have
introduced a log error, to spot inconsistent hashcode but indeed it
turn out is getting exercised also during a normal code path,
basically when:
 <command orginator> --> <key owner> --> <all the other members>

all the other members when getting the replication from the non-key
owner will end up in this case. So that cannot be actually considered
an error.

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

Fix: Flows are deleted when another controller is started

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

Strong password check to consider underscore as a special character

ISSUE:
Current strong password check regular expression does not consider '_' as a special character

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

Merge "Add to jboss repository snapshot and release properties"

Merge "Remove unused class member SAVE"

Add to jboss repository snapshot and release properties

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

Timeout ForwardingRulesManager workOrder

- Before this patch is an FRM request was submitted and no answer was
received by a remote controller, that would indefinitively stuck the
caller, clearly a bad practice. This patch is introducing a default
timeout of 30 second (very bad scenario) and a log.error is being
raised if that happens.
- Cleaned workMonitor cache on workStatus coming back or timeout.
- Make timeout tunable via system properties

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

Merge "Action mapping To|From SAL conversion."

Merge "Added sal-compability to build"

Action mapping To|From SAL conversion.

- action was defined in flow-type.yang file according to actions in SAL
- action mapping was added to FromSalConversionsUtils.actionFrom
- action mapping was added to ToSalConversionsUtils.actionFrom

Change-Id: I3bd1c2eba896fb3a305da6ad698db9f0275d6f50
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Added sal-compability to build

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

Add project.reporting.outputEncoding property to prevent maven warning

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

Merge "Enhance debug capabilities"

Remove unused class member SAVE

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

Moved MD SAL from sal/yang-prototype to md-sal

Moved MD SAL sources, artefacts from
 opendaylight/sal/yang-prototype to opendaylight/md-sal

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

Enhance debug capabilities

- Make sure infinispan complains very loud when it think an hashcode
is not consistent in the cluster.
- For now this patch is controller specific, but i will propose to
upstream in infinispan as well.
- Given BaseDistributionInterceptor.java came from version 5.3.0.Final
enforce it in the build

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

Fix occasional NPEs in Connection manager

- This fixes an issue where ConnectionManager bundle start/stop
  callbacks don't have the required dependenicies available, which
  causes occasional NPEs.

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

Merge "UI: Fix broken html literals"

Merge "Minor log msg changes"

Added null check in FRM error handling and also formatting of OF ErrorMessage in the log statement

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

Minor log msg changes

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

UI: Fix broken html literals

- Fix html literals that don't have correct apostrophes,
  resulting in breakage when user enters strings with spaces

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

Merge "Remove unused  spring dependency and wrong version in web pom"

Remove unused  spring dependency and wrong version in web pom

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

Initial push of Neutron interface

Two bundles included in this:
-networkconfig.neutron (contains JAXB annotated classes, *CRUD
interfaces with an implementation [for storing objects], and
Aware interfaces [for other bundles to register services])
-networkconfig.neutron.northbound (contains the JAXRS annotated
classes along with JAXB annotated request specific classes)

Commit amended to refactor .internal package to separate
implementation bundle and package

Commit re-amended to clean up issues in commons/opendaylight
and distribution/opendaylight poms

Commit re-re-amended to fix conflicts on rebase (I hope)

Getting close to just abandoning the change and starting over

Commit re-re-re-amended to fix parent issues in pom

amend to remove leaked CRs in distribution pom

- Cleanup of some duplicates artifact and of commons-net in modules
dependencies, anyway they get in via the commons.opendaylight

Change-Id: I15724c55e843237c010dfaec1e1e523560174f51
Signed-off-by: Ryan Moats <rmoats@us.ibm.com>

Move mockito to dependencyManagement section in parent pom.

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

Merge "Declare osgi.core.version in one pom and reuse it."

Merge "Move init and destroy empty impl from Activator classes. Have only one empty implementation in parent class instead of having it in more than 20 classes. Only one class actually has a non empty implementation and it overrides the empty implementation from the parent."

Declare osgi.core.version in one pom and reuse it.

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

Move init and destroy empty impl from Activator classes. Have only one
empty implementation in parent class instead of having it in more than
20 classes. Only one class actually has a non empty implementation and
it overrides the empty implementation from the parent.

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

Merge "Removing the Container Aware dependency from Clustering Services."

Merge "FRM to remove flows on switchport admin down"

Merge "Conversion class FromSalConversionsUtils."

Removing the Container Aware dependency from Clustering Services.

Clustering Services implementation is ContainerAware to handle the case of Container deletion.
More appropriate way to handle it is to have an explicit API in global cluster manager to expose
a functionality to clean all the caches assocaited with a Container. This API can be used by
the Container Manager to clean up the caches.

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

Fix NPE in visual topology

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

FRM to remove flows on switchport admin down

- When a swithport is shutdown FRM should uninstall the flows which have that port as input/output
- Similarly when a port is moved to admin up state, FRM should install the interested flows from the config

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

Change the member isValid in class MatchField non-transient so that it can be synced properly.

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

FRM uses "Start Level Thread"  to handle container flow changes during bootup

Modify the current design: when FRM receives the notification, it puts it into its own event queue and returns immediately. The event is eventually handled (in the same way) but through FRM's own thread.

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

Couple of changes

1. StaticRoutingImplementation now updates cache after updating nodeconnector for   \
the static route

2. The clone method of MatchField now clones the valid flag and the type

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

Cherry-picked and merged modified JavaDocs for northbound apis to fix JSON/XML representation

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

Merge "Few debug enhancements to debug hashing issues"

Merge "Fix: IPProtocols not parsing correctly"

Merge "Fix multi-host caching in visual topology"

Reverting the FRM WorkOrder caches to NON_TRANSACTIONAL

After Giovanni's recent hashCode fix, most of the cache issue issues are resolved and it also renders itself to making
FRM WorkOrder back NON_TRANSACTIONAL. Also reverted the need to have a seperate thread to handle this processing.
Now it is back to normal and how it is supposed to be.

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

Conversion class FromSalConversionsUtils.

SAL -> YANG conversion

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

Fix: IPProtocols not parsing correctly

- Add IP protocol field validation where appropriate
- Enabled all IP protocols in enum
- Added unit tests

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

Merge "In Firefox DropDown sub menus will have the first option selected"

In Firefox DropDown sub menus will have the first option selected

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

Fix for Refresh Port Details and Flow Details in Troubleshoot makes search hard
Replace the auto refresh with a refresh button.

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

Fix multi-host caching in visual topology

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

Controller User Audit log to track activities

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

Few debug enhancements to debug hashing issues

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

Merge "Provide consistent hash code for enums and use it too"

Merge "Validate user input in FlowSepc Configuration"

Merge "Sal Models for meter table. Removed tab space from group models and formatted the same."

Merge "Aligning JSON marshalled output for Properties similar to the XML counterpart."

Merge "Added validation for protocol field while adding flow"

Provide consistent hash code for enums and use it too

- Java enums if used as key, works well in a single JVM, but across
JVM the hashcode generated by them is not guaranteed to be the same
and that breaks the calculation of a cluster wide consistent hash
code, needed by the clustering services. The fix is to provide a
consistent hashcode for the Enum and to use it in the dependent data
structures.

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

Merge "UI: Allow admin to reset password for other users"

Merge "Simple changes to eliminate pmd warnings"

Merge "Add strong password check for users"

Merge "Prepare the TopologyManager code for integration testing. Need to have the code in a structure where topologymanager is the top level directory and has sub directories - api - implementation - integrationtest"

Validate user input in FlowSepc Configuration

Existing flow spec validation was missing range check for IP Protocol and Protocol
source and destination ports. Also, port number were not being checked for invalid
input

Change-Id: I11a5c6b90c4d5193f4e1950df684363e22b0e33a
Signed-off-by: Maurice Qureshi <maquresh@cisco.com>

Aligning JSON marshalled output for Properties similar to the XML counterpart.

After the Jackson migration (https://git.opendaylight.org/gerrit/#/c/1131), the marshalled JSON output
got misaligned from its XML counterpart. This is because of the "special" handling of XmlElementRef by
Jettison.
Please refer to http://pastebin.com/6kCSuddg for various marshalled JSON output for Jettison, Jackson and Moxy.

Having tried & exhausted various mechanisms (using Jackson and Moxy), finally decided to use the Jackson
annotations to take care of this requirement. Changes involves :

1. Modified NodeConnectProperties, EdgProperties and NodeProperties to use @JsonIgnore and @JsonProperty
2. Reverted a few NorthboundIT changes as prior to the gerrit#1131 version to align to the JSON format.
3. Added @XmlAccessorType(XmlAccessType.NONE) in all the Property classes to avoid including unnecessary data.

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

Sal Models for meter table.
Removed tab space from group models and formatted the same.

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

Added validation for protocol field while adding flow

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

UI: Allow admin to reset password for other users

- Allow admin to reset password for other users
  without providing current password.
- enforce re-login if changed own password

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

Switched odl-model-project archetype to use yangtools 0.5.8-SNAPSHOT

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

Merge "Remove maven repo2 from poms because no need for 2 maven repos, especially since only repo1 is the official supported one."

Merge "Fix "FileNotFound" error for jacoco.exec and the path to the integration tests."

Merge "Conversion class ToSalConversionsUtils."

Merge "Configure jackson to turn off FAIL_ON_UNKNOWN_PROPERTIES. This ensures that Jackson is more lenient when it sees extra attributes in the request payload. Add custom exception mapper for handling jackson json exceptions during deserialization"

Merge "Adding Tri-states to the connection manager locality check."

Merge "Added Router code generation and unit tests for runtime code generator"

Merge "MD-SAL model for group feature in OF 1.3."

Conversion class ToSalConversionsUtils.

YANG -> SAL conversion

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

Adding Tri-states to the connection manager locality check.

As of now, the connection manager's isLocal() provides a binary true/false for a locality.
But is missing a key state of not-connected. This is now included as part of a new API (getLocalityStatus).
Retaining the isLocal as a deprecated method to be removed later once all the users have migrated.

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

Merge "In o.o.c.switchmanager.SpanConfig, handle node connector string format correctly."

Merge "Add missing validation on ContainerFlowConfig objects"

Fix "FileNotFound" error for jacoco.exec and the path to the integration
tests.

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

Declare jacoco.version in parent pom and reuse it in other pom files.

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

Remove maven repo2 from poms because no need for 2 maven repos,
especially since only repo1 is the official supported one.

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

Add strong password check for users

- User password must be at least 8 characters long, contain upper and lower
  case letters, at least one number and one non-alphanumeric character. Max lenght 256.
- Disabled by default, can be enabled through config.ini

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

Added Router code generation and unit tests for runtime code generator

Added implementation of Router generation
  - SAL implementation of service responsible for routing requests
    to southbound components based on ContextReference field,
    which is conceptually same as NodeType in HARD SAL, but more
    flexible and does allow selecting implementation on different
    fields than only NodeType.

Moved runtime code generation code to separate packages .codegen

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

Configure jackson to turn off FAIL_ON_UNKNOWN_PROPERTIES. This ensures that Jackson is
more lenient when it sees extra attributes in the request payload. Add custom exception
mapper for handling jackson json exceptions during deserialization

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

MD-SAL model for group feature in OF 1.3.

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