Statistics Northbound Integration Test

Added northbound tests for node specific
requests

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

NodeFactory service

Added INodeFactory interface in SAL to
handle creation of different Node types;

Node will use INodeFactory to create Nodes
that are not one of the preset types;

StubNodeFactory for plugin stub creates
Nodes of STUB type;

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

Extend getJsonResult() to accommodate HTTP methods other than "GET".
Also pass the HTTP response code to calling program.

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

Merge "Fixed dependency management."

Merge "Fixed yang grouping resolution. Fixed refine parsing. Updated tests."

Merge "Fixed whitespace for Gerrit 393"

Merge "Fix for bug 24."

Fixed dependency management.

Fixed dependency management for Binding Generator and Yang projects.

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

Fixed yang grouping resolution.
Fixed refine parsing. Updated tests.

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

Fixed whitespace for Gerrit 393

This commit is primarily about experimenting
with commits against not yet merged Gerrits.
I took as an opportunity doing this patch to fix
some tabs and trailing whitespaces for
https://git.opendaylight.org/gerrit/#/c/393/

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

Fix for bug 24.

Description: Ping between host hosts takes more than 1second
Solution: Make modification in ArpHandler to handle the followings
- When a ARP Request is received from an host (requestor) to an unknown target host, an requestor entry is added. This will be used by ArpHandler to relay the ARP Reply back to this requestor once the target host replies.
- These target hosts  are saved and timed for up to 2sec. If the target host does not reply within up to 2sec, the host will be removed. This housekeeping process is implemented in a periodic timer task.

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

Refactored yang-model-parser-impl to improve readability. SchemaContextImpl moved to separate class. Fixed parsing of refine statement. Fixed bug in YangParser.g4 in min-elements statement.
Updated tests. Improved code generation: added import statemets to generated code.

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

Merge "Statistics Northbound Test"

Merge "ICMP fix and Packet class should store and provide access to the raw payload in case no packet class is available to parse it."

Merge "CodeGenerator update."

Merge "Refactored yang-maven-plugin. Updated tests. Removed backup files."

ICMP fix and Packet class should store and provide access to the raw payload in case no packet class is available to parse it.

- Packet to store a copy of the raw payload if no packet class is present to parse it
- Added missing code for ICMP
- Refactored PAcket, ICMP and IPv4 code
- Fixed corrupted flag setting when comparing checksum after deserializing
- Added Junit for IPv4.computeChecksum() and for serilaize/deserialize ICMP message
- Addressed some eclipse and static analysis warnings

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

Statistics Northbound Test

Added to stub plugin's ReadService
to return a flow for each action type

Adding additional tests for checking
that actions are being passed up
properly to the northbound

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

CodeGenerator update.

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

Refactored yang-maven-plugin. Updated tests.
Removed backup files.

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

Equals/HashCode/StringBuilder removal from Supported/PeerBandwidth

Supported/PeerBandwidth does not add any new non-static members, so it should be
safe to simply fall back to the parent classes equals/hashcode.

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

Equals/HashCode/StringBuilder removal

AdvertisedBandwidth is a subclass of Bandwidth.  Bandwidth has equals/hashcode.
AsvertisedBandwidth does not add any new non-static members, so it should be
safe to simply fall back to the parent classes equals/hashcode.

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

Two 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.

Eleventh set.

Change-Id: I45b20ae7c5d285598a1e1a0bfa9fbbfa7c1b617a
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.

Tenth set.

Change-Id: I62a1b9724ed235c6b406fbbf2a005606cfd9d87e
Signed-off-by: Ed Warnicke <eaw@cisco.com>
Signed-off-by: Giovanni Meo <gmeo@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.

Ninth set.

Change-Id: Icf3ea474ed0313ab92e93717658f20f6bc7c228c
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.

Eighth set.

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

Merge "Five more Equals/HashCode/StringBuilder replacements"

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.

Seventh set.

Fixed to no longer use NodeIDString in Node or NodeConnectorIDString in
NodeConnector when computing equality or hashcode.

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

Enable multiple roles per user (backend)

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

Northbound IntegrationTest

Initial base for writing integration
tests for northbound bundles.

All northbound integration tests can be
done within this bundle.  A full controller
using the stub protocol plugin will be started.

A test for getting all flow statistics is also
done.

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

plugin-stub: InventoryService to Stub

Changing the InventoryService to use the
node STUB type instead of OF.  Changed the
SwitchManagerIntegrationTest to match the
stub type changes.

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

Merge "- Respond to switch Echo Request immediately. It bypasses transmit queue and sends the message over the socket directly. - Send PACKET_OUT (lldp/arp etc) at high priority. When the controller sends down large amount of flowmod messages, the switch may take some time to install/update all the rules in HW. At high priority, the PACKET_OUT messages do not have to wait for the whole flowmod operation to be complete."

Merge "Added support for resolving augmentations."

Refactored equals in AugmentationSchemaImpl.

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

- Respond to switch Echo Request immediately. It bypasses transmit queue and sends the message over the socket directly.
- Send PACKET_OUT (lldp/arp etc) at high priority. When the controller sends down large amount of flowmod messages, the switch may take some time to install/update all the rules in HW. At high priority, the PACKET_OUT messages do not have to wait for the whole flowmod operation to be complete.

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

More javadocs added to yang-model-api.
Deleted not used KeyDefinition class.

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

Added support for resolving augmentations.

Added implementation into BindingGeneratorImpl to support generation of augmentati statements from yang models.
Added AugmentedTypeTest and augmented-test-models for testing of agumentation functionality.

Added ability to generate Java Interfaces that extends generic types.
Added Augmentation and Augmentable interfaces into yang-binding project.

Fixed bug in bad equals resolving in AbstractBaseType in binding-generator-util;
Added ReferencedTypeImpl into binding-generator-util;

Added findParentModule static method into SchemaContextUtil for resolving Module from SchemaNode in Schema Tree;
Added DATA_OBJECT, augmentableTypeFor and augmentationTypeFor into Types for referencing of Augmentable<T>, Augmentation<T> and DataObject interfaces defined in yang-binding;

Fixed incorrect resolving of namespaces and package names during generation of interfaces from yang lists;
Fixed bug in TypeProvider in resolving of extended types extended from multiple extended types;
Added ControllerTest and controller-models as test resources;
Changed position of revision date during package generation -> Now revision date comes after namespace;

Change-Id: I687925a622853bb6c9a809f17efd980c2960ae39
Signed-off-by: Lukas Sedlak <lsedlak@cisco.com>
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Merge "Updated SCM location"

Updated SCM location

Change-Id: I6c59e59713366e77825270e4f0f5265a88ed628d
Signed-off-by: David Erickson <daviderickson@cs.stanford.edu>

Merge "Formatting cleanup of POM files"

Merge changes I02c2af4b,If6adaabc,I559b92ff

* changes:
  Change generated MANIFEST.MF location
  Remove superfluous groupId.
  Add PDE OSGi launcher for the controller project

Merge "Fixed tests. Added javadocs to yang-model-api. Minor formatting changes performed in yang-model-parser-impl."

Formatting cleanup of POM files

Replaced tabs with two spaces and removed
some extraneous lines and tail whitespace.

Change-Id: I3dbc766b3ffabf76a17a04454beb69933f67f8a0
Signed-off-by: David Erickson <daviderickson@cs.stanford.edu>

Change generated MANIFEST.MF location

This patch moves the generated MANIFEST.MF for
OSGi from the target/classes/META-INF folder to
./META-INF for non-web bundles, and to
./src/main/resources/META-INF for web bundles.
This fixes a problem in Eclipse where it would
think that the project is always out of sync,
indicating this on the OSGi run configuration.

Change-Id: I02c2af4bf80a7f1fe66f53ddf3ae24232d8b83ec
Signed-off-by: David Erickson <daviderickson@cs.stanford.edu>

Remove superfluous groupId.

The groupId is already specified in the
parent element, and the additional declaration
causes warnings in Eclipse.

Change-Id: If6adaabc4407a4ea5100ba8af666b05c333dae8d
Signed-off-by: David Erickson <daviderickson@cs.stanford.edu>

Fixed tests. Added javadocs to yang-model-api.
Minor formatting changes performed in yang-model-parser-impl.

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

Add PDE OSGi launcher for the controller project

- Created a p2site distribution which will place in a p2 repository
all the artifacts that are part of the
opendaylight/commons/opendaylight/pom.xml parent pom. The p2site will
be built by the controller-merge job on jenkins and will be used as
update site for the "opendaylight" target platform.
- Created a target-platform "opendaylight" to be used by PDE when used
in eclipse, this target platform is supposed to fetch the artifact
from a remote P2 site that will be hosted on jenkins after this patch
will get in.
- Create an OSGi PDE launcher "opendaylight-osgi-launcher" to be able
to launch the controller without the assembly phase to be required,
this one works with the target-platform "opendaylight"
- Created a target-platform "opendaylight-local" to be used by PDE
with the assumptions that the p2 site reside local and has been built
in the p2site distribution.
All this enhancements are to address some of the eclipse usability
issues as documented in:
https://bugs.opendaylight.org/show_bug.cgi?id=21

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

Fix for exception seen on selecting device tab, after stopping switchmgr bundle.

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

Added support for Modules to ModuleDependencySort (beside ModuleBuilders).
ModuleDependencySort now provides only static methods.

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

Merge "Fixing minor build errors"

Merge "Five more Equals/HashCode/StringBuilder replacements"

Merge "Seven  more Equals/HashCode/StringBuilder replacements"

Merge "Six  more Equals/HashCode/StringBuilder replacements"

Merge "Five more Equals/HashCode/StringBuilder replacements"

Merge "Five more Equals/HashCode/StringBuilder replacements"

Merge "Five more Equals/HashCode/StringBuilder replacements"

Merge "Fixed YANG union type parsing. Changed module builders storing from HashMap to LinkedHashMap."

Merge "Added test to yang-maven-plugin-it."

Fixing minor build errors

The POM of the northbound integration test was
referencing the onecontroller name instead of
opendaylight.  And /logger/log/ in a test file.

Signed-off-by: David Erickson <daviderickson@cs.stanford.edu>

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.

Sixth set.

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

Seven  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.

Fifth set.

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

Six  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.

Fourth in set.

Change-Id: Ic24cd900b1b88a1496c5f0a14f8f24650c00b297
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.

Third in the set.

Change-Id: If8008c7e9db88022edd50fbc36f82cd32cdb6bad
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.

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>

Fixed YANG union type parsing. Changed module builders storing from HashMap to LinkedHashMap.

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

Added test to yang-maven-plugin-it.

Signed-off-by: Martin Vitez <mvitez@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"