Browse Source

Rename fetch-translations-stestr-output

The role name is misleading. It's not fetching stestr output, it's
fetching subunit output.

Add a note to the README explaining what it does and that it's not using
fetch-subunit-output on purpose.

Use  "{{ zuul.project.src_dir }}" instead of
"src/{{ zuul.project.canonical_name }}".

Change-Id: I1b5ab65f9697ec184969ca7899fb40294cdb853c
changes/49/535849/2
Monty Taylor 4 years ago
committed by Andreas Jaeger
parent
commit
63404d3d0d
  1. 4
      playbooks/translation/propose-translation-update.yaml
  2. 4
      playbooks/translation/upstream-translation-update.yaml
  3. 12
      roles/fetch-translations-stestr-output/README.rst
  4. 2
      roles/fetch-translations-stestr-output/defaults/main.yaml
  5. 14
      roles/fetch-translations-subunit-output/README.rst
  6. 2
      roles/fetch-translations-subunit-output/defaults/main.yaml
  7. 0
      roles/fetch-translations-subunit-output/tasks/main.yaml

4
playbooks/translation/propose-translation-update.yaml

@ -8,8 +8,8 @@
- name: Run propose_translation_update.sh script
command: "{{ ansible_user_dir }}/scripts/propose_translation_update.sh {{ zuul.project.short_name }} {{ zuul.branch }} {{ zuul.job }}"
args:
chdir: "src/{{ zuul.project.canonical_name }}"
chdir: "{{ zuul.project.src_dir }}"
- hosts: all
roles:
- fetch-translations-stestr-output
- fetch-translations-subunit-output

4
playbooks/translation/upstream-translation-update.yaml

@ -8,9 +8,9 @@
- name: Run upstream_translation_update.sh script
command: "{{ ansible_user_dir }}/scripts/upstream_translation_update.sh {{ zuul.project.short_name }} {{ zuul.job }} {{zuul.branch}}"
args:
chdir: "src/{{ zuul.project.canonical_name }}"
chdir: "{{ zuul.project.src_dir }}"
- hosts: all
roles:
- fetch-translation-output
- fetch-translations-stestr-output
- fetch-translations-subunit-output

12
roles/fetch-translations-stestr-output/README.rst

@ -1,12 +0,0 @@
Collect stestr output from translation jobs
This role collects only the testrepository.subunit file which has to
be compressed already.
**Role Variables**
.. zuul:rolevar:: zuul_work_dir
:default: {{ zuul.project.src_dir }}
Directory where ``testrepository.subunit.gz`` file is.

2
roles/fetch-translations-stestr-output/defaults/main.yaml

@ -1,2 +0,0 @@
---
zuul_work_dir: "src/{{ zuul.project.canonical_name }}"

14
roles/fetch-translations-subunit-output/README.rst

@ -0,0 +1,14 @@
Collect subunit output from translation jobs
This role collects only the testrepository.subunit file which has
already been compressed. The common_translation_update.sh script
uses generate-subunit to create a subunit file containing success or
failure, thus :zuul:role:`fetch-subunit-output` is not appropriate.
**Role Variables**
.. zuul:rolevar:: zuul_work_dir
:default: {{ zuul.project.src_dir }}
Directory where ``testrepository.subunit.gz`` file is.

2
roles/fetch-translations-subunit-output/defaults/main.yaml

@ -0,0 +1,2 @@
---
zuul_work_dir: "{{ zuul.project.src_dir }}"

0
roles/fetch-translations-stestr-output/tasks/main.yaml → roles/fetch-translations-subunit-output/tasks/main.yaml

Loading…
Cancel
Save