Skip test-setup.sh in pep8 jobs
pep8 jobs should not require database setup, etc, so we can skip running test-setup.sh to save time. Also, address comments from https://review.opendev.org/670120 and use a block to not run the stat if can we will skip using the file. Change-Id: Ia140b331ae1502fcf1f685faa6dace117d77d35d
This commit is contained in:
parent
9d8e765881
commit
7513629be7
@ -1,14 +1,15 @@
|
|||||||
- name: "Check if project's tools/test-setup.sh exists"
|
- when: not test_setup_skip
|
||||||
stat:
|
block:
|
||||||
path: "{{ zuul_work_dir }}/tools/test-setup.sh"
|
- name: "Check if project's tools/test-setup.sh exists"
|
||||||
register: p
|
stat:
|
||||||
|
path: "{{ zuul_work_dir }}/tools/test-setup.sh"
|
||||||
|
register: p
|
||||||
|
|
||||||
- name: Run tools/test-setup.sh
|
- name: Run tools/test-setup.sh
|
||||||
command: tools/test-setup.sh {{ test_setup_args }}
|
command: tools/test-setup.sh {{ test_setup_args }}
|
||||||
environment: "{{ test_setup_environment }}"
|
environment: "{{ test_setup_environment }}"
|
||||||
args:
|
args:
|
||||||
chdir: "{{ zuul_work_dir }}"
|
chdir: "{{ zuul_work_dir }}"
|
||||||
when:
|
when:
|
||||||
- p.stat.exists
|
- p.stat.exists
|
||||||
- p.stat.executable
|
- p.stat.executable
|
||||||
- not test_setup_skip
|
|
||||||
|
@ -147,6 +147,7 @@
|
|||||||
Uses tox with the ``pep8`` environment.
|
Uses tox with the ``pep8`` environment.
|
||||||
vars:
|
vars:
|
||||||
tox_envlist: pep8
|
tox_envlist: pep8
|
||||||
|
test_setup_skip: true
|
||||||
|
|
||||||
- job:
|
- job:
|
||||||
name: tox-cover
|
name: tox-cover
|
||||||
|
Loading…
Reference in New Issue
Block a user