Sam Hague [Wed, 22 Jun 2016 21:18:15 +0000 (17:18 -0400)]
ovsdb enable checkstyle on error
Change-Id: Ie3105f3cac8b0db4762c68df36c650f6cefdca00
Signed-off-by: Sam Hague <shague@redhat.com>
Anil Vishnoi [Thu, 23 Jun 2016 06:01:27 +0000 (06:01 +0000)]
Merge changes from topic 'blueprint'
* changes:
Add blueprint wiring to library
Add blueprint wiring for southbound
Add blueprint wiring for hwvtepsouthbound
Alexis de Talhouët [Tue, 21 Jun 2016 21:46:12 +0000 (17:46 -0400)]
Add blueprint wiring to library
Change-Id: Ieb404cc0ada8fef56f9ce63ec04ffbb40029a47e
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
Alexis de Talhouët [Tue, 21 Jun 2016 22:39:49 +0000 (18:39 -0400)]
Add blueprint wiring for southbound
Change-Id: I6e5cea49f584cc93a2842533287e950eadfabb40
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
Vishal Thapar [Wed, 22 Jun 2016 12:37:46 +0000 (18:07 +0530)]
BUG 5945: Tunnel updates through Genius not working
When creating tunnels through Genius, bridge information is present only
in operational and not in config. So any data chages that come in will not
have any augmentation. This was fixed as part of BUG 4644 but changes for
DataTreeListener refactored the code and this scenario was missed.
Change-Id: Ie4982d887e555a40984a5f4ff1211b785fe5ee6d
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
Anil Vishnoi [Wed, 22 Jun 2016 00:46:58 +0000 (00:46 +0000)]
Merge "BUG 5746 - Ovsdb QoS and Queue model enhancements"
Alexis de Talhouët [Tue, 21 Jun 2016 22:03:17 +0000 (18:03 -0400)]
Add blueprint wiring for hwvtepsouthbound
Change-Id: I973476d85f890f701b2b668dcebef61372e7aa58
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
Sam Hague [Mon, 20 Jun 2016 18:24:58 +0000 (14:24 -0400)]
Remove verbose logging in TransactUtils
The two logs removed are repeated constantly for every transaction and don't add much value since this is a utility.
Change-Id: I8d66c9b87e42ad93f3f91da1a5a76e9cb70e12f6
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Mon, 20 Jun 2016 10:21:39 +0000 (10:21 +0000)]
Merge "Bug:5938 Added log message for OvsdbDataTreeChangeListener."
Anil Vishnoi [Mon, 20 Jun 2016 07:00:58 +0000 (07:00 +0000)]
Merge ".gitignore .factorypath created by m2e-apt"
Anil Vishnoi [Mon, 20 Jun 2016 06:51:12 +0000 (06:51 +0000)]
Merge "Patch set 2 ------------ 1. Break this up into 2 patches, as Sam suggested. This patch will add new schema dependencies and fix ovsdb code to handle ovsdb node when both schemas are present."
ananthip [Thu, 16 Jun 2016 17:10:33 +0000 (13:10 -0400)]
Bug:5938 Added log message for OvsdbDataTreeChangeListener.
Change-Id: I4ae256f2564055bac0d79abb275d1f3f371c128f
Signed-off-by: ananthip <ananthip@hcl.com>
Michael Vorburger [Sun, 19 Jun 2016 15:58:46 +0000 (17:58 +0200)]
.gitignore .factorypath created by m2e-apt
https://github.com/vorburger/opendaylight-eclipse-setup now bundles
https://github.com/jbosstools/m2e-apt, which creates .factorypath files
all over the place.
Change-Id: I34a8edc9a0a4cd65f5e273014a232c0cba106c51
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Anil Vishnoi [Fri, 17 Jun 2016 23:05:16 +0000 (23:05 +0000)]
Merge "Support for IT Ping feature"
Victor Pickard [Mon, 13 Jun 2016 22:51:17 +0000 (18:51 -0400)]
Patch set 2
------------
1. Break this up into 2 patches, as Sam
suggested. This patch will add new schema
dependencies and fix ovsdb code to handle
ovsdb node when both schemas are present.
2. Only create transaction handlers for
Open-vSwitch database.
3. Revert change to hwvtepsouthbound-it/pom.xml,
so that hwvtepIT test are skipped, until
docker image is published that contains
both Open_vSwitch and hardware_vtep schemas.
See TODO itmes below.
4. Next patch will address items in TODO below.
When the ovsdb node has other schemas, such as
hardware_vtep, the code was creating transaction
handlers for hardware_vtep database, which the ovsdb
code cannot handle.
With this change, all IT tests pass when running
docker image loaded with both Open_vSwitch schema
and hardware_vtep schema.
Additionally, all IT tests pass when running docker
image loaded with only Open_vSwitch schema.
TODO
----
1. Publish docker image that has support for both schemas.
2. Replace assumes in hardware vtep IT to asserts if/when
hardware_vtep schema is not present.
3. Enable HwvtepSouthboundIT tests, in hwvtepsouthbound-it/pom.xml.
4. Sam suggested additional testing to ensure that both ovsdb southbound
and hwvtep southbound can connect at same time to ovsdb node running
both schemas.
5. Confirm deployment model when both schemas are present on
ovsdb node.
Patch set 1
-----------
Add hardwarevtep schema to pom and features.xml.
With these changes, and my docker image with both
ovsschema and hardwarevtep schema(to be published),
library/it tests pass, as well as hwvtepsouthbound-it
tests pass.
Change-Id: I94d24d524dcafaa38c69d129190ea29140efd5c9
Signed-off-by: Victor Pickard <vpickard@redhat.com>
Sam Hague [Thu, 16 Jun 2016 14:37:23 +0000 (10:37 -0400)]
Make SchemaVersionMismatchException logging consistent
The logging for the exception was not consistent: The log level varied,
the output varied and the logging of the stack trace varied.
Now a single utility method is called for the log. The level is set
to debug, same text, and the exception message rather than the trace
is logged. In all cases these are not errors because it is expected
that different schema versions may not have support for some columns.
Change-Id: I29ef9d30e2044483da1dfd9dc573d13715d222f1
Signed-off-by: Sam Hague <shague@redhat.com>
Josh [Wed, 15 Jun 2016 09:10:37 +0000 (11:10 +0200)]
Support for IT Ping feature
Change-Id: I12860f66d0abcd521bd07756232ede9ccdebfc60
Signed-off-by: Josh <jhershbe@redhat.com>
Eric Multanen [Fri, 25 Mar 2016 16:06:29 +0000 (09:06 -0700)]
BUG 5746 - Ovsdb QoS and Queue model enhancements
Update the QoS and Queue model to improve managability
of QoS and Queue tables via MD-SAL config model.
Adds code to update operational md-sal using updated
model.
Adds code to allow creation of queue and qos in same
transaction and assign queue to the qos row via
named-uuid.
QoS and Queue entries can be managed by ODL ID
values instead of needing to know the operational
UUID value.
Adds code to support assignment of QoS to port
by MD-SAL QoS ID.
Adds a QoS entry list to the termination point
model so updates and delete operations can be made
directly on the list instead of updating the
termination point.
Support for ingress policing rate and burst configuration
for interfaces is added.
Includes updated Postman examples.
v2 - clarify some logic based on comment feedback
add additional qos types: linux-sfq, linux-codel
linux-fq_codel, egress-policer
v3 - rebase, whitespace
v4 - address review comments
v5 - Changes to address review comments:
Use a queue-ref (instance identifier instead
of queue-id in the qos queue list.
Use a qos-ref (instance identifier instead
of qos-id in the port qos list.
Use a key of '1' for the single item port qos list.
Add instance identifer to the external-ids column
of the qos and queue tables instead of just id
value. Use key of 'opendaylight-iid'.
Updated the postman collection to account for changes
Clarified the comment and code of method
bytesToHexString in TransactUtils.java.
Still to do:
- update unit tests
- update integration tests
Change-Id: I71ac462afe5a881c621fad9a3c5603e1aab8d021
Signed-off-by: Eric Multanen <eric.w.multanen@intel.com>
Patrick Roanhouse [Tue, 14 Jun 2016 18:43:40 +0000 (14:43 -0400)]
removed unused imports in HwvtepSouthboundUtil.java, 'org.opendaylight.controller.md.sal.common.api.data.TransactionCommitFailedException' and org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.hwvtep.rev150901.HwvtepLogicalSwitchAttributes'.
Change-Id: Ia3afae16031f1d43e5c4ccfc54913d636cf4facb
Signed-off-by: Patrick Roanhouse <Patrick.M.Roanhouse@intel.com>
Sam Hague [Tue, 14 Jun 2016 14:35:32 +0000 (14:35 +0000)]
Merge "Bug 5976 - Thread leak when connecting/disconnecting ovs nodes"
Sam Hague [Tue, 14 Jun 2016 12:05:17 +0000 (12:05 +0000)]
Merge "Remove OvsdbDataChangeListener"
Sam Hague [Mon, 13 Jun 2016 23:30:31 +0000 (19:30 -0400)]
Add IT for port del dtcl
Change-Id: I2db8c5a6f8d56ffe9a6609b79ffe34dcbabba9c1
Signed-off-by: Sam Hague <shague@redhat.com>
Anil Vishnoi [Wed, 8 Jun 2016 02:15:29 +0000 (19:15 -0700)]
Bug 5992 - GBP + SFC integration is broken
When Bridge and termination points are created together, it generates two data tree change
notifications. Termination point changes are part of second data change notification,
but that does not contain bridge and node augmentation, because there was no change
in those augmentations, and because of that plugin was not able to find the connection
instance for the ovsdb node. As a last resort plugin tries to find it based on the
rootNode iid from operational data store, but because in this scenario rootNode is
bridge node, and bridge creation request was process just before the second data
change notification, operational data was not updated with the bridge node details
by the time plugin checks in the operational data store. So final attempt to find
the connection instance also fails. To fix this and also optimize it a bit, this
patch added an additional attempt to get the connection instances using the
managed-by ref node of the bridge node and check for that in the operational data
store. That node is the connection node, so it must be there if device is connected
to the controller, otherwise configuration should fail, which is expected outcome.
Change-Id: I743269b5ca2f36f761bbb4a118a2837631e1847d
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Dileep [Mon, 6 Jun 2016 16:21:12 +0000 (09:21 -0700)]
Bug 5976 - Thread leak when connecting/disconnecting ovs nodes
ExecutorService in OvsdbClientImpl is not shutdown properly
When the del-manager is called along with closing the channel,
the executor service is shutdown.
Set the threadfactory to distinguish the Ovsdbclient threadpool.
Change-Id: I13473ca40456ba78162591ea23cfd907d2f5b34f
Signed-off-by: Dileep Ranganathan <dileep.ranganathan@intel.com>
Stephen Kitt [Mon, 13 Jun 2016 10:09:18 +0000 (12:09 +0200)]
Remove OvsdbDataChangeListener
This has been obsoleted by OvsdbDataTreeChangeListener.
Change-Id: I741584d94fb6bae7c6a456cdd812d093c8f9974b
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Patrick Roanhouse [Wed, 8 Jun 2016 16:59:00 +0000 (12:59 -0400)]
Removed unused import 'org.opendaylight.ovsdb.lib.impl.OvsdbConnectionService'." from HwvtepSouthboundProvider.java
Change-Id: I0542e44fbc94b9dca691f4aa4e5fc01c9f1dc497
Signed-off-by: Patrick Roanhouse <Patrick.M.Roanhouse@intel.com>
David Suarez [Tue, 7 Jun 2016 10:50:19 +0000 (12:50 +0200)]
Replace wildcard imports
Change-Id: Icb2884d8d3785cbe551dc0fbd8a71b1c3446782e
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
Anil Vishnoi [Fri, 3 Jun 2016 08:06:58 +0000 (08:06 +0000)]
Merge "Add ability to create VxLAN-GPE tunnels"
Rashmi Pujar [Thu, 2 Jun 2016 14:26:19 +0000 (10:26 -0400)]
Bug 5944 - Looping INFO messages for UNSUPPORTED AutoAttach OVS
Change-Id: I93704b2a0997da85a3cf2373cbb0d5c77b313db6
Signed-off-by: Rashmi Pujar <rpujar@inocybe.com>
David Suarez [Tue, 31 May 2016 15:05:06 +0000 (17:05 +0200)]
Add ability to create VxLAN-GPE tunnels
Change-Id: Ic352bebeed0a7cbfdbdc92b49ac6fcf0db2b615f
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
Josh [Thu, 26 May 2016 09:14:30 +0000 (11:14 +0200)]
System property awareness in DockerOvs
Change-Id: If039cd49c18f9ba55181918c66fce1eaf973df82
Signed-off-by: Josh <jhershbe@redhat.com>
Anil Vishnoi [Sat, 28 May 2016 22:34:33 +0000 (22:34 +0000)]
Merge "removed unused import 'org.opendaylight.yang.gen.v1.urn.ietf.params.xml. ns.yang.ietf.inet.types.rev100924.Ipv4Address' in HwvtepSouthboundMapper.java"
Patrick Roanhouse [Fri, 27 May 2016 22:40:26 +0000 (18:40 -0400)]
removed unused import 'org.opendaylight.yang.gen.v1.urn.ietf.params.xml. ns.yang.ietf.inet.types.rev100924.Ipv4Address' in HwvtepSouthboundMapper.java
Change-Id: I5e8ec077386e0cfc63877f01c218c52f23e73b29
Signed-off-by: Patrick Roanhouse <Patrick.M.Roanhouse@intel.com>
Anil Vishnoi [Fri, 27 May 2016 23:28:07 +0000 (16:28 -0700)]
BUG 5885 - OVSDB plugin failure to update passive ovsdb nodes
Updates the ovsdb southbound so that configuration to ovsdb
nodes which have connected to the plugin passively can be
updated (e.g. such as qos or queue entries) without requiring
connection-info to be supplied in the configuration.
Similar to what already works when adding bridges or
termination points to passively connected ovsdb nodes.
The onDataChanged() method executes the following
sequence of operations on changes to the Ovsdb data tree:
- connect(changes)
- updateConnections(changes)
- updateData(changes)
- disconnect(changes)
When an OVSDB node has connected passively, or more
generally, when the changes to the OVSDB node do not
include 'connection-info', it is not necessary to
attempt to execute connection related operations for
the specified host. This patch checks for the absence
of the connection-info and skips trying to connect or
update a connection in the connect() and updateConnections()
methods.
v2 - add in data tree change patch from master branch
now that data tree change listener is merged.
Change-Id: I8238711076a74cfa06422e5cacb127ba95fd23b7
Signed-off-by: Eric Multanen <eric.w.multanen@intel.com>
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Vishal Thapar [Thu, 26 May 2016 13:50:22 +0000 (19:20 +0530)]
BUG 5506: OVSDB server doesn't close connection after peer is power down
https://git.opendaylight.org/gerrit/#/c/35436/ added code to detect when a
peer goes down without sending a FIN. However, it only did it for switch
initiated connection. In case of ODL initiated connection we are unable to
detect peer being down and reconciliation is never triggered.
Change-Id: I62d0b2cef75b889fc75b43fe51be5156f921fcfe
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Fri, 27 May 2016 16:14:55 +0000 (16:14 +0000)]
Merge "BUG 5479: HWVtep Southbound doesn't retry connection"
Vishal Thapar [Wed, 25 May 2016 04:17:51 +0000 (09:47 +0530)]
BUG 5479: HWVtep Southbound doesn't retry connection
This fix is hwvtep counterpart of
https://git.opendaylight.org/gerrit/#/c/36028/ which fixes BUG 5221.
Wiki:
https://wiki.opendaylight.org/view/OVSDB_Integration:OVSDB_SB_Reconciliation
Change-Id: I497dbfa339de157d12afe94ce5c36b35ffda5dd9
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
(cherry picked from commit
9d80362e33fb2d6a2c0dc4b7994c976e46e8799f)
Jamo Luhrsen [Thu, 19 May 2016 05:49:54 +0000 (22:49 -0700)]
BUG 5769
bailing on connection if get databases is not returning anything
and then using disconnected() which seems to have the effect of
letting our threadpool service more connections. The condition
before would block any new (and good) connections if we initiated
just 10 incorrect TCP connections toward port 6640
Change-Id: I5a59762049a0460cc39ebbfce0c93d002ec24d68
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Signed-off-by: Sam Hague <shague@redhat.com>
Vishal Thapar [Fri, 27 May 2016 11:48:59 +0000 (17:18 +0530)]
Fix for verification build failures
Change-Id: Ib1b6a920042bb977ca952320cf50d53af7288e75
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
Anil Vishnoi [Mon, 23 May 2016 22:16:20 +0000 (22:16 +0000)]
Merge "Infra for running ovs dockers"
Anil Vishnoi [Sun, 22 May 2016 05:07:39 +0000 (05:07 +0000)]
Merge "Patch - https://git.opendaylight.org/gerrit/#/c/39061/ is broken .This patch fixes the breakage"
Patrick Roanhouse [Wed, 11 May 2016 21:46:40 +0000 (17:46 -0400)]
Minor sonar listed bug fixed for coding style in HwvtepConnectionManager.java
Change-Id: I28bae105be378010ac109c524977705afb6e4aa0
Signed-off-by: Patrick Roanhouse <patrick.m.roanhouse@intel.com>
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Anil Vishnoi [Sun, 22 May 2016 01:15:19 +0000 (18:15 -0700)]
Patch - https://git.opendaylight.org/gerrit/#/c/39061/ is broken
.This patch fixes the breakage
Change-Id: I78034a55facd10f5921000e86bf451a90faa0691
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Sam Hague [Sun, 22 May 2016 00:35:02 +0000 (00:35 +0000)]
Merge "Remove logging entire exception to avoid a noisy stack trace"
Rashmi Pujar [Wed, 18 May 2016 18:21:52 +0000 (14:21 -0400)]
Remove logging entire exception to avoid a noisy stack trace
Change-Id: I8b8efe61742505b6b89b66684e482b9d4e9e550b
Signed-off-by: Rashmi Pujar <rpujar@inocybe.com>
vinh.nguyen [Fri, 8 Apr 2016 19:14:46 +0000 (12:14 -0700)]
BUG-5177 - bridge not created if it's configured northbound while ovs node is disconnected
Add bridge configuration reconciliation logic to southbound plugin.
When ovsdb southbound plugin receives connection request from an
OVS node, the plugin pushes all existing bridge configurations
for that node in the config data store to the node.
This is the solution for reconciliation scenarios #4 and #6 in:
https://wiki.opendaylight.org/view/OVSDB_Integration:OVSDB_SB_Reconciliation
Change-Id: I436cd15e2ff3231551fec4d4175983cec7bc2186
Signed-off-by: vinh.nguyen <vinh.nguyen@hcl.com>
Josh [Mon, 16 May 2016 13:43:15 +0000 (15:43 +0200)]
Infra for running ovs dockers
Added a class called DockerOvs.
See documentation in the class
Added component:
<groupId>com.esotericsoftware.yamlbeans</groupId>
<artifactId>yamlbeans</artifactId>
this is used to parse the docker-compose files to
extract the port mappings
Change-Id: I978a9a6bc6296f7d8e1246c37c3895b2f36c5a6d
Signed-off-by: Josh <jhershbe@redhat.com>
Sam Hague [Wed, 18 May 2016 20:57:46 +0000 (16:57 -0400)]
Get bridge details for delete when using dataTree
Change-Id: Id9f74a8cffb058142daa303f70d7fe570d493bef
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Wed, 18 May 2016 20:57:06 +0000 (16:57 -0400)]
Remove redudnant calls for bridge updates
Change-Id: I34ed01cbdf870351707bb6daf990a5fc88609027
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Mon, 16 May 2016 22:02:42 +0000 (22:02 +0000)]
Merge "Bug 5876 - OVSDB library: Retry when SSL handshake doesn't begin yet"
Hsin-Yi Shen [Wed, 13 Apr 2016 20:11:07 +0000 (13:11 -0700)]
Bug 5876 - OVSDB library: Retry when SSL handshake doesn't begin yet
Sometimes the listener get notified before SSL handshake is done.
The root cause is we don't handle the handshake state "NOT_HANDSHAKING" properly.
"NOT_HANDSHAKING" could mean handshake process is finished or doesn't begin yet.
We should handle latter case(handshake process doesn't begin yet) and retry before
notifying lsteners.
Change-Id: Ibdd82beb2aa876313113211951a4347cc601331a
Signed-off-by: Hsin-Yi Shen <syshen66@gmail.com>
Sam Hague [Fri, 13 May 2016 21:21:22 +0000 (21:21 +0000)]
Merge "Add MdsalUtilsAsync to make transactions asyncrhonous"
Sam Hague [Fri, 13 May 2016 19:22:06 +0000 (15:22 -0400)]
Remove missed netvirt files
Change-Id: I1bc0e9767be9c74aaaa1c7c1f042d185419dd062
Signed-off-by: Sam Hague <shague@redhat.com>
Alexis de Talhouët [Tue, 19 Apr 2016 16:52:45 +0000 (12:52 -0400)]
Add MdsalUtilsAsync to make transactions asyncrhonous
Change-Id: I8bb5d82f019ad62beb4b3c944640070227f38632
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
Sam Hague [Fri, 6 May 2016 17:21:02 +0000 (13:21 -0400)]
Remove the remaining netvirt code from ovsdb
Change-Id: Icaf75e219380c3b17c4479d64c49ee6862c2a9c6
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Fri, 6 May 2016 13:25:43 +0000 (13:25 +0000)]
Merge "Use ${project.version} for internal dependencies"
Daniel Farrell [Wed, 4 May 2016 14:45:25 +0000 (10:45 -0400)]
Fix Puppet install in OVSDB Vagrantfile
The Vagrantfile's initial shell provisioner call attempted to
install Puppet via an old method that now fails. The repos
for installing Puppet on Ubuntu seem to have changed.
http://pastebin.com/6A4eBzM2
Puppet is now included in the default repos, so a simple apt-get
install works as expected.
Change-Id: Id5bb8e4d9d966ac69397fc7ab877bc5495e2dce5
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>
Stephen Kitt [Wed, 4 May 2016 14:47:20 +0000 (16:47 +0200)]
Use ${project.version} for internal dependencies
Correct the version on southbound-impl's southbound-utils
dependency...
Change-Id: I12c9aaa94670031a0b3a323facfd1dafecc90317
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Robert Varga [Wed, 4 May 2016 12:26:02 +0000 (14:26 +0200)]
Move TpId allocation out of loop
The TpId being looked up is loop-invariant, hence it makes
sense to allocate it once rather than in a tight loop.
Change-Id: I8242754c61fa56b0779f164b001705fe252fe60e
Signed-off-by: Robert Varga <rovarga@cisco.com>
Vishal Thapar [Wed, 4 May 2016 06:53:12 +0000 (12:23 +0530)]
Fix Iid for locator-refs in HwvtepTunnelUpdate
Local and remote locator ref IIDs are off NodeID, not physical switch IID.
The iid for tunnels was generated correctly but locators stored in data
were using incorrect Iid.
Change-Id: Ida585feeb5c1bf419d1123c5ecbe0dbdf99486af
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
Sam Hague [Wed, 4 May 2016 01:33:29 +0000 (01:33 +0000)]
Merge "Fixed hard-coded port 12345 in TestClient"
Michael Vorburger [Tue, 3 May 2016 23:36:52 +0000 (01:36 +0200)]
Fixed hard-coded port 12345 in TestClient
This avoids java.net.BindException: Address already in use
Change-Id: I15ca5fe4f5e381c40b4bf5eaddcc3ada4ca4d886
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Sam Hague [Mon, 2 May 2016 21:33:51 +0000 (21:33 +0000)]
Merge "Use DataTreeChangeListener instead of DataChangeListener"
Sam Hague [Mon, 2 May 2016 21:33:36 +0000 (21:33 +0000)]
Merge "BUG-5737: enable OVSDB Maven site"
Vishal Thapar [Wed, 20 Apr 2016 07:44:21 +0000 (13:14 +0530)]
BUG5764: Hwvtep tunnel update/delete not reflected correctly
1. Adds HwvtepTunnelUpdate/Delete command patterns
2. Removes tunnel handling from PhysicalSwitchUpdate
3. Adds PhysicalSwitch and locator information to HwvtepDeviceInfo
4. Reorders Aggregator to take care of dependencies
5. Renames classes to distinguish from ones in transact. This makes easier
to troubleshoot.
Change-Id: I1269127118c11961f552df286292ed8f5b4ef50d
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
Jamo Luhrsen [Fri, 29 Apr 2016 00:04:24 +0000 (17:04 -0700)]
BUG 5809: allow TP updates from ovs to be reflected in operational
Change-Id: Ibafac754c62adc13c3931cbf3e985e44af26f93c
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>
Vishal Thapar [Tue, 29 Mar 2016 20:11:35 +0000 (01:41 +0530)]
BUG 5556: Unable to delete mac entries
This adds a new class that acts as cache for rows received from hwvtep
device. Currently it only stores LogicalSwitches to be used for mac entry
operations and creatg VlanBindings in PhysicalPortUpdate
Change-Id: Ie9ad19d62020e100be533553b9394167d449c939
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
Stephen Kitt [Thu, 31 Mar 2016 14:58:00 +0000 (16:58 +0200)]
Use DataTreeChangeListener instead of DataChangeListener
For now this patch sticks to the old approach, where we just listen
for all Node events and rebuild the changes and re-extract them.
Change-Id: I0f8e4ed09998a28326d39306cbb1b3486260e7f8
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Stephen Kitt [Thu, 14 Apr 2016 12:21:05 +0000 (14:21 +0200)]
BUG-5737: enable OVSDB Maven site
This is a first pass:
* enable the Maven site
* fix up all the URLs
* fix the Javadoc error
We need to give useful names to all the modules next.
Change-Id: Ia9f1ad00210a58c8cc786f68dd8c0047de9a4b48
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Anil Vishnoi [Fri, 15 Apr 2016 07:45:43 +0000 (00:45 -0700)]
Bug 4644: TerminationPoint not remove from switch
if termination point is added to the existing bridge
which was already present before OVSDB node got connected
to the controller. Plugin was not able to delete the termination
point because it was not able to find the connection client and
the parent bridge from the data store change notification on
termination point delete.
Change-Id: I1f720f77aa8005274674903ddb5fa1daf962af5d
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Josh [Tue, 12 Apr 2016 11:50:27 +0000 (13:50 +0200)]
split it-utils: ovsdb-it-utils + netvirt-it-utils
Change-Id: I5b512b65c3502431f919028a6383178c4f2dd140
Signed-off-by: Josh <jhershbe@redhat.com>
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Mon, 11 Apr 2016 00:04:32 +0000 (20:04 -0400)]
Disable netvirt
Change-Id: I6255bd971af8dc7282e8361e3010dff4812655d8
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Fri, 8 Apr 2016 00:02:50 +0000 (00:02 +0000)]
Merge "Switch from fig to docker-compose"
Sam Hague [Thu, 7 Apr 2016 21:02:37 +0000 (21:02 +0000)]
Merge "BUG-5291:Flows are not removed from control and compute nodes."
Sam Hague [Thu, 7 Apr 2016 21:02:28 +0000 (21:02 +0000)]
Merge "Test for NetvirtSFC config"
hari.pr [Tue, 5 Apr 2016 15:22:58 +0000 (20:52 +0530)]
BUG-5291:Flows are not removed from control and compute nodes.
* When the VM instance is deleted from control and compute nodes,
* Found still flows are existed in table-40 and table-90 in L2 scenario.
Signed-off-by: hari.pr <hari.pr@hcl.com>
Change-Id: I6e0873e979cdbd8dc3037fe012477413faf95a92
(cherry picked from commit
ad36780546ecbe1adbf9f081fca0f1eca7287713)
Sam Hague [Thu, 7 Apr 2016 19:42:02 +0000 (15:42 -0400)]
Test for NetvirtSFC config
Change-Id: I3fd13785ca51a55f8137679a68add283d3e3df5c
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Thu, 7 Apr 2016 19:04:35 +0000 (15:04 -0400)]
Bug 5651 - southbound-impl failed to find utils.yang-utils in Boron
Change-Id: I5b7ac318f701bf8d410103a5218a1c69931b57d2
Signed-off-by: Sam Hague <shague@redhat.com>
Stephen Kitt [Thu, 7 Apr 2016 14:40:02 +0000 (16:40 +0200)]
Switch from fig to docker-compose
docker-compose no longer supports fig.yml files, so finish the switch
completely.
Change-Id: I2faec3ebd04a129368a3c40f99ff7b0389c998eb
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Sam Hague [Wed, 6 Apr 2016 21:56:43 +0000 (21:56 +0000)]
Merge "OVS 2.5.1 docker image in fig.yml"
Josh [Tue, 5 Apr 2016 08:39:53 +0000 (10:39 +0200)]
OVS 2.5.1 docker image in fig.yml
Change-Id: Iefab500c304c99742c5ecd0877404c8ebbb10efb
Signed-off-by: Josh <jhershbe@redhat.com>
Stephen Kitt [Tue, 29 Mar 2016 11:56:37 +0000 (13:56 +0200)]
Use stateless transaction commands
Changing the transaction command model to one where all the required
information is passed to the execute() method makes the commands much
simpler to convert to a DataTreeChangeListener-based model.
Change-Id: I7458ed91646a6f0848afda86f72cf003c9704466
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Sam Hague [Mon, 4 Apr 2016 23:22:44 +0000 (23:22 +0000)]
Merge "BUG-5648 - delete qos and queues from udpates"
Stephen Kitt [Fri, 1 Apr 2016 15:10:07 +0000 (17:10 +0200)]
Split listeners from data processors
Listening for changes is different from processing them, so separate
the two concerns. This should make the data processors easier to
test.
Change-Id: I7a385b13c37ac2d02f12799042fc956084bd3670
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Mon, 15 Feb 2016 17:46:52 +0000 (12:46 -0500)]
Bug 5354 - getPortNumberFromNode is not correctly calculating the port number from the port name
Signed-off-by: Sam Hague <shague@redhat.com>
Change-Id: If1844e95560b0c2dee853b19e9e472b8fcab2915
Sam Hague [Thu, 11 Feb 2016 23:13:31 +0000 (18:13 -0500)]
Bug 4996 - Wrong flows when using SFC coexistence
Signed-off-by: Sam Hague <shague@redhat.com>
Josh [Mon, 4 Apr 2016 12:47:20 +0000 (14:47 +0200)]
BUG-5648 - delete qos and queues from udpates
Even when the update does not have updates to
the OpenVSwitch table.
See comments in Bug.
Change-Id: Ibf041e0e3315408112138b6e72a42782fac3e0a5
Signed-off-by: Josh <jhershbe@redhat.com>
Sam Hague [Sun, 3 Apr 2016 04:33:00 +0000 (04:33 +0000)]
Merge "reorder feature install for netvirtsfc"
Sam Hague [Sun, 3 Apr 2016 04:32:29 +0000 (00:32 -0400)]
reorder feature install for netvirtsfc
Change-Id: I7926afdd17fb720c1e008b1f105b4aeac3090d78
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Sun, 3 Apr 2016 02:47:27 +0000 (02:47 +0000)]
Merge "Use PerClass to force a new container per test class"
Sam Hague [Sun, 3 Apr 2016 02:46:46 +0000 (22:46 -0400)]
Use PerClass to force a new container per test class
Change-Id: I72460583d501eeaa62a74a5ffb3bffaa633e995b
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Sat, 2 Apr 2016 22:39:38 +0000 (22:39 +0000)]
Merge "Fixed NPE thrown while switch initiated connection disconnects from controller"
Anil Vishnoi [Sat, 2 Apr 2016 07:25:05 +0000 (00:25 -0700)]
Fixed NPE thrown while switch initiated connection disconnects from controller
Change-Id: Ic9cd502729ab9fa4f67e8937e47bb57efdc6db9e
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Sam Hague [Sat, 2 Apr 2016 21:46:59 +0000 (17:46 -0400)]
Fixes for LibraryIT timing out
Something recently changed in the karaf builds such that
it is causing the five core karaf bundles to only go to
an installed state. The LibraryIT tests will not run unless
they see all bundles active, so the installed state bundles
cause the test to timeout.
Reworked the tests to not use the bundle states and to use what all
the other tests in OBSDB use which is to wait until the config is up.
Added these methods to the abstract test class so that it could be
shared across the five differetn library IT sets.
Also cleaned up a bunchof System.Out.println's and Assert's.
Change-Id: Iddd31eede1de1c5f3cde88bb1c3f7c96a5489051
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Sat, 2 Apr 2016 00:46:40 +0000 (20:46 -0400)]
Add IT for netvirt neutron
Change-Id: Id2846d3156562cdda0216c6b3685365ae7902b85
Signed-off-by: Sam Hague <shague@redhat.com>
Anil Vishnoi [Thu, 10 Mar 2016 07:35:37 +0000 (23:35 -0800)]
Bug 5221 - passive connection not reconnected if ovs service is restarted
Change-Id: I830aeeed4ba7e51adb0ad952c11fd1603d207b1b
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Sam Hague [Wed, 30 Mar 2016 22:06:46 +0000 (22:06 +0000)]
Merge "it-utils should be only included for IT"
Sam Hague [Wed, 30 Mar 2016 19:53:42 +0000 (15:53 -0400)]
it-utils should be only included for IT
The it-utils bundle was being included with the odl-ovsdb-openstack
feature. When the feature was installed karaf would error out
looking for org.junit - which is a dependency for it-utils since
that is a test bundle.
Fix: Create a IT feature and include the bundle there and use
that feature for the IT test.
Change-Id: I4fc0395ca177d98b8433857b384cddbc033057fb
Signed-off-by: Sam Hague <shague@redhat.com>
Rashmi Pujar [Tue, 29 Mar 2016 22:42:18 +0000 (18:42 -0400)]
Skip IT tests for Autoattach CRUD for supported schema
- will be uncommented when CRUD is supported for AA table in ovs
Change-Id: Iaa9d3e518f207a609a8da10dc6ffea5c301971eb
Signed-off-by: Rashmi Pujar <rpujar@inocybe.com>
Sam Hague [Tue, 29 Mar 2016 13:01:36 +0000 (13:01 +0000)]
Merge "Added conntrack matches and Refactored code"