Skip to content

Commit 92437b3

Browse files
authored
Merge pull request #184 from YPCrumble/feature/upgrade-molecule
Upgrade Molecule to v3.3.4.
2 parents 7dd9370 + 5ef97c5 commit 92437b3

File tree

11 files changed

+58
-20
lines changed

11 files changed

+58
-20
lines changed
File renamed without changes.

molecule/default/molecule.yml

+2-4
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@ dependency:
33
name: galaxy
44
driver:
55
name: docker
6-
lint:
7-
name: yamllint
6+
lint: |
7+
yamllint .
88
platforms:
99
- name: instance-xenial
1010
image: ubuntu
@@ -20,8 +20,6 @@ provisioner:
2020
name: ansible-lint
2121
env:
2222
ANSIBLE_ROLES_PATH: ../../roles/
23-
scenario:
24-
name: default
2523
verifier:
2624
name: testinfra
2725
lint:

requirements-dev.txt

+4-2
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
11
-r requirements.txt
22

3-
docker==3.6.0
4-
molecule==2.22.0
3+
molecule==3.3.4
4+
molecule-docker==0.2.4
5+
testinfra==3.2.1
6+
yamllint==1.19.0

requirements.txt

+1-1
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
ansible==2.9.20
1+
ansible==4.2.0

roles/celery/tasks/setup_supervisor.yml

+8-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,14 @@
11
---
22

33
- name: Ensure the Supervisor service is running
4-
service: name=supervisor state=started enabled=yes
4+
service:
5+
name: supervisor
6+
state: started
7+
enabled: yes
8+
# TODO: This is likely due to a bug in Ansible.
9+
# Remove this line in the future.
10+
# See https://github.com/ansible/ansible/issues/75005
11+
use: sysvinit
512

613
- name: Create the Supervisor config file for {{ celery_application_name }}
714
template: src=supervisor_{{ celery_application_name }}.conf.j2

roles/db/tasks/main.yml

+8-1
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,14 @@
2323
tags: packages
2424

2525
- name: Ensure the PostgreSQL service is running
26-
service: name=postgresql state=started enabled=yes
26+
service:
27+
name: postgresql
28+
state: started
29+
enabled: yes
30+
# TODO: This is likely due to a bug in Ansible.
31+
# Remove this line in the future.
32+
# See https://github.com/ansible/ansible/issues/75005
33+
use: sysvinit
2734

2835
- name: Ensure database is created
2936
become: true

roles/memcached/tasks/main.yml

+8-1
Original file line numberDiff line numberDiff line change
@@ -13,4 +13,11 @@
1313
- restart memcached
1414

1515
- name: Ensure the Memcached service is running
16-
service: name=memcached state=started enabled=yes
16+
service:
17+
name: memcached
18+
state: started
19+
enabled: yes
20+
# TODO: This is likely due to a bug in Ansible.
21+
# Remove this line in the future.
22+
# See https://github.com/ansible/ansible/issues/75005
23+
use: sysvinit

roles/nginx/tasks/main.yml

+8-1
Original file line numberDiff line numberDiff line change
@@ -35,4 +35,11 @@
3535
notify: reload nginx
3636

3737
- name: Ensure Nginx service is started
38-
service: name=nginx state=started enabled=yes
38+
service:
39+
name: nginx
40+
state: started
41+
enabled: yes
42+
# TODO: This is likely due to a bug in Ansible.
43+
# Remove this line in the future.
44+
# See https://github.com/ansible/ansible/issues/75005
45+
use: sysvinit

roles/rabbitmq/handlers/main.yml

+3-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
11
---
22

33
- name: restart rabbitmq-server
4-
service: name=rabbitmq-server state=restarted
4+
service:
5+
name: rabbitmq-server
6+
state: restarted

roles/rabbitmq/tasks/main.yml

+8-7
Original file line numberDiff line numberDiff line change
@@ -28,19 +28,20 @@
2828
tags:
2929
- packages
3030

31-
- name: Enable the RabbitMQ Management Console
32-
rabbitmq_plugin: names=rabbitmq_management state=enabled
33-
notify: restart rabbitmq-server
34-
3531
- name: Make sure rabbitmq-server is enabled and running
3632
service:
3733
name: rabbitmq-server
3834
state: started
3935
enabled: true
36+
# TODO: This is likely due to a bug in Ansible.
37+
# Remove this line in the future.
38+
# See https://github.com/ansible/ansible/issues/75005
39+
use: sysvinit
40+
41+
- name: Enable the RabbitMQ Management Console
42+
rabbitmq_plugin: names=rabbitmq_management state=enabled
43+
notify: restart rabbitmq-server
4044

4145
- include: setup_vhosts.yml
4246

4347
- include: setup_users.yml
44-
45-
- name: Ensure that the RabbitMQ service is running
46-
service: name=rabbitmq-server state=started enabled=yes

roles/web/tasks/setup_supervisor.yml

+8-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,14 @@
11
---
22

33
- name: Ensure the Supervisor service is running
4-
service: name=supervisor state=started enabled=yes
4+
service:
5+
name: supervisor
6+
state: started
7+
enabled: yes
8+
# TODO: This is likely due to a bug in Ansible.
9+
# Remove this line in the future.
10+
# See https://github.com/ansible/ansible/issues/75005
11+
use: sysvinit
512

613
- name: Create the Supervisor config file
714
template: src=supervisor_config.j2

0 commit comments

Comments
 (0)