data.coop/ansible
8
3
Fork 7
Code Issues 52 Pull requests 10 Projects 1 Releases Wiki Activity

WIP: Initial work on woodpecker ci. #152

Draft
valberg wants to merge 3 commits from woodpeckerci into main
pull from: woodpeckerci
merge into: data.coop:main
Conversation 1 Commits 3 Files changed 2 +3 -4
2 changed files with 3 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit ae52815876 - Show all commits

3
roles/docker/defaults/main.yml
View file

@ -167,8 +167,7 @@ services:
file: woodpecker.yml
domain: "ci.{{ base_domain }}"
volume_folder: "{{ volume_root_folder }}/woodpecker"
server_version: v0.15
agent_version: v0.15
version: v0.15
mailu:
file: mailu.yml

4
roles/docker/tasks/services/woodpecker.yml
View file

@ -8,7 +8,7 @@
version: "3.8"
services:
server:
image: "woodpeckerci/woodpecker-server:{{ services.woodpecker.server_version }}"
image: "woodpeckerci/woodpecker-server:{{ services.woodpecker.version }}"
restart: always
networks:
- default
samsapti marked this conversation as resolved
reynir commented 2023-02-18 20:32:07 +00:00
Review
Copy link

Unless you add purge_networks: yes the default network is implicitly added (IMHO an annoying default). I think we should either add purge_networks: yes or leave out default.

Unless you add `purge_networks: yes` the `default` network is implicitly added (IMHO an annoying default). I think we should either add `purge_networks: yes` or leave out `default`.
samsapti commented 2023-12-12 20:11:33 +00:00
Review
Copy link

default is needed for server to be able to communicate with the other Compose services. (Why is it an annoying default?)

`default` is needed for `server` to be able to communicate with the other Compose services. (Why is it an annoying default?)
@ -28,7 +28,7 @@
LETSENCRYPT_EMAIL: "{{ letsencrypt_email }}"
agent:
image: "woodpeckerci/woodpecker-agent:{{ services.woodpecker.agent_version }}"
image: "woodpeckerci/woodpecker-agent:{{ services.woodpecker.version }}"
restart: always
volumes:
- "/var/run/docker.sock:/var/run/docker.sock"