From 5836ffe2db39fa0ae6c6f89387150336fdd1399e Mon Sep 17 00:00:00 2001 From: Kyr Shatskyy Date: Wed, 7 Aug 2024 00:54:00 +0200 Subject: [PATCH] suite/util: list_lock() once for get_arch() We only want to try to list_locks() for getting arch by machine_type when scheduling a suite. Otherwise we've got into an infinite loop and console log flooded with useless error messages. Signed-off-by: Kyr Shatskyy --- teuthology/suite/test/test_util.py | 4 ++-- teuthology/suite/util.py | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/teuthology/suite/test/test_util.py b/teuthology/suite/test/test_util.py index 20a7dcc88..daa583023 100644 --- a/teuthology/suite/test/test_util.py +++ b/teuthology/suite/test/test_util.py @@ -103,7 +103,7 @@ def test_get_branch_info(self, m_get): def test_get_arch_fail(self, m_query): m_query.list_locks.return_value = False util.get_arch('magna') - m_query.list_locks.assert_called_with(machine_type="magna", count=1) + m_query.list_locks.assert_called_with(machine_type="magna", count=1, tries=1) @patch('teuthology.lock.query') def test_get_arch_success(self, m_query): @@ -111,7 +111,7 @@ def test_get_arch_success(self, m_query): result = util.get_arch('magna') m_query.list_locks.assert_called_with( machine_type="magna", - count=1 + count=1, tries=1 ) assert result == "arch" diff --git a/teuthology/suite/util.py b/teuthology/suite/util.py index 8999f9eaf..db0cf11cb 100644 --- a/teuthology/suite/util.py +++ b/teuthology/suite/util.py @@ -263,7 +263,7 @@ def get_arch(machine_type): :returns: A string or None """ - result = teuthology.lock.query.list_locks(machine_type=machine_type, count=1) + result = teuthology.lock.query.list_locks(machine_type=machine_type, count=1, tries=1) if not result: log.warning("No machines found with machine_type %s!", machine_type) else: