BUG-2658: Remove adsal and migrate to mdsal

This commit covers removing adsal-compatibility.

patchset 2: Remove adsal-compatibility from ovs-sfc features
patchset 3: Really remove the adsal-compatibility from ovs-sfc features.
patchset 4: Apparently not
patchset 5: Trying again

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

Enable OVSDB northbound V2 integration tests.

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

Move library integration tests to integrationtest dir

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

Move bridgedomain it to the intergationtest bundle

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

Fix broken ovsdb build due to commit in openflowplugin

See:  https://git.opendaylight.org/gerrit/#/c/13234/
      https://git.opendaylight.org/gerrit/#/c/14289/

NOTE: ovs-sfc feature is being disabled for the time being, as it is
      not used for anything.

Patch 1: fix build
Patch 2: complete the cleanup
Patch 3: Enable ovs-sfc, add feedback from code review
Patch 4: Coalesce openflowplugin.version-* constants into openflowplugin.version
Patch 5: Fix ovs-sfc single feature test

Change-Id: I3d307e41165c5872844ed76447c083dd84bd5522
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Remove deprecated BindingAwareProvider methods

https://git.opendaylight.org/gerrit/#/c/13806/

will remove deprecated BindingAwareProvider methods.
This patch simply removes them from
implmementation of that interface so when 13806 is
merged we can merge this to fix.

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

Set root pom.xml <name> for Sonar

As mentioned on the mailing list Sonar uses the <name> field of the
pom.xml that is passed to the mvn command as the name of the project in
Sonar. In most cases this is the root pom.xml file in a project. This
patch sets the name to the project shortname.

https://lists.opendaylight.org/pipermail/discuss/2014-November/004024.html

Change-Id: I34549ed7631b183128b025859bd641d5ec0c8927
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Merge "Enable SSL two way authentication"

Merge "Updated error catch for deleteRow ConfigurationServiceImpl.java"

Enable SSL two way authentication

With this fix the ovsdb server will provide ssl connection with two way authentication.
Both server and client will authenticate peer certificate.
The method of getting peer certificate is also provided in ovsdb connection info.

Signed-off-by: Hsin-Yi Shen <syshen66@gmail.com>

Updated error catch for deleteRow ConfigurationServiceImpl.java

Logged more specific information from caught error and updated
error message format to match best practices.

Change-Id: If43e49b0544a5efba85a71dc0d227eca627854b5
Signed-off-by: Marcus Koontz <marcus.g.koontz@intel.com>

Fix logs in how it should use exceptions.

While debugging issue with openstack CI, I identified a number of places where
exception was not doing the logging properly. Thus, this change set.

Ref link on logging:
https://wiki.opendaylight.org/view/Logging_Best_Practices

Patch 2: minimize the changes, but yet follow the Logging_Best_Practices.
Patch 3: missed change in same file, to keep changes consistent on patch 2.

Change-Id: Ie3dae66f24467ee2dd1113e9c0dd6a5d6ab0df5a
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Add fixes for SFC data model changes.

This fix is really just commenting the code out until we get around to completing the OVS-SFC feature. The SFC model will likely change before then so it is best just to comment out the existing code to avoid any other build failures.

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

Restructure plugin IT tests to reuse TestBase methods

Patchset 2: rebase

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

Add TestWatcher so we know what test is running

Patch 2:
  - move testWatcher to base class
  - convert System.out.println in OvsdbPluginV3IT to use logger

Patch 3:
  - remove logs from mocked class (they never get called due to mock)

Change-Id: I21902bcd7595b16c8951ca04d1883c0745da6ba0
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Bug 2460 - Converting ipv4 to ipv4Prefix requires mask

Starting on fixes for Bug 1953, the validation of IPv4Prefix
will make sure mask is being provided.

Patch Set 2: Move the +"/32" to a common utility function.

Ref gerrit: https://git.opendaylight.org/gerrit/#/c/11384/

Change-Id: I84771d087d7ed1ec5cc8045e3efe3673e6969d5c
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Merge "Add OVSDB Northbound v3 postman collection"

Add OVSDB Northbound v3 postman collection

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

Merge "printCache: Error executing command: java.lang.NullPointerException"

Merge "Add template unit test for schemas/hardwarevtep"

Merge "Add template unit test for schema/openvswitch"

Update README with instructions on how to update bundles in karaf

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

printCache: Error executing command: java.lang.NullPointerException

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

Add template unit test for schemas/hardwarevtep

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

Add template unit test for schema/openvswitch

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

Merge "Fix critical Sonar Issue: ConfigurationServiceImpl.java"

Fix critical Sonar Issue: ConfigurationServiceImpl.java

Fix for the following issue:
- Throwable.printStackTrace(...) should never be called

https://sonar.opendaylight.org/component/index#component=org.opendaylight.ovsdb%3Aplugin%3Amaster%3Asrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fovsdb%2Fplugin%2Fimpl%2FConfigurationServiceImpl.java&settings=issues&blocks=959%2C967&tab=issues&item=.js-filter-CRITICAL-issues&line=963

patch 2: replace 'e.fillInStackTrace()' with 'e'
patch 3: set log level to error

Change-Id: I91af7d1548b3c7563126ace9ad33a0f1b8877379
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Fix Critcal Sonar Issue: ConfigurationServiceImpl.java.

Fixes the following issues:
- Use a logger to log this exception.

https://sonar.opendaylight.org/component/index#component=org.opendaylight.ovsdb%3Aplugin%3Amaster%3Asrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fovsdb%2Fplugin%2Fimpl%2FConfigurationServiceImpl.java&settings=issues&blocks=959%2C967&tab=issues&item=.js-filter-CRITICAL-issues

Change-Id: I37465b04134122d932d48ea26655057bbf5ea3ca
Signed-off-by: Marcus Koontz <marcus.g.koontz@intel.com>

Fix Major Sonar Issues: SouthboundEvent.java.

Fixes the following issues:
- Removed semi-colon to fix empty statement error
- 12 Avoid using If/If else/For statements without curly braces

https://sonar.opendaylight.org/component#component=org.opendaylight.ovsdb:openstack.net-virt:master:src/main/java/org/opendaylight/ovsdb/openstack/netvirt/SouthboundEvent.java

Change-Id: Ic3adee50ec4a3f6f0d48c3d8330cfde9654d423f
Signed-off-by: Marcus Koontz <marcus.g.koontz@intel.com>

Fix Major Sonar Issues: NorthboundEvent.java.

Fixes the following issues:
- 22 Avoid using If/If else/For statements without curly braces

https://sonar.opendaylight.org/component#component=org.opendaylight.ovsdb:openstack.net-virt:master:src/main/java/org/opendaylight/ovsdb/openstack/netvirt/NorthboundEvent.java

Change-Id: I84271baabf37d732fbaf34869b4a60ba697789e8
Signed-off-by: Marcus Koontz <marcus.g.koontz@intel.com>

Fix Major Sonar Issues: LBaaSPoolHandler.java.

Fixes the following issues:
- 13 Avoid using If/If else/For statements without curly braces

https://sonar.opendaylight.org/component#component=org.opendaylight.ovsdb:openstack.net-virt:master:src/main/java/org/opendaylight/ovsdb/openstack/netvirt/LBaaSPoolHandler.java

Change-Id: I7655947f05731f213a54fee2c39b00a0ae8c945a
Signed-off-by: Marcus Koontz <marcus.g.koontz@intel.com>

Fix Major & Minor Sonar Issues: LBaaSHandler.java.

Fixes the following issues:
- Comments should not be located at the end of lines of code
- 9 Avoid using If/If else/For statements without curly braces

https://sonar.opendaylight.org/component#component=org.opendaylight.ovsdb:openstack.net-virt:master:src/main/java/org/opendaylight/ovsdb/openstack/netvirt/LBaaSHandler.java

Change-Id: I30152177b433cc3de4034e174fd4a42641f81994
Signed-off-by: Marcus Koontz <marcus.g.koontz@intel.com>

Add log to indicate the schema version

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

Make method names consistent

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

Fix tests where schema version is not supported

Tests were failing on earlier ovs versions because the schema version did not support certain tables. In these cases a bridge was created and not deleted because only part of the test ran.

Add assumeTrue() around the schema version check to all tests with a problem schema version to allow the test to continue.

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

Use features-integration as base for ovsdb karaf distro

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

Merge "Remove unused schema code."

Merge changes Ibe8d0d9f,I31935e1e,I0f1f3bee

* changes:
  Add checkstyle to utils.mdsal-openflow
  Add checkstyle to utils.config
  Add checkstyle to plugin-shell

Merge changes I8fe016c2,I75842f8e

* changes:
  Add assumeTrue around hardware_vtep schema check
  Add README.txt file with details on how to use hardware_vtep schema

Add checkstyle to utils.mdsal-openflow

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

Add checkstyle to utils.config

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

Add checkstyle to plugin-shell

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

Remove unused schema code.

The existing integration tests in schema were adapted to pax-exam and moved to the integrationtest bundle.

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

Add assumeTrue around hardware_vtep schema check

The images used in the jenkins jobs do not support hardware_vtep so this is needed to pass the test.

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

Merge "Add pax-exam integration-tests for hardware_vtep"

Add README.txt file with details on how to use hardware_vtep schema

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

Add pax-exam integration-tests for hardware_vtep

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

Merge "Add pax exam schema integration tests"

Add pax exam schema integration tests

Patch Set 4: Fix verify error

Patch Set 3: checkstyle fixes

Patch Set 2:
- Adapted old tests to use pax exam
- Fixed some issues in existing tests that were not correct. In most cases this would be cleaning up orphaned columns in tables.
- Added new tests to cover untested code
- Added monitoring code to all tests to test that integration
- Refactored some methods to be common

Patch Set 1: Early draft, not to be merged

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

Fix for Sonar Issue(A catch statement should never catch throwable)

Signed-off-by: Bindya Narayan <bindya.narayan@intel.com>

Fix for Sonar "Avoid Catching Throwable" error.

This patch fixes the error caught by sonar by catching exception
and not throwing 'Throwable' in processGetTableSchema method which
seems to be redundant.

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

Fix Sonar issue: Utility classes should not have a public constructor

Fix name of artifact to be less generic.

Change-Id: Ib09fd1c49d2d666ef2d5f19bbe4f6971301a7d8d
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Merge "Complete PrintCache unit-test"

Merge "Bug 2021 - NORMAL and CONTROLLER flows should not use hard coded strings"

Add jacoco-maven-plugin to the rest of the projects in OVSDB

adding the plugin to the rest of the projects will allow Sonar to get coverage information on the projects.

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

Complete PrintCache unit-test

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

Bug 2021 - NORMAL and CONTROLLER flows should not use hard coded strings

The output port value should be using the node connector id assembled
from the sal-compatibility library, which is provided by the controller.

Patch 3: Add feedback from code review

Change-Id: I3aa503bfd54e3cfcd7fa6dd730849b88f90a38c1
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Updating the control.vm.box in Vagrantfile to use the right type

Change-Id: I18a4e14767318475abad245c8deeb3ead3e27090
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.com>

Merge "Bug 2316 - Fixing equals() in LoadBalancerMember class to be compliant and also use .equals instead of == for comparing Strings"

Bug 2316 - Fixing equals() in LoadBalancerMember class to be compliant
and also use .equals instead of == for comparing Strings

Change-Id: I5a4cad3f66540abe9c9d8ff5b2d2211aa9b6d0f4
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.com>

Add adsal-compatibility to ovs-sfc features since sal-compatibility was added to md-sal-openflow

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

Merge "Fixes to get ovsdb running in eclipse"

Fixes to get ovsdb running in eclipse

This gerritt fixes the lifecycle and plugin maven errors. With this fix I don't see any more major errors. There are still some warnings that I will look at in a later commit.

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

Add unit-tests for ConfigProperties

Patch 4: Implement feedback from shague

Change-Id: I23cc7b70032e3b884171b00d52fa5c8630fda587
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Merge "Fix issue found by sonar: Method call on object which may be null"

Add workaround because sfc models changed.

This workaround simply comments out the offending lines. The ovs-sfc code will be updated when the models settle down.

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

Fix issue found by sonar: Method call on object which may be null

Change-Id: I937dddbe08fa12cbe38ad01a5990a90c4ac19df1
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Bug 2214 - Unable to set the external_ids field of the Qos object

Fix typo on external_ids for QoS table, to conform with
http://openvswitch.org/ovs-vswitchd.conf.db.5.pdf page 38.

Change-Id: I97ac25bf1355b18d9944de3e78bbe6edadd3399d
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Merge changes I89f90cf4,I248403b7

* changes:
  Add unit-tests for OvsdbNorthboundV3
  Add printCache unit tests.

Update readme to include karaf run description.

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

Add unit-tests for OvsdbNorthboundV3

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

Add printCache unit tests.

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

Merge "Ignore testGetOpenflowVersion: it is no longer applicable"

Merge "Enable SSL connection for ovsdb server."

Enable SSL connection for ovsdb server.

Change-Id: I61213343be3aec2907b9b5d3b1f3446335d55fde
Signed-off-by: Hsin-Yi Shen <syshen66@gmail.com>

Ignore testGetOpenflowVersion: it is no longer applicable

The openFlow version is now determined by configuration's getProperty("ovsdb.of.version", "1.3").
Thus, the ovs version attribute (returned by ovs' OpenVSwitch table) is not used to determine what is
the openFlow version chosen by netVirtConfigurationService.

See: https://git.opendaylight.org/gerrit/#/c/11084/
     https://github.com/opendaylight/ovsdb/commit/2bc58c9cca16dc3e389cdfc18593578748fd52d5

Patch 2: include Sam's feedback

Change-Id: Icf4f3f633b2131a4521ef2509c02bcb0c7223abf
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Merge "Add template unit-tests for bundles that don't have any."

Add template unit-tests for bundles that don't have any.

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

Bug 2192 - Part 3 - Make ovsdb library not depend on ovsdb.utils.config

The ovsdb library is designed as Java modular component that can work in
both OSGi and non-OSGi environments. Thus, it should depend on as few
bundles/jar as possible.

Patch 4: Implement feedback from Sam H.

Change-Id: If2aca5ac2f2e883907e28d9ddb7986f0a5c8c1ab
Signed-off-by: Hsin-Yi Shen <syshen66@gmail.com>
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Fix NeutronIT testGetTunnelEndpoint

The test was broken in that it attempted to update all fields of the OpenVSwitch
row. That is not valid due to immutable columns, such as _version.

Change-Id: I214e96914d85a0ca8321bf7c3e498f9a46f24f0e
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Cleaning up code of InventoryServiceImpl to remove function updateOFBridgeName() that is never used.

Change-Id: I68e32b090ac8701cd08e252df294ef83785a6055
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.com>

Update readme with minor corrections.

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

Bug 2192 - Part2 - Move getProperty() to a common util module

With this change, the calls to obtain non-default configurations for OVSDB
have been moved to a single function.

Patch 2: Rename utils.common to utils.config
         Remove utils.config dependency from common/parent/pom.xml (not needed by all)

Change-Id: Ibf7e1ee2747c5cafc59da97b5f1f8a4390d95ca2
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Revert "Retructuring the utils folder to make it a broader toolkit beyond just a mdsal toolkit. This will make room for adding the config prop utils."

This reverts commit 353b7ec925f3a82aabf52bb21a40189397c5176e.

Change-Id: I73a57bc8d6c2fee5bb79838e99af1500e0b99cc1
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Retructuring the utils folder to make it a broader toolkit beyond just
a mdsal toolkit. This will make room for adding the config prop utils.

Change-Id: I3df261ece22b9674c8e7a9cbd145ae766c961e64
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.com>

Merge "Add missing feature dependencies to ovsdb karaf distro"

Add missing feature dependencies to ovsdb karaf distro

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

Bug 2076 - Routing does not work unless node contains vms on all vlans (segIds)

The logic for determining whether or not entries should be added to table 70 was
not taking into consideration the fact that vms on non-local vlans are also needed.

The fix simply fixes the logic used to filter out L3 fwd entries.

Change-Id: I0b7836a8f267c6346c24dad0adae6af87471dd57
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Bug 2192 - Fix access to custom.properties, so non-default values can be provided

When OVSDB changed to use Karaf, the custom variables in config.ini got moved to
custom.properties. With that change, OVSDB was not grabbing its custom values from
the right place, causing all OVSDB related properties to get ignored.

Change-Id: Ic43d547e3d30e69c86718ac03314c4b74c61b561
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
(cherry picked from commit ff43d8c79ab0157a6d1b36f3c75c350878c66083)

Merge "Fix some compiler warnings."

Fix some compiler warnings.

1. Duplicated dependencies in many poms.
2. Dynamic versions for artifact ids.

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

Update README to include all current projects.

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

Add import statements for o.o.o.o.n.a.ConfigurationService

Change-Id: Id72133738a78dea3f4fb6b9c87bf726b1f6aeb04
Signed-off-by: Cédric Ollivier <ollivier.cedric@gmail.com>

Incrementing versions by 0.1.0 for post-Helium master branch

Change-Id: I83919752fe6ba2cbb2fbfd2ace0e060dd6b3f019
Signed-off-by: Colin Dixon <colin@colindixon.com>

Bug 2096 - LBaaS pool member deletion is not resetting rules properly

The LBConfig cloning is losing the segmentationID. This is fixed here.

Change-Id: If6f828fca09c3a02fa9bfc16ef97c439a702ae4a
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.com>

Bug 2092 - Invalid code in LBaaS handlers that extracts MAC address of member without paying attention to the subnet-ID.

This prevent ID address overlap of LBaaS pools and wrong rules written to switch.

Change-Id: I9624a7e503810a5cbc8db0ca00b2cfc9d1dc8191
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.com>

Merge "Bug 1806 - Null destination InetAddress in OF13Provider.handleInterfaceUpdate"

Bug 1806 - Null destination InetAddress in OF13Provider.handleInterfaceUpdate

The error is caused when ovsdb can not find the local_ip address from the open_vSwitch table other_config row. this address is used to get the src or dst ip address to be used for tunnels.

Added null chcks around the src and dst ip addresses. Log a warning if either is missing.

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

Merge "Bug 2013 Vlan flows needs merging from traditional rules to a single L2Fwding Service"

Bug 2013 Vlan flows needs merging from traditional rules to a single L2Fwding Service

Add fixes for the interface delete cases:
- include the latest rules that needed to be removed
- rework the bcast rule in the piece where a port is removed from the output list. The existing method assumed that the only actions in the apply actions case are all output actions, but the vlan bcast rule has an output to ethPort, followed by popVlan and then the output to flood ports. Also the md-sal does not maintain the order that rules were entered so that cannot be relied upon. This required creating anew method to handle the cases and ensure the proper actions are recreated.

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