reformatting packet creation to be more readable

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

Fixed UDP traffic load balancing

Signed-off-by: Anilkumar Vishnoi <avishnoi@in.ibm.com>

Merge "Mavenified generation of ANTLR YANG parser"

Redirecting Caught and Uncaught Exceptions to OSGI Console and Log File

The existing mechanism int the Controller allows the exceptions to be printed only
the console. This applies to both caught and uncaught exception. If the console
buffer is not too large and gets overwritten or gets cleared, there is no way for
the user to know what exceptions occurred. This commit implements a new mechanism
by which both types of exceptions will get printed on the console as well as logged
to the file.

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

Added Yang validator.
Initial implementation committed that validates Module and Submodule statements.
It also validates a few additional yang statements e.g. revision, import.

Rules that must be obeyed are documented in code as javadoc.
Added tests to test implemented rules.

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

Added protocol_plugins.stub that implements
IPluginInReadService

Modified statisticsmanager structure so that
api and implementation are separate modules.

Added Integration test for statisticsmanager
Modified pom.xml for sonar integration test
inclusion.  When going to the Statistics
Manager unit test, should also show
integration percentage.  If Sonar's integration
widget is enabled, should also see coverage on
dashboard.

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

Mavenified generation of ANTLR YANG parser

Added ANTLR sources for YANG parser and lexer,
Added antlr4-maven-plugin configuration and lifecycle
mappings for the Eclipse.

Removed original sources of YANG parser and lexer.

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

Merge "Refactor frontend JS"

Refactor frontend JS

Refactored one.js into lib.js and open.js. lib.js contains all the common library functions
one.js and other page.js can use. open.js is the core engine that runs the web UI.

Frontend is now extensible by automatically including  lib.js and overriding default JS engine.

Use spaces instead of tabs.

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

Merge "Topology Service related enhancement and Style changes. Signed-off-by: Madhavan Kasthurirangan <mkasthur@cisco.com>"

Merge "- Added the username/password criteria in usermanager - Relaxed the criteria for nodename/gateway name in devices"

Topology Service related enhancement and Style changes.
Signed-off-by: Madhavan Kasthurirangan <mkasthur@cisco.com>

Merge "Controller to listen to expired flow removal message - Added plugin-to-FM async notification path for Flow Programmer so that functional modules can receive notification of flow removal sent by switch - On flow programming phase, instruct switch to notify controller about flow removal only if hard or idle timeout is set - Have FRM listen to flow removal message so that it updates the sw view of installed flows and the static flow config installation status accordingly - Changed equals logic in MatchField as a null mask object for net address is equivalent to a /32 (or /128) bitmask - Removed strict dependencies in SAL Activator. SAL services startup should not depend on external services - Enforced the sun_doding_style.xml in newly added or modified files Change-Id: Ifc913ceef87aac0b4694bcd7c47f29d5cb10c6ed Signed-off-by: Alessandro Boch <aboch@cisco.com>"

Controller to listen to expired flow removal message
- Added plugin-to-FM async notification path for Flow Programmer so that functional modules can receive notification of flow removal sent by switch
- On flow programming phase, instruct switch to notify controller about flow removal only if hard or idle timeout is set
- Have FRM listen to flow removal message so that it updates the sw view of installed flows and the static flow config installation status accordingly
- Changed equals logic in MatchField as a null mask object for net address is equivalent to a /32 (or /128) bitmask
- Removed strict dependencies in SAL Activator. SAL services startup should not depend on external services
- Enforced the sun_doding_style.xml in newly added or modified files
Change-Id: Ifc913ceef87aac0b4694bcd7c47f29d5cb10c6ed
Signed-off-by: Alessandro Boch <aboch@cisco.com>

- Added the username/password criteria in usermanager
- Relaxed the criteria for nodename/gateway name in devices

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

Merge "Refactored YANG types resolving."

Merge "Improve Vendor Extension Error Handling. Signed-off-by: Madhavan Kasthurirangan <mkasthur@cisco.com>"

Improve Vendor Extension Error Handling.
Signed-off-by: Madhavan Kasthurirangan <mkasthur@cisco.com>

Refactored YANG types resolving.

Fixed YANG union type parsing. Refactored code after PMD check.
Fixed organization statement in test files.

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

Merge "Fixed bug in HostTracker.retrieveCache() where incorrect variable is being checked for null."

Merge "Added javadoc to maven-yang-plugin."

Merge "Removed forgotten references to yang-to-sources test jar in maven-yang-plugin integration tests. Replaced with reference to maven-yang test jar (new name for yang-to-sources)."

- Added OSGI cmd to display TLS configuration
- Added null pointer checks and more debug msgs
Signed-off-by: Jason Ye <yisye@cisco.com>

Fixed bug in HostTracker.retrieveCache()
where incorrect variable is being checked
for null.

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

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

Merge "Fix Logging for Topology Manager"

Fix Logging for Topology Manager

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

Fix comments in Path.java. Also, Max Throughput dijkstra modification.
Signed-off-by: Madhavan Kasthurirangan <mkasthur@cisco.com>

Added javadoc to maven-yang-plugin.

Documentation can be generated using mvn site command.
Useful information about goals and cofiguration is located under
Project Reports / Plugin Documentation in generated docs.

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

Removed forgotten references to yang-to-sources test jar in maven-yang-plugin integration tests.
Replaced with reference to maven-yang test jar (new name for yang-to-sources).

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

Update the API generation code and code generation sample

Fixed bug in generation of package names.

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

YANG model parser refactoring
-fix YANG types constraints resolving
-added tests
-removed unused test files

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

Renamed yang-to-sources-plugin to maven-yang-plugin.
Updated all references in tests.

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

Merge "Moving a recently added trim() call in SwitchHandler after Null check."

Moving a recently added trim() call in SwitchHandler after Null check.

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

Fix issue seen with enunciate plugin when importing in eclipse the
project with m2e

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

Merge "- Add stop() method in IMessageReadWrite for proper clean up when connection is closed. - Be able to handle whitespace in TLS config lines Signed-off-by: Jason Ye <yisye@cisco.com>"

- Add stop() method in IMessageReadWrite for proper clean up when connection is closed.
- Be able to handle whitespace in TLS config lines
Signed-off-by: Jason Ye <yisye@cisco.com>

Changing NB API authentication/authorization to use Tomcat Valve.
Affected modules
1. Flow Programmer
2. Host Tracker
3. Static Roting
4. Statistics
5. Subnets
6. Switch Manager
7. Topology
8. Loadbalancer
Change-Id: I94267ed1878c49b9615e956aa816473361ded8f2
Signed-off-by: Venkatraghavan Sankarasubbu <vensanka@cisco.com>

ISSUE: Topology does not show Node description as learned by the OFA

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

Fix stale cached links in visual topology

Links weren't properly updated everytime there was a topology state change.

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

Fix disappearing nodes

Minor fix in jit.js, this bug was only seen in Webkit-based browsers such as Chrome/Chromium

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

- If a port is already under OF control and then hw LLDP transmit is enabled on it, controller should discard the hw LLDP packet.
- When we change the name of the Production switch, stale entries should be removed.
- Added Backend APIs to add/delete/get user link configs
- Northbound REST APIs to GET/POST/DELETE user link configs
- Endpoint connected as Production Switch to OF port. If the system-name TLV exists in the hw LLDP packet, it should be used for display.
- Some misc osgi debugging commands.

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

Enable sal.core.Node to be an XML root element

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

Auth Principal to work with Remote authorization
Also added some helpful OSGi console commands for remote auth.

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

ISSUE: After controller restart, proactive flows are not installed
This is because of an incongruence between Devices.web and SwitchManagerImpl code, where the
former only consider the Node.nodeID string form and the latter expects the full Node string form.
CHANGE:
Have Devices.web passing Node.toString() form to front-end

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

Merge "fixed method name"

Merge "Call per page clean up if that is defined."

fixed method name

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

Simple Load Balancer Application - Recommitting the source codewith the whole source code of the application in single commit

Signed-off-by: Anilkumar Vishnoi <avishnoi@in.ibm.com>

fixing spelling errors in comments

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

Call per page clean up if that is defined.

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

Branding support for the newly added error page.

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

Replacing Spring Security framework with Apache Tomcat Valve for Web Security (UI Only).
Northbound will be handled as part of a different commit.
Updated Web UI modules :
1. Root
2. Flows
3. Devices
4. Topology
5. Troublehshooting

Change-Id: I09fa1968a184bf6ae0c6248e8eea3c05a40433d8
Signed-off-by: Venkatraghavan Sankarasubbu <vensanka@cisco.com>

Merge "Web UI Branding support This fix provides an extensible branding support with the help of OSGi Fragment. Also included a sample branding fragment which can be used to create custom brands."

Web UI Branding support
This fix provides an extensible branding support with the help of OSGi Fragment.
Also included a sample branding fragment which can be used to create custom brands.

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

Merge "Add TLS support in the Opendaylight Controller: - TLS configuration is specified in file ./configuration/config.ini. To enable the TLS feature, set secureChannelEnabled=true and specify the location of controller Java KeyStore and TrustStore files. The Java KeyStore contains controller's private key and certificate. The Java TrustStore contains the trusted certificate entries, including switches' Certification Authority (CA) certificates. Here is the sample configuration,"

Adding ability to customize onClick on topology graph.

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

Add TLS support in the Opendaylight Controller:
- TLS configuration is specified in file ./configuration/config.ini. To enable the TLS feature, set secureChannelEnabled=true and specify the location of controller Java KeyStore and TrustStore files. The Java KeyStore contains controller's private key and certificate. The Java TrustStore contains the trusted certificate entries, including switches' Certification Authority (CA) certificates. Here is the sample configuration,

secureChannelEnabled=true
controllerKeyStore=./configuration/ctlKeyStore
controllerKeyStorePassword=xxxxx (this password should match the password used for KeyStore generation)
controllerTrustStore=./configuration/ctlTrustStore
controllerTrustStorePassword=xxxxx (this password should match the password used for TrustStore generation)

- Added two message read/write services, one for clear text, one for secure communication.

- Added priority queue for message transmission. The system critical messages, such as Hello, Echo Reply etc will be treated as high priority and will be served ahead of other messages, like statistics request, discovery packets etc.

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

Gerritt #107 introduced a fix that caused Routing Listener registration issue.
Fixing it here. Also removed the unused DS xmls.

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

Fix for bug 7

Change-Id: I62e5725e0712e07fcb2fe8c0425cfca0fa6da865
Signed-off-by: Anilkumar Vishnoi <avishnoi@in.ibm.com>

Merge "Added skeleton for yang maven plugin."

Added skeleton for yang maven plugin.

Added implementation of yang-to-sources maven plugin
which is responsible for parsing YANG files and invoking
configured code providers to generate sources from YANG
during maven generate-sources and generate-resources phase.

Added JUNIT tests testing the integration with maven and
following scenarios:

No generators for generate-sources and generate-resources
Unknown generators for generate-sources and generate-resources
Non-existant root folder with yang sources
No yang sources under yang root folder
Correct configuration for both generate-sources and generate-resources

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

Added support for RPCs to Binding-Aware SAL

Added support for RPCs to Binding-Aware SAL. The implementations
of the RPCs are mapped to interfaces describing RPC contract.

The registered implementation is registered for all RPCs described
in the interface and only one registered implementation of the
interface could exists at-the time.

It is possible to register implementation of derived interface,
but client must request the derived interface to access its
implementation of rpcs defined in super interface.

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

Added support for binding-independent RPCs

Implemented the support for binding-independent RPCs directly into BrokerImpl,
and removed unused placeholders for functionality.

Cleanup:
  - Moved Session Implementations directly into BrokerImpl as private inner
    classes, because they are BrokerImpl specific.
  - Removed unnecessary packages.
  - Extracted RPC utils from sal-broker-impl to sal-common-utils to be reusable
    for other components without introducing dependency on sal-broker-impl.

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

Merge "ISSUE: Some changes to Authorization"

ISSUE: Some changes to Authorization

CHANGE:
- Fixed enum string in AppRoleLevel and added fromString() method
- Added couple of methods to IResourceAuthorization
- Have some IResourceAuthorization methods to return Status
- Changed UserManagerImpl.getUserLevel() to check against local configured users as well, instead of against only local authenticated users
- Added a couple of Junits tests

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

Renaming OneWeb to DaylightWeb

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

Removing references to a prior branded controller

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

ISSUE:
Controller does not format TOS match field as specified by OF1.0 spec.
CHANGE:
- In protocol_plugin when converting SAL Match to OFMatch have TOS bits to be msb and viceversa
- Added utilities functions in NetUtils for deriving unsigned values for primitive
  types byte and short when needed for comparisons and bit shifting operations.
  Made use of them throughout the code.

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

Refactored YangModelParserImpl
Renamed Identityref to IdentityrefType
Renamed IdentityType to IdentitySchemaNode and moved from yang-model-util to yang-model-api + refactored as interface
Added tests

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

Fixed bug in naming conventions.
In yang-data-util changed package name from
org.opendaylight.controller.data.util to
org.opendaylight.controller.yang.data.util;
In yang-model-api changed package name from
org.opendaylight.controller.model.api.type to
org.opendaylight.controller.yang.model.api.type;
In yang-model-util changed package name from
org.opendaylight.controller.model.util to
org.opendaylight.controller.yang.model.util;

Refactored and updated all imports dependant on projects
yang-model-util, yang-model-api and yang-model-util

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

Fixed bugs in naming conventions of packages.

In yang-data-util changed package name from
org.opendaylight.controller.data.util to
org.opendaylight.controller.yang.data.util;
In yang-model-api changed package name from
org.opendaylight.controller.model.api.type to
org.opendaylight.controller.yang.model.api.type;
In yang-model-util changed package name from
org.opendaylight.controller.model.util to
org.opendaylight.controller.yang.model.util;

Added junit test for resolving Yang types from multiple Yang modules;
Fixed typo in QName of Uint64.java to uint64 QName;

Refactored and updated all imports dependant on projects
yang-model-util, yang-model-api and yang-model-util

Change-Id: I13779030a9823a821fc06d5b9a90e195da3e0a1a
Signed-off-by: lsedlak <lsedlak@cisco.com>

Merge "Fix for messags at the boot up time This commit proposes following two sets of changes"

Merge "Fixed YANG union type resolving in yang model parser."

Merge "Added Google Guava as dependency. Replaced Map<K,List<V> for SAL listeners."

Logging Related Enhancements.
Signed-off-by: Madhavan Kasthurirangan <mkasthur@cisco.com>

Fix for messags at the boot up time
This commit proposes following two sets of changes

1. Since the logger level has been changed from "WARN" to "INFO" for most
   of the components, change the logger messages in HostTracker.java
   accordingly

2. Change the usage of "+" to "{}" in logger messages for better efficiency

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

Consistent URL Scheme for Northbound and Web UI
Changing the URLs that is currently used to be more consistent and an added hierarchy : controller/nb, controller/web

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

Fixed YANG union type resolving in yang model parser.

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

Added Google Guava as dependency. Replaced Map<K,List<V> for SAL listeners.

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

Merge "Partial fix for bug 8. Fixing many generics-related warnings and some typos."

Merge "ISSUE Opendaylight controller to get node description from OF description statistics datapath description field"

ISSUE
Opendaylight controller to get node description from OF description
statistics datapath description field

CHANGE
- Switch Mgr to expose a getNodeDescription() method which returns the
  description configured by user if any, otherwise the one learnt from
  the proto plugin
- Web bunldes to make use of the above api (Currently they are ignoring
  the info learnt by the plugin)
- OFStatisticsManager to implement a IStatistics interface for informing
  listeners that the description statistics info has been refreshed
- Removing current logic where OFStatisticsManager was explicitely
  invoking the description property update on InventoryService through
  IOFInventoryService
- Removed logic where InventoryServiceShim queries OFStatisticsManager
  about node description as information is not available at that time
  and IStatisticsListener updates will serve same purpose
- Removing statistics request timeouts from OFStatisticsManager as the
  timeout is compeltely handled by core.internal.StatisticsCollector

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

Partial fix for bug 8. Fixing many generics-related warnings and some typos.

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

OpenDaylight uses Jersey framework to support JAX-RS compliant Northbound API.
OSGi makes sure the dependent modules are loaded first and before moving a module
into RESOLVED state. But the module implementation can result in one bundle become
ACTIVE before the other. This can be resolved properly by having these sequentialized
by monitoring the service bringup order.
In this case, com.sun.jersey.spi.container expects the jersey-server and core to be
active too soon causing the exceptions as seen in Bug 5.
But, as per OSGi specification, any bundle can be independently shutdown and restored
and the dependent bundles must be robust in handling this. Fortunately, the jersey
and dependent framework bundles handles this scenario properly and hence this exception
is just informational and has no impact on the running system.

OSGi also provides another mechanism called startlevel, which forces some infrastructure
bundles to be loaded sooner than the actual dependency is resolved. This accelerates the
bundle loading and solves problems like these. But, using this mechanism is discouraged,
because a well-made OSGi bundle need not depend on such bundle load parameters for its
operation. We are anyways adding this fix as a temporary measure to check the status
of the issue in various runtime environments. This fix can be removed after the current
work on Apache Valve integration is in place.

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

Merge "ISSUE Currently ForwardingRulesManager installs the proactive mode flows through the static flow configuration path. This allows these controller generated flows to be listed in the flow programmer UI. The drawback of this approach is that now the proactive flows are saved to FRM startup configuration, when user saves the config. So when FRM boots up, it will program the proactive flows on the switch whenever the switch connects to the controller, regardless of the actual switch forwarding mode configured on switch mgr."

ISSUE
Currently ForwardingRulesManager installs the proactive mode flows through the
static flow configuration path. This allows these controller generated flows to
be listed in the flow programmer UI. The drawback of this approach is that now
the proactive flows are saved to FRM startup configuration, when user saves the
config. So when FRM boots up, it will program the proactive flows on the switch
whenever the switch connects to the controller, regardless of the actual switch
forwarding mode configured on switch mgr.

This is not correct as the notion of proactive forwarding mode associated to a
switch is controlled by switch manager, through sm UI. The trigger to install the
proactive flows should come from switch manager, FRM should not save these internal
generated flows to its startup config, nor program them on its own.

CHANGES
1) On FRM config save, the internal generated static flows will not be added to the
list that will be actually saved.
2) In Switch Mgr, when the switch connects, while reading the respective switch
configuration (if any) the mode is learnt. If it is set to proactive, the mode change
event is being played.
3) When a switch disconnect, FRM has to remove the internal static flow configurations
for that switch.
4) Switch Mgr isSpecial method to correctly report special node conenctor (method should
be provided by NodeConnector class in future).
5) Adding the startup directory to the distribution to allow write/read of startup
configuration files.
6) Have FlowConfig class to expose a method to retrieve the configured forwarding mode,
so that internal coding is hidden to clients.

Signed-off-by: aboch <aboch@cisco.com>
Change-Id: Id628042b7d656d95ace87a1e72c13f2950e6795c
Signed-off-by: aboch <aboch@cisco.com>

Adding additional JUnit tests for SwitchManager
module

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

Merge "Refactoring SubnetConfig: -Use NetUtils IP validation methods instead of own regex and duplicate method -Remove unused method getFieldsNames()"

Refactoring SubnetConfig:
-Use NetUtils IP validation methods
instead of own regex and duplicate method
-Remove unused method getFieldsNames()

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

Merge "Ability to configure some system parameters on the controller"

Ability to configure some system parameters on the controller

This commit allows the controller to be brought up with the following parameters that can be specified in the config.ini file:
- of.listenPort=6633. TCP port on which the controller is listening (default 6633)
- of.messageResponseTimer=5000. The time (in milliseconds) the controller will wait for a response after sending a Barrier Request or a Statistic Request message (default 2000 msec)
If the paramater is not specified in the config.ini, the controller will use the default value.

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

JUnit for StatisticsNorthbound

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

JUnit tests for commons.northbound

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

Merge "JUnit Test for Topology Northbound"

JUnit Test for Topology Northbound

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

JUnit tests for flowprogrammer.northbound,
hosttracker.northbound, and subnets.northbound

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

JUnit test for forwarding.staticrouting.northbound

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

Merge "JUnit tests for V6Match Signed-off-by: Kalvin Hom <kahom@cisco.com>"

JUnit tests for V6Match
Signed-off-by: Kalvin Hom <kahom@cisco.com>
Change-Id: I246edb222d08c08181d26071544d4bfc349daf73
Signed-off-by: Kalvin Hom <kahom@cisco.com>