Remove unrelated health check for sriov driver

Remove the ovs default bridge presence check from sriov health
checks. It is unrelated to sriov and adds overhead.
Additionally, derived from HealthHandler to staisfy base
driver requirement.

Change-Id: I8283c2931336250c0dc1f69e1e841c591fcf8cf5
This commit is contained in:
ashish.billore 2020-03-04 16:12:29 +09:00 committed by Ashish Billore
parent 44c0a307b5
commit ca0c7a80eb
1 changed files with 3 additions and 12 deletions

View File

@ -27,13 +27,15 @@ from kuryr_kubernetes.cni.binding import base as b_base
from kuryr_kubernetes import config
from kuryr_kubernetes import constants
from kuryr_kubernetes import exceptions
from kuryr_kubernetes.handlers import health
from kuryr_kubernetes import utils
LOG = logging.getLogger(__name__)
CONF = cfg.CONF
class VIFSriovDriver(b_base.BaseBindingDriver):
class VIFSriovDriver(health.HealthHandler, b_base.BaseBindingDriver):
def __init__(self):
super().__init__()
self._lock = None
@ -383,17 +385,6 @@ class VIFSriovDriver(b_base.BaseBindingDriver):
vf_index, pf)
raise
def is_alive(self):
bridge_name = CONF.neutron_defaults.ovs_bridge
try:
with b_base.get_ipdb() as h_ipdb:
h_ipdb.interfaces[bridge_name]
return True
except Exception:
LOG.warning("Default OVS bridge %s does not exist on "
"the host.", bridge_name)
return False
def _get_device_pf_mapping(self):
"""Return a mapping in format {<physnet_name>:[<pf_name>, ...]}"""