octavia 12.0.0.0rc2__py3-none-any.whl → 13.0.0.0rc1__py3-none-any.whl

octavia/tests/unit/amphorae/backends/agent/api_server/test_amphora_info.py

@@ -185,6 +185,7 @@ class TestAmphoraInfo(base.TestCase):
                          u'packages': {},
                          u'topology': u'SINGLE',
                          u'topology_status': u'OK'}
+        self.useFixture(test_utils.OpenFixture('/etc/tuned/active_profile'))
         actual = self.amp_info.compile_amphora_details()
         self.assertEqual(expected_dict, actual.json)
         m_count.assert_called_once_with(sorted(mget_loadbalancers()))
@@ -277,6 +278,7 @@ class TestAmphoraInfo(base.TestCase):
                          u'packages': {},
                          u'topology': u'SINGLE',
                          u'topology_status': u'OK'}
+        self.useFixture(test_utils.OpenFixture('/etc/tuned/active_profile'))
         actual = self.amp_info.compile_amphora_details(self.lvs_driver)
         self.assertEqual(expected_dict, actual.json)
         api_server.VERSION = original_version

octavia/tests/unit/amphorae/backends/agent/api_server/test_osutils.py

@@ -106,7 +106,7 @@ class TestOSUtils(base.TestCase):
                                                  '192.0.2.2', 16)
 
         mock_interface_file.assert_called_once_with(
-            name='eth1',
+            name='eth1', if_type="lo",
             addresses=[{"address": "192.0.2.2", "prefixlen": 16}])
         mock_interface.write.assert_called_once()
 

octavia/tests/unit/amphorae/backends/agent/api_server/test_plug.py

@@ -265,7 +265,10 @@ class TestPlug(base.TestCase):
                 "BaseOS.write_port_interface_file")
     @mock.patch("octavia.amphorae.backends.agent.api_server.osutils."
                 "BaseOS.bring_interface_up")
-    def test_plug_network(self, mock_if_up, mock_write_port_interface,
+    @mock.patch("octavia.amphorae.backends.agent.api_server.util."
+                "send_member_advertisements")
+    def test_plug_network(self, mock_send_member_adv,
+                          mock_if_up, mock_write_port_interface,
                           mock_netns, mock_iproute,
                           mock_by_mac, mock_interface_exists, mock_webob):
         fixed_ips = [
@@ -282,12 +285,13 @@ class TestPlug(base.TestCase):
             self.test_plug.plug_network(FAKE_MAC_ADDRESS, fixed_ips, 1400)
 
         mock_write_port_interface.assert_called_once_with(
-            interface='eth0', fixed_ips=fixed_ips, mtu=mtu)
-        mock_if_up.assert_called_once_with('eth0', 'network')
+            interface='eth2', fixed_ips=fixed_ips, mtu=mtu)
+        mock_if_up.assert_called_once_with('eth2', 'network')
+        mock_send_member_adv.assert_called_once_with(fixed_ips)
 
         mock_webob.Response.assert_any_call(
             json={'message': 'OK',
-                  'details': 'Plugged on interface eth0'},
+                  'details': 'Plugged on interface eth2'},
             status=202)
 
     @mock.patch.object(plug, "webob")
@@ -300,7 +304,10 @@ class TestPlug(base.TestCase):
                 "BaseOS.write_port_interface_file")
     @mock.patch("octavia.amphorae.backends.agent.api_server.osutils."
                 "BaseOS.bring_interface_up")
-    def test_plug_network_existing_interface(self, mock_if_up,
+    @mock.patch("octavia.amphorae.backends.agent.api_server.util."
+                "send_member_advertisements")
+    def test_plug_network_existing_interface(self, mock_send_member_adv,
+                                             mock_if_up,
                                              mock_write_port_interface,
                                              mock_netns, mock_by_mac,
                                              mock_interface_exists,
@@ -327,6 +334,7 @@ class TestPlug(base.TestCase):
         mock_write_port_interface.assert_called_once_with(
             interface=FAKE_INTERFACE, fixed_ips=fixed_ips, mtu=mtu)
         mock_if_up.assert_called_once_with(FAKE_INTERFACE, 'network')
+        mock_send_member_adv.assert_called_once_with(fixed_ips)
 
         mock_webob.Response.assert_any_call(
             json={'message': 'OK',
@@ -344,8 +352,10 @@ class TestPlug(base.TestCase):
                 "BaseOS.write_vip_interface_file")
     @mock.patch("octavia.amphorae.backends.agent.api_server.osutils."
                 "BaseOS.bring_interface_up")
+    @mock.patch("octavia.amphorae.backends.agent.api_server.util."
+                "send_member_advertisements")
     def test_plug_network_on_vip(
-            self, mock_if_up, mock_write_vip_interface,
+            self, mock_send_member_adv, mock_if_up, mock_write_vip_interface,
             mock_netns, mock_by_mac, mock_interface_exists, mock_webob):
         fixed_ips = [
             {'ip_address': FAKE_IP_IPV4,
@@ -394,9 +404,47 @@ class TestPlug(base.TestCase):
             fixed_ips=fixed_ips, mtu=mtu)
 
         mock_if_up.assert_called_once_with(FAKE_INTERFACE, 'vip')
+        mock_send_member_adv.assert_called_once_with(fixed_ips)
 
         mock_webob.Response.assert_any_call(
             json={'message': 'OK',
                   'details': 'Updated existing interface {}'.format(
                       FAKE_INTERFACE)},
             status=202)
+
+    @mock.patch('pyroute2.NetNS', create=True)
+    def test__netns_get_next_interface(self, mock_netns):
+        netns_handle = mock_netns.return_value.__enter__.return_value
+
+        netns_handle.get_links.return_value = [
+            {'attrs': [['IFLA_IFNAME', 'lo']]},
+        ]
+
+        ifname = self.test_plug._netns_get_next_interface()
+        self.assertEqual('eth2', ifname)
+
+        netns_handle.get_links.return_value = [
+            {'attrs': [['IFLA_IFNAME', 'lo']]},
+            {'attrs': [['IFLA_IFNAME', 'eth1']]},
+            {'attrs': [['IFLA_IFNAME', 'eth2']]},
+            {'attrs': [['IFLA_IFNAME', 'eth3']]},
+        ]
+
+        ifname = self.test_plug._netns_get_next_interface()
+        self.assertEqual('eth4', ifname)
+
+        netns_handle.get_links.return_value = [
+            {'attrs': [['IFLA_IFNAME', 'lo']]},
+            {'attrs': [['IFLA_IFNAME', 'eth1']]},
+            {'attrs': [['IFLA_IFNAME', 'eth3']]},
+        ]
+
+        ifname = self.test_plug._netns_get_next_interface()
+        self.assertEqual('eth2', ifname)
+
+        netns_handle.get_links.return_value = [
+            {'attrs': [['IFLA_IFNAME', f'eth{idx}']]}
+            for idx in range(2, 1000)]
+
+        ifname = self.test_plug._netns_get_next_interface()
+        self.assertEqual('eth1000', ifname)

octavia/tests/unit/amphorae/backends/agent/api_server/test_util.py

@@ -406,3 +406,38 @@ class TestUtil(base.TestCase):
         util.send_vip_advertisements(LB_ID1)
         mock_get_int_name.assert_not_called()
         mock_send_advert.assert_not_called()
+
+    @mock.patch('octavia.amphorae.backends.utils.ip_advertisement.'
+                'send_ip_advertisement')
+    @mock.patch('octavia.amphorae.backends.utils.network_utils.'
+                'get_interface_name')
+    def test_send_member_advertisements(self, mock_get_int_name,
+                                        mock_send_advert):
+        # IPv4 fixed_ips
+        mock_get_int_name.side_effect = ['fake0', 'fake1']
+        fixed_ips = [{'ip_address': '192.0.2.1'},
+                     {'ip_address': '192.2.0.2'}]
+        util.send_member_advertisements(fixed_ips)
+        mock_send_advert.assert_has_calls(
+            [mock.call('fake0', fixed_ips[0]['ip_address'],
+                       net_ns=consts.AMPHORA_NAMESPACE),
+             mock.call('fake1', fixed_ips[1]['ip_address'],
+                       net_ns=consts.AMPHORA_NAMESPACE)])
+
+        # Mixed IPv4/IPv6
+        mock_send_advert.reset_mock()
+        mock_get_int_name.side_effect = ['fake0', 'fake1']
+        fixed_ips = [{'ip_address': '192.0.2.1'},
+                     {'ip_address': '2001:db8::2'}]
+        util.send_member_advertisements(fixed_ips)
+        mock_send_advert.assert_has_calls(
+            [mock.call('fake0', fixed_ips[0]['ip_address'],
+                       net_ns=consts.AMPHORA_NAMESPACE),
+             mock.call('fake1', fixed_ips[1]['ip_address'],
+                       net_ns=consts.AMPHORA_NAMESPACE)])
+
+        # Exception
+        mock_send_advert.reset_mock()
+        mock_get_int_name.side_effect = Exception('ERROR')
+        util.send_member_advertisements(fixed_ips)
+        mock_send_advert.assert_not_called()

octavia/tests/unit/amphorae/backends/health_daemon/test_health_daemon.py

@@ -319,6 +319,14 @@ class TestHealthDaemon(base.TestCase):
         stats_query_mock.show_stat.assert_called_once_with()
         stats_query_mock.get_pool_status.assert_called_once_with()
 
+    @mock.patch('octavia.amphorae.backends.utils.haproxy_query.HAProxyQuery')
+    def test_get_stats_exception(self, mock_query):
+        mock_query.side_effect = Exception('Boom')
+
+        stats, pool_status = health_daemon.get_stats('TEST')
+        self.assertEqual([], stats)
+        self.assertEqual({}, pool_status)
+
     @mock.patch('octavia.amphorae.backends.agent.api_server.'
                 'util.is_lb_running')
     @mock.patch('octavia.amphorae.backends.health_daemon.'

octavia/tests/unit/amphorae/backends/health_daemon/test_health_sender.py

@@ -93,6 +93,24 @@ class TestHealthSender(base.TestCase):
 
         sendto_mock.reset_mock()
 
+        # Test IPv6 path enclosed within square brackets ("[" and "]").
+        self.conf.config(group="health_manager",
+                         controller_ip_port_list=['[2001:0db8::f00d]:80'])
+        mock_getaddrinfo.return_value = [(socket.AF_INET6,
+                                          socket.SOCK_DGRAM,
+                                          socket.IPPROTO_UDP,
+                                          '',
+                                          ('2001:db8::f00d', 80, 0, 0))]
+
+        sender = health_sender.UDPStatusSender()
+
+        sender.dosend(SAMPLE_MSG)
+
+        sendto_mock.assert_called_once_with(SAMPLE_MSG_BIN,
+                                            ('2001:db8::f00d', 80, 0, 0))
+
+        sendto_mock.reset_mock()
+
         # Test IPv6 link-local address path
         self.conf.config(
             group="health_manager",

octavia/tests/unit/amphorae/backends/utils/test_interface.py

@@ -74,6 +74,7 @@ class TestInterface(base.TestCase):
                    '],\n'
                    '"mtu": 1450,\n'
                    '"name": "eth1",\n'
+                   '"if_type": "mytype",\n'
                    '"routes": [\n'
                    '{"dst": "0.0.0.0/0",\n'
                    '"gateway": "10.0.0.1"},\n'
@@ -107,6 +108,7 @@ class TestInterface(base.TestCase):
 
         expected_dict = {
             consts.NAME: "eth1",
+            consts.IF_TYPE: "mytype",
             consts.MTU: 1450,
             consts.ADDRESSES: [{
                 consts.ADDRESS: "10.0.0.2",
@@ -331,6 +333,7 @@ class TestInterface(base.TestCase):
                 mock_link, mock_addr, mock_route, mock_rule):
         iface = interface_file.InterfaceFile(
             name="eth1",
+            if_type="vip",
             mtu=1450,
             addresses=[{
                 consts.ADDRESS: '1.2.3.4',
@@ -445,6 +448,78 @@ class TestInterface(base.TestCase):
             mock.call(["post-up", "eth1"])
         ])
 
+    @mock.patch('pyroute2.IPRoute.rule')
+    @mock.patch('pyroute2.IPRoute.route')
+    @mock.patch('pyroute2.IPRoute.addr')
+    @mock.patch('pyroute2.IPRoute.link')
+    @mock.patch('pyroute2.IPRoute.get_links')
+    @mock.patch('pyroute2.IPRoute.link_lookup')
+    @mock.patch('pyroute2.IPRoute.get_rules')
+    @mock.patch('subprocess.check_output')
+    def test_up_backend(self, mock_check_output, mock_get_rules,
+                        mock_link_lookup, mock_get_links, mock_link, mock_addr,
+                        mock_route, mock_rule):
+        iface = interface_file.InterfaceFile(
+            name="eth1",
+            if_type="backend",
+            mtu=1450,
+            addresses=[{
+                consts.ADDRESS: '1.2.3.4',
+                consts.PREFIXLEN: 24
+            }],
+            routes=[],
+            rules=[],
+            scripts={
+                consts.IFACE_UP: [{
+                    consts.COMMAND: "post-up eth1"
+                }],
+                consts.IFACE_DOWN: [{
+                    consts.COMMAND: "post-down eth1"
+                }],
+            })
+
+        idx = mock.MagicMock()
+        mock_link_lookup.return_value = [idx]
+
+        mock_get_links.return_value = [{
+            consts.STATE: consts.IFACE_DOWN
+        }]
+        mock_get_rules.return_value = [{
+            'src_len': 32,
+            'attrs': {
+                'FRA_SRC': '1.1.1.1',
+                'FRA_TABLE': 20,
+                'FRA_PROTOCOL': 0
+            }
+        }]
+
+        controller = interface.InterfaceController()
+        controller.up(iface)
+
+        mock_link.assert_called_once_with(
+            controller.SET,
+            index=idx,
+            state=consts.IFACE_UP,
+            mtu=1450)
+
+        mock_addr.assert_has_calls([
+            mock.call(controller.ADD,
+                      index=idx,
+                      address='1.2.3.4',
+                      prefixlen=24,
+                      family=socket.AF_INET),
+        ])
+
+        mock_route.assert_called_once_with(
+            'dump', family=mock.ANY, match=mock.ANY)
+
+        # for 'backend' iface, we don't update the rules
+        mock_rule.assert_not_called()
+
+        mock_check_output.assert_has_calls([
+            mock.call(["post-up", "eth1"])
+        ])
+
     @mock.patch('pyroute2.IPRoute.rule')
     @mock.patch('pyroute2.IPRoute.route')
     @mock.patch('pyroute2.IPRoute.addr')
@@ -463,6 +538,7 @@ class TestInterface(base.TestCase):
                        mock_route, mock_rule):
         iface = interface_file.InterfaceFile(
             name="eth1",
+            if_type="vip",
             mtu=1450,
             addresses=[{
                 consts.ADDRESS: '1.2.3.4',
@@ -658,6 +734,7 @@ class TestInterface(base.TestCase):
                      mock_route, mock_rule):
         iface = interface_file.InterfaceFile(
             name="eth1",
+            if_type="vip",
             mtu=1450,
             addresses=[{
                 consts.DHCP: True,
@@ -722,6 +799,7 @@ class TestInterface(base.TestCase):
                   mock_link, mock_addr, mock_route, mock_rule):
         iface = interface_file.InterfaceFile(
             name="eth1",
+            if_type="vip",
             mtu=1450,
             addresses=[{
                 consts.ADDRESS: '1.2.3.4',
@@ -848,6 +926,7 @@ class TestInterface(base.TestCase):
                               mock_addr, mock_route, mock_rule):
         iface = interface_file.InterfaceFile(
             name="eth1",
+            if_type="vip",
             mtu=1450,
             addresses=[{
                 consts.ADDRESS: '1.2.3.4',
@@ -997,6 +1076,7 @@ class TestInterface(base.TestCase):
                                mock_route, mock_rule):
         iface = interface_file.InterfaceFile(
             name="eth1",
+            if_type="vip",
             mtu=1450,
             addresses=[{
                 consts.ADDRESS: '1.2.3.4',
@@ -1069,6 +1149,7 @@ class TestInterface(base.TestCase):
                        mock_addr, mock_route, mock_rule):
         iface = interface_file.InterfaceFile(
             name="eth1",
+            if_type="vip",
             mtu=1450,
             addresses=[{
                 consts.DHCP: True,

octavia/tests/unit/amphorae/backends/utils/test_interface_file.py

@@ -691,6 +691,7 @@ class TestInterfaceFile(base.TestCase):
                    '"prefixlen": 26}\n'
                    '],\n'
                    '"mtu": 1450,\n'
+                   '"if_type": "mytype",\n'
                    '"name": "eth1",\n'
                    '"routes": [\n'
                    '{"dst": "0.0.0.0/0",\n'
@@ -717,6 +718,7 @@ class TestInterfaceFile(base.TestCase):
 
         expected_dict = {
             consts.NAME: "eth1",
+            consts.IF_TYPE: "mytype",
             consts.MTU: 1450,
             consts.ADDRESSES: [{
                 consts.ADDRESS: "10.0.0.181",

octavia/tests/unit/amphorae/backends/utils/test_keepalivedlvs_query.py

@@ -52,6 +52,24 @@ KERNAL_FILE_SAMPLE_V6 = (
     "  -> [fd79:35e2::8f3f]:115C       "
     "Masq    2      0          0")
 
+KERNEL_FILE_SAMPLE_MIXED = (
+    "IP Virtual Server version 1.2.1 (size=4096)\n"
+    "Prot LocalAddress:Port Scheduler Flags\n"
+    "  -> RemoteAddress:Port Forward Weight ActiveConn InActConn\n"
+    "UDP  [fd79:35e2:9963:0000:f816:3eff:fe6d:7a2a]:1E61 rr\n"
+    "  -> [fd79:35e2:9963:0000:f816:3eff:feca:b7bf]:08AE      "
+    "Masq    3      0          0\n"
+    "  -> [fd79:35e2:9963:0000:f816:3eff:fe9d:94df]:0D05      "
+    "Masq    2      0          0\n"
+    "  -> [fd79:35e2::8f3f]:115C       "
+    "Masq    2      0          0\n"
+    "UDP  0A000025:1E61 rr\n"
+    "  -> 0A000023:0D05      Masq    2      0          0\n"
+    "  -> 0A000019:08AE      Masq    3      0          0\n"
+    "UDP  0A000025:FF12 rr\n"
+    "  -> 0A000023:12AB      Masq    2      0          0\n"
+    "  -> 0A000019:BF2E      Masq    3      0          0")
+
 CFG_FILE_TEMPLATE_v4 = (
     "# Configuration for Listener %(listener_id)s\n\n"
     "net_namespace %(ns_name)s\n\n"
@@ -190,6 +208,37 @@ CFG_FILE_TEMPLATE_mixed = (
     "    # Member %(member_id5)s is disabled\n\n"
     "}")
 
+CFG_FILE_TEMPLATE_mixed_no_ipv6_member = (
+    "# Configuration for Listener %(listener_id)s\n\n"
+    "net_namespace %(ns_name)s\n\n"
+    "virtual_server_group ipv4-group {\n"
+    "   10.0.0.37 7777\n"
+    "}\n\n"
+    "virtual_server group ipv4-group {\n"
+    "    lb_algo rr\n"
+    "    lvs_method NAT\n"
+    "    protocol udp\n\n\n"
+    "    # Configuration for Pool %(pool_id)s\n"
+    "    # Configuration for Member %(member_id1)s\n"
+    "    real_server 10.0.0.25 2222 {\n"
+    "        weight 3\n"
+    "        MISC_CHECK {\n\n"
+    "            misc_path \"/usr/bin/check_script.sh\"\n\n"
+    "            misc_timeout 5\n\n"
+    "        }\n\n"
+    "    }\n\n"
+    "    # Member %(member_id2)s is disabled\n\n"
+    "}\n"
+    "virtual_server_group ipv6-group {\n"
+    "   fd79:35e2:9963:0:f816:3eff:fe6d:7a2a 7777\n"
+    "}\n\n"
+    "virtual_server group ipv6-group {\n"
+    "    lb_algo rr\n"
+    "    lvs_method NAT\n"
+    "    protocol udp\n\n\n"
+    "    # Configuration for Pool %(pool_id)s\n"
+    "}")
+
 CFG_FILE_TEMPLATE_DISABLED_LISTENER = (
     "# Listener %(listener_id)s is disabled \n\n"
     "net_namespace %(ns_name)s\n\n"
@@ -233,6 +282,7 @@ class LvsQueryTestCase(base.TestCase):
         self.member_id4_v6 = uuidutils.generate_uuid()
         self.member_id5_v6 = uuidutils.generate_uuid()
         self.listener_id_mixed = uuidutils.generate_uuid()
+        self.listener_id_mixed_no_ipv6_member = uuidutils.generate_uuid()
         self.pool_id_mixed = uuidutils.generate_uuid()
         self.disabled_listener_id = uuidutils.generate_uuid()
         cfg_content_v4 = CFG_FILE_TEMPLATE_v4 % {
@@ -264,6 +314,15 @@ class LvsQueryTestCase(base.TestCase):
             'member_id4': self.member_id4_v6,
             'member_id5': self.member_id5_v6
         }
+        cfg_content_mixed_no_ipv6_member = (
+            CFG_FILE_TEMPLATE_mixed_no_ipv6_member % {
+                'listener_id': self.listener_id_mixed,
+                'ns_name': constants.AMPHORA_NAMESPACE,
+                'pool_id': self.pool_id_mixed,
+                'member_id1': self.member_id1_v4,
+                'member_id2': self.member_id2_v4
+            }
+        )
         cfg_content_disabled_listener = (
             CFG_FILE_TEMPLATE_DISABLED_LISTENER % {
                 'listener_id': self.listener_id_v6,
@@ -277,6 +336,10 @@ class LvsQueryTestCase(base.TestCase):
         self.useFixture(test_utils.OpenFixture(
             util.keepalived_lvs_cfg_path(self.listener_id_mixed),
             cfg_content_mixed))
+        self.useFixture(test_utils.OpenFixture(
+            util.keepalived_lvs_cfg_path(
+                self.listener_id_mixed_no_ipv6_member),
+            cfg_content_mixed_no_ipv6_member))
         self.useFixture(test_utils.OpenFixture(
             util.keepalived_lvs_cfg_path(self.disabled_listener_id),
             cfg_content_disabled_listener))
@@ -300,7 +363,7 @@ class LvsQueryTestCase(base.TestCase):
                                        'Weight': '2',
                                        'ActiveConn': '0',
                                        'InActConn': '0'}}
-        self.assertEqual((True, expected), result)
+        self.assertEqual(expected, result)
 
         # Ipv6 resolver
         input_listener_ip_port = [
@@ -327,7 +390,45 @@ class LvsQueryTestCase(base.TestCase):
                          'Weight': '2',
                          'ActiveConn': '0',
                          'InActConn': '0'}}
-        self.assertEqual((True, expected), result)
+        self.assertEqual(expected, result)
+
+        # mixed resolver
+        input_listener_ip_port = [
+            '[fd79:35e2:9963:0:f816:3eff:fe6d:7a2a]:7777',
+            '10.0.0.37:7777']
+        mock_check_output.return_value = KERNEL_FILE_SAMPLE_MIXED
+        result = lvs_query.get_listener_realserver_mapping(
+            target_ns, input_listener_ip_port,
+            health_monitor_enabled=True)
+        expected = {'[fd79:35e2:9963:0:f816:3eff:feca:b7bf]:2222':
+                    {'status': constants.UP,
+                     'Forward': 'Masq',
+                     'Weight': '3',
+                     'ActiveConn': '0',
+                     'InActConn': '0'},
+                    '[fd79:35e2:9963:0:f816:3eff:fe9d:94df]:3333':
+                        {'status': constants.UP,
+                         'Forward': 'Masq',
+                         'Weight': '2',
+                         'ActiveConn': '0',
+                         'InActConn': '0'},
+                    '[fd79:35e2::8f3f]:4444':
+                        {'status': constants.UP,
+                         'Forward': 'Masq',
+                         'Weight': '2',
+                         'ActiveConn': '0',
+                         'InActConn': '0'},
+                    '10.0.0.25:2222': {'status': 'UP',
+                                       'Forward': 'Masq',
+                                       'Weight': '3',
+                                       'ActiveConn': '0',
+                                       'InActConn': '0'},
+                    '10.0.0.35:3333': {'status': 'UP',
+                                       'Forward': 'Masq',
+                                       'Weight': '2',
+                                       'ActiveConn': '0',
+                                       'InActConn': '0'}}
+        self.assertEqual(expected, result)
 
         # negetive cases
         mock_check_output.return_value = KERNAL_FILE_SAMPLE_V4
@@ -335,7 +436,7 @@ class LvsQueryTestCase(base.TestCase):
             result = lvs_query.get_listener_realserver_mapping(
                 target_ns, [listener_ip_port],
                 health_monitor_enabled=True)
-            self.assertEqual((False, {}), result)
+            self.assertEqual({}, result)
 
         mock_check_output.return_value = KERNAL_FILE_SAMPLE_V6
         for listener_ip_port in [
@@ -344,7 +445,7 @@ class LvsQueryTestCase(base.TestCase):
             result = lvs_query.get_listener_realserver_mapping(
                 target_ns, [listener_ip_port],
                 health_monitor_enabled=True)
-            self.assertEqual((False, {}), result)
+            self.assertEqual({}, result)
 
     def test_get_lvs_listener_resource_ipports_nsname(self):
         # ipv4
@@ -526,7 +627,7 @@ class LvsQueryTestCase(base.TestCase):
             mock.Mock(st_mtime=1234),
             mock.Mock(st_mtime=1234),
         )
-        mock_get_mapping.return_value = (False, {})
+        mock_get_mapping.return_value = {}
         res = lvs_query.get_lvs_listener_pool_status(self.listener_id_v4)
         expected = {
             'lvs':
@@ -629,6 +730,29 @@ class LvsQueryTestCase(base.TestCase):
         res = lvs_query.get_lvs_listeners_stats()
         self.assertEqual({}, res)
 
+        # listener for both ipv4 and ipv6, but no member in the ipv6 pool
+        mock_is_running.return_value = True
+        mock_get_listener.return_value = [
+            self.listener_id_mixed_no_ipv6_member]
+        output_list = list()
+        output_list.append(IPVSADM_OUTPUT_TEMPLATE % {
+            "listener_ipport": "10.0.0.37:7777",
+            "member1_ipport": "10.0.0.25:2222",
+            "member2_ipport": "10.0.0.35:3333"})
+        output_list.append(IPVSADM_STATS_OUTPUT_TEMPLATE % {
+            "listener_ipport": "10.0.0.37:7777",
+            "member1_ipport": "10.0.0.25:2222",
+            "member2_ipport": "10.0.0.35:3333"})
+        mock_check_output.side_effect = output_list
+        res = lvs_query.get_lvs_listeners_stats()
+        # We can check the expected result reference the stats sample,
+        # that means this func can compute the stats info of single listener.
+        expected = {self.listener_id_mixed_no_ipv6_member: {
+            'status': constants.OPEN,
+            'stats': {'bin': 6387472, 'stot': 5, 'bout': 7490,
+                      'ereq': 0, 'scur': 0}}}
+        self.assertEqual(expected, res)
+
     @mock.patch('subprocess.check_output')
     @mock.patch("octavia.amphorae.backends.agent.api_server.util."
                 "is_lvs_listener_running", return_value=True)