Skip to content

Commit e75d19f

Browse files
fix linting again
1 parent dcfba48 commit e75d19f

File tree

1 file changed

+19
-10
lines changed

1 file changed

+19
-10
lines changed

tasks/install_runner.yml

Lines changed: 19 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@
2626
dest: "{{ runner_dir }}/"
2727
owner: "{{ runner_user_id.stdout }}"
2828
group: "{{ runner_user_group_id.stdout }}"
29-
remote_src: yes
29+
remote_src: true
3030
mode: "0755"
3131
environment:
3232
PATH: /usr/local/bin:/opt/homebrew/bin/:{{ ansible_env.HOME }}/bin:{{ ansible_env.PATH }}
@@ -37,7 +37,7 @@
3737
path: "{{ runner_dir }}/.env"
3838
block: "{{ custom_env }}"
3939
owner: "{{ runner_user }}"
40-
create: yes
40+
create: true
4141
mode: "0755"
4242
marker_begin: "# BEGIN ANSIBLE MANAGED BLOCK"
4343
marker_end: "# END ANSIBLE MANAGED BLOCK"
@@ -77,6 +77,7 @@
7777
{{ runner_extra_config_args }}"
7878
args:
7979
chdir: "{{ runner_dir }}"
80+
changed_when: true
8081
become_user: "{{ runner_user }}"
8182
no_log: "{{ hide_sensitive_logs | bool }}"
8283
when: runner_name not in registered_runners.json.runners|map(attribute='name')|list
@@ -95,14 +96,19 @@
9596
--replace"
9697
args:
9798
chdir: "{{ runner_dir }}"
99+
changed_when: true
98100
become_user: "{{ runner_user }}"
99101
no_log: "{{ hide_sensitive_logs | bool }}"
100-
when: runner_name in registered_runners.json.runners|map(attribute='name')|list and reinstall_runner and not runner_org
102+
when: >
103+
runner_name in registered_runners.json.runners|map(attribute='name')|list and
104+
reinstall_runner and
105+
not runner_org
101106
102107
- name: Install service
103108
ansible.builtin.command: "./svc.sh install {{ runner_user }}"
104109
args:
105110
chdir: "{{ runner_dir }}"
111+
changed_when: true
106112
become: "{{ 'false' if ansible_system == 'Darwin' else 'true' }}"
107113
when: not runner_service_file_path.stat.exists
108114

@@ -117,7 +123,11 @@
117123
chdir: "{{ runner_dir }}"
118124
no_log: "{{ hide_sensitive_logs | bool }}"
119125
ignore_errors: "{{ ansible_check_mode }}"
120-
when: ansible_system != 'Darwin' and runner_state|lower == "started" and ansible_facts.services[(runner_service.content | b64decode) | trim ]['state'] != 'running'
126+
changed_when: true
127+
when: >
128+
ansible_system != 'Darwin' and
129+
runner_state|lower == "started" and
130+
ansible_facts.services[(runner_service.content | b64decode) | trim ]['state'] != 'running'
121131
122132
- name: START and enable Github Actions Runner service (macOS) # TODO: Idempotence
123133
ansible.builtin.command: "./svc.sh start"
@@ -126,25 +136,24 @@
126136
become: false
127137
no_log: "{{ hide_sensitive_logs | bool }}"
128138
ignore_errors: "{{ ansible_check_mode }}"
139+
changed_when: true
129140
when: ansible_system == 'Darwin' and runner_state|lower
130141

131142
- name: STOP and disable Github Actions Runner service
132-
ansible.builtin.shell: "./svc.sh stop"
143+
ansible.builtin.command: "./svc.sh stop"
133144
args:
134145
chdir: "{{ runner_dir }}"
146+
changed_when: true
135147
become: "{{ 'false' if ansible_distribution == 'MacOS' else 'true' }}"
136148
no_log: "{{ hide_sensitive_logs | bool }}"
137149
ignore_errors: "{{ ansible_check_mode }}"
138150
when: runner_state|lower == "stopped"
139151

140152
- name: Version changed - RESTART Github Actions Runner service
141-
ansible.builtin.shell:
142-
cmd: |
143-
./svc.sh stop
144-
sleep 5
145-
./svc.sh start
153+
ansible.builtin.command: "./svc.sh stop && sleep 5 && ./svc.sh start"
146154
args:
147155
chdir: "{{ runner_dir }}"
156+
changed_when: true
148157
become: "{{ 'false' if ansible_system == 'Darwin' else 'true' }}"
149158
no_log: "{{ hide_sensitive_logs | bool }}"
150159
ignore_errors: "{{ ansible_check_mode }}"

0 commit comments

Comments
 (0)