zuul/zuul-jobs
Code Issues Proposed changes

Update bindep role defaults to be more sane

Browse Source 
The default for bindep dir should really be zuul.project.src_dir.

Also, this is a CI system. Let's default profile to test.

Change-Id: Ibc0e7617195865c50ea2bb0e969963f0525ea9b1
This commit is contained in:
Monty Taylor 2017-10-17 16:17:35 +02:00
parent 879a5bafa8
commit 1ea07ceee2
No known key found for this signature in database
GPG Key ID: 7BAE94BC7141A594
2 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
roles/bindep/README.rst
View File

@ -8,11 +8,12 @@ then install the missing packages.
**Role Variables** **Role Variables**
.. zuul:rolevar:: bindep_dir .. zuul:rolevar:: bindep_dir
:default: . :default: {{ zuul.project.src_dir }}
The directory to look for bindep files in. The directory to look for bindep files in.
.. zuul:rolevar:: bindep_profile .. zuul:rolevar:: bindep_profile
:default: test
A specific bindep profile to request. A specific bindep profile to request.

4
roles/bindep/defaults/main.yaml
View File

@ -1,3 +1,3 @@
--- ---
bindep_dir: . bindep_dir: "{{ zuul.project.src_dir }}"
bindep_profile: "" bindep_profile: test