Skip to content

Commit a9aae92

Browse files
mnasiadkaAlex-Welsh
authored andcommitted
ansible-lint: Fix name[template], remove package-latest
Change-Id: I7c693403b1768c6c97eea90c3dfe9326764c3230 Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
1 parent c453068 commit a9aae92

File tree

10 files changed

+12
-16
lines changed

10 files changed

+12
-16
lines changed

.ansible-lint

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,13 +14,9 @@ skip_list:
1414
- unnamed-task
1515
# disable experimental rules
1616
- experimental
17-
# Package installs should not use latest
18-
- package-latest
1917
# NOTE(frickler): Agreed at Zed PTG not to use FQCN for builtin actions for now, due to
2018
# conflicts with open patches and backports.
2119
- fqcn-builtins
22-
# Allow Jinja templating inside task and play names
23-
- name[template]
2420
# FQCNs again, now for module actions
2521
- fqcn[action]
2622
# TODO(frickler): Discuss these in detail, skipping for now to unblock things

ansible/roles/ceilometer/tasks/config.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@
5454
- should_copy_custom_meter_definitions
5555
with_dict: "{{ ceilometer_services | select_services_enabled_and_mapped_to_host }}"
5656

57-
- name: Check if the folder ["{{ node_custom_config }}/ceilometer/{{ ceilometer_dynamic_pollsters_local_folder }}"] for dynamic pollsters definitions exist
57+
- name: Check if the folder for dynamic pollsters definitions exist
5858
stat:
5959
path: "{{ node_custom_config }}/ceilometer/{{ ceilometer_dynamic_pollsters_local_folder }}"
6060
delegate_to: localhost

ansible/roles/cloudkitty/tasks/config.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,13 +28,13 @@
2828
when:
2929
- cloudkitty_policy.results | length > 0
3030

31-
- name: Check if custom {{ cloudkitty_custom_metrics_yaml_file }} exists
31+
- name: Check if custom metrics file exists
3232
stat:
3333
path: "{{ node_custom_config }}/cloudkitty/{{ cloudkitty_custom_metrics_yaml_file }}"
3434
delegate_to: localhost
3535
register: cloudkitty_custom_metrics_file
3636

37-
- name: Copying {{ cloudkitty_custom_metrics_yaml_file }} if it exists
37+
- name: Copying custom metrics file
3838
copy:
3939
src: "{{ node_custom_config }}/cloudkitty/{{ cloudkitty_custom_metrics_yaml_file }}"
4040
dest: "{{ node_config_directory }}/{{ item.key }}/{{ cloudkitty_custom_metrics_yaml_file }}"

ansible/roles/mariadb/tasks/backup.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
check_mode: false
1010
register: container_facts
1111

12-
- name: Taking {{ mariadb_backup_type }} database backup via Mariabackup
12+
- name: Taking database backup via Mariabackup - backup type {{ mariadb_backup_type }}
1313
vars:
1414
cmd: "{{ 'kolla_mariadb_backup.sh' if mariadb_backup_target == 'active' else 'kolla_mariadb_backup_replica.sh' }}"
1515
become: true

ansible/roles/octavia-certificates/tasks/check_expiry.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
- "client_ca.cert.pem"
1212
- "client.cert-and-key.pem"
1313

14-
- name: Check whether certificates are valid within {{ octavia_certs_expiry_limit }} days
14+
- name: Check whether certificates are valid within days limit - {{ octavia_certs_expiry_limit }}
1515
assert:
1616
that:
1717
- item.valid_at.point_1

ansible/roles/octavia-certificates/tasks/main.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,13 +34,13 @@
3434

3535
- import_tasks: client_cert.yml
3636

37-
- name: Ensure {{ node_custom_config }}/octavia directory exists
37+
- name: Ensure octavia directory exists under {{ node_custom_config }}
3838
file:
3939
path: "{{ node_custom_config }}/octavia"
4040
state: "directory"
4141
mode: "0770"
4242

43-
- name: Copy the to-be-deployed keys and certs to {{ node_custom_config }}/octavia
43+
- name: Copy the to-be-deployed keys and certs
4444
copy:
4545
src: "{{ octavia_certs_work_dir }}/{{ item.src }}"
4646
dest: "{{ node_custom_config }}/octavia/{{ item.dest }}"

ansible/roles/octavia/tasks/get_resources_info.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
delegate_to: "{{ groups['octavia-api'][0] }}"
1616
register: flavor_results
1717

18-
- name: Get {{ octavia_service_auth_project }} project id
18+
- name: Get project id for project {{ octavia_service_auth_project }}
1919
become: true
2020
kolla_toolbox:
2121
container_engine: "{{ kolla_container_engine }}"

ansible/roles/octavia/tasks/hm-interface.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@
8686
enabled: true
8787
state: started
8888

89-
- name: Wait for interface {{ octavia_network_interface }} ip appear
89+
- name: Wait for interface ip appear on {{ octavia_network_interface }}
9090
vars:
9191
port_ip: "{{ port_info.port.fixed_ips[0].ip_address }}"
9292
command: ip address show dev {{ octavia_network_interface }}

ansible/roles/octavia/tasks/prepare.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@
3939
run_once: true
4040
delegate_to: "{{ groups['octavia-api'][0] }}"
4141

42-
- name: Get {{ octavia_service_auth_project }} project id
42+
- name: Get project id for project {{ octavia_service_auth_project }}
4343
become: true
4444
kolla_toolbox:
4545
container_engine: "{{ kolla_container_engine }}"

ansible/roles/service-rabbitmq/tasks/main.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
tags:
77
- service-rabbitmq
88
block:
9-
- name: "{{ project_name }} | Ensure RabbitMQ vhosts exist"
9+
- name: "Ensure RabbitMQ vhosts exist for {{ project_name }}"
1010
kolla_toolbox:
1111
container_engine: "{{ kolla_container_engine }}"
1212
module_name: rabbitmq_vhost
@@ -19,7 +19,7 @@
1919
retries: "{{ service_rabbitmq_retries }}"
2020
delay: "{{ service_rabbitmq_delay }}"
2121

22-
- name: "{{ project_name }} | Ensure RabbitMQ users exist"
22+
- name: "Ensure RabbitMQ users exist for {{ project_name }}"
2323
kolla_toolbox:
2424
container_engine: "{{ kolla_container_engine }}"
2525
module_name: rabbitmq_user

0 commit comments

Comments
 (0)