jens/ansible-role-drone
1
0
Fork 1
Code Issues Pull requests Releases Wiki Activity

Update 'tasks/main.yml' #7

Merged
jens merged 10 commits from jens-patch-2 into main 2021-09-09 10:01:49 +00:00
Conversation 0 Commits 10 Files changed 3 +1 -2
1 changed files with 1 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit fc2f3a77d7 - Show all commits

3
tasks/main.yml
View file

@ -22,7 +22,7 @@
force: true
become: true
become_user : "{{ drone_user }}"
register: gitclone
notify: "Build drone"
jens commented 2021-08-26 16:44:34 +00:00
Review
Copy link

this doesn't really help, this always returns changed, even when running withing 1 second,
perhaps the 'forced' is doing this? But forced is needed to discard local changes...

this doesn't really help, this always returns changed, even when running withing 1 second, perhaps the 'forced' is doing this? But forced is needed to discard local changes...
eludum commented 2021-09-09 09:59:28 +00:00
Review
Copy link

Forced does this, but is needed as stated, the notify here could indeed be omitted.

Forced does this, but is needed as stated, the notify here could indeed be omitted.
- name: "Setup systemd service"
template:
@ -43,7 +43,6 @@
notify: "Restart drone"
args:
chdir: "{{ drone_home }}/drone/cmd/drone-server"
when: gitclone.changed # noqa no-handler
- name: "Clone runner"
get_url: