Merge "Log k8s endpoints being ignored by Kuryr"
This commit is contained in:
commit
f0b3f6a775
|
@ -50,9 +50,10 @@ class LBaaSSpecHandler(k8s_base.ResourceEventHandler):
|
|||
lbaas_spec = self._get_lbaas_spec(service)
|
||||
|
||||
if self._should_ignore(service):
|
||||
LOG.debug("Skiping Kubernetes service of an unsupported kind or "
|
||||
"without a selector as Kubernetes does not create an "
|
||||
"endpoint object for it.")
|
||||
LOG.debug("Skipping Kubernetes service %s of an unsupported kind "
|
||||
"or without a selector as Kubernetes does not create "
|
||||
"an endpoint object for it.",
|
||||
service['metadata']['name'])
|
||||
return
|
||||
|
||||
if self._has_lbaas_spec_changes(service, lbaas_spec):
|
||||
|
@ -242,6 +243,8 @@ class LoadBalancerHandler(k8s_base.ResourceEventHandler):
|
|||
def on_present(self, endpoints):
|
||||
lbaas_spec = self._get_lbaas_spec(endpoints)
|
||||
if self._should_ignore(endpoints, lbaas_spec):
|
||||
LOG.debug("Ignoring Kubernetes endpoints %s",
|
||||
endpoints['metadata']['name'])
|
||||
return
|
||||
|
||||
lbaas_state = self._get_lbaas_state(endpoints)
|
||||
|
|
|
@ -90,7 +90,7 @@ class TestLBaaSSpecHandler(test_base.TestCase):
|
|||
m_handler._set_lbaas_spec.assert_not_called()
|
||||
|
||||
def test_on_present_no_selector(self):
|
||||
svc_event = mock.sentinel.svc_event
|
||||
svc_event = {'metadata': {'name': 'dummy_name'}}
|
||||
old_spec = mock.sentinel.old_spec
|
||||
|
||||
m_handler = mock.Mock(spec=h_lbaas.LBaaSSpecHandler)
|
||||
|
|
Loading…
Reference in New Issue