Merge "Fixed schema path of types in augment statements. Updated tests."

Merge "Fixed platform dependant bugs in maven plugin"

Merge "removing DataPacketService's receive queue"

Merge "UserManager as role coordinator"

removing DataPacketService's receive queue

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

Fixed schema path of types in augment statements.
Updated tests.

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

SwitchNorthbound tests

Added tests for all available SwitchManager REST APIs.
Made changes to stub InventoryService to allow for updating nodeProps and nodeConnectorProps.
Made changes to SwitchNorthbound.java to check for both nodeType and nodeId when getting a Node.
Made changes in all northbound tests to throw JSONException instead of simple assertion failure.

Change-Id: I816dcbaea2ed10833e3270395fe894e5c152f5d8
Signed-off-by: patrichu <patrichu@cisco.com>

Fixed platform dependant bugs in maven plugin

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

UserManager as role coordinator

- UM to expose API to check if a role name is already in use
- Included search in local authorization configurations when querying for roles on user, getUserRoles()
- Added local authorization code to Junit to exercise the above
- Minor code refactoring to resuse getUserRoles() to avoid code duplication
- Downgrade and changed log level on cache allocation when cache is already present
- Removed new line in UM log messages to be in line with other bundles

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

Added Support for Union Type def resolving and bug fixes.

Added implementation into TypeProviderImpl to proper resolve union type definitions;
Added UnionDependencySort for correct resolution of multiple unions in same yang model;
Added support for resolving of imported union types from multiple modules;
Added tests for Union Type def resolving;

Fixed Resolving of typedef Enumerations - now the Enumeration is resolved as standalone enum typedef name .java;
Fixed resolving of Inner Enumerations;
Fixed resolving of referenced Inner Enumerations through leafref definition;
Added EnumGenerator class for generating of Enum definitions and inner Enum definitions;
Fixed generation of Enumeration type through code writer;
Fixed generation of Enumeration package name in case of inner enum definition in Class and Generated Transfer Objects;
Added capability to generate inner enums into ClassCodeGenerator and InterfaceGenerator;

Fixed of missing apostrophe in createToString method in GeneratorUtil.java in case when more than one property should be generated in toString method;

Fixed platform specific bug in maven-yang-plugin incorrect retieval of maven dependency in YangToSourcesProcessor.java;
Fixed platform specific bug in maven-yang-plugin-it - hard coded resource paths replaced for getClass().getResource calls;

Added support into BindingGenerator API to suppor of generation of Types with restriction of specified modules that will be generated;
Added implementation of generateTypes in BindingGeneratorImpl;

Added tests and test resources for testing Enumeration gen. types
Added test resources for augment relative xpath testing;

Fixed improper implementation of hashCode and equals in ExtendedType in yang-model-util;
Fixed findParentModule method in SchemaContextUtil in yang-model-util project;

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

Merge "Checkstyle enforcer"

Merge "Yang-maven-plugin refactored and config attributes and spi adjusted."

Checkstyle enforcer

Started to enforce checkstyle, if the code style doesn't match the
checkstyle settings build will break. This is needed in order to make
sure folks comply to one code style and don't start diverging.

For now the enforcement is only limited to:
- make sure no TABS are present in the java files
- make sure that no trailing whitespaces are present

Along with code style enforcement updates all the files that didn't
comply else compilation would not be successful.

Added new exclusion in .gitignore

Added new parent for commons.opendaylight, please read the README in
the commons.parent project for the reason for it.

Make sure the violation of checkstyle are reported on the console to
avoid for a developer to be able to dig through the log in the target
directory

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

Fixed bug in parser in augment resolving process.

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

Fixed parsing of typedef statement. Refactored YangParserImpl to improve code readability.
Added methods to LeafSchemaNode interface. Updated tests.

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

Yang-maven-plugin refactored and config attributes and spi adjusted.

Removed project base dir parameter from CodeGenerator
since it can be acquired via additionalConfiguration.
Added resourceBaseDir config attribute that allows resources
to be added to packaged jar.
Added mavenProject config attribute that provides any additional
configuration to code generators if necessary.
Adjusted log levels for the plugin.

Plugin refactored and generate sources test fixed.

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

Merge "Updated example with OF model done during HackFest."

Merge "Samples/maven-code-gen-sample pom.xml updated after recent changes."

Merge "Modified construction of built-in yang types."

Merge "Adding Set Next Hop action"

Samples/maven-code-gen-sample pom.xml updated after recent changes.

Added insepctDependencies config attribute.
Yang-maven-plugin integration tests marked as IT and now run by failsafe.

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

Merge "Manual link addition feature assumes Long for Switch identifier and Short for Port identifier. This commit fixes that assumption."

Fixed Openflow InventoryService to return the property cache.

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

Manual link addition feature assumes Long for Switch identifier and Short for Port identifier. This commit fixes that assumption.

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

Merge "Revert "Checkstyle enforcer""

Revert "Checkstyle enforcer"

This reverts commit 541d0a36997f292bb037a2199463431eee538358

We are going to talk a bit more about this before doing it.

Change-Id: I788ecea18796c6eb325285dfaa83a6aae300634c

Merge "Checkstyle enforcer"

Checkstyle enforcer

Started to enforce checkstyle, if the code style doesn't match the
checkstyle settings build will break. This is needed in order to make
sure folks comply to one code style and don't start diverging.

For now the enforcement is only limited to:
- make sure no TABS are present in the java files
- make sure that no trailing whitespaces are present

Along with code style enforcement updates all the files that didn't
comply else compilation would not be successful.

Added new exclusion in .gitignore

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

Fixed a bug in removeSpanPorts

Signed-off-by: Youcef Laribi <Youcef.Laribi@citrix.com>

Updated example with OF model done during HackFest.

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

Merge changes I029118f9,I4a0320b9

* changes:
  Fixed typo in transform package
  Cleaned up Integration Tests

Fixed typo in transform package

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

Merge "Yang-maven-plugin refactored + fixed bugs. Removed resourceProviders argument(unused). Other arguments marked as optional. Added argument to set whether use yang files from dependencies or not. Added packaging of yang files from project to META-INF. Added additionalConfiguration argument for code generators."

Merge "Finalize the OSGi launcher for the opendaylight distribution"

Modified construction of built-in yang types.

Base yang types can not be restricted now. Every restriction causes creation of new ExtendedType with default yang type as base type.
Updated tests.

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

Removing invalid casts from hosttracker.

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

Adding Set Next Hop action

- It will be ignored by openflow plugin

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

Cleaned up Integration Tests

- Removed dependency version declarations from Integration Tests.
  The version of controller bundles and dependencies is loaded
  from integration test pom files.

- Moved integration tests from test phase to verify.
  The integration tests could be skipped by -DskipITs option.

- Added common base POM for integration tests with pax-exam
  configuration.

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

Merge "Fix exception in HostTracker when non-OF node is removed."

Merge "FlowProgrammer.Northbound test"

Merge "Fix to maintain Java 6 compatability"

Merge "Custom Node Type jaxb string to id"

Merge "Refactored ModuleBuilder to avoid name conflicts. Fixed implementation of ChoiceBuilder. Refactored parsing of refine statement. Added tests. Renamed ChildNodeBuilder -> DataNodeContainerBuilder. Renamed AbstractChildNodeBuilder -> AbstractDataNodeContainerBuilder. Renamed TypedefBuilder -> TypeDefinitionBuilderImpl."

Removed Equals/HashCodeBuilder for ARP/LLDPTLV

This also entailed fixing equals and HashCodeBuilder for Packet.

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

Yang-maven-plugin refactored + fixed bugs.
Removed resourceProviders argument(unused).
Other arguments marked as optional.
Added argument to set whether use yang files from dependencies or not.
Added packaging of yang files from project to META-INF.
Added additionalConfiguration argument for code generators.

Yang-maven-plugin SPI, ResourceProvider interface removed since
it was unused. Code generator interface improved.

YangParserImpl API changed to provide a map<File, Module>. Users will
know which module belongs to which file and can utilize this information
when parsing yang files from multiple sources. Yang-maven-plugin needs
this information since it parses yang files from project as well as its
dependencies.

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

Finalize the OSGi launcher for the opendaylight distribution

In this commit following changes are being done:
1) Adjusted opendaylight-osgi-launcher-local.launch not to include
bundles that are clustering.stub and protocol_plugin.stub because
those are only meant for integration testing
2) Modifed the opendaylight.target to refer to an http repository so
eventual problem with certificates will not be seen, using the https
url of jenkins on some system has been causing problems because the
root certificate for jenkins was not known.
3) Adjusted the opendaylight-osgi-launcher.launch not to include the
bundles that are clustering.stub and protocol_plugin.stub because
those are only meant for integration testing
4) Adjust the opendaylight-osgi-launcher.launch to make sure the
jersey libraries startup soon else an harmless yet annoying error
would be printed
5) Moved clusterin.test bundle off Declarative service else the
OSGI-INF/component.xml location even though works for built done via
maven will not work for builds done via eclipse PDE.

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

Fix exception in HostTracker when non-OF node is removed.

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

FlowProgrammer.Northbound test

Added northbound integration tests
for adding, getting, and deleting
flows.

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

Merge "Fixed a bug in deriving Switch Mac-address and a null check on set payload on ip packet Change-Id:I91cd687f3177829862b6aac4a8bce8bd53f7baa5 Signed-off-by: Madhu Venugopal <vmadhu@cisco.com>"

Fixed a bug in deriving Switch Mac-address and a null check on set payload on ip packet
Change-Id:I91cd687f3177829862b6aac4a8bce8bd53f7baa5
Signed-off-by: Madhu Venugopal <vmadhu@cisco.com>

Merge "Principal to contain all the user roles"

Fix to maintain Java 6 compatability

Swapped out the use of Integer.compare (which requires Java 7)
for Integer.valueOf(x).compareTo(Integer.valueOf(y)) which
is promised to return the identical value here:
http://docs.oracle.com/javase/7/docs/api/java/lang/Integer.html#compare(int, int)

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

Custom Node Type jaxb string to id

When using the web services, nodeIDString
needs to be parsed into the correct
type to get the nodeID when it is not
one of the preset Node types.

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

Refactored ModuleBuilder to avoid name conflicts. Fixed implementation of ChoiceBuilder.
Refactored parsing of refine statement. Added tests.
Renamed ChildNodeBuilder -> DataNodeContainerBuilder.
Renamed AbstractChildNodeBuilder -> AbstractDataNodeContainerBuilder.
Renamed TypedefBuilder -> TypeDefinitionBuilderImpl.

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

Merge "- Added osgi cmds so that user is able to configure a link. The link is added to the main Topology if it's not a Production link."

Principal to contain all the user roles

- Added IUserManager.getUserLevels() which returns all the controller levels for the passed user
- Modified CustomRealm to return a principal containing all the controller roles for the passed user
- Code style and format changes for the touched files

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

- Added osgi cmds so that user is able to configure a link. The link is added to the main Topology if it's not a Production link.

Usage:

addUserLink <name> <node connector string> <node connector string>
deleteUserLink <name>
printUserLink

- Fixed an exception in SwitchManager osgi cmd pns

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

Merge "Replaced Equals/HashcodeBuilder for DatapacketListener"

Node MAC address to be derived inside the protocol plugin

- Moved network node's MAC computation from switch mgr to InventoryShimService in protocol plugin
- Modified MAC property to only carry network node MAC
- Some automatic coding style changes
- Some static analysis fix

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

Fix exceptions in SwitchManager Web in case of non-OF nodes and node connectors

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

Replaced Equals/HashcodeBuilder for DatapacketListener

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

Merge "Add 'TableStatistics' to SAL and Northbound Statistics API."

Add 'TableStatistics' to SAL and Northbound Statistics API.

It is similar to the FlowStats and PortStats (NodeConnectorStats) that are present now.

Change-Id: Ibecb95c9c4a4f911e5fd3847bbf1f2e77eb16be1
Signed-off-by: Aditya Prakash Vaja <aditya.vaja@bigswitch.com>

- SAL Inventory needs to be modified to be able to take multiple IPluginInInventoryService providers. Same applies to IListenInventoryUpdates.
- Plugin needs to introduce a wrapper of SAL interface if it's going to be used locally within the plugin.

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

Merge "Add hosttracker northbound integration test. This modification depends on change-id I760800: NodeConnector custom type"

Merge "Refactored SchemaPath for yang java types. Fixed SchemaPath for augmented nodes types. Updated yang java types constructors. Updated tests accordingly."

Merge "NodeConnector custom type"

Refactored SchemaPath for yang java types. Fixed SchemaPath for augmented nodes types.
Updated yang java types constructors. Updated tests accordingly.

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

Fix method getFlowEntriesForGroup in forwardingrulesmanager

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

Add hosttracker northbound integration test.
This modification depends on change-id I760800: NodeConnector custom type

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

NodeConnector custom type

Changes for NodeConnector to create
new NodeConnector of custom type;

Created STUB implementation for
protocol plugin;

Reordered NodeFactory's arguments
to be consistent with other methods;

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

Merge "Bumped version of Maven YANG tools to 0.5.1-SNAPSHOT"

Bumped version of Maven YANG tools to 0.5.1-SNAPSHOT

 - Removed unnecessary dependencies from YANG Maven Tools integration tests.
 - Added Attach Sources and Javadoc to deploy phase

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

Merge "Added extensible implementation of Transformators"

Merge "Correct maven dependency for MavenProject"

Added extensible implementation of Transformators

- set of generic Transformers - factories responsible for producing
  outputs based on input objects
  - Ideal for development of translation layers between protocol
  specific objects and abstract transfer objects such as Actions in SAL.

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

Correct maven dependency for MavenProject

The name of the artifact which exports
org.apache.maven.project.MavenProject has changed between maven 2.x and
3.x to mave-core. Update the dependency to pick up the changes, which
include generics -- allowing for safer codegen.

Change-Id: I6237eb935e837e8ff887f8e072be12dd8ddb4c8c
Signed-off-by: Martin Vitez <mvitez@cisco.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.sk>

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>