Fixes all tasks should be named rule
This prepared for ansible-lint v5 which no longer has exceptions for task naming, requiring all to be named. Change-Id: I5e761d1e3836fa270d7afdcf01780320001f820d Part-Of: #773245
This commit is contained in:
parent
a31d70e6df
commit
ad3167bd56
@ -1,6 +1,7 @@
|
||||
- hosts: all
|
||||
tasks:
|
||||
- include_role:
|
||||
- name: Install helm charts
|
||||
include_role:
|
||||
name: helm-template
|
||||
vars:
|
||||
helm_release_name: "{{ zj_item.key }}"
|
||||
|
@ -11,7 +11,8 @@
|
||||
include_tasks: remote-windows.yaml
|
||||
when: ansible_os_family == "Windows"
|
||||
|
||||
- import_role:
|
||||
- name: Remove zuul master ssh key
|
||||
import_role:
|
||||
name: remove-zuul-sshkey
|
||||
|
||||
- name: Add back temp key
|
||||
|
@ -9,7 +9,8 @@
|
||||
return_content: true
|
||||
register: bazel_installer_checksum
|
||||
|
||||
- debug:
|
||||
- name: Display checksum
|
||||
debug:
|
||||
msg: "Checksum is {{ bazel_installer_checksum.content.split(' ')[0] }}"
|
||||
|
||||
- name: Download bazel installer
|
||||
@ -19,9 +20,11 @@
|
||||
mode: 0755
|
||||
checksum: "sha256:{{ bazel_installer_checksum.content.split(' ')[0] }}"
|
||||
|
||||
- debug:
|
||||
- name: Display distribution
|
||||
debug:
|
||||
msg: "Distribution is {{ ansible_distribution }}"
|
||||
- debug:
|
||||
- name: Display OS family
|
||||
debug:
|
||||
msg: "OS family is {{ ansible_os_family }}"
|
||||
|
||||
- name: Install bazel and platform-specific dependencies
|
||||
|
@ -5,7 +5,8 @@
|
||||
failed_when: false
|
||||
register: installed_bazel_version
|
||||
|
||||
- debug:
|
||||
- name: Display Bazel version
|
||||
debug:
|
||||
msg: "Current installed Bazel version is {{ installed_bazel_version.stdout }}"
|
||||
|
||||
- name: Download and install Bazel if needed
|
||||
|
@ -1 +1 @@
|
||||
git_cache_root: "{{ansible_user_dir }}/git-cache"
|
||||
git_cache_root: "{{ ansible_user_dir }}/git-cache"
|
||||
|
@ -3,7 +3,8 @@
|
||||
msg: npm_command is required for this role
|
||||
when: npm_command is not defined
|
||||
|
||||
- include_role:
|
||||
- name: Run npm
|
||||
include_role:
|
||||
name: js-package-manager
|
||||
vars:
|
||||
js_build_command: '{{ npm_command }}'
|
||||
|
@ -3,7 +3,8 @@
|
||||
msg: yarn_command is required for this role
|
||||
when: yarn_command is not defined
|
||||
|
||||
- include_role:
|
||||
- name: Run yarn
|
||||
include_role:
|
||||
name: js-package-manager
|
||||
vars:
|
||||
js_build_command: '{{ yarn_command }}'
|
||||
|
Loading…
Reference in New Issue
Block a user