controller.git
8 years agoMerge "- Added the username/password criteria in usermanager - Relaxed the criteria...
Giovanni Meo [Fri, 19 Apr 2013 00:43:06 +0000 (00:43 +0000)]
Merge "- Added the username/password criteria in usermanager - Relaxed the criteria for nodename/gateway name in devices"

8 years agoMerge "Controller to listen to expired flow removal message - Added plugin-to-FM...
Jason Ye [Thu, 18 Apr 2013 23:01:30 +0000 (23:01 +0000)]
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>"

8 years agoController to listen to expired flow removal message 12/212/2
Alessandro Boch [Thu, 18 Apr 2013 01:48:44 +0000 (18:48 -0700)]
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>
8 years ago- Added the username/password criteria in usermanager 08/208/2
dibhatia [Thu, 18 Apr 2013 19:55:33 +0000 (12:55 -0700)]
- 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>
8 years agoMerge "Refactored YANG types resolving."
Giovanni Meo [Thu, 18 Apr 2013 05:37:51 +0000 (05:37 +0000)]
Merge "Refactored YANG types resolving."

8 years agoMerge "Improve Vendor Extension Error Handling. Signed-off-by: Madhavan Kasthuriranga...
Jason Ye [Wed, 17 Apr 2013 23:16:56 +0000 (23:16 +0000)]
Merge "Improve Vendor Extension Error Handling. Signed-off-by: Madhavan Kasthurirangan <mkasthur@cisco.com>"

8 years agoImprove Vendor Extension Error Handling. 06/206/1
Madhavan Kasthurirangan [Wed, 17 Apr 2013 19:15:39 +0000 (12:15 -0700)]
Improve Vendor Extension Error Handling.
Signed-off-by: Madhavan Kasthurirangan <mkasthur@cisco.com>
8 years agoRefactored YANG types resolving. 01/201/1
Martin Vitez [Tue, 16 Apr 2013 08:43:34 +0000 (10:43 +0200)]
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>
8 years agoMerge "Fixed bug in HostTracker.retrieveCache() where incorrect variable is being...
Giovanni Meo [Wed, 17 Apr 2013 06:10:11 +0000 (06:10 +0000)]
Merge "Fixed bug in HostTracker.retrieveCache() where incorrect variable is being checked for null."

8 years agoMerge "Added javadoc to maven-yang-plugin."
Giovanni Meo [Wed, 17 Apr 2013 05:55:47 +0000 (05:55 +0000)]
Merge "Added javadoc to maven-yang-plugin."

8 years agoMerge "Removed forgotten references to yang-to-sources test jar in maven-yang-plugin...
Giovanni Meo [Wed, 17 Apr 2013 05:47:16 +0000 (05:47 +0000)]
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)."

8 years ago- Added OSGI cmd to display TLS configuration 00/200/1
Jason Ye [Tue, 16 Apr 2013 21:29:52 +0000 (14:29 -0700)]
- Added OSGI cmd to display TLS configuration
- Added null pointer checks and more debug msgs
Signed-off-by: Jason Ye <yisye@cisco.com>
8 years agoFixed bug in HostTracker.retrieveCache() 99/199/1
Kalvin Hom [Tue, 16 Apr 2013 22:45:30 +0000 (15:45 -0700)]
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>
8 years agoMerge "Fix Logging for Topology Manager"
Jason Ye [Tue, 16 Apr 2013 21:05:50 +0000 (21:05 +0000)]
Merge "Fix Logging for Topology Manager"

8 years agoFix Logging for Topology Manager 97/197/1
Pramila Singh [Tue, 16 Apr 2013 20:54:43 +0000 (13:54 -0700)]
Fix Logging for Topology Manager

Signed-off-by: Pramila Singh <pramisin@cisco.com>
8 years agoFix comments in Path.java. Also, Max Throughput dijkstra modification. 96/196/1
Madhavan Kasthurirangan [Tue, 16 Apr 2013 19:47:07 +0000 (12:47 -0700)]
Fix comments in Path.java. Also, Max Throughput dijkstra modification.
Signed-off-by: Madhavan Kasthurirangan <mkasthur@cisco.com>
8 years agoAdded javadoc to maven-yang-plugin. 93/193/1
Maros Marsalek [Mon, 15 Apr 2013 12:12:42 +0000 (14:12 +0200)]
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>
8 years agoRemoved forgotten references to yang-to-sources test jar in maven-yang-plugin integra... 69/169/1
Maros Marsalek [Mon, 15 Apr 2013 10:23:34 +0000 (12:23 +0200)]
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>
8 years agoUpdate the API generation code and code generation sample 39/139/3
Lukas Sedlak [Thu, 11 Apr 2013 21:47:40 +0000 (14:47 -0700)]
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>
8 years agoYANG model parser refactoring 26/126/3
Martin Vitez [Mon, 8 Apr 2013 08:31:04 +0000 (10:31 +0200)]
YANG model parser refactoring
-fix YANG types constraints resolving
-added tests
-removed unused test files

Signed-off-by: Martin Vitez <mvitez@cisco.com>
8 years agoRenamed yang-to-sources-plugin to maven-yang-plugin. 34/134/2
Maros Marsalek [Thu, 11 Apr 2013 06:37:23 +0000 (08:37 +0200)]
Renamed yang-to-sources-plugin to maven-yang-plugin.
Updated all references in tests.

Signed-off-by: Maros Marsalek <mmarsale@cisco.com>
8 years agoMerge "Moving a recently added trim() call in SwitchHandler after Null check."
Alessandro Boch [Sat, 13 Apr 2013 01:44:15 +0000 (01:44 +0000)]
Merge "Moving a recently added trim() call in SwitchHandler after Null check."

8 years agoMoving a recently added trim() call in SwitchHandler after Null check. 47/147/1
Madhu Venugopal [Sat, 13 Apr 2013 01:36:35 +0000 (18:36 -0700)]
Moving a recently added trim() call in SwitchHandler after Null check.

Change-Id: Ib7f5787d612b0cca2b04383e684b6d91401ebb73
Signed-off-by: Madhu Venugopal <vmadhu@cisco.com>
8 years agoFix issue seen with enunciate plugin when importing in eclipse the 35/135/2
Giovanni Meo [Thu, 11 Apr 2013 17:27:58 +0000 (19:27 +0200)]
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>
8 years agoMerge "- Add stop() method in IMessageReadWrite for proper clean up when connection...
Alessandro Boch [Fri, 12 Apr 2013 23:25:23 +0000 (23:25 +0000)]
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>"

8 years ago- Add stop() method in IMessageReadWrite for proper clean up when connection is closed. 45/145/1
Jason Ye [Fri, 12 Apr 2013 23:08:25 +0000 (16:08 -0700)]
- 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>
8 years agoChanging NB API authentication/authorization to use Tomcat Valve. 36/136/2
Venkatraghavan Sankarasubbu [Fri, 12 Apr 2013 22:54:17 +0000 (15:54 -0700)]
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>
8 years agoISSUE: Topology does not show Node description as learned by the OFA 44/144/1
Alessandro Boch [Fri, 12 Apr 2013 20:40:59 +0000 (13:40 -0700)]
ISSUE: Topology does not show Node description as learned by the OFA

Signed-off-by: Alessandro Boch <aboch@cisco.com>
8 years agoFix stale cached links in visual topology 43/143/1
Andrew Kim [Fri, 12 Apr 2013 19:24:05 +0000 (12:24 -0700)]
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>
8 years agoFix disappearing nodes 41/141/1
Andrew Kim [Fri, 12 Apr 2013 19:08:02 +0000 (12:08 -0700)]
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>
8 years ago- If a port is already under OF control and then hw LLDP transmit is enabled on it... 40/140/1
Jason Ye [Thu, 11 Apr 2013 21:53:07 +0000 (14:53 -0700)]
- 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>
8 years agoEnable sal.core.Node to be an XML root element 38/138/1
Yevgeny Khodorkovsky [Thu, 11 Apr 2013 20:55:34 +0000 (13:55 -0700)]
Enable sal.core.Node to be an XML root element

Change-Id: Iec1d94864a33e7e175c5180d84c1f8b5c2ccc641
Signed-off-by: Yevgeny Khodorkovsky <ykhodork@cisco.com>
8 years agoAuth Principal to work with Remote authorization 37/137/1
Madhu Venugopal [Thu, 11 Apr 2013 19:06:05 +0000 (12:06 -0700)]
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>
8 years agoISSUE: After controller restart, proactive flows are not installed 32/132/2
Alessandro Boch [Wed, 10 Apr 2013 23:30:35 +0000 (16:30 -0700)]
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>
8 years agoMerge "fixed method name"
Giovanni Meo [Thu, 11 Apr 2013 17:07:53 +0000 (17:07 +0000)]
Merge "fixed method name"

8 years agoMerge "Call per page clean up if that is defined."
Madhu Venugopal [Thu, 11 Apr 2013 03:59:45 +0000 (03:59 +0000)]
Merge "Call per page clean up if that is defined."

8 years agofixed method name 33/133/1
Youcef Laribi [Thu, 11 Apr 2013 00:05:56 +0000 (17:05 -0700)]
fixed method name

Change-Id: I18ff595c6fe84eed59c2ef138b0e783ba00ba1f0
Signed-off-by: Youcef Laribi <Youcef.Laribi@citrix.com>
8 years agoSimple Load Balancer Application - Recommitting the source codewith the whole source... 30/130/1
Anilkumar Vishnoi [Wed, 10 Apr 2013 22:17:12 +0000 (03:47 +0530)]
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>
8 years agofixing spelling errors in comments 29/129/1
Colin Dixon [Wed, 10 Apr 2013 15:13:35 +0000 (10:13 -0500)]
fixing spelling errors in comments

Change-Id: Iecac10313c1cb7a88c5de3848b4506f591f88dca
Signed-off-by: Colin Dixon <ckd@us.ibm.com>
8 years agoCall per page clean up if that is defined. 28/128/1
lakshya [Wed, 10 Apr 2013 17:59:46 +0000 (10:59 -0700)]
Call per page clean up if that is defined.

Signed-off-by: lakshya <lakshya@cisco.com>
8 years agoBranding support for the newly added error page. 23/123/1
Madhu Venugopal [Sat, 6 Apr 2013 16:40:44 +0000 (09:40 -0700)]
Branding support for the newly added error page.

Change-Id: I49f0a4641bc7f6522006fa0794f6448997d0f9bd
Signed-off-by: Madhu Venugopal <vmadhu@cisco.com>
8 years agoReplacing Spring Security framework with Apache Tomcat Valve for Web Security (UI... 22/122/1
Venkatraghavan Sankarasubbu [Sat, 6 Apr 2013 14:19:52 +0000 (07:19 -0700)]
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>
8 years agoMerge "Web UI Branding support This fix provides an extensible branding support with...
Andrew Kim [Sat, 6 Apr 2013 01:45:23 +0000 (01:45 +0000)]
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."

8 years agoWeb UI Branding support 21/121/1
Madhu Venugopal [Sat, 6 Apr 2013 01:38:08 +0000 (18:38 -0700)]
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>
8 years agoMerge "Add TLS support in the Opendaylight Controller: - TLS configuration is specifi...
Alessandro Boch [Sat, 6 Apr 2013 00:50:18 +0000 (00:50 +0000)]
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,"

8 years agoAdding ability to customize onClick on topology graph. 20/120/1
Author Name [Fri, 5 Apr 2013 22:28:23 +0000 (15:28 -0700)]
Adding ability to customize onClick on topology graph.

Change-Id: Ie4d5abf02e9a704ec66ca5cd40633d09153e98c1
Signed-off-by: lakshya <lakshya@cisco.com>
8 years agoAdd TLS support in the Opendaylight Controller: 19/119/1
Jason Ye [Fri, 5 Apr 2013 23:58:47 +0000 (16:58 -0700)]
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>
8 years agoGerritt #107 introduced a fix that caused Routing Listener registration issue. 15/115/1
Madhu Venugopal [Fri, 5 Apr 2013 21:19:35 +0000 (14:19 -0700)]
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>
8 years agoFix for bug 7 14/114/1
Anilkumar Vishnoi [Fri, 5 Apr 2013 18:15:30 +0000 (23:45 +0530)]
Fix for bug 7

Change-Id: I62e5725e0712e07fcb2fe8c0425cfca0fa6da865
Signed-off-by: Anilkumar Vishnoi <avishnoi@in.ibm.com>
8 years agoMerge "Added skeleton for yang maven plugin."
Giovanni Meo [Fri, 5 Apr 2013 13:19:55 +0000 (13:19 +0000)]
Merge "Added skeleton for yang maven plugin."

8 years agoAdded skeleton for yang maven plugin. 11/111/2
Maros Marsalek [Fri, 5 Apr 2013 12:03:55 +0000 (14:03 +0200)]
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>
8 years agoAdded support for RPCs to Binding-Aware SAL 13/113/1
Tony Tkacik [Fri, 5 Apr 2013 12:59:46 +0000 (14:59 +0200)]
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>
8 years agoAdded support for binding-independent RPCs 12/112/1
Tony Tkacik [Fri, 5 Apr 2013 11:11:04 +0000 (13:11 +0200)]
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>
8 years agoMerge "ISSUE: Some changes to Authorization"
Andrew Kim [Fri, 5 Apr 2013 03:06:41 +0000 (03:06 +0000)]
Merge "ISSUE: Some changes to Authorization"

8 years agoISSUE: Some changes to Authorization 09/109/2
Alessandro Boch [Fri, 5 Apr 2013 02:43:34 +0000 (19:43 -0700)]
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>
8 years agoRenaming OneWeb to DaylightWeb 08/108/2
Colin Dixon [Thu, 4 Apr 2013 22:38:42 +0000 (17:38 -0500)]
Renaming OneWeb to DaylightWeb

Change-Id: Ic8f89bc249aadd81fec3416322122259ef51b89a
Signed-off-by: Colin Dixon <ckd@us.ibm.com>
8 years agoRemoving references to a prior branded controller 07/107/1
Colin Dixon [Thu, 4 Apr 2013 22:27:28 +0000 (17:27 -0500)]
Removing references to a prior branded controller

Change-Id: Ie5518714964fd1713ae64274228bf7269e32d728
Signed-off-by: Colin Dixon <ckd@us.ibm.com>
8 years agoISSUE: 06/106/1
Alessandro Boch [Thu, 4 Apr 2013 17:25:14 +0000 (10:25 -0700)]
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>
8 years agoRefactored YangModelParserImpl 05/105/1
mvitez [Thu, 4 Apr 2013 13:48:56 +0000 (15:48 +0200)]
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>
8 years agoFixed bug in naming conventions. 04/104/1
lsedlak [Thu, 4 Apr 2013 13:04:19 +0000 (15:04 +0200)]
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>
8 years agoFixed bugs in naming conventions of packages. 03/103/3
lsedlak [Thu, 4 Apr 2013 13:04:19 +0000 (15:04 +0200)]
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>
8 years agoMerge "Fix for messags at the boot up time This commit proposes following two sets...
Giovanni Meo [Thu, 4 Apr 2013 08:04:08 +0000 (08:04 +0000)]
Merge "Fix for messags at the boot up time This commit proposes following two sets of changes"

8 years agoMerge "Fixed YANG union type resolving in yang model parser."
Giovanni Meo [Thu, 4 Apr 2013 08:02:28 +0000 (08:02 +0000)]
Merge "Fixed YANG union type resolving in yang model parser."

8 years agoMerge "Added Google Guava as dependency. Replaced Map<K,List<V> for SAL listeners."
Giovanni Meo [Thu, 4 Apr 2013 07:57:15 +0000 (07:57 +0000)]
Merge "Added Google Guava as dependency. Replaced Map<K,List<V> for SAL listeners."

8 years agoLogging Related Enhancements. 02/102/1
Madhavan Kasthurirangan [Wed, 3 Apr 2013 21:31:31 +0000 (14:31 -0700)]
Logging Related Enhancements.
Signed-off-by: Madhavan Kasthurirangan <mkasthur@cisco.com>
8 years agoFix for messags at the boot up time 01/101/1
Maurice Qureshi [Wed, 3 Apr 2013 19:17:02 +0000 (12:17 -0700)]
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>
8 years agoConsistent URL Scheme for Northbound and Web UI 00/100/1
Madhu Venugopal [Wed, 3 Apr 2013 14:45:30 +0000 (07:45 -0700)]
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>
8 years agoFixed YANG union type resolving in yang model parser. 99/99/1
mvitez [Wed, 3 Apr 2013 13:28:38 +0000 (15:28 +0200)]
Fixed YANG union type resolving in yang model parser.

Signed-off-by: mvitez <mvitez@cisco.com>
8 years agoAdded Google Guava as dependency. Replaced Map<K,List<V> for SAL listeners. 98/98/1
Tony Tkacik [Wed, 3 Apr 2013 08:51:04 +0000 (10:51 +0200)]
Added Google Guava as dependency. Replaced Map<K,List<V> for SAL listeners.

Signed-off-by: Tony Tkacik <ttkacik@cisco.com>
8 years agoMerge "Partial fix for bug 8. Fixing many generics-related warnings and some typos."
Giovanni Meo [Wed, 3 Apr 2013 09:26:01 +0000 (09:26 +0000)]
Merge "Partial fix for bug 8. Fixing many generics-related warnings and some typos."

8 years agoMerge "ISSUE Opendaylight controller to get node description from OF description...
Madhu Venugopal [Tue, 2 Apr 2013 21:51:26 +0000 (21:51 +0000)]
Merge "ISSUE Opendaylight controller to get node description from OF description statistics datapath description field"

8 years agoISSUE 96/96/1
Alessandro Boch [Tue, 2 Apr 2013 21:05:14 +0000 (14:05 -0700)]
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>
8 years agoPartial fix for bug 8. Fixing many generics-related warnings and some typos. 95/95/1
Colin Dixon [Tue, 2 Apr 2013 20:31:19 +0000 (15:31 -0500)]
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>
8 years agoOpenDaylight uses Jersey framework to support JAX-RS compliant Northbound API. 94/94/2
Madhu Venugopal [Tue, 2 Apr 2013 11:31:43 +0000 (04:31 -0700)]
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>
8 years agoMerge "ISSUE Currently ForwardingRulesManager installs the proactive mode flows throu...
Jason Ye [Mon, 1 Apr 2013 22:42:08 +0000 (22:42 +0000)]
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."

8 years agoISSUE 93/93/1
aboch [Mon, 1 Apr 2013 20:36:56 +0000 (13:36 -0700)]
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>
8 years agoAdding additional JUnit tests for SwitchManager 92/92/1
Kalvin Hom [Sat, 30 Mar 2013 00:46:50 +0000 (17:46 -0700)]
Adding additional JUnit tests for SwitchManager
module

Change-Id: I9235a653d1740faf3a143b7fea48d80c91584124
Signed-off-by: Kalvin Hom <kahom@cisco.com>
8 years agoMerge "Refactoring SubnetConfig: -Use NetUtils IP validation methods instead of own...
Alessandro Boch [Fri, 29 Mar 2013 20:53:19 +0000 (20:53 +0000)]
Merge "Refactoring SubnetConfig: -Use NetUtils IP validation methods instead of own regex and duplicate method -Remove unused method getFieldsNames()"

8 years agoRefactoring SubnetConfig: 91/91/1
Kalvin Hom [Fri, 29 Mar 2013 20:40:31 +0000 (13:40 -0700)]
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>
8 years agoMerge "Ability to configure some system parameters on the controller"
Giovanni Meo [Fri, 29 Mar 2013 07:52:55 +0000 (07:52 +0000)]
Merge "Ability to configure some system parameters on the controller"

8 years agoAbility to configure some system parameters on the controller 90/90/1
Chi-Vien Ly [Fri, 29 Mar 2013 03:51:14 +0000 (20:51 -0700)]
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>
8 years agoJUnit for StatisticsNorthbound 81/81/3
Kalvin Hom [Thu, 28 Mar 2013 06:13:58 +0000 (23:13 -0700)]
JUnit for StatisticsNorthbound

Change-Id: I2b876ac1b2a22e6599edcb25e28b665f3f083500
Signed-off-by: Kalvin Hom <kahom@cisco.com>
8 years agoJUnit tests for commons.northbound 89/89/1
Kalvin Hom [Fri, 29 Mar 2013 00:28:44 +0000 (17:28 -0700)]
JUnit tests for commons.northbound

Change-Id: Ie450b2b8227809dac2dde5bed5c5b15dfc7cd260
Signed-off-by: Kalvin Hom <kahom@cisco.com>
8 years agoMerge "JUnit Test for Topology Northbound"
Alessandro Boch [Thu, 28 Mar 2013 23:01:30 +0000 (23:01 +0000)]
Merge "JUnit Test for Topology Northbound"

8 years agoJUnit Test for Topology Northbound 88/88/1
patrichu [Thu, 28 Mar 2013 22:05:29 +0000 (15:05 -0700)]
JUnit Test for Topology Northbound

Change-Id: I1b380cf5a54da2394660d7422fc2c28576b222f5
Signed-off-by: patrichu <patrichu@cisco.com>
8 years agoJUnit tests for flowprogrammer.northbound, 87/87/1
Kalvin Hom [Thu, 28 Mar 2013 20:59:05 +0000 (13:59 -0700)]
JUnit tests for flowprogrammer.northbound,
hosttracker.northbound, and subnets.northbound

Change-Id: I10644b8c08448492ff30e1d9f2773b95bd9916cc
Signed-off-by: Kalvin Hom <kahom@cisco.com>
8 years agoJUnit test for forwarding.staticrouting.northbound 86/86/1
Kalvin Hom [Thu, 28 Mar 2013 20:54:29 +0000 (13:54 -0700)]
JUnit test for forwarding.staticrouting.northbound

Change-Id: I8c69f7e74aabfac429977087804d5e7bc182bbbc
Signed-off-by: Kalvin Hom <kahom@cisco.com>
8 years agoMerge "JUnit tests for V6Match Signed-off-by: Kalvin Hom <kahom@cisco.com>"
Madhu Venugopal [Thu, 28 Mar 2013 18:04:01 +0000 (18:04 +0000)]
Merge "JUnit tests for V6Match Signed-off-by: Kalvin Hom <kahom@cisco.com>"

8 years agoJUnit tests for V6Match 80/80/3
Kalvin Hom [Thu, 28 Mar 2013 00:47:49 +0000 (17:47 -0700)]
JUnit tests for V6Match
Signed-off-by: Kalvin Hom <kahom@cisco.com>
Change-Id: I246edb222d08c08181d26071544d4bfc349daf73
Signed-off-by: Kalvin Hom <kahom@cisco.com>
8 years agoMerge "- Removed sitebuildsettings, the deployment of this and manteinance in openday...
Giovanni Meo [Thu, 28 Mar 2013 16:30:26 +0000 (16:30 +0000)]
Merge "- Removed sitebuildsettings, the deployment of this and manteinance in opendaylight infra is troublesome and also the settings are isolated that can be manually tweaked in the parent POM:  - opendaylight/commons/opendaylight/pom.xl  - third-party/commons/thirdparty - Adjusted the deployment of third-party snapshots to happen in opendaylight-snapshot nexus repo, given there is only deployment access to that on jenkins and also because there is not reason to distinguish it, anyway the groupID already avoid collision."

8 years ago- Removed sitebuildsettings, the deployment of this and manteinance in 85/85/1
Giovanni Meo [Thu, 28 Mar 2013 15:07:04 +0000 (16:07 +0100)]
- Removed sitebuildsettings, the deployment of this and manteinance in
opendaylight infra is troublesome and also the settings are isolated
that can be manually tweaked in the parent POM:
- opendaylight/commons/opendaylight/pom.xl
- third-party/commons/thirdparty
- Adjusted the deployment of third-party snapshots to happen in
opendaylight-snapshot nexus repo, given there is only deployment
access to that on jenkins and also because there is not reason to
distinguish it, anyway the groupID already avoid collision.

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

8 years agoAdded YANG typedef support in the YANG parser 84/84/2
Martin Vitez [Thu, 28 Mar 2013 13:19:31 +0000 (14:19 +0100)]
Added YANG typedef support in the YANG parser

Added javadoc to builders
Added JUnit test
Fixed hashCode and equals implementation (now only based on QName) for schema Node TOs.

Change-Id: I0d064391ac8de2c77258d3817ce4f7f50ea90350
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>
8 years agoUpdated YANG Type definitions and documentation in YANG Model Utils. 83/83/1
Lukas Sedlak [Thu, 28 Mar 2013 12:58:52 +0000 (13:58 +0100)]
Updated YANG Type definitions and documentation in YANG Model Utils.

Splitted Integer Type into Unsigned and Signed Integer Type.
Updated documentation.

Signed-off-by: Tony Tkacik <ttkacik@cisco.com>
8 years agoAdded javadoc and RFC references to Java YANG model. 82/82/1
Lukas Sedlak [Thu, 28 Mar 2013 12:09:04 +0000 (13:09 +0100)]
Added javadoc and RFC references to Java YANG model.

Signed-off-by: Tony Tkacik <ttkacik@cisco.com>
8 years agoMerge "simple JUnit test for switchmanager.northbound Signed-off-by: Kalvin Hom ...
Bhushan Kanekar [Thu, 28 Mar 2013 01:10:50 +0000 (01:10 +0000)]
Merge "simple JUnit test for switchmanager.northbound Signed-off-by: Kalvin Hom <kahom@cisco.com>"

8 years agoMerge "Issue in flow programming: Add a flow, uninstall it, try to remove it. Remove...
Madhu Venugopal [Wed, 27 Mar 2013 23:19:57 +0000 (23:19 +0000)]
Merge "Issue in flow programming:  Add a flow, uninstall it, try to remove it. Remove seems to fail from gui behavior.  Try to install the flow. It will fail and uncaught exception in forwardingrulesmanager is printed. Fix:  Handle invalid flow status toggle request in forwardingrulesmanager.  IForwardingrulesmanager to exposed a per node and per flow name api for toggling the flow status.  Flow.web to evaluate api return status by status code and not by status description,  because a Status with StatusCode.SUCCESS does not imply the description is "Success"."

8 years agosimple JUnit test for switchmanager.northbound 78/78/1
Kalvin Hom [Wed, 27 Mar 2013 21:58:36 +0000 (14:58 -0700)]
simple JUnit test for switchmanager.northbound
Signed-off-by: Kalvin Hom <kahom@cisco.com>
8 years agoThis commit fixes the followings 77/77/2
Chi-Vien Ly [Wed, 27 Mar 2013 18:39:20 +0000 (11:39 -0700)]
This commit fixes the followings
- Change routing.dijkstra_implementation to implement ITopologyManagerAware instead of IListenTopoUpdates
- Correct the bundle bringup sequence using the init(), start() called by dependency manager
- Add routing.dijkstra_implementation restartability through OSGI stop/start

Change-Id: Ia624ab4ea51f4d35c5ffe31d14b1aed58ca123c3
Signed-off-by: Chi-Vien Ly <chivly@cisco.com>
8 years agoIssue in flow programming: 76/76/1
aboch [Wed, 27 Mar 2013 16:57:49 +0000 (09:57 -0700)]
Issue in flow programming:
Add a flow, uninstall it, try to remove it. Remove seems to fail from gui behavior.
Try to install the flow. It will fail and uncaught exception in forwardingrulesmanager is printed.
Fix:
Handle invalid flow status toggle request in forwardingrulesmanager.
IForwardingrulesmanager to exposed a per node and per flow name api for toggling the flow status.
Flow.web to evaluate api return status by status code and not by status description,
because a Status with StatusCode.SUCCESS does not imply the description is "Success".

Change-Id: I33567b473828721bd6548ea222f4b7bcf83c0306
Signed-off-by: aboch <aboch@cisco.com>
8 years agoResolving http://bugs.opendaylight.org/show_bug.cgi?id=4 75/75/1
Chi-Vien Ly [Tue, 26 Mar 2013 17:43:28 +0000 (10:43 -0700)]
Resolving bugs.opendaylight.org/show_bug.cgi?id=4

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