Skip to content

Commit

Permalink
Use branch centric job template for check/gate.
Browse files Browse the repository at this point in the history
Fixup psutils patching to not use a removed attribute.

Raised upper version bound for psutils.

Change-Id: I0367363bf0d1d3b1a48ff8682a979b5dd1a98a45
  • Loading branch information
javacruft committed Aug 14, 2024
1 parent 0ed0272 commit 53bdfe8
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 9 deletions.
2 changes: 1 addition & 1 deletion .zuul.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
- project:
templates:
- openstack-python3-charm-yoga-jobs
- openstack-python3-charm-jobs
- openstack-cover-jobs
2 changes: 1 addition & 1 deletion requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -26,4 +26,4 @@ six>=1.9.0

dnspython

psutil>=1.1.1,<2.0.0
psutil>=1.1.1,<6.0.0
2 changes: 1 addition & 1 deletion unit_tests/test_rabbit_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -1015,7 +1015,7 @@ def test_check_cluster_memberships(self, mock_relation_ids,
rabbit_utils.check_cluster_memberships())

@mock.patch.object(rabbit_utils, 'cmp_pkgrevno')
@mock.patch('rabbitmq_context.psutil.NUM_CPUS', 2)
@mock.patch('rabbitmq_context.psutil.cpu_count', lambda *args: 2)
@mock.patch('rabbitmq_context.relation_ids')
@mock.patch('rabbitmq_context.config')
def test_render_rabbitmq_env(self, mock_config, mock_relation_ids,
Expand Down
12 changes: 6 additions & 6 deletions unit_tests/test_rabbitmq_context.py
Original file line number Diff line number Diff line change
Expand Up @@ -230,7 +230,7 @@ def test_partition_handling(self, config, mock_cmp_pkgrevno,

class TestRabbitMQEnvContext(unittest.TestCase):

@mock.patch.object(rabbitmq_context.psutil, 'NUM_CPUS', 2)
@mock.patch.object(rabbitmq_context.psutil, 'cpu_count', lambda *args: 2)
@mock.patch.object(rabbitmq_context, 'relation_ids', lambda *args: [])
@mock.patch.object(rabbitmq_context, 'service_name')
@mock.patch.object(rabbitmq_context, 'config')
Expand All @@ -254,7 +254,7 @@ def fake_config(key):
'RABBITMQ_SERVER_ADDITIONAL_ERL_ARGS':
"'+A 72'"})

@mock.patch.object(rabbitmq_context.psutil, 'NUM_CPUS', 2)
@mock.patch.object(rabbitmq_context.psutil, 'cpu_count', lambda *args: 2)
@mock.patch.object(rabbitmq_context, 'relation_ids')
@mock.patch.object(rabbitmq_context, 'service_name')
@mock.patch.object(rabbitmq_context, 'config')
Expand Down Expand Up @@ -290,7 +290,7 @@ def fake_relation_ids(key):
{'RABBITMQ_SERVER_ADDITIONAL_ERL_ARGS':
"'+A 48'"})

@mock.patch.object(rabbitmq_context.psutil, 'NUM_CPUS', 2)
@mock.patch.object(rabbitmq_context.psutil, 'cpu_count', lambda *args: 2)
@mock.patch.object(rabbitmq_context, 'relation_ids')
@mock.patch.object(rabbitmq_context, 'service_name')
@mock.patch.object(rabbitmq_context, 'config')
Expand All @@ -317,7 +317,7 @@ def fake_relation_ids(key):
"'+A 48'"})

@mock.patch.object(rabbitmq_context, 'relation_ids', lambda *args: [])
@mock.patch.object(rabbitmq_context.psutil, 'NUM_CPUS', 40)
@mock.patch.object(rabbitmq_context.psutil, 'cpu_count', lambda *args: 40)
@mock.patch.object(rabbitmq_context, 'service_name')
@mock.patch.object(rabbitmq_context, 'config')
def test_rabbitmqenv_in_container(self, mock_config, mock_service_name):
Expand Down Expand Up @@ -351,7 +351,7 @@ def fake_config(key):
{'RABBITMQ_SERVER_ADDITIONAL_ERL_ARGS':
"'+A 960'"})

@mock.patch.object(rabbitmq_context.psutil, 'NUM_CPUS', 48)
@mock.patch.object(rabbitmq_context.psutil, 'cpu_count', lambda *args: 48)
@mock.patch.object(rabbitmq_context, 'relation_ids', lambda *args: [])
@mock.patch.object(rabbitmq_context, 'service_name')
@mock.patch.object(rabbitmq_context, 'config')
Expand All @@ -375,7 +375,7 @@ def fake_config(key):
{'RABBITMQ_SERVER_ADDITIONAL_ERL_ARGS':
"'+A 480'"})

@mock.patch.object(rabbitmq_context.psutil, 'NUM_CPUS', 48)
@mock.patch.object(rabbitmq_context.psutil, 'cpu_count', lambda *args: 48)
@mock.patch.object(rabbitmq_context, 'relation_ids', lambda *args: [])
@mock.patch.object(rabbitmq_context.rabbit_utils, 'use_long_node_name')
@mock.patch.object(rabbitmq_context, 'service_name')
Expand Down

0 comments on commit 53bdfe8

Please sign in to comment.