Skip to content

Commit a4df840

Browse files
authored
Fix package module in Molecule verification tests (#364)
1 parent 6ff33cf commit a4df840

File tree

9 files changed

+29
-14
lines changed

9 files changed

+29
-14
lines changed

.github/ISSUE_TEMPLATE/bug_report.md

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5,22 +5,22 @@ title: ''
55
labels: ''
66
assignees: ''
77
---
8-
**Describe the bug**
8+
### Describe the bug
99
A clear and concise description of what the bug is.
1010

11-
**To reproduce**
11+
### To reproduce
1212
Steps to reproduce the behavior:
1313
1. Deploy NGINX role using playbook.yml
1414
2. View output/logs/configuration on '...'
1515
3. See error
1616

17-
**Expected behavior**
17+
### Expected behavior
1818
A clear and concise description of what you expected to happen.
1919

20-
**Your environment:**
20+
### Your environment:
2121
- Version of the NGINX role or specific commit
2222
- Version of Ansible
2323
- Target deployment platform
2424

25-
**Additional context**
25+
### Additional context
2626
Add any other context about the problem here.

.github/ISSUE_TEMPLATE/feature_request.md

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5,14 +5,14 @@ title: ''
55
labels: ''
66
assignees: ''
77
---
8-
**Is your feature request related to a problem? Please describe.**
9-
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
8+
### Is your feature request related to a problem? Please describe
9+
A clear and concise description of what the problem is. Ex. I'm always frustrated when ...
1010

11-
**Describe the solution you'd like**
11+
### Describe the solution you'd like
1212
A clear and concise description of what you want to happen.
1313

14-
**Describe alternatives you've considered**
14+
### Describe alternatives you've considered
1515
A clear and concise description of any alternative solutions or features you've considered.
1616

17-
**Additional context**
17+
### Additional context
1818
Add any other context or screenshots about the feature request here.

.github/PULL_REQUEST_TEMPLATE/pull_request_template.md renamed to .github/PULL_REQUEST_TEMPLATE/pull_request.md

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,10 @@
1+
---
2+
name: Pull request
3+
about: Submit a pull request to contribute to this role
4+
title: ''
5+
labels: ''
6+
assignees: ''
7+
---
18
### Proposed changes
29
Describe the use case and detail of the change. If this PR addresses an issue on GitHub, make sure to include a link to that issue using one of the [supported keywords](https://docs.github.com/en/github/managing-your-work-on-github/linking-a-pull-request-to-an-issue) here in this description (not in the title of the PR).
310

File renamed without changes.

CHANGELOG.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,14 @@ ENHANCEMENTS:
1010

1111
* Only run GitHub actions Galaxy CI/CD workflow when a new release is published.
1212
* Specify GitHub actions Ubuntu release.
13+
* Minor GitHub template tweaks, including the creation of a SECURITY doc.
1314
* Update list of supported platforms.
1415
* Update Ansible base to `2.10.5`, Ansible to `2.10.6`, Molecule to `3.2.3` and yamllint to `1.26.0`.
1516

17+
BUG FIXES:
18+
19+
Add `state` parameter to package module in Molecule verification tests.
20+
1621
## 0.19.1 (January 11, 2021)
1722

1823
ENHANCEMENTS:

molecule/common/playbooks/default_verify.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
- name: Check if NGINX is installed
66
package:
77
name: nginx
8+
state: present
89
check_mode: yes
910
register: install
1011
failed_when: (install is changed) or (install is failed)

molecule/common/playbooks/module_verify.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
- name: Check if NGINX is installed
66
package:
77
name: nginx
8+
state: present
89
check_mode: yes
910
register: install
1011
failed_when: (install is changed) or (install is failed)

molecule/common/playbooks/plus_converge.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
- name: Converge
33
hosts: all
44
tasks:
5-
- name: Install NGINX
5+
- name: Install NGINX Plus
66
include_role:
77
name: ansible-role-nginx
88
vars:

molecule/common/playbooks/plus_verify.yml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,14 +2,15 @@
22
- name: Verify
33
hosts: all
44
tasks:
5-
- name: Check if NGINX is installed
5+
- name: Check if NGINX Plus is installed
66
package:
77
name: nginx-plus
8+
state: present
89
check_mode: yes
910
register: install
1011
failed_when: (install is changed) or (install is failed)
1112

12-
- name: Check if NGINX service is running
13+
- name: Check if NGINX Plus service is running
1314
service:
1415
name: nginx
1516
state: started
@@ -18,7 +19,7 @@
1819
register: service
1920
failed_when: (service is changed) or (service is failed)
2021

21-
- name: Verify NGINX is up and running
22+
- name: Verify NGINX Plus is up and running
2223
uri:
2324
url: http://localhost
2425
status_code: 200

0 commit comments

Comments
 (0)