From dbfc6b8256911162f720de0dbb6fe9b1eea4cca5 Mon Sep 17 00:00:00 2001
From: Robert Kaussow <mail@geeklabor.de>
Date: Thu, 4 Jun 2020 18:44:52 +0200
Subject: [PATCH 01/10] remove systemd and use native docker-compose

---
 defaults/main.yml                             | 67 ++++++++++-----
 handlers/main.yml                             |  9 --
 molecule/centos7/converge.yml                 |  7 +-
 molecule/centos7/requirements.yml             |  7 +-
 molecule/centos7/tests/test_default.py        |  2 +-
 tasks/setup.yml                               | 29 +++----
 templates/etc/systemd/system/drone.service.j2 | 20 -----
 templates/services/droneci-compose.yml.j2     | 85 +++++++++----------
 8 files changed, 106 insertions(+), 120 deletions(-)
 delete mode 100644 handlers/main.yml
 delete mode 100644 templates/etc/systemd/system/drone.service.j2

diff --git a/defaults/main.yml b/defaults/main.yml
index 986b8c9..d80134a 100644
--- a/defaults/main.yml
+++ b/defaults/main.yml
@@ -1,35 +1,57 @@
 ---
 droneci_version: 1.6.0
-droneci_service_directory: /var/lib/docker/services/drone-ci
-droneci_docker_socket_path: /var/run/docker.sock
+droneci_image: "drone/drone:{{ droneci_version }}"
+
+droneci_service_directory: /var/lib/docker/services/droneci
+droneci_container_name: droneci
+droneci_restart_policy: always
+droneci_service_stopped: False
+
 droneci_host: http://localhost
 droneci_secret: myveryownsecret
 
 # @var droneci_license_key:description Path to the license key file
 # @var droneci_license_key: $ "_unset_"
 
-droneci_restart_policy: always
-
-droneci_server_container_name: drone-server
-droneci_server_image: "drone/drone:{{ droneci_version }}"
-droneci_server_exposed_port: 8080
-droneci_server_exposed_ip: 127.0.0.1
+droneci_exposed_ports:
+  - "127.0.0.1:8080:80"
 
-# @var droneci_server_memory_limit: $ "_unset_"
-# @var droneci_server_memory_limit:example: $ "512m"
-# @var droneci_server_memory_reservation: $ "_unset_"
-# @var droneci_server_memory_reservation:example: $ "256m"
+# @var droneci_volumes:description: > Define required docker volumes.
+# If you don't use sqlite you could remove the default volume. To enable a Drone licences
+# you will need to configure a bind mount to `/etc/drone.key`.
+# @end
+# @var droneci_volumes:example: >
+# droneci_volumes:
+#   # Instead of the name you could specify a path on the container host system,
+#   # but you also have to enable bind mount for this volume
+#   - name: droneci-data
+#     # target location inside the container
+#     dest: /var/lib/drone
+#     # enable bind mount, if false volume will be configured as named volume
+#     # keep in mind you MUST set bind in any case
+#     bind: True
+# @end
+droneci_volumes:
+  - name: droneci-data
+    dest: /var/lib/drone
+    bind: False
 
-droneci_server_extra_hosts: []
+# @var droneci_networks:example: >
+# droneci_networks:
+#   - name: droneci
+#     # optional network driver, defaults to 'bride'
+#     driver: host
+# @end
+droneci_networks:
+  - name: droneci
 
-droneci_agent_container_name: drone-agent
-droneci_agent_image: "drone/drone-runner-docker:1.0.1"
-droneci_agent_capacity: 2
+droneci_networks_applied:
+  - droneci
 
-# @var droneci_agent_memory_limit:example: $ "512m"
-# @var droneci_agent_memory_limit: $ "_unset_"
-# @var droneci_agent_memory_reservation:example: $ "256m"
-# @var droneci_agent_memory_reservation: $ "_unset_"
+# @var droneci_memory_limit: $ "_unset_"
+# @var droneci_memory_limit:example: $ "512m"
+# @var droneci_memory_reservation: $ "_unset_"
+# @var droneci_memory_reservation:example: $ "256m"
 
 droneci_db_type: sqlite
 droneci_db_server: localhost
@@ -48,11 +70,12 @@ droneci_gitea_skip_verify: False
 # droneci_gitea_oauth_client_id: 1111-222-33333-44444 # defaults to not set
 # droneci_gitea_oauth_client_secret: 1234abcd5678efgh # defaults to not set
 
-# @var droneci_admin: $ "_unset_"
+# @var droneci_admin_user: $ "_unset_"
 # @var droneci_http_proxy: $ "_unset_"
 # @var droneci_https_proxy: $ "_unset_"
 
 # @var droneci_no_proxy: $ ["drone-server", "drone-agent"]
 droneci_no_proxy: []
 
-droneci_docker_compose_bin: /usr/local/bin/docker-compose
+droneci_docker_socket_path: /var/run/docker.sock
+droneci_extra_hosts: []
diff --git a/handlers/main.yml b/handlers/main.yml
deleted file mode 100644
index 64b13de..0000000
--- a/handlers/main.yml
+++ /dev/null
@@ -1,9 +0,0 @@
----
-- name: Restart container
-  systemd:
-    state: restarted
-    daemon_reload: yes
-    name: drone
-  listen: __drone_restart
-  become: True
-  become_user: root
diff --git a/molecule/centos7/converge.yml b/molecule/centos7/converge.yml
index 1bc8d51..69e68f2 100644
--- a/molecule/centos7/converge.yml
+++ b/molecule/centos7/converge.yml
@@ -2,9 +2,12 @@
 - name: Converge
   hosts: all
   vars:
-    droneci_admin: dummy
+    dockerengine_packages_extra:
+      - epel-release
+      - python-pip
+    droneci_admin_user: dummy
+    droneci_service_stopped: True
 
   roles:
-    - role: xoxys.python3
     - role: xoxys.docker_engine
     - role: xoxys.droneci
diff --git a/molecule/centos7/requirements.yml b/molecule/centos7/requirements.yml
index c1e62b1..8cc34d3 100644
--- a/molecule/centos7/requirements.yml
+++ b/molecule/centos7/requirements.yml
@@ -1,10 +1,5 @@
 ---
-- src: https://gitea.rknet.org/ansible/xoxys.python3.git
-  name: xoxys.python3
-  scm: git
-  version: master
-
 - src: https://gitea.rknet.org/ansible/xoxys.docker_engine.git
   name: xoxys.docker_engine
   scm: git
-  version: master
+  version: refactoring
diff --git a/molecule/centos7/tests/test_default.py b/molecule/centos7/tests/test_default.py
index b455207..e2f223d 100644
--- a/molecule/centos7/tests/test_default.py
+++ b/molecule/centos7/tests/test_default.py
@@ -10,6 +10,6 @@ testinfra_hosts = testinfra.utils.ansible_runner.AnsibleRunner(
 
 
 def test_droneci_compose_file(host):
-    f = host.file('/var/lib/docker/services/drone-ci/docker-compose.yml')
+    f = host.file('/var/lib/docker/services/droneci/droneci.yml')
 
     assert f.exists
diff --git a/tasks/setup.yml b/tasks/setup.yml
index e0306c8..315fd35 100644
--- a/tasks/setup.yml
+++ b/tasks/setup.yml
@@ -13,19 +13,11 @@
     - name: Deploy compose file to '{{ droneci_service_directory }}'
       template:
         src: "services/droneci-compose.yml.j2"
-        dest: "{{ droneci_service_directory }}/docker-compose.yml"
+        dest: "{{ droneci_service_directory }}/droneci.yml"
         owner: root
         group: root
         mode: 0644
-        validate: "{{ droneci_docker_compose_bin }} -f %s config -q"
-      notify: __drone_restart
-
-    - name: Create systemd unit files
-      template:
-        src: "etc/systemd/system/drone.service.j2"
-        dest: "/etc/systemd/system/drone.service"
-        mode: 0644
-      notify: __drone_restart
+        validate: "docker-compose -f %s config -q"
 
     - name: Copy license key file is defined
       copy:
@@ -34,11 +26,16 @@
         mode: 0600
       when: droneci_license_key is defined
 
-    - name: Ensure drone service is up and running
-      systemd:
-        state: started
-        daemon_reload: yes
-        enabled: yes
-        name: drone
+    - name: Ensure Drone is up and running
+      docker_compose:
+        project_src: "{{ droneci_service_directory }}"
+        files:
+          - droneci.yml
+        pull: yes
+        remove_orphans: yes
+        stopped: "{{ droneci_service_stopped }}"
+        state: present
+      # temp. disable changes; breaks idempotency for whatever reason
+      changed_when: False
   become: True
   become_user: root
diff --git a/templates/etc/systemd/system/drone.service.j2 b/templates/etc/systemd/system/drone.service.j2
deleted file mode 100644
index c66b4e4..0000000
--- a/templates/etc/systemd/system/drone.service.j2
+++ /dev/null
@@ -1,20 +0,0 @@
-#jinja2:lstrip_blocks: True
-{{ ansible_managed | comment }}
-[Unit]
-Description=Drone CI compose service
-Requires=docker.service
-After=docker.service
-
-[Service]
-Restart={{ droneci_restart_policy }}
-WorkingDirectory={{ droneci_service_directory }}
-
-ExecStartPre={{ droneci_docker_compose_bin }} pull --quiet --ignore-pull-failures
-ExecStart={{ droneci_docker_compose_bin }} up --remove-orphans
-
-ExecStop={{ droneci_docker_compose_bin }} down --remove-orphans
-
-ExecReload={{ droneci_docker_compose_bin }} pull --quiet --ignore-pull-failures
-
-[Install]
-WantedBy=multi-user.target
diff --git a/templates/services/droneci-compose.yml.j2 b/templates/services/droneci-compose.yml.j2
index 95b651d..f12d06a 100644
--- a/templates/services/droneci-compose.yml.j2
+++ b/templates/services/droneci-compose.yml.j2
@@ -1,25 +1,34 @@
 #jinja2:lstrip_blocks: True
 {{ ansible_managed | comment }}
-version: '2'
+version: "3"
 
 services:
-  droneserver:
-    container_name: {{ droneci_server_container_name }}
-    image: {{ droneci_server_image }}
+  droneci:
+    container_name: {{ droneci_container_name }}
+    image: {{ droneci_image }}
+    {% if droneci_exposed_ports | default([]) %}
     ports:
-      - {{ droneci_server_exposed_ip + ':' if droneci_server_exposed_ip is defined else '' }}{{ droneci_server_exposed_port }}:80
+      {% for port in droneci_exposed_ports %}
+      - {{ port | quote }}
+      {% endfor %}
+    {% endif %}
+    {% if droneci_volumes | default([]) %}
     volumes:
-      {% if droneci_db_type == "sqlite" %}
-      - droneserver-data:/var/lib/drone/
-      {% endif %}
-      {% if droneci_license_key is defined %}
-      - {{ droneci_license_key }}:/etc/drone.key
-      {% endif %}
-    {% if droneci_server_extra_hosts | default([]) %}
+      {% for volume in droneci_volumes %}
+      - "{{ volume.name }}:{{ volume.dest }}"
+      {% endfor %}
+    {% endif %}
+    {% if droneci_networks_applied | default([]) %}
+    networks:
+      {% for network in droneci_networks_applied %}
+      - {{ network }}
+      {% endfor %}
+    {% endif %}
+    {% if droneci_extra_hosts | default([]) %}
     extra_hosts:
-    {% for host in droneci_server_extra_hosts %}
-      - {{ '"' + host + '"' }}
-    {% endfor %}
+      {% for host in droneci_extra_hosts %}
+      - {{ host | quote }}
+      {% endfor %}
     {% endif %}
     privileged: true
     environment:
@@ -27,7 +36,7 @@ services:
       - DRONE_SERVER_PROTO={{ droneci_host | urlsplit('scheme') }}
       - DRONE_AGENTS_ENABLED=true
       - DRONE_RPC_SECRET={{ droneci_secret }}
-      - DRONE_USER_CREATE=username:{{ droneci_admin }},admin:true
+      - DRONE_USER_CREATE=username:{{ droneci_admin_user }},admin:true
       - DRONE_LOGS_COLOR=true
       - DRONE_LOGS_PRETTY=true
       - DRONE_TLS_AUTOCERT=false
@@ -53,36 +62,24 @@ services:
       {% endif %}
       - NO_PROXY={{ droneci_no_proxy | join(',') }}
       - no_proxy={{ droneci_no_proxy | join(',') }}
-    {% if droneci_server_memory_limit is defined %}
-    mem_limit: {{ droneci_server_memory_limit }}
-    {% endif %}
-    {% if droneci_server_memory_reservation is defined %}
-    mem_reservation: {{ droneci_server_memory_reservation }}
-    {% endif %}
-
-  droneagent:
-    container_name: {{ droneci_agent_container_name }}
-    image: {{ droneci_agent_image }}
-    depends_on:
-      - droneserver
-    volumes:
-      - {{ droneci_docker_socket_path }}:/var/run/docker.sock
-    privileged: true
-    environment:
-      - DRONE_RPC_PROTO=http
-      - DRONE_RPC_HOST=droneserver
-      - DRONE_RPC_SECRET={{ droneci_secret }}
-      - DRONE_RUNNER_CAPACITY={{ droneci_agent_capacity }}
-      # TODO: temp static value, needs to be replaced by a custom fact
-      - DOCKER_API_VERSION=1.26
-    {% if droneci_agent_memory_limit is defined %}
-    mem_limit: {{ droneci_agent_memory_limit }}
+    {% if droneci_memory_limit is defined %}
+    mem_limit: {{ droneci_memory_limit }}
     {% endif %}
-    {% if droneci_agent_memory_reservation is defined %}
-    mem_reservation: {{ droneci_agent_memory_reservation }}
+    {% if droneci_memory_reservation is defined %}
+    mem_reservation: {{ droneci_memory_reservation }}
     {% endif %}
 
-{% if droneci_db_type == "sqlite" %}
+{% if droneci_volumes | default([]) | rejectattr("bind") %}
 volumes:
-  droneserver-data:
+  {% for volume in droneci_volumes | rejectattr("bind") %}
+  {{ volume.name }}:
+  {% endfor %}
+{% endif %}
+
+{% if droneci_networks | default([]) %}
+networks:
+  {% for network in droneci_networks %}
+  {{ network.name }}:
+    driver: {{ network.backend | default("bridge") }}
+  {% endfor %}
 {% endif %}

From d2ca52aecbbcc12e92d7fbe60af8daea61373887 Mon Sep 17 00:00:00 2001
From: Robert Kaussow <mail@geeklabor.de>
Date: Sun, 14 Jun 2020 23:03:58 +0200
Subject: [PATCH 02/10] fix filename

---
 tasks/setup.yml                                               | 4 ++--
 .../{droneci-compose.yml.j2 => droneci_compose.yml.j2}        | 0
 2 files changed, 2 insertions(+), 2 deletions(-)
 rename templates/services/{droneci-compose.yml.j2 => droneci_compose.yml.j2} (100%)

diff --git a/tasks/setup.yml b/tasks/setup.yml
index 315fd35..f333544 100644
--- a/tasks/setup.yml
+++ b/tasks/setup.yml
@@ -12,11 +12,11 @@
 
     - name: Deploy compose file to '{{ droneci_service_directory }}'
       template:
-        src: "services/droneci-compose.yml.j2"
+        src: "services/droneci_compose.yml.j2"
         dest: "{{ droneci_service_directory }}/droneci.yml"
         owner: root
         group: root
-        mode: 0644
+        mode: 0640
         validate: "docker-compose -f %s config -q"
 
     - name: Copy license key file is defined
diff --git a/templates/services/droneci-compose.yml.j2 b/templates/services/droneci_compose.yml.j2
similarity index 100%
rename from templates/services/droneci-compose.yml.j2
rename to templates/services/droneci_compose.yml.j2

From 902d85f49805488f95f2551765eab41ef9476757 Mon Sep 17 00:00:00 2001
From: Robert Kaussow <mail@geeklabor.de>
Date: Mon, 15 Jun 2020 23:54:20 +0200
Subject: [PATCH 03/10] fix compose v3 syntax

---
 defaults/main.yml                         | 14 ++++++++++----
 templates/services/droneci_compose.yml.j2 |  8 +++-----
 2 files changed, 13 insertions(+), 9 deletions(-)

diff --git a/defaults/main.yml b/defaults/main.yml
index d80134a..d1432f2 100644
--- a/defaults/main.yml
+++ b/defaults/main.yml
@@ -48,10 +48,16 @@ droneci_networks:
 droneci_networks_applied:
   - droneci
 
-# @var droneci_memory_limit: $ "_unset_"
-# @var droneci_memory_limit:example: $ "512m"
-# @var droneci_memory_reservation: $ "_unset_"
-# @var droneci_memory_reservation:example: $ "256m"
+# @var droneci_docker_resources: $ "_unset_"
+# @var droneci_docker_resources:example: >
+# droneci_docker_resources:
+#   limits:
+#     cpus: "0.50"
+#     memory: 50M
+#   reservations:
+#     cpus: "0.25"
+#     memory: 20M
+# @end
 
 droneci_db_type: sqlite
 droneci_db_server: localhost
diff --git a/templates/services/droneci_compose.yml.j2 b/templates/services/droneci_compose.yml.j2
index f12d06a..9947965 100644
--- a/templates/services/droneci_compose.yml.j2
+++ b/templates/services/droneci_compose.yml.j2
@@ -62,11 +62,9 @@ services:
       {% endif %}
       - NO_PROXY={{ droneci_no_proxy | join(',') }}
       - no_proxy={{ droneci_no_proxy | join(',') }}
-    {% if droneci_memory_limit is defined %}
-    mem_limit: {{ droneci_memory_limit }}
-    {% endif %}
-    {% if droneci_memory_reservation is defined %}
-    mem_reservation: {{ droneci_memory_reservation }}
+    {% if droneci_docker_resources is defined %}
+    resources:
+      {{ droneci_docker_resources | to_nice_yaml | indent(6) }}
     {% endif %}
 
 {% if droneci_volumes | default([]) | rejectattr("bind") %}

From 01ea1cf575c05208763b6141de2e3533cb8c6b36 Mon Sep 17 00:00:00 2001
From: Robert Kaussow <mail@geeklabor.de>
Date: Tue, 16 Jun 2020 00:02:40 +0200
Subject: [PATCH 04/10] Revert "fix compose v3 syntax"

This reverts commit fb715e3459bf8e7d253204e818baedff6d67fa21.
---
 defaults/main.yml                         | 14 ++++----------
 templates/services/droneci_compose.yml.j2 |  8 +++++---
 2 files changed, 9 insertions(+), 13 deletions(-)

diff --git a/defaults/main.yml b/defaults/main.yml
index d1432f2..d80134a 100644
--- a/defaults/main.yml
+++ b/defaults/main.yml
@@ -48,16 +48,10 @@ droneci_networks:
 droneci_networks_applied:
   - droneci
 
-# @var droneci_docker_resources: $ "_unset_"
-# @var droneci_docker_resources:example: >
-# droneci_docker_resources:
-#   limits:
-#     cpus: "0.50"
-#     memory: 50M
-#   reservations:
-#     cpus: "0.25"
-#     memory: 20M
-# @end
+# @var droneci_memory_limit: $ "_unset_"
+# @var droneci_memory_limit:example: $ "512m"
+# @var droneci_memory_reservation: $ "_unset_"
+# @var droneci_memory_reservation:example: $ "256m"
 
 droneci_db_type: sqlite
 droneci_db_server: localhost
diff --git a/templates/services/droneci_compose.yml.j2 b/templates/services/droneci_compose.yml.j2
index 9947965..f12d06a 100644
--- a/templates/services/droneci_compose.yml.j2
+++ b/templates/services/droneci_compose.yml.j2
@@ -62,9 +62,11 @@ services:
       {% endif %}
       - NO_PROXY={{ droneci_no_proxy | join(',') }}
       - no_proxy={{ droneci_no_proxy | join(',') }}
-    {% if droneci_docker_resources is defined %}
-    resources:
-      {{ droneci_docker_resources | to_nice_yaml | indent(6) }}
+    {% if droneci_memory_limit is defined %}
+    mem_limit: {{ droneci_memory_limit }}
+    {% endif %}
+    {% if droneci_memory_reservation is defined %}
+    mem_reservation: {{ droneci_memory_reservation }}
     {% endif %}
 
 {% if droneci_volumes | default([]) | rejectattr("bind") %}

From 42a82a6adb16d4695acb3b9bd59f30a9233afa90 Mon Sep 17 00:00:00 2001
From: Robert Kaussow <mail@geeklabor.de>
Date: Tue, 16 Jun 2020 00:03:26 +0200
Subject: [PATCH 05/10] switch back to compose v2 to allow resource constraints

---
 templates/services/droneci_compose.yml.j2 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/templates/services/droneci_compose.yml.j2 b/templates/services/droneci_compose.yml.j2
index f12d06a..9fac3e7 100644
--- a/templates/services/droneci_compose.yml.j2
+++ b/templates/services/droneci_compose.yml.j2
@@ -1,6 +1,6 @@
 #jinja2:lstrip_blocks: True
 {{ ansible_managed | comment }}
-version: "3"
+version: "2"
 
 services:
   droneci:

From fa18029fdf5f89a99d81343bf3bec91b2456a261 Mon Sep 17 00:00:00 2001
From: Robert Kaussow <mail@geeklabor.de>
Date: Tue, 16 Jun 2020 00:41:55 +0200
Subject: [PATCH 06/10] move to docker-compose.yml

---
 molecule/centos7/tests/test_default.py | 2 +-
 tasks/setup.yml                        | 4 +---
 2 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/molecule/centos7/tests/test_default.py b/molecule/centos7/tests/test_default.py
index e2f223d..5b62ff6 100644
--- a/molecule/centos7/tests/test_default.py
+++ b/molecule/centos7/tests/test_default.py
@@ -10,6 +10,6 @@ testinfra_hosts = testinfra.utils.ansible_runner.AnsibleRunner(
 
 
 def test_droneci_compose_file(host):
-    f = host.file('/var/lib/docker/services/droneci/droneci.yml')
+    f = host.file('/var/lib/docker/services/droneci/docker-compose.yml')
 
     assert f.exists
diff --git a/tasks/setup.yml b/tasks/setup.yml
index f333544..4540dfc 100644
--- a/tasks/setup.yml
+++ b/tasks/setup.yml
@@ -13,7 +13,7 @@
     - name: Deploy compose file to '{{ droneci_service_directory }}'
       template:
         src: "services/droneci_compose.yml.j2"
-        dest: "{{ droneci_service_directory }}/droneci.yml"
+        dest: "{{ droneci_service_directory }}/docker-compose.yml"
         owner: root
         group: root
         mode: 0640
@@ -29,8 +29,6 @@
     - name: Ensure Drone is up and running
       docker_compose:
         project_src: "{{ droneci_service_directory }}"
-        files:
-          - droneci.yml
         pull: yes
         remove_orphans: yes
         stopped: "{{ droneci_service_stopped }}"

From c960f62e2252c5dfcf06087f972dca29ae5031a0 Mon Sep 17 00:00:00 2001
From: Robert Kaussow <mail@geeklabor.de>
Date: Tue, 16 Jun 2020 21:46:28 +0200
Subject: [PATCH 07/10] fix volume checks

---
 templates/services/droneci_compose.yml.j2 | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/templates/services/droneci_compose.yml.j2 b/templates/services/droneci_compose.yml.j2
index 9fac3e7..5f94a33 100644
--- a/templates/services/droneci_compose.yml.j2
+++ b/templates/services/droneci_compose.yml.j2
@@ -68,15 +68,15 @@ services:
     {% if droneci_memory_reservation is defined %}
     mem_reservation: {{ droneci_memory_reservation }}
     {% endif %}
+{% if droneci_volumes | default([]) | rejectattr("bind") | list | length > 0 %}
 
-{% if droneci_volumes | default([]) | rejectattr("bind") %}
 volumes:
   {% for volume in droneci_volumes | rejectattr("bind") %}
   {{ volume.name }}:
   {% endfor %}
 {% endif %}
+{% if droneci_networks | default([]) | length > 0 %}
 
-{% if droneci_networks | default([]) %}
 networks:
   {% for network in droneci_networks %}
   {{ network.name }}:

From 42c799a2dd04d0297db38f0674cc838597cd302a Mon Sep 17 00:00:00 2001
From: Robert Kaussow <mail@geeklabor.de>
Date: Tue, 16 Jun 2020 22:04:08 +0200
Subject: [PATCH 08/10] add restart policy

---
 defaults/main.yml                         | 3 +--
 templates/services/droneci_compose.yml.j2 | 1 +
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/defaults/main.yml b/defaults/main.yml
index d80134a..9ab62f4 100644
--- a/defaults/main.yml
+++ b/defaults/main.yml
@@ -3,6 +3,7 @@ droneci_version: 1.6.0
 droneci_image: "drone/drone:{{ droneci_version }}"
 
 droneci_service_directory: /var/lib/docker/services/droneci
+droneci_docker_socket_path: /var/run/docker.sock
 droneci_container_name: droneci
 droneci_restart_policy: always
 droneci_service_stopped: False
@@ -76,6 +77,4 @@ droneci_gitea_skip_verify: False
 
 # @var droneci_no_proxy: $ ["drone-server", "drone-agent"]
 droneci_no_proxy: []
-
-droneci_docker_socket_path: /var/run/docker.sock
 droneci_extra_hosts: []
diff --git a/templates/services/droneci_compose.yml.j2 b/templates/services/droneci_compose.yml.j2
index 5f94a33..60fad1d 100644
--- a/templates/services/droneci_compose.yml.j2
+++ b/templates/services/droneci_compose.yml.j2
@@ -6,6 +6,7 @@ services:
   droneci:
     container_name: {{ droneci_container_name }}
     image: {{ droneci_image }}
+    restart: {{ droneci_restart_policy }}
     {% if droneci_exposed_ports | default([]) %}
     ports:
       {% for port in droneci_exposed_ports %}

From 122fbe3a2b815942fd478b3c1237c554d5b82efb Mon Sep 17 00:00:00 2001
From: Robert Kaussow <mail@geeklabor.de>
Date: Wed, 17 Jun 2020 21:20:54 +0200
Subject: [PATCH 09/10] switch to compose v2.4

---
 tasks/setup.yml                           | 4 ----
 templates/services/droneci_compose.yml.j2 | 2 +-
 vars/main.yml                             | 4 ----
 3 files changed, 1 insertion(+), 9 deletions(-)
 delete mode 100644 vars/main.yml

diff --git a/tasks/setup.yml b/tasks/setup.yml
index 4540dfc..7520ac0 100644
--- a/tasks/setup.yml
+++ b/tasks/setup.yml
@@ -1,8 +1,4 @@
 ---
-- name: Concat lists for proxy setup
-  set_fact:
-    droneci_no_proxy: "{{ __droneci_default_no_proxy }} + {{ droneci_no_proxy | default([]) }}"
-
 - block:
     - name: Ensure service directory exists
       file:
diff --git a/templates/services/droneci_compose.yml.j2 b/templates/services/droneci_compose.yml.j2
index 60fad1d..a51e79e 100644
--- a/templates/services/droneci_compose.yml.j2
+++ b/templates/services/droneci_compose.yml.j2
@@ -1,6 +1,6 @@
 #jinja2:lstrip_blocks: True
 {{ ansible_managed | comment }}
-version: "2"
+version: "2.4"
 
 services:
   droneci:
diff --git a/vars/main.yml b/vars/main.yml
deleted file mode 100644
index 0b18f80..0000000
--- a/vars/main.yml
+++ /dev/null
@@ -1,4 +0,0 @@
----
-__droneci_default_no_proxy:
-  - droneserver
-  - droneagent

From 5c4de44c6af8aa9abef918f1502e0cfade893800 Mon Sep 17 00:00:00 2001
From: Robert Kaussow <mail@geeklabor.de>
Date: Thu, 18 Jun 2020 16:07:23 +0200
Subject: [PATCH 10/10] rename volumes and networks

---
 defaults/main.yml | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/defaults/main.yml b/defaults/main.yml
index 9ab62f4..a640e29 100644
--- a/defaults/main.yml
+++ b/defaults/main.yml
@@ -25,7 +25,7 @@ droneci_exposed_ports:
 # droneci_volumes:
 #   # Instead of the name you could specify a path on the container host system,
 #   # but you also have to enable bind mount for this volume
-#   - name: droneci-data
+#   - name: data
 #     # target location inside the container
 #     dest: /var/lib/drone
 #     # enable bind mount, if false volume will be configured as named volume
@@ -33,21 +33,21 @@ droneci_exposed_ports:
 #     bind: True
 # @end
 droneci_volumes:
-  - name: droneci-data
+  - name: data
     dest: /var/lib/drone
     bind: False
 
 # @var droneci_networks:example: >
 # droneci_networks:
-#   - name: droneci
+#   - name: default
 #     # optional network driver, defaults to 'bride'
 #     driver: host
 # @end
 droneci_networks:
-  - name: droneci
+  - name: default
 
 droneci_networks_applied:
-  - droneci
+  - default
 
 # @var droneci_memory_limit: $ "_unset_"
 # @var droneci_memory_limit:example: $ "512m"