Changed sun_coding_style.xml to have 120 line wrap instead of 80

See thread on controller-dev starting here:
https://lists.opendaylight.org/pipermail/controller-dev/2013-June/000673.html

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

Merge "Disable unit tests on integration tests plugin"

Disable unit tests on integration tests plugin

- This patch disable unit tests to be even looked on the integration
tests bundles, so that we are sure that integration tests are only
looked by failsafe and so they can be disabled without impacting unit
tests.

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

Merge "Parents pom distribution"

Add IfNewHostNotify to DeviceManager

Small changes so that DeviceManager uses
IfNewHostNotify as listener to pass new
host info.  SimpleForwardingManager app should
now function, creating flows and allowing hosts
to ping each other.

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

Merge "Enable checkstyle for other non-source files"

Subnets Northbound integration tests.

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

Enable checkstyle for other non-source files

Just to make sure there is uniformity of threatment, also other files
are being exposed to checkstyle, limited to space and tabs rules.
Along with the rules, bulk changing the code to avoid breakages in the
build.

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

Merge "Bumped version of YANG tools to 0.5.2-SNAPSHOT"

Bumped version of YANG tools to 0.5.2-SNAPSHOT

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

Merge "Configuration integrationtest to test save config functionality"

Merge "Rooted out the last of the ReflectionToStringBuilder usage"

Configuration integrationtest to test save config functionality

- Renamed to make sure the integration test runs only as part of
failsafe phase, so it can be skipped

Change-Id: If69d30f1f653ec35e96861df541c94114c6ce791
Signed-off-by: Dheeraj Uppalapati <duppalap@cisco.com>

Merge "Added support for source code generation from identity statement, grouping statement and identityref type statement. Introduce new BaseIdentity class as base for all classes generated from identity statements. Introduce new WildcardType interface to add possibility to generate wildcard java type parameters in generated code. Updated tests."

Merge "Generation of Notifications and RPCs from YANG"

Merge "Added Static Routing Northbound integration tests for all available REST APIs."

Merge "HostTracker Fixes"

Added support for source code generation from identity statement, grouping statement and identityref type statement.
Introduce new BaseIdentity class as base for all classes generated from identity statements.
Introduce new WildcardType interface to add possibility to generate wildcard java type parameters in generated code.
Updated tests.

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

Rooted out the last of the ReflectionToStringBuilder usage

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

Correction for TopologyManager jUNIT test. Arguments for Edge() was in
wrong order.

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

Merge "Separate UserManager into api and implementation bundles"

Added Static Routing Northbound integration tests for all available REST APIs.

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

Generation of Notifications and RPCs from YANG

 - Added generation of RPC Service interfaces
 - Added generation of RPC Messages
 - Added generation of Notifications
 - Updated JUnit tests to reflect changes

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

Separate UserManager into api and implementation bundles

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

HostTracker Fixes

Fixes the following:
Hosts not being added due to SwitchManager
not being set and NullPointerException

Conversion added from Device to HostNodeConnector

REST Service nb/v2/host/default will show hosts

Known problems still need fixing:
- hosts cannot ping each other.
- When mininet is already running and switches are there,
  startup for controller is fine.  If controller already
  running, then mininet started and new switches added,
  controller is throwing errors.
- GUI isn't showing hosts in the diagram.

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

Merge "Fix bulk sync in SwitchMananger."

Merge "HostTracker Bundle Separation"

Fix bulk sync in SwitchMananger.

When node connector is added to a container, notify node addition if it was not present in the container.

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

Merge "Added Support for conversion yang binary type to java type;"

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

Merge "Fixed platform dependant bugs in maven plugin"

Added Support for conversion yang binary type to java type;

Added support for generation of primitive java types;
Added support for conversion yang binary type to java byte array;

Updated toString method in AbstractBaseType in case that primitive type does not contain package name;

Added documentation into TypeProvider javaTypeForSchemaDefinitionType method;
Updated GeneratorUtil methods to not generate fully qualified names or imports in case of generating of java primitive types;

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

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>

HostTracker Bundle Separation

Moved the new HostTracker into
hostracker_new bundles.

Added exclusion to bin.xml so it doesn't
get assembled into distribution.

Added original hosttracker back.

Goal is to allow collaboration and
incremental check-ins for this
module without breaking anything else.

Original information from Rob Adams:
    Author: Rob Adams <rob.adams@bigswitch.com>
    Date:   Wed Jun 5 14:58:02 2013 -0700

    Initial port of device manager to cisco controller.  Currently capable
    of tracking devices but still a long list of TODOs:
    - No VLAN support - Ethernet frame parser can't parse VLAN tags and
      would be a bit of work to add it since current code can't handle
      anything but fixed-format header field which doesn't cover 802.1Q
    - No shared framework for asynchronous tasks.  For now device manager
      just allocates its own scheduled thread pool but a shared thread
      pool would be much better.
    - No clustering support in this code.  There is code that uses an
      eventually-consistent data store to support this but it's commented
      out since this is not available in daylight.  It would be possible
      to port this to the consistent store but this would lose valuable
      semantics and a component in the core should handle eventual
      consistency.
    - No web API support.  Would need to write a new implementation of the
      API; shouldn't be too hard presumably
    - Debug counters not ported
    - No mechanism for documenting log messages.  This doesn't exist in
      ODL.
    - DHCP parsing not supported; disable pulling hostname from DHCP.
    - Node/Port IDs are derived from Object and have no useful semantics.
      Device manager needs them to be comparable in order to get
      attachment points correct and consistent in all cases.  At the
      moment it just blindly casts them
    - Topology module doesn't implement the needed semantics.  First need
      to port net-virt-platform topology code, but there are impedance
      mismatches in the APIs to resolve first
    - Need to merge host tracker and device manager APIs to a new unified
      API and port all existing code to the new API, or adopt the device
      manager API.
    - Need to replace ARP handler to just handle ARPs and not be
      responsible for trying to discover devices
    - Unit tests need to be ported as well; currently commented out

Change-Id: If65e6582599a9c6fafaabfeb4b93b1d931965ab1
Signed-off-by: Kalvin Hom <kahom@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>

Parents pom distribution

- Introduced a distribution that gather all the parents module and
allow for their deployment in one shot rather in multiple single calls
as done right now by jenkins

Change-Id: I11c485c2d50e7de69cea7a8091cd0e62068a3641
Signed-off-by: Giovanni Meo <gmeo@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>