From 2f8ebf4180a17c8b58b8f7659e7eaa086d413b7a Mon Sep 17 00:00:00 2001 From: Artom Lifshitz Date: Thu, 7 Nov 2019 16:31:44 -0500 Subject: [PATCH] Skip test_rx_queue_size test_rx_queue_size requires configuration that it is not setting itself and relying on the deployment to set it. We can either refactor it to set the configuration that it needs, or remove it altogether if it adds no value. Until one of those two things happen, skip it. Story: 2006820 Task: 37386 Change-Id: I22c73c023dcc6b9f4c5c67bd5bb8c24ef1edbb80 --- .../api/compute/test_rx_tx_queue_size.py | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/whitebox_tempest_plugin/api/compute/test_rx_tx_queue_size.py b/whitebox_tempest_plugin/api/compute/test_rx_tx_queue_size.py index 7d79f3b3..4fdfa0bb 100644 --- a/whitebox_tempest_plugin/api/compute/test_rx_tx_queue_size.py +++ b/whitebox_tempest_plugin/api/compute/test_rx_tx_queue_size.py @@ -18,6 +18,7 @@ from oslo_log import log as logging from tempest.common import waiters from tempest import config from tempest.lib.common.utils import data_utils +from tempest.lib import decorators from whitebox_tempest_plugin.api.compute import base @@ -69,10 +70,14 @@ class RxTxQueueSizeTest(base.BaseWhiteboxComputeTest): # Required in /etc/nova/nova.conf # [libvirt] # rx_queue_size = 1024 + # TODO(artom) We either need to refactor this to use the + # ServiceManager.config_option context manager, or remove this test + # altogether if it adds no value. + @decorators.skip_because(bug='2006820', bug_type='storyboard') def test_rx_queue_size(self): domain = self.get_server_xml(self.server_id) driver = domain.find( "devices/interface[@type='bridge']/driver[@name='vhost']") self.assertEqual( - driver.attrib['rx_queue_size'], '1024', + driver.attrib('rx_queue_size'), '1024', "Can't find interface with the proper rx_queue_size")