From 58887d06456890bc173019644765ef42ea4cb287 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fran=C3=A7ois=20Charlier?= Date: Wed, 8 Nov 2017 17:25:44 +0100 Subject: [PATCH] validate-tempest: better workers default tempest_workers used `ansible_processor_count` which is in fact the number of physical CPUs - rarely more than two nowadays - when it should in fact use number of CPU cores/threads which is `ansible_processor_vcpus`. Change-Id: I7aa0dd29df5130e15bf6c1cf97774744e2f46921 --- roles/validate-tempest/README.md | 2 +- roles/validate-tempest/tasks/pre-tempest.yml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/roles/validate-tempest/README.md b/roles/validate-tempest/README.md index d25c17d54..5d0849625 100644 --- a/roles/validate-tempest/README.md +++ b/roles/validate-tempest/README.md @@ -24,7 +24,7 @@ Role Variables to empty if you don't want to run with blacklist_file option. Default value is an empty list * `skip_file_src`: path to skip tests file, set it empty if running without skip list: `skip_file_src=''` -* `tempest_workers`: int - how many parallel workers to run (default is number of cores) +* `tempest_workers`: int - how many parallel workers to run (default is half the number of cores/threads) * `tempest_until_failure`: false/true - default is false, repeat the run again and again until failure occurs * `tempest_exit_on_failure`: true/false - whether to exit from role with tempest exit code (default: true) * `tempestmail_config`: config.yaml - name of config file for tempestmail script diff --git a/roles/validate-tempest/tasks/pre-tempest.yml b/roles/validate-tempest/tasks/pre-tempest.yml index 712c161dc..131e72637 100644 --- a/roles/validate-tempest/tasks/pre-tempest.yml +++ b/roles/validate-tempest/tasks/pre-tempest.yml @@ -3,7 +3,7 @@ - name: Set the number of workers in tempest set_fact: - tempest_workers: "{{ ansible_processor_count|int // 2 }}" + tempest_workers: "{{ ansible_processor_vcpus|int // 2 }}" when: tempest_workers is not defined - name: Create overcloud tempest setup script