ovsdb.git
9 years agoFirst step in unifying datachange events 70/17870/3
Ed Warnicke [Tue, 7 Apr 2015 18:39:49 +0000 (11:39 -0700)]
First step in unifying datachange events

What works:
OVSDB connect/disconnect
Bridge Create/Delete
TerminationPoint Create

Change-Id: I15cf4a5fc3b6d3d514575c035d44675b2737afe7
Signed-off-by: Ed Warnicke <eaw@cisco.com>
9 years agoOpenFlow port values are uint32, correcting the size in the model for that 03/17803/2
Ed Warnicke [Mon, 6 Apr 2015 18:07:14 +0000 (11:07 -0700)]
OpenFlow port values are uint32, correcting the size in the model for that

Change-Id: I1c810137d5c2f103b2679a4aa15cf32fa38e019e
Signed-off-by: Ed Warnicke <eaw@cisco.com>
9 years agoMerge "Small fix to handle the fact that ovsdb sometimes returns ofport of -1"
Flavio Fernandes [Mon, 6 Apr 2015 17:30:44 +0000 (17:30 +0000)]
Merge "Small fix to handle the fact that ovsdb sometimes returns ofport of -1"

9 years agoMerge "Check for trunks (if not present, don't throw an NPE ;) )"
Flavio Fernandes [Mon, 6 Apr 2015 17:21:32 +0000 (17:21 +0000)]
Merge "Check for trunks (if not present, don't throw an NPE ;) )"

9 years agoSmall fix to handle the fact that ovsdb sometimes returns ofport of -1 94/17794/2
Ed Warnicke [Mon, 6 Apr 2015 16:30:41 +0000 (09:30 -0700)]
Small fix to handle the fact that ovsdb sometimes returns ofport of -1

It turns out that sometimes ovsdb returns an ofport of -1 to indicate that
there is no associated ofport value.  We should represent that by *not*
having an ofport value

Change-Id: I2a28a34333ff1b92728286672499714079346444
Signed-off-by: Ed Warnicke <eaw@cisco.com>
9 years agoCheck for trunks (if not present, don't throw an NPE ;) ) 98/17798/1
Ed Warnicke [Mon, 6 Apr 2015 16:58:34 +0000 (09:58 -0700)]
Check for trunks (if not present, don't throw an NPE ;) )

Change-Id: I9198ea669823e6ea36dc34fffc95269feedccf76
Signed-off-by: Ed Warnicke <eaw@cisco.com>
9 years agoModified postman collection to test vlan-tag and trunks. 70/17670/3
Gabriel Robitaille-Montpetit [Mon, 6 Apr 2015 16:11:39 +0000 (12:11 -0400)]
Modified postman collection to test vlan-tag and trunks.

Change-Id: I54c089fc8c69bcf4d88d57f97a971b5538696a39
Signed-off-by: Gabriel Robitaille-Montpetit <grmontpetit@inocybe.com>
9 years agoMerge "Fixed if/else/for/while/do curly braces for LBaaSPoolMemberHandler.java"
Flavio Fernandes [Mon, 6 Apr 2015 15:45:31 +0000 (15:45 +0000)]
Merge "Fixed if/else/for/while/do curly braces for LBaaSPoolMemberHandler.java"

9 years agoMerge changes I99c0e84f,I0e035323
Flavio Fernandes [Mon, 6 Apr 2015 15:44:50 +0000 (15:44 +0000)]
Merge changes I99c0e84f,I0e035323

* changes:
  Add sonar for southbound
  Add copyright's to southbound-it files.

9 years agoAdded trunks code using Long 66/17666/8
Gabriel Robitaille-Montpetit [Thu, 2 Apr 2015 18:05:29 +0000 (14:05 -0400)]
Added trunks code using Long

Change-Id: Ib2cb185d3cc87ee34cbcf66dabd008c45001b52f
Signed-off-by: Gabriel Robitaille-Montpetit <grmontpetit@inocybe.com>
9 years agoAdd sonar for southbound 89/17789/1
Sam Hague [Mon, 6 Apr 2015 13:48:03 +0000 (09:48 -0400)]
Add sonar for southbound

Change-Id: I99c0e84f1b52e7d295f485b954160d1593d72c7d
Signed-off-by: Sam Hague <shague@redhat.com>
9 years agoFixed if/else/for/while/do curly braces for LBaaSPoolMemberHandler.java 76/17676/2
Leo Cavalcanti [Thu, 2 Apr 2015 22:37:41 +0000 (19:37 -0300)]
Fixed if/else/for/while/do curly braces for LBaaSPoolMemberHandler.java

This is also a try out for me just to make sure my Gerrit is ok.

Change-Id: I9efbc0a8cd2157d15b3f06ab2d0888c692d494c4
Signed-off-by: Leo Cavalcanti <leozinho@gmail.com>
9 years agoAdd copyright's to southbound-it files. 85/17785/2
Sam Hague [Mon, 6 Apr 2015 09:36:18 +0000 (05:36 -0400)]
Add copyright's to southbound-it files.

Change-Id: I0e0353233b068873bd1ba983eb11c7efde83bc1d
Signed-off-by: Sam Hague <shague@redhat.com>
9 years agoMerge "Revert "Updated ovsdb.yang to include external-ids for bridges.""
Sam Hague [Mon, 6 Apr 2015 10:39:06 +0000 (10:39 +0000)]
Merge "Revert "Updated ovsdb.yang to include external-ids for bridges.""

9 years agoRevert "Updated ovsdb.yang to include external-ids for bridges." 88/17788/1
Sam Hague [Mon, 6 Apr 2015 10:38:46 +0000 (10:38 +0000)]
Revert "Updated ovsdb.yang to include external-ids for bridges."

This reverts commit e19e24bee91dca1e3d03420977e84e81d5240c4f.

Change-Id: Ibec4b568044929967dcc725aaa5dfa547b8e3b0e
Signed-off-by: Sharad Mishra <sharad.d.mishra@intel.com>
Signed-off-by: Sam Hague <shague@redhat.com>
9 years agoMerge "Updated ovsdb.yang to include external-ids for bridges."
Sam Hague [Sat, 4 Apr 2015 01:20:04 +0000 (01:20 +0000)]
Merge "Updated ovsdb.yang to include external-ids for bridges."

9 years agoMerge "support for Interface external_ids"
Sam Hague [Sat, 4 Apr 2015 00:36:31 +0000 (00:36 +0000)]
Merge "support for Interface external_ids"

9 years agosupport for Interface external_ids 18/17718/7
Ryan Goulding [Fri, 3 Apr 2015 16:38:09 +0000 (12:38 -0400)]
support for Interface external_ids

Adds support for Interface external_ids.  "external_ids" are configured on
Termination Point creation.  "external_ids" are persisted to the Operational
data store on Ovsdb Port update.  If external-id-value is not provided, the
port is still created as YANG has no way to enforce the field as mandatory.

Change-Id: Ibdac48f8d5144ddefd79278d5587c5a63bca0869
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
9 years agoUpdated ovsdb.yang to include external-ids for bridges. 43/17743/1
Sharad Mishra [Fri, 3 Apr 2015 16:54:04 +0000 (09:54 -0700)]
Updated ovsdb.yang to include external-ids for bridges.

Change-Id: Ia639165fe05be5f4e1f236893780b177465b0d9a
Signed-off-by: Sharad Mishra <sharad.d.mishra@intel.com>
9 years agoAdd external-ids and other-config to the OVSDB node model 30/17730/2
Eric Multanen [Tue, 31 Mar 2015 11:25:49 +0000 (04:25 -0700)]
Add external-ids and other-config to the OVSDB node model

Add external-ids and other-config to the OVSDB node
for capturing information in the Open_vSwitch table.

Added other-config as a grouping so it can be used
by other tables which need other-config.

v2 - move the external-ids and other-config to the
     ovsdb node (not the bridge node - for another patch)

Change-Id: I551f2a9e786460941cae23d1c3e3862c11d1bb07
Signed-off-by: Eric Multanen <eric.w.multanen@intel.com>
9 years agoMerge "net-virt-providers: install flow twice unnecessarily"
Sam Hague [Fri, 3 Apr 2015 00:50:05 +0000 (00:50 +0000)]
Merge "net-virt-providers: install flow twice unnecessarily"

9 years agonet-virt-providers: install flow twice unnecessarily 78/17678/1
Isaku Yamahata [Thu, 2 Apr 2015 23:56:41 +0000 (16:56 -0700)]
net-virt-providers: install flow twice unnecessarily

some net-virt-providers installs flows twice unnecessarily.
It seems like copy-n-paste bug.
Remove unnecessary writeFlow() call.

Change-Id: Iaa73fac7eb09c45f598634257210a293c10f40b6
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
9 years agoMerge "Add JUnit testing for FloatingIPHandler class."
Flavio Fernandes [Thu, 2 Apr 2015 23:36:15 +0000 (23:36 +0000)]
Merge "Add JUnit testing for FloatingIPHandler class."

9 years agoMerge "Add IT for southbound"
Flavio Fernandes [Thu, 2 Apr 2015 23:35:30 +0000 (23:35 +0000)]
Merge "Add IT for southbound"

9 years agoMerge "ofport and ofport_request operational data store persistence"
Sam Hague [Thu, 2 Apr 2015 21:55:51 +0000 (21:55 +0000)]
Merge "ofport and ofport_request operational data store persistence"

9 years agoofport and ofport_request operational data store persistence 67/17467/7
Ryan Goulding [Thu, 2 Apr 2015 16:37:35 +0000 (12:37 -0400)]
ofport and ofport_request operational data store persistence

Persists ofport and ofport_request fields in the operational data store.  The
yang model utilizes uint16 for these fields, while the library code utilizes
Long for consistency reasons.

Change-Id: Ib9390d19736cd1aba5710161ee477799dc5ba466
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
9 years agoAdd IT for southbound 99/17599/2
Sam Hague [Thu, 2 Apr 2015 02:25:40 +0000 (22:25 -0400)]
Add IT for southbound

Change-Id: Iff82a08c08cae8013417031f48f32d6639b94b8c
Signed-off-by: Sam Hague <shague@redhat.com>
9 years agoAdd JUnit testing for FloatingIPHandler class. 73/17673/1
Alexis de Talhouët [Thu, 2 Apr 2015 20:40:42 +0000 (16:40 -0400)]
Add JUnit testing for FloatingIPHandler class.

Change-Id: I1a5dc5fb7c7bc9b4a856925bb0ff731bf0d48e8b
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
9 years agoMerge "Add JUnit tesing for NeutronL3Adapter class."
Flavio Fernandes [Thu, 2 Apr 2015 18:51:05 +0000 (18:51 +0000)]
Merge "Add JUnit tesing for NeutronL3Adapter class."

9 years agoMerge "Add JUnit testing for AbstractHandler class."
Flavio Fernandes [Thu, 2 Apr 2015 18:50:23 +0000 (18:50 +0000)]
Merge "Add JUnit testing for AbstractHandler class."

9 years agoMerge "Add JUnit testing for AbstractEvent class."
Flavio Fernandes [Thu, 2 Apr 2015 18:49:59 +0000 (18:49 +0000)]
Merge "Add JUnit testing for AbstractEvent class."

9 years agoMerge "Revert "Add break in iteration over vlan tags for a given ovsdbTerminationPoint""
Flavio Fernandes [Thu, 2 Apr 2015 18:01:10 +0000 (18:01 +0000)]
Merge "Revert "Add break in iteration over vlan tags for a given ovsdbTerminationPoint""

9 years agoRevert "Add break in iteration over vlan tags for a given ovsdbTerminationPoint" 53/17653/1
Flavio Fernandes [Thu, 2 Apr 2015 15:28:06 +0000 (15:28 +0000)]
Revert "Add break in iteration over vlan tags for a given ovsdbTerminationPoint"

This reverts commit 38c634b2cada040ccd98d91c11d21326a4eba197.

The break is actually out of place; removing it.

Ref: https://gist.github.com/3d1dab38f9dde757fe83

Change-Id: If0d91bd762450d79d9c897947ba1fe48bbb854df
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
9 years agoAdd JUnit tesing for NeutronL3Adapter class. 49/17649/2
Alexis de Talhouët [Thu, 2 Apr 2015 15:21:56 +0000 (11:21 -0400)]
Add JUnit tesing for NeutronL3Adapter class.

Change-Id: Icb54db19c31a222ad7a4c8e8a4a5de82d28471f4
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
9 years agoAdd JUnit testing for AbstractHandler class. 45/17645/1
Alexis de Talhouët [Thu, 2 Apr 2015 14:52:39 +0000 (10:52 -0400)]
Add JUnit testing for AbstractHandler class.

Change-Id: If172dff7ad5d98a6cabdb78a6c4067c3327e1f10
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
9 years agoAdd JUnit testing for AbstractEvent class. 37/17637/1
Alexis de Talhouët [Thu, 2 Apr 2015 14:03:33 +0000 (10:03 -0400)]
Add JUnit testing for AbstractEvent class.

Change-Id: Iebc184f92e724887ed6e796cd0f0beefbd467352
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
9 years agoMerge "Model update to add external-id to port/interface."
Sam Hague [Thu, 2 Apr 2015 00:15:18 +0000 (00:15 +0000)]
Merge "Model update to add external-id to port/interface."

9 years agoMerge "Add break in iteration over vlan tags for a given ovsdbTerminationPoint"
Sam Hague [Thu, 2 Apr 2015 00:13:59 +0000 (00:13 +0000)]
Merge "Add break in iteration over vlan tags for a given ovsdbTerminationPoint"

9 years agoFix: bug wherein the bridge with name having a - cannot be created. e.g. br-int canno... 94/17594/1
Prateek Garg [Wed, 1 Apr 2015 23:31:27 +0000 (16:31 -0700)]
Fix: bug wherein the bridge with name having a - cannot be created. e.g. br-int cannot be created

Change-Id: I620f8255100fdfcd9cef57a926280c58520d76fa
Signed-off-by: Prateek Garg <prategar@cisco.com>
9 years agoAdd break in iteration over vlan tags for a given ovsdbTerminationPoint 88/17588/1
Flavio Fernandes [Wed, 1 Apr 2015 22:03:41 +0000 (18:03 -0400)]
Add break in iteration over vlan tags for a given ovsdbTerminationPoint

Since we expect no more than a single vlan tag for a given termination point,
there is no need to call setVlanTag multiple times on termination point builder.
Added a TODO to mark the fact that the iteration/builder will need to be enhanced
if more than a single vlan tag support is needed.

Change-Id: I1ff45ccea8fb2b061157ca7b9af0ad2e072abdcb
Ref: https://gist.github.com/1e07ddc7e663710c2ac7
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
9 years agoadding Interface type for dpdkvhostuser interfaces in yang and southboundConstants 77/17577/1
Praveen Mala [Wed, 1 Apr 2015 18:30:09 +0000 (14:30 -0400)]
adding Interface type for dpdkvhostuser interfaces in yang and southboundConstants

Change-Id: I5731264de24fca852d282b60719f0f4bf1924a8d
Signed-off-by: Praveen Mala <praveen.mala@intel.com>
9 years agoFix bug when a bridge is removed. 70/17570/1
Ed Warnicke [Wed, 1 Apr 2015 17:05:10 +0000 (13:05 -0400)]
Fix bug when a bridge is removed.

The external_ids opendaylight-iid was not being used
for bridge remove.  Now it is.

Change-Id: I2cde45a07f255bbc5f1efb7d2c3b1bd2b63f98f5
Signed-off-by: Ed Warnicke <eaw@cisco.com>
9 years agoFix for case when datapath/iface types are not supported 67/17567/1
Ed Warnicke [Wed, 1 Apr 2015 16:46:01 +0000 (12:46 -0400)]
Fix for case when datapath/iface types are not supported

If we attempt to get datapath/iface types from ovs
versions prior to their support, an exception is thrown,
this patch handles it correctly.

Change-Id: I8315bf3a2f250c412cdf51c385754be409dc2257
Signed-off-by: Ed Warnicke <eaw@cisco.com>
9 years agoMerge "Revert "Added trunks code to go with the model.""
Sam Hague [Wed, 1 Apr 2015 16:06:28 +0000 (16:06 +0000)]
Merge "Revert "Added trunks code to go with the model.""

9 years agoRevert "Added trunks code to go with the model." 62/17562/1
Gabriel Robitaille-Montpetit [Wed, 1 Apr 2015 15:54:43 +0000 (15:54 +0000)]
Revert "Added trunks code to go with the model."

This reverts commit 436e0cbe741db71cd645bda565a7a83a8f722cad.

Change-Id: If9ba1d0c7ef7a30d6d0aef3575e240f52a180780
Signed-off-by: Gabriel Robitaille-Montpetit <grmontpetit@inocybe.com>
9 years agoModel update to add external-id to port/interface. 79/17579/3
Sharad Mishra [Wed, 1 Apr 2015 10:52:34 +0000 (03:52 -0700)]
Model update to add external-id to port/interface.

Change-Id: I099111e7ef09428516473ad4fb998d88835e321c
Signed-off-by: Sharad Mishra <sharad.d.mishra@intel.com>
9 years agoMerge "Added trunks code to go with the model."
Sam Hague [Wed, 1 Apr 2015 10:00:08 +0000 (10:00 +0000)]
Merge "Added trunks code to go with the model."

9 years agoRevert: Revert "Add JUnit testing for ConfigurationServiceImpl class." 08/17508/1
Flavio Fernandes [Tue, 31 Mar 2015 21:08:13 +0000 (17:08 -0400)]
Revert: Revert "Add JUnit testing for ConfigurationServiceImpl class."

This commit re-introduces commit eb362a3e296c10ad8ac7b5cb626cd2466f4d558b.

The issue was a minor tweak missing in pom.xml, so that powermock would
stop generating npe.

Change-Id: I4b3b7aa6fa8a597a3d0d84e648e2c0dd21038c68
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
Also-by: Alexis de Talhouët <adetalhouet@inocybe.com>
9 years agoRevert "Add JUnit testing for ConfigurationServiceImpl class." 00/17500/2
Flavio Fernandes [Tue, 31 Mar 2015 19:54:34 +0000 (19:54 +0000)]
Revert "Add JUnit testing for ConfigurationServiceImpl class."

This reverts commit eb362a3e296c10ad8ac7b5cb626cd2466f4d558b.

This is to stop regression introduced, where hit the following failure:

https://jenkins.opendaylight.org/releng/view/ovsdb/job/ovsdb-daily-full-integration/OVS_VERSION=1.9.0,jdk=openjdk7,nodes=dynamic_docker/23/consoleFull

apiTests(org.opendaylight.ovsdb.integrationtest.plugin.OvsdbPluginIT)
Tests run: 3, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 53.545 sec <<< FAILURE! - in org.opendaylight.ovsdb.integrationtest.plugin.OvsdbPluginIT
testInventoryListeners(org.opendaylight.ovsdb.integrationtest.plugin.OvsdbPluginIT)  Time elapsed: 10.368 sec  <<< ERROR!
java.lang.NullPointerException: null
at org.mockito.internal.exceptions.stacktrace.StackTraceFilter.<clinit>(StackTraceFilter.java:21)
at org.mockito.internal.exceptions.stacktrace.ConditionalStackTraceFilter.<init>(ConditionalStackTraceFilter.java:17)
at org.mockito.exceptions.base.MockitoException.filterStackTrace(MockitoException.java:30)
at org.mockito.exceptions.base.MockitoException.<init>(MockitoException.java:19)
at org.mockito.exceptions.misusing.MockitoConfigurationException.<init>(MockitoConfigurationException.java:18)
at org.mockito.internal.configuration.ClassPathLoader.loadImplementations(ClassPathLoader.java:145)
at org.mockito.internal.configuration.ClassPathLoader.findPluginImplementation(ClassPathLoader.java:110)
at org.mockito.internal.configuration.ClassPathLoader.findPlatformMockMaker(ClassPathLoader.java:106)
at org.mockito.internal.configuration.ClassPathLoader.<clinit>(ClassPathLoader.java:59)
at org.mockito.internal.util.MockUtil.<clinit>(MockUtil.java:21)
at org.mockito.internal.MockitoCore.<init>(MockitoCore.java:40)
at org.mockito.Mockito.<clinit>(Mockito.java:932)
at org.opendaylight.ovsdb.integrationtest.plugin.OvsdbPluginIT.testInventoryListeners(OvsdbPluginIT.java:171)

Change-Id: I13d0c3193e297eb9b825bd657478955fc7876733
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
9 years agoAdded trunks code to go with the model. 75/17375/8
Gabriel Robitaille-Montpetit [Mon, 30 Mar 2015 18:01:04 +0000 (14:01 -0400)]
Added trunks code to go with the model.

Change-Id: I514f8d8ad7d786901061b14909735cee9bb412f6
Signed-off-by: Gabriel Robitaille-Montpetit <grmontpetit@inocybe.com>
9 years agoAdd <relativePath/> to ensure Maven pulls artifact from Nexus 69/17469/1
Thanh Ha [Tue, 31 Mar 2015 15:48:37 +0000 (11:48 -0400)]
Add <relativePath/> to ensure Maven pulls artifact from Nexus

Needed by autorelease to be able to find and replace this value
prebuild.

Change-Id: Ifad9a6c514c2aa47e52d18ebbfeb1bf71b121496
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
9 years agoSmall fix to insure correct running offline. 44/17444/2
Ed Warnicke [Tue, 31 Mar 2015 14:03:37 +0000 (10:03 -0400)]
Small fix to insure correct running offline.

The southbound-impl features depend on the config file, not
just on the southbound-impl bundle.

<groupId>org.opendaylight.ovsdb</groupId>
<artifiactId>southbound-impl</artifactId>

is the *bundle* dependency.

<groupId>org.opendaylight.ovsdb</groupId>
<artifiactId>southbound-impl</artifactId>
<type>xml</type>
<classifier>config</classifier>

is a distinct artifact for the config file.

By failing to depend on the config file, the
southbound-impl feature will not work correctly in
offline mode.

Change-Id: Iab4ba7225f30aac19b45267da33c3a6219488b4a
Signed-off-by: Ed Warnicke <eaw@cisco.com>
9 years agoMerge "Add datapath types to vswitch."
Flavio Fernandes [Tue, 31 Mar 2015 13:06:20 +0000 (13:06 +0000)]
Merge "Add datapath types to vswitch."

9 years agoMerge "Add multiple options to port via MDSAL"
Flavio Fernandes [Tue, 31 Mar 2015 13:03:52 +0000 (13:03 +0000)]
Merge "Add multiple options to port via MDSAL"

9 years agoAdd multiple options to port via MDSAL 01/17401/1
Eric Multanen [Mon, 30 Mar 2015 23:29:15 +0000 (16:29 -0700)]
Add multiple options to port via MDSAL

Only one option was getting added to the port when
multiple options were supplied in the MDSAL config
operation.  This patch adds all of them to the
port.

Change-Id: Iee139510d6d396c410641130b5aeee1fedca4c72
Signed-off-by: Eric Multanen <eric.w.multanen@intel.com>
9 years agoAllow port to be added via MDSAL without VLAN tag 00/17400/1
Eric Multanen [Mon, 30 Mar 2015 21:44:26 +0000 (14:44 -0700)]
Allow port to be added via MDSAL without VLAN tag

Check if VLAN tag was provided before trying to add
it when creating a port to prevent a null pointer exception.

Change-Id: Iac0582ee5e151e446383130d108ef8c0e182f34d
Signed-off-by: Eric Multanen <eric.w.multanen@intel.com>
9 years agoUse dependencyManagement imports 74/17374/2
Robert Varga [Mon, 30 Mar 2015 18:00:54 +0000 (20:00 +0200)]
Use dependencyManagement imports

Some upstream projects already provide artifacts, so use them to
simplify poms.

Change-Id: I58a45b3bcd8301383fd6c7a6a4bdea2e3e7838f4
Signed-off-by: Robert Varga <rovarga@cisco.com>
9 years agoMerge "Remove port/interface from operational datastore."
Flavio Fernandes [Mon, 30 Mar 2015 21:24:21 +0000 (21:24 +0000)]
Merge "Remove port/interface from operational datastore."

9 years agoMerge "Enabled the use of opendaylight-iid as external_ids for bridges"
Flavio Fernandes [Mon, 30 Mar 2015 21:21:38 +0000 (21:21 +0000)]
Merge "Enabled the use of opendaylight-iid as external_ids for bridges"

9 years agoMerge "Add ability to serialize/deserialize iids to/from Strings"
Sam Hague [Mon, 30 Mar 2015 21:07:01 +0000 (21:07 +0000)]
Merge "Add ability to serialize/deserialize iids to/from Strings"

9 years agoEnabled the use of opendaylight-iid as external_ids for bridges 82/17382/2
Ed Warnicke [Mon, 30 Mar 2015 17:59:13 +0000 (13:59 -0400)]
Enabled the use of opendaylight-iid as external_ids for bridges

Change-Id: Ie95bedab4fc0bc367e0581a64c1084c807bd390c
Signed-off-by: Ed Warnicke <eaw@cisco.com>
9 years agoAdd ability to serialize/deserialize iids to/from Strings 81/17381/2
Ed Warnicke [Mon, 30 Mar 2015 17:43:22 +0000 (13:43 -0400)]
Add ability to serialize/deserialize iids to/from Strings

Change-Id: I17df9cb64f53f7084bf439b4d6ae2203af8a2e19
Signed-off-by: Ed Warnicke <eaw@cisco.com>
9 years agoMerge "Add JUnit testing for ConfigurationServiceImpl class."
Sam Hague [Mon, 30 Mar 2015 20:25:41 +0000 (20:25 +0000)]
Merge "Add JUnit testing for ConfigurationServiceImpl class."

9 years agoAdded code for vlan-tags on termination points. 56/17256/6
Gabriel Robitaille-Montpetit [Fri, 27 Mar 2015 17:44:09 +0000 (13:44 -0400)]
Added code for vlan-tags on termination points.

Change-Id: Id57d5b157e505a1b6a73cb8b32022be0d64e4d2b
Signed-off-by: Gabriel Robitaille-Montpetit <grmontpetit@inocybe.com>
9 years agoAdd JUnit testing for ConfigurationServiceImpl class. 37/17237/6
Alexis de Talhouët [Fri, 27 Mar 2015 14:09:53 +0000 (10:09 -0400)]
Add JUnit testing for ConfigurationServiceImpl class.

I had to add dependency to PowerMockito in the pom file in order to mock final class and static method.

Change-Id: Ib5de1a79acc78b4c42e27cbcbbc9e137fa963b46
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
9 years agoMerge "Fail build on checkstyle errors"
Flavio Fernandes [Mon, 30 Mar 2015 17:08:44 +0000 (17:08 +0000)]
Merge "Fail build on checkstyle errors"

9 years agoMerge "Added trunk model."
Sam Hague [Mon, 30 Mar 2015 16:32:07 +0000 (16:32 +0000)]
Merge "Added trunk model."

9 years agoAdd datapath types to vswitch. 92/17392/1
Sharad Mishra [Mon, 30 Mar 2015 15:24:45 +0000 (08:24 -0700)]
Add datapath types to vswitch.

Change-Id: If19b5d42361b1af291d79ec3f83920a97a16981e
Signed-off-by: Sharad Mishra <sharad.d.mishra@intel.com>
9 years agoFail build on checkstyle errors 68/17368/1
Sam Hague [Mon, 30 Mar 2015 15:59:13 +0000 (11:59 -0400)]
Fail build on checkstyle errors

Change-Id: Ib4015597e55291754278e56810f6676f408fd03f
Signed-off-by: Sam Hague <shague@redhat.com>
9 years agoRemove port/interface from operational datastore. 80/17380/1
Sharad Mishra [Mon, 30 Mar 2015 11:05:41 +0000 (04:05 -0700)]
Remove port/interface from operational datastore.

Change-Id: Id28b6129de6879d3e9b2b141ae801ba0741bdb7e
Signed-off-by: Sharad Mishra <sharad.d.mishra@intel.com>
9 years agoRemove uneeded version property from distribution pom 14/17314/1
Flavio Fernandes [Sun, 29 Mar 2015 23:20:48 +0000 (19:20 -0400)]
Remove uneeded version property from distribution pom

Change-Id: I0e7732a2e3d1fe63c59ecf38fe4dc3274e244c10
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
9 years agoUpdates due to using latest checkstyle 84/17284/1
Sam Hague [Sat, 28 Mar 2015 14:34:53 +0000 (10:34 -0400)]
Updates due to using latest checkstyle

Change-Id: I5e38107256902e29bb07d88e08dac7e72d1745b0
Signed-off-by: Sam Hague <shague@redhat.com>
9 years agoUse latest checktyle version 83/17283/1
Sam Hague [Sat, 28 Mar 2015 14:33:44 +0000 (10:33 -0400)]
Use latest checktyle version

Some generics warnings were fixed in the latest version.

Change-Id: I57520444a183152e0b90b1b287915730814d5c39
Signed-off-by: Sam Hague <shague@redhat.com>
9 years agoUpdate libary with checkstyle corrections 68/17268/1
Sam Hague [Fri, 27 Mar 2015 23:45:49 +0000 (19:45 -0400)]
Update libary with checkstyle corrections

Change-Id: I4408dd37290495a2df949ed538b9bf53090415a8
Signed-off-by: Sam Hague <shague@redhat.com>
9 years agoUse local checkstyle rules to allow incremental corrections 67/17267/1
Sam Hague [Fri, 27 Mar 2015 23:44:23 +0000 (19:44 -0400)]
Use local checkstyle rules to allow incremental corrections

Change-Id: I03481afe0f224b6133bdb0b3b6e4beca1f2d3c0e
Signed-off-by: Sam Hague <shague@redhat.com>
9 years agoAdded trunk model. 64/17264/2
Gabriel Robitaille-Montpetit [Fri, 27 Mar 2015 21:24:43 +0000 (17:24 -0400)]
Added trunk model.

Change-Id: I81270948e49f291c5d8927a64c3b5466cd9d02c5
Signed-off-by: Gabriel Robitaille-Montpetit <grmontpetit@inocybe.com>
9 years agoUpdate doc of class EventDispatcherImplTest. 58/17258/1
Alexis de Talhouët [Fri, 27 Mar 2015 18:19:25 +0000 (14:19 -0400)]
Update doc of class EventDispatcherImplTest.

Change-Id: Ib781c519f2be1e99764de1358c0ab087f77cb36b
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
9 years agoMerge "Added vlan-tag to port-interface (wip)."
Sam Hague [Fri, 27 Mar 2015 17:36:24 +0000 (17:36 +0000)]
Merge "Added vlan-tag to port-interface (wip)."

9 years agoSmall fix to model 53/17253/1
Ed Warnicke [Fri, 27 Mar 2015 17:15:06 +0000 (13:15 -0400)]
Small fix to model

Change-Id: I705c29b42887c10e05f97e02717abfb99a4697a5
Signed-off-by: Ed Warnicke <eaw@cisco.com>
9 years agoFixed interface options attribute model. Replaced single string value to key-value... 31/17231/1
Anil Vishnoi [Fri, 27 Mar 2015 12:31:19 +0000 (18:01 +0530)]
Fixed interface options attribute model. Replaced single string value to key-value pair

Change-Id: I249f40c6cbd7e0aadf7a743fc00dd32452415329
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
9 years agoMerge "Update OpenVSwitch schema to include new datapath and port types."
Sam Hague [Thu, 26 Mar 2015 21:49:04 +0000 (21:49 +0000)]
Merge "Update OpenVSwitch schema to include new datapath and port types."

9 years agoAdded vlan-tag to port-interface (wip). 00/17200/2
Gabriel Robitaille-Montpetit [Thu, 26 Mar 2015 20:23:39 +0000 (16:23 -0400)]
Added vlan-tag to port-interface (wip).

Change-Id: Iccd6eefb10da36225c0f981b71d64a09865aa003
Signed-off-by: Gabriel Robitaille-Montpetit <grmontpetit@inocybe.com>
9 years agoMerge "Add JUnit testing for EventDispatcherImpl class."
Sam Hague [Thu, 26 Mar 2015 20:59:49 +0000 (20:59 +0000)]
Merge "Add JUnit testing for EventDispatcherImpl class."

9 years agoAdd JUnit testing for EventDispatcherImpl class. 99/17199/1
Alexis de Talhouët [Thu, 26 Mar 2015 19:51:11 +0000 (15:51 -0400)]
Add JUnit testing for EventDispatcherImpl class.

Change-Id: I4b382bb681af4e6d805094666657d81688166b9d
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
9 years agoInterface ofport and ofport_request support 55/17155/2
Ryan Goulding [Wed, 25 Mar 2015 18:48:32 +0000 (14:48 -0400)]
Interface ofport and ofport_request support

Added support to yang model for ofport and ofport_request.  These fields are
settable at the time of endpoint creation.  Changed the Interface schema ofport
field to utilize the type Integer; ofport is defined as an integer in the
Open_vSwitch specification.  Changed references to Interface ofport column to
use Integer data type.  Enforced a Yang model range for ofport_request to
ensure the value is in the range [1,65279] as defined by the Open_vSwitch
specification. Future work will persist these settings to the data store.

Change-Id: Ic8f1bab8d8fefe6b102cfceb50ba74f8cfa0b9b7
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
9 years agoMerge "Add JUnit testing for ProviderNetworkManagerImpl."
Sam Hague [Thu, 26 Mar 2015 17:44:59 +0000 (17:44 +0000)]
Merge "Add JUnit testing for ProviderNetworkManagerImpl."

9 years agoMerge "Bug2901: Fix Inventory ofport_request column in schema"
Sam Hague [Thu, 26 Mar 2015 17:43:25 +0000 (17:43 +0000)]
Merge "Bug2901: Fix Inventory ofport_request column in schema"

9 years agoAdd JUnit testing for ProviderNetworkManagerImpl. 77/17177/1
Alexis de Talhouët [Thu, 26 Mar 2015 16:49:07 +0000 (12:49 -0400)]
Add JUnit testing for ProviderNetworkManagerImpl.

Change-Id: Ie195e64d2e6f80fea11dee7ea6bfc1d3635567da
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
9 years agoBug2901: Fix Inventory ofport_request column in schema 53/17153/2
Ryan Goulding [Thu, 26 Mar 2015 11:51:03 +0000 (07:51 -0400)]
Bug2901: Fix Inventory ofport_request column in schema

Fixes ofport_request column setter.  Open_vSwitch defines ofport_request as an
integer, not a String.  The setOpenFlowPortRequest() method was modified to
take Set<Integer> instead of String.

Change-Id: I95e4a015aa53015fee2a9802e4f034951bed2b47
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
9 years agoUpdate OpenVSwitch schema to include new datapath and port types. 01/17201/2
Sharad Mishra [Thu, 26 Mar 2015 13:24:50 +0000 (06:24 -0700)]
Update OpenVSwitch schema to include new datapath and port types.

This update includes OVS schema changes made in http://openvswitch.org/pipermail/dev/2015-March/052794.html
These changes were made to OVS to include DPDK datapath and port types.

Change-Id: Ic45c60b91666f19d993df0bcd3049ca727d43d89
Signed-off-by: Sharad Mishra <sharad.d.mishra@intel.com>
9 years agoMerge "BridgeRemovedCommand"
Sam Hague [Thu, 26 Mar 2015 13:23:40 +0000 (13:23 +0000)]
Merge "BridgeRemovedCommand"

9 years agoAdd copyright to SecurityServicesImplTest and VlanConfigurationCacheImplTest 51/17151/2
Alexis de Talhouët [Thu, 26 Mar 2015 12:18:33 +0000 (08:18 -0400)]
Add copyright to SecurityServicesImplTest and VlanConfigurationCacheImplTest

Change-Id: Ifd7ab76ce7e72eecb0707cb65c63737c43450e1d
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
9 years agoMerge "Add JUnit testing for SecurityServicesImpl class."
Sam Hague [Thu, 26 Mar 2015 03:49:35 +0000 (03:49 +0000)]
Merge "Add JUnit testing for SecurityServicesImpl class."

9 years agoBridgeRemovedCommand 14/17114/3
Ed Warnicke [Wed, 25 Mar 2015 15:11:34 +0000 (11:11 -0400)]
BridgeRemovedCommand

We can now delete bridges

Change-Id: I2c89b65c5410172cdb4f5e103bfa0da5b6ab8915
Signed-off-by: Ed Warnicke <eaw@cisco.com>
9 years agoAdd JUnit testing for SecurityServicesImpl class. 26/17126/2
Alexis de Talhouët [Wed, 25 Mar 2015 21:19:55 +0000 (17:19 -0400)]
Add JUnit testing for SecurityServicesImpl class.

Change-Id: I9fa3af98acc89d4ea10764f573c0bcc2050f0e09
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
9 years agoMerge "Allow port/interface creation via MDSAL without supplying options"
Sam Hague [Wed, 25 Mar 2015 21:12:43 +0000 (21:12 +0000)]
Merge "Allow   port/interface creation via MDSAL without supplying options"

9 years agoMerge "Add JUnit testing for VlanConfigurationCacheImpl class."
Sam Hague [Wed, 25 Mar 2015 21:10:16 +0000 (21:10 +0000)]
Merge "Add JUnit testing for VlanConfigurationCacheImpl class."

9 years agoMerge "Add JUnit testing of TenantNetworkManagerImpl class"
Sam Hague [Wed, 25 Mar 2015 21:06:59 +0000 (21:06 +0000)]
Merge "Add JUnit testing of TenantNetworkManagerImpl class"

9 years agoAdded version leaf to ovsdb-node-attributes of type yang::uuid 02/17002/8
Gabriel Robitaille-Montpetit [Mon, 23 Mar 2015 15:22:47 +0000 (11:22 -0400)]
Added version leaf to ovsdb-node-attributes of type yang::uuid

Signed-off-by: Gabriel Robitaille-Montpetit <grmontpetit@inocybe.com>
Change-Id: I51b913cd6be634f240e158334715e150da4b4bf0

9 years agoAdd JUnit testing for VlanConfigurationCacheImpl class. 15/17115/5
Alexis de Talhouët [Wed, 25 Mar 2015 15:53:54 +0000 (11:53 -0400)]
Add JUnit testing for VlanConfigurationCacheImpl class.

Change-Id: Ia29f89d8e2bc68da6d6dbcd4e7c4587918e3776a
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>