Skip to content

Commit 6976aec

Browse files
authored
Use local role name in Molecule tests (#682)
1 parent 12838c7 commit 6976aec

File tree

14 files changed

+14
-13
lines changed

14 files changed

+14
-13
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@ BUG FIXES:
1818
CI/CD:
1919

2020
- Add Molecule tests for NGINX Amplify.
21+
- Use the local role name (`ansible-role-nginx`) instead of the fully qualified role name (`nginxinc.nginx`) in Molecule to ensure tests always work as intended in environments where the role has been already installed beforehand.
2122

2223
## 0.24.2 (October 3rd, 2023)
2324

molecule/default/converge.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
tasks:
55
- name: Install NGINX
66
ansible.builtin.include_role:
7-
name: nginxinc.nginx
7+
name: ansible-role-nginx
88
vars:
99
nginx_modules:
1010
- geoip

molecule/distribution/converge.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,6 @@
44
tasks:
55
- name: Install NGINX
66
ansible.builtin.include_role:
7-
name: nginxinc.nginx
7+
name: ansible-role-nginx
88
vars:
99
nginx_install_from: os_repository

molecule/downgrade-plus/converge.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
tasks:
2222
- name: Install NGINX
2323
ansible.builtin.include_role:
24-
name: nginxinc.nginx
24+
name: ansible-role-nginx
2525
vars:
2626
nginx_type: plus
2727
nginx_license:

molecule/downgrade/converge.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,6 @@
2121
tasks:
2222
- name: Install NGINX
2323
ansible.builtin.include_role:
24-
name: nginxinc.nginx
24+
name: ansible-role-nginx
2525
vars:
2626
nginx_version: "{{ version }}"

molecule/plus/converge.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
tasks:
55
- name: Install NGINX Plus
66
ansible.builtin.include_role:
7-
name: nginxinc.nginx
7+
name: ansible-role-nginx
88
vars:
99
nginx_type: plus
1010
nginx_license:

molecule/source-version/converge.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
tasks:
55
- name: Install NGINX from source
66
ansible.builtin.include_role:
7-
name: nginxinc.nginx
7+
name: ansible-role-nginx
88
vars:
99
nginx_install_from: source
1010
nginx_branch: stable

molecule/source/converge.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
tasks:
55
- name: Install NGINX from source
66
ansible.builtin.include_role:
7-
name: nginxinc.nginx
7+
name: ansible-role-nginx
88
vars:
99
nginx_install_from: source
1010
nginx_branch: stable

molecule/stable/converge.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,6 @@
44
tasks:
55
- name: Install NGINX
66
ansible.builtin.include_role:
7-
name: nginxinc.nginx
7+
name: ansible-role-nginx
88
vars:
99
nginx_branch: stable

molecule/uninstall-plus/converge.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
tasks:
55
- name: Uninstall NGINX
66
ansible.builtin.include_role:
7-
name: nginxinc.nginx
7+
name: ansible-role-nginx
88
vars:
99
nginx_type: plus
1010
nginx_setup_license: false

0 commit comments

Comments
 (0)