Five more Equals/HashCode/StringBuilder replacements

I'm intentionally breaking these up into small groups so they can
be reasonably reviewed, rather than one giant group that can't.
Contributes to fixing bug 20.

Second in the set.

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

Five more Equals/HashCode/StringBuilder replacements

I'm intentionally breaking these up into small groups so they can
be reasonably reviewed, rather than one giant group that can't.
Contributes to fixing bug 20.

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

Replace EqualsBuilder and HashCodeBuilder in transient member case.

This partially addresses bug 20, and provides the example of the
case where we have transient member.  I'm breaking it out
so that the transient member case can be reviewed simply.

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

Merge "Fixed generation of Transfer Objects;"

Merge "Removed unused private variable containerAwareRegistration and unused import "

Merge "Removed unused private variable containerAwareRegistration - please review this change"

Fixed generation of Transfer Objects;

Fiexed bad generating of parametrized constructors when constructor had
more than two parameters;

Fixed generation of hashCode, equals and toString when
GeneratedTransferObject had not specified equals, hashCode and toString
properties;

Added capability to generate setters if property has flag isReadOnly set
to false.
Parameters which contains flag isReadOnly == false will not apear as
parameters in generated constructor;

Fixed generation of subdirs from package when used generateToFile with
File as parent directory;

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

Removed unused private variable containerAwareRegistration and unused
import

Change-Id: I6b3bd2a85d66f73408a90a3adfe114216df94743
Signed-off-by: Natarajan P<natp@cisco.com>

Replace EqualsBuilder, HashBuilder, ReflectionToString

In one file, as a test case, replace the EqualsBuilder,
HashiCodeBuilder, and ReflectionToString
with code generated by Eclipse. This is a
very partial fix for bug 20 (more work to be done).

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

Removed unused private variable containerAwareRegistration - please
review this change

Change-Id: I4c6e16dcc3fe08ff44f2ad225238391165b1fcd9
Signed-off-by: Natarajan P<natp@cisco.com>

Merge "Session invalidation is done after the custom error handling"

Session invalidation is done after the custom error handling

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

Remove 'model' from package names in yang-model-parser-impl.

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

Fixed relative/absolute yang files directory resolving.

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

Merge "Refactored yang-maven-plugin."

Merge "new LinkedList() mostly redundant with Arrays.asList()"

Merge "Logging improvements to improve efficiency - specifically in critical path. Signed-off-by: Madhavan Kasthurirangan <mkasthur@cisco.com>"

Logging improvements to improve efficiency - specifically in critical path.
Signed-off-by: Madhavan Kasthurirangan <mkasthur@cisco.com>

Fix single node in visual topology

When staging a node from the cache, forgot to update the adjacencies correctly.

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

Merge "- Discard discovery packet if the ingress port is down - SwitchManager osgi cmds are useful tools for debugging. But the input node id currently has to be in long format. Change it to take node id string. Example: pncs "OF|00:01:44:03:a7:7a:54:e0" Signed-off-by: Jason Ye <yisye@cisco.com>"

Refactored yang-maven-plugin.

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

- Discard discovery packet if the ingress port is down
- SwitchManager osgi cmds are useful tools for debugging. But the input node id currently has to be in long format. Change it to take node id string. Example: pncs "OF|00:01:44:03:a7:7a:54:e0"
Signed-off-by: Jason Ye <yisye@cisco.com>

new LinkedList() mostly redundant with Arrays.asList()

General code clean up.

You get a ArrayList by default and that's the right thing in most
places.  This commit removes a bunch of missing Java generic warnings
and shouldn't affect any code.

Change-Id: I777cdf6a3fd1f2c883aac188d544ebc5a4420f55
Signed-off-by: Rob Sherwood <Rob.Sherwood@bigswitch.com>

Removed unused imports

Change-Id: Ief3f6879e3e4ed9cb743a699a8614fff4d5d9a59
Signed-off-by: Natarajan <natp@cisco.com>

Correct NodeConnetor insertion for edgeMap.
Signed-off-by: Madhavan Kasthurirangan <mkasthur@cisco.com>

Authorization fixes for Northbound bundles

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

Fix user changing their password

Wrong user was being sent to change the password (it was the session user).

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

Adding plugin stub:
FlowProgrammerService

Change-Id: Ia5a09a1dfc1547b4016296473e7751f0a23ccf50
Signed-off-by: Kalvin Hom <kahom@cisco.com>

Merge "- Added osgi cmds to enable/disable LLDP snooping on each individual node connector - Fixed TopologyManager nodeConnectorsDB which should only contain ISL ports"

Merge "Fixed augment schema path resolving."

Merge "Fix NPE and enhance flows table"

- Added osgi cmds to enable/disable LLDP snooping on each individual node connector
- Fixed TopologyManager nodeConnectorsDB which should only contain ISL ports

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

Sanity check missing in Description Stats Converter constructor

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

Fix NPE and enhance flows table

NPE in devices web caused loop in JS
Added ingress port to flows table

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

Merge "Prevent stub and integrationtest plugins to go into the distribution of opendaylight"

Fixed augment schema path resolving.

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

Added support for implements and extends of GeneratedType;

Added functionality to store List of implementing Generation Types into
GeneratedType and GeneratedTypeBuilder;
Added functionality to store Extending Transfer Object for
GeneratedTransferObject and GeneratedTOBuilder;

Fixed:
DataNodeIterator no longer returns augmented containers, lists, leafs
and leaflists - only non augmented module schema tree;
Added findDataSchemaNode for given SchemaPath into SchemaContextUtil;

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

Merge "Primitive support to save the topology coordinates and cleanup of some unused tiering code"

Merge "Setting the auto refresh rate for flows and ports based on the number of flows and ports."

Primitive support to save the topology coordinates and cleanup of some unused tiering code

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

Merge "protocol plugin stub node type changed to STUB Modified integration test to use STUB type nodes."

User manager to properly handle authenticated users with no authorization

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

Setting the auto refresh rate for flows and ports based on the number of flows and ports.

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

Prevent stub and integrationtest plugins to go into the distribution
of opendaylight

Fixed a spelling error in the switchmanager api bundle and provided a
logback.xml needed for integration test of switchmanager.

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

Add generic container logic to web bundles

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

Merge "Add LLDP retry count to config.ini"

Add LLDP retry count to config.ini

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

protocol plugin stub node type
changed to STUB
Modified integration test to use STUB type nodes.

Change-Id: Ic6c23457400f4bbf0b6898c5238a7580ffd649ee
Signed-off-by: Kalvin Hom <kahom@cisco.com>

Fix in ForwardingRulesManager async path

- Fix in modifyOrAddFlowEntryAsync() path
- Protecting concurrent modification to nodeFlows entries through synchronized methods for now

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

Change to update node properties

- inventoryServiceShim to trigger node description update
- Inventory Service to accept node properties updates for known nodes only
- Inventory Service to publish only effective node properties update and on default container only
- Minor changes to the Property classes toString() methods
- Removed toString() overridden method in UpdateType as default toString() which prints the enum name is self explicative and serves better in logs
Signed-off-by: Alessandro Boch <aboch@cisco.com>

Fixed problem with incorrect subnet validation with container flow
Also added unit test cases to verify the validation

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

Authorization failures return 403 with cookie populated. This forces the user to clear the cache
before reauthenticating. Added a custom 403 page with session invalidated.

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

Logging enhancement related to Packet
Signed-off-by: Madhavan Kasthurirangan <mkasthur@cisco.com>

Merge "API 2 Documentation for opendaylight.controller.usermanager"

Added line number to error messages.

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

Merge "Updated POM files and versions to SNAPSHOT"

API 2 Documentation for opendaylight.controller.usermanager

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

Merge "Add password edit and verify"

Add password edit and verify

Add feature to edit and verify password.

Minor fix in devices where NPE caused a JS infinite loop.

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

As part of performance enhancements, Topology updates are now batched as much as possible. This will effectively reduce the number of Routing updates to the listeners.
Signed-off-by: Jason Ye <yisye@cisco.com>

Changed sal version to 0.5.0-SNAPSHOT
Moved switchmanager.integrationteststub into protoco_plugins.stub
Removed non-source code files.
Added integration test for switchmanager bundle

Change-Id: I2dd2cbffe9e30224dcf2e03b3f11da465baec09f
Signed-off-by: taochang <taochang@cisco.com>

Updated POM files and versions to SNAPSHOT

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

Merge "Adding integration tests for clustering service. Added to pom file for sonar integration."

Merge "Change HostTracker.IntegrationTest to use clustering.stub instead of services-implementation."

Merge "Bug fix for ForwardingRulesManager: addFlow and addFlowAsync are reversed."

Bug fix for ForwardingRulesManager:
addFlow and addFlowAsync are
reversed.

https://bugs.opendaylight.org/show_bug.cgi?id=16

Change-Id: If119f64e637401ea022b092af44192bdef3db986
Signed-off-by: Kalvin Hom <kahom@cisco.com>

Adding integration tests for clustering service.
Added to pom file for sonar integration.

Change-Id: I69555941a1d8aa7487090793aed946c1711dfe67
Signed-off-by: Patrick Chu <patrichu@cisco.com>

Fix Device Port Status

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

Change HostTracker.IntegrationTest
to use clustering.stub instead
of services-implementation.

Fixed typos for jacoco.exec

Change-Id: Ie699afad7c7c109b517db592c99a20606c8dc867
Signed-off-by: Kalvin Hom <kahom@cisco.com>

edgeUpdate API enhancement and related changes.
Signed-off-by: Madhavan Kasthurirangan <mkasthur@cisco.com>

Merge "Added generate To File for specified directory."

Merge "Implemented module dependency sort, that returns modules in order in which they should be processed. If module A imports module B, sort returns ordered list [B,A]."

Added generate To File for specified directory.

Added overloading method generateToFile with File as input parameter and
List of Files as result;
Removed unused import in BaseTypeProvider test and BaseYangTypes;

Changed binding-generator, yang and sal maven pom files to use java 1.7;

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

Implemented module dependency sort, that returns modules in order in which they should be processed.
If module A imports module B, sort returns ordered list [B,A].

Implemented utility topological sort that is used by module dependency sort.
Implemented tests for both topological sort and module sort.

Removed circular dependencies from test yang files.

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

Fixed parsing of unknown nodes for identity statemens.

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

Merge "Added reserved port descriptions to comments matching them to the OF Spec in NodeConnector.java and fixed a couple of comment typos"

Merge "pom changes for FRM integration tests; added integrationtest bundle with a small simple test"

Added reserved port descriptions to comments matching them to the OF Spec in NodeConnector.java and fixed a couple of comment typos

Signed-off-by: Brent Salisbury <brent.salisbury@gmail.com>

pom changes for FRM integration
tests;
added integrationtest bundle with
a small simple test

Change-Id: I1c0591286cb8cea4dcb1ff72f02d2d3a25ba922d
Signed-off-by: Kalvin Hom <kahom@cisco.com>

- Add flow and port stats polling intervals in config.ini
- Add DESC stats polling interval in CONFIG.ini and thru OSGI cmd
- When the switch port is changed to admin down or the link is changed to down state, the switch sends port_status msg with OFPPR_MODIFY type. The existing logic is to map OFPPR_MODIFY to UpdateType.CHANGED. But we should exam the Config state and Link state fields within the port_status packet. If they are "down", we should fire an UpdateType.REMOVED event instead and remove both edges associated with the port.

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

Merge "Description: - Made changes in devices.web bundle and sal.core.Config - files changed for this: pages.js, Devices.java and Config.java - changes seen in the diff in StatusJsonBean.java are purely sun xml style coding    \ related - Port status is correctly shown as "Up" or "Down".  In case the port status cannot  \ be determined, it will be displayed as "Undefined""

Merge "Hide production nodes in visual topology"

Merge "Javadoc for controller.web Change-Id: Ie687bbb842391bf94f7c23a3c58833cf53a2e438"

Description:
- Made changes in devices.web bundle and sal.core.Config
- files changed for this: pages.js, Devices.java and Config.java
- changes seen in the diff in StatusJsonBean.java are purely sun xml style coding    \
related
- Port status is correctly shown as "Up" or "Down".  In case the port status cannot  \
be determined, it will be displayed as "Undefined"

Change-Id: I15f4281fb0a00c466942d7c4effab6fe4d25a7b9
Signed-off-by: dibhatia <dibhatia@cisco.com>

Merge "Update comments for JavaDoc Added comments for the followings interfaces: IForwardingStaticRouting, IRouting. Removed the method saveConfig() from IForwardingStaticRouting since it is not a public method and is only use privately."

Hide production nodes in visual topology

At least for now, we don't want to display production nodes within the visual topology.

However, if there are production nodes, then the user can still view them using the
northbound interface.

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

UI Flow configuration fix

ISSUE: Flows configured through UI do not honor selected actions

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

Update comments for JavaDoc
Added comments for the followings interfaces: IForwardingStaticRouting, IRouting.
Removed the method saveConfig() from IForwardingStaticRouting since it is not a public method and is only use privately.

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

Javadoc for controller.web
Change-Id: Ie687bbb842391bf94f7c23a3c58833cf53a2e438

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

changing directory structure to fix errors

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

Expanded UnknownSchemaNode implementation, refactored java source code generation.
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Fixed NPE in maven-yang-plugin

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

Added Maven plugin for generation SAL APIs from YANG

Added sample project.

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

Added generation of Transfer Objects from Type Definitions.

Extended functionality of Type provider to resolve Type Definitions in schema context as GeneratedTransfeObjects with proper package names and
class names;
Updated generateTypes in BindingGeneratorImpl to generate also Transfer Objects from Type Definitions;
Added BindingGeneratorUtil to replace functionality of CodeGeneratorHelper;
Modified EnumerationBuilderImpl - changed visibility from package protected to public due to need for generation of stand alone enums;
Refactored GeneratorUtil with use of BindingGeneratorUtil methods;

Fixed bug in where SchemaPath was incorectly generated for yang types in yang-model-util project;

Changed YangModelParser API - removed parseYangModel method and changed parseYangModels param from vararg String to List of Files and from vararg InputStream to List of Input Streams;

Modified YangToSourcesMojoj and GenerateSourcesTest in maven-yang-plugin due to change in YangModelParser API;
Modified YangToSourcesPluginTest due to change in YangModelParser API;
Modified Demo in binding-model-demo due to change in YangModelParser API;
Modified GeneratedTypesTest in binding-generator-impl -> changed load of yang files due to change in YangModelParser API;

Added comment generation for interface and class definition in binding-java-api-generator -> GeneratorUtil;

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

Resubmitted with source code synchronized.  Added integration test for hosttracker, split switchmanager
into api and implementation bundles.

Change-Id: I04d5d89562316cafa89a5ee46e08eb724733711a
Signed-off-by: taochang <taochang@cisco.com>

- Application is no longer blocked when programming hundreds of flows. The Barrier message is now sent asynchronously.
- Waiting for Barrier reply is still blocked in low level plugin, but it's per switch basis.
Signed-off-by: Jason Ye <yisye@cisco.com>

Fixed SchemaPath resolution for base YANG types.
Refactored refine statement parsing.

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

Merge "Convert various result into Status Signed-off-by: Jason Ye <yisye@cisco.com>"

Convert various result into Status
Signed-off-by: Jason Ye <yisye@cisco.com>

Merge "Gen Type and Generated TO builders moved to Util project."