Adding revision for ietf types.
It can conflict with if some other project uses a newer version of the ietf types.

Change-Id: I38229f571dacd14c5a569e802e5eb8019fe322a9
Signed-off-by: Amit Mandke <ammandke@cisco.com>

Merge "Add JUnit testing for PortSecurityHandler class."

Merge "Add JUnit testing for RouterHandler class."

Merge "Add JUnit testing for SubnetHandler class."

Merge "Add JUnit testing for NodeCacheManagerImpl class."

Turn down logging to appropriate debug level to reduce log pollution

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

Add JUnit testing for NodeCacheManagerImpl class.

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

Add JUnit testing for SubnetHandler class.

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

Add JUnit testing for RouterHandler class.

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

Add JUnit testing for PortSecurityHandler class.

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

Merge "Add JUnit testing for NodeCacheManagerEvent class."

Merge "Add JUnit testing for NetworkHandler class."

Merge "Add JUnit testing for PortHandler class."

Add JUnit testing for PortHandler class.

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

Add JUnit testing for NodeCacheManagerEvent class.

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

Add JUnit testing for NetworkHandler class.

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

Add JUnit testing for LBaaSPoolHandlerTest class.

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

Support other_config for OVSDB Nodes

Add support to set other_config via MDSAL config and
update operational MDSAL from the Open_vSwitch table
on the OVS host.

v2 - add NPE to the LOG warning.
v3 - rebase
v4 - fix logging of exception
v5 - remove whitespace

Change-Id: If684e89185fe5660263c3e7ff8b1f5c7087764ee
Signed-off-by: Eric Multanen <eric.w.multanen@intel.com>

Merge "Add support for external-ids in the OVSDB node"

Merge "port/interface-external-ids refactoring"

port/interface-external-ids refactoring

This change aggregates separate, appropriately named, list to store port
and interface related external_ids.  ovsdb.yang model changes were made to
rename the list and eliminate the more generic ovsdb-external-ids-attributes
grouping.  Java files were modified to compensate for the model changes.

Change-Id: Ieaa0ac82c8e0efe930cd837dab26fff4b0a572fe
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Merge "changed getManagingNode(..) to expect augmented path. In OvsdbBridgeAttributes 'managedBy' can be path to 'OvsdbNodeAugmentation' as well. This can happen if the attribute is set via binding aware code."

Merge "Move UuidUtilsTest to the right package (netvirt.api instead of netvirt)."

Merge "Add JUnit testing for LBaaSPoolMemberHandler class."

Merge "Add JUnit testing for NeutronCacheUtilsTest class."

Merge "Fixed postman collection for create termination point restconf request"

Fixed postman collection for create termination point restconf request

Change-Id: I27268940f506b10b3e2d8f0e48e5f40ecf6e5709
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

Add JUnit testing for NeutronCacheUtilsTest class.

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

Add JUnit testing for LBaaSPoolMemberHandler class.

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

Merge "Implemented port/interface delete command to delete interface from ovsdb bridge *.*Fixed tp-ip : We should not append bridge id to the tp-id, as it's already augmented on bridge node. Also removed the utility method to avoid further uses. *.* removed attached-to from termination-point attributes *.* Fixed restconf *.* Rebased patch"

Implemented port/interface delete command to delete interface from ovsdb bridge
*.*Fixed tp-ip : We should not append bridge id to the tp-id, as it's already
augmented on bridge node. Also removed the utility method to avoid further
uses.
*.* removed attached-to from termination-point attributes
*.* Fixed restconf
*.* Rebased patch

Change-Id: Iedd87b386a3b2c6acd2e33ab26d17fbeec6f35ff
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

Merge "Close transaction so we don't leak them."

Merge "Introducing the extractCreatedOrUpdated method"

Merge "Update netvirt.impl unit tests."

Move UuidUtilsTest to the right package (netvirt.api instead of netvirt).

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

Update netvirt.impl unit tests.

Basically I added tests, mainly Mockito.verify(). And I refactored, updated and cleaned up the classes.

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

Merge "Add support for bridge external-ids."

changed getManagingNode(..) to expect augmented path.
In OvsdbBridgeAttributes 'managedBy' can be path to 'OvsdbNodeAugmentation' as well.
This can happen if the attribute is set via binding aware code.

Signed-off-by: Amit Mandke <ammandke@cisco.com>

Close transaction so we don't leak them.

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

Modified postman collection to test vlan-mode.

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

Merge topic 'unification'

* changes:
  Switch to generic extract methods in TransactUtils
  Use SouthboundMapper.getRandomUUID() in TerminationPoints

Merge "Add JUnit testing for FWaasHandler class."

Merge "Add JUnit testing for LBaaSHandler class."

Modify test case testProcessEvent() for FloatingIPHandlerTest class.

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

Add JUnit testing for LBaaSHandler class.

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

Add support for external-ids in the OVSDB node

Write external-ids supplied in an update to an OVSDB
node via the MDSAL.  Update the Open_vSwitch table
on the OVS host.

TODO - handle external-ids provided on OVSDB Node create

v2 - update the operational MDSAL with received external-ids
v3 - add NPE to LOG warning
v4 - rebase against new data change listener
v5 - remove unused methods in TransactUtils.java from earlier versions
v6 - more rebase
v7 - formatting fix

Change-Id: Ifcb6b433272e43ee9542521e4688b2d4fcd356e7
Signed-off-by: Eric Multanen <eric.w.multanen@intel.com>

Merge topic 'unification'

* changes:
  Remove need for termination-point to have bridge-name
  Step 2: Migrate termination point stuff under the same transact as bridge stuff.

Add support for bridge external-ids.

updated the yang model and impl to use "bridge-external-ids".

Change-Id: I6fbf9383557ddf115408fe398a4ba2aeb7134738
Signed-off-by: Sharad Mishra <sharad.d.mishra@intel.com>
Add support for bridge external-ids

Change-Id: I850d32c55485acf566b5b419aaf22e782af37df7
Signed-off-by: Sharad Mishra <sharad.d.mishra@intel.com>

Introducing the extractCreatedOrUpdated method

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

Switch to generic extract methods in TransactUtils

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

Use SouthboundMapper.getRandomUUID() in TerminationPoints

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

Remove need for termination-point to have bridge-name

bridge-name can be dug out of the change event :)

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

Merge "Add test case for enqueueEvent() for AbstractHandler class."

Step 2: Migrate termination point stuff under the same transact as bridge stuff.

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

Merge "First step in unifying datachange events"

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>

Merge "Added model for vlan mode."

Added model for vlan mode.

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

Add test case for enqueueEvent() for AbstractHandler class.

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

Add JUnit testing for FWaasHandler class.

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

Interface options operational data store persistence

Stores the interface options to the operational data store.

Change-Id: I8e6956036f2a43ee404c0e30e19e7264ad345138
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

OpenFlow port values are uint32, correcting the size in the model for that

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

Merge "Small fix to handle the fact that ovsdb sometimes returns ofport of -1"

Merge "Check for trunks (if not present, don't throw an NPE ;) )"

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>

Check for trunks (if not present, don't throw an NPE ;) )

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

Modified postman collection to test vlan-tag and trunks.

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

Merge "Fixed if/else/for/while/do curly braces for LBaaSPoolMemberHandler.java"

Merge changes I99c0e84f,I0e035323

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

Added trunks code using Long

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

Add sonar for southbound

Change-Id: I99c0e84f1b52e7d295f485b954160d1593d72c7d
Signed-off-by: Sam Hague <shague@redhat.com>

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>

Add copyright's to southbound-it files.

Change-Id: I0e0353233b068873bd1ba983eb11c7efde83bc1d
Signed-off-by: Sam Hague <shague@redhat.com>

Merge "Revert "Updated ovsdb.yang to include external-ids for bridges.""

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>

Merge "Updated ovsdb.yang to include external-ids for bridges."

Merge "support for Interface external_ids"

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>

Updated ovsdb.yang to include external-ids for bridges.

Change-Id: Ia639165fe05be5f4e1f236893780b177465b0d9a
Signed-off-by: Sharad Mishra <sharad.d.mishra@intel.com>

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>

Merge "net-virt-providers: install flow twice unnecessarily"

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>

Merge "Add JUnit testing for FloatingIPHandler class."

Merge "Add IT for southbound"

Merge "ofport and ofport_request operational data store persistence"

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>

Add IT for southbound

Change-Id: Iff82a08c08cae8013417031f48f32d6639b94b8c
Signed-off-by: Sam Hague <shague@redhat.com>

Add JUnit testing for FloatingIPHandler class.

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

Merge "Add JUnit tesing for NeutronL3Adapter class."

Merge "Add JUnit testing for AbstractHandler class."

Merge "Add JUnit testing for AbstractEvent class."

Merge "Revert "Add break in iteration over vlan tags for a given ovsdbTerminationPoint""

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>

Add JUnit tesing for NeutronL3Adapter class.

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

Add JUnit testing for AbstractHandler class.

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

Add JUnit testing for AbstractEvent class.

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

Merge "Model update to add external-id to port/interface."

Merge "Add break in iteration over vlan tags for a given ovsdbTerminationPoint"

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>

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>