Skip to content

Commit f777868

Browse files
Bump ansible-lint from 4.3.7 to 5.0.4 in /.github/workflows/requirements (#70)
1 parent bea9070 commit f777868

File tree

11 files changed

+61
-51
lines changed

11 files changed

+61
-51
lines changed

.ansible-lint

+3-2
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
---
12
skip_list:
2-
- '106'
3-
- '204'
3+
- '106'
4+
- '204'
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
ansible-base==2.10.7
2-
ansible-lint==4.3.7
2+
ansible-lint==5.0.4
33
yamllint==1.26.0
44
molecule[docker]==3.2.4
55
docker==4.4.4

CHANGELOG.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ ENHANCEMENTS:
1919
* Consolidate dependencies into a single tasks file.
2020
* Remove requirement for `package_facts` module when using this role.
2121
* Update Signatures repository URL.
22-
* Update Ansible base to `2.10.7`, Molecule to `3.2.4`, yamllint to `1.26.0` and Docker Python SDK to `4.4.4`.
22+
* Update Ansible base to `2.10.7`, Ansible Lint to `5.0.4`, Molecule to `3.2.4`, yamllint to `1.26.0` and Docker Python SDK to `4.4.4`.
2323
* Specify GitHub actions Ubuntu release.
2424
* Minor GitHub template tweaks, including the creation of a SECURITY doc.
2525
* Only run GitHub actions Galaxy CI/CD workflow when a new release is published.

meta/main.yml

+1
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ galaxy_info:
33
author: Daniel Edgar
44
description: Official Ansible role for NGINX App Protect
55
role_name: nginx_app_protect
6+
namespace: nginxinc
67
company: F5 Networks, Inc.
78

89
license: Apache License, Version 2.0

molecule/advanced/molecule.yml

+8
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,8 @@
11
---
2+
dependency:
3+
name: galaxy
4+
options:
5+
role-file: molecule/advanced/requirements.yml
26
driver:
37
name: docker
48
lint: |
@@ -74,3 +78,7 @@ provisioner:
7478
defaults:
7579
no_target_syslog: true
7680
log: false
81+
playbooks:
82+
prepare: prepare.yml
83+
converge: converge.yml
84+
verify: verify.yml

molecule/advanced/prepare.yml

+8-6
Original file line numberDiff line numberDiff line change
@@ -19,12 +19,14 @@
1919

2020
- name: Set up rsyslog server for verifying NAP syslog events
2121
hosts: nap
22-
roles:
23-
- role: robertdebock.rsyslog
24-
vars:
25-
rsyslog_receiver: true
26-
rsyslog_remote_tcp: true
27-
rsyslog_remote_port: 514
22+
tasks:
23+
- name: Set up rsyslog
24+
include_role:
25+
name: robertdebock.rsyslog
26+
vars:
27+
rsyslog_receiver: true
28+
rsyslog_remote_tcp: true
29+
rsyslog_remote_port: 514
2830

2931
- name: Prepare workload for tests
3032
hosts: workload

molecule/default/molecule.yml

+4
Original file line numberDiff line numberDiff line change
@@ -43,3 +43,7 @@ platforms:
4343
command: "/sbin/init"
4444
provisioner:
4545
name: ansible
46+
playbooks:
47+
prepare: prepare.yml
48+
converge: converge.yml
49+
verify: verify.yml

molecule/specific-version/converge.yml

+20-2
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,17 @@
11
---
22
- name: Converge
33
hosts: all
4-
vars_files:
5-
- scenario-vars.yml
4+
vars:
5+
specify_app_protect_signatures_version: true
6+
specify_app_protect_threat_campaigns_version: true
7+
app_protect_signature_version_matrix:
8+
alpine: "=2021.01.20-r1"
9+
debian: "=2019.07.16-1"
10+
redhat: "-2019.07.16"
11+
app_protect_threat_campaigns_version_matrix:
12+
alpine: "=2021.01.03-r1"
13+
debian: "=2020.08.20-1"
14+
redhat: "-2020.08.20"
615
tasks:
716
- name: Set NGINX App Protect signature version fact
817
set_fact:
@@ -17,3 +26,12 @@
1726
- name: Install NGINX App Protect
1827
include_role:
1928
name: ansible-role-nginx-app-protect
29+
vars:
30+
nginx_app_protect_license:
31+
certificate: license/nginx-repo.crt
32+
key: license/nginx-repo.key
33+
nginx_app_protect_remove_license: false
34+
nginx_app_protect_install_signatures: true
35+
nginx_app_protect_install_threat_campaigns: true
36+
nginx_app_protect_configure: false
37+
nginx_app_protect_state: present

molecule/specific-version/molecule.yml

+4
Original file line numberDiff line numberDiff line change
@@ -43,3 +43,7 @@ platforms:
4343
command: "/sbin/init"
4444
provisioner:
4545
name: ansible
46+
playbooks:
47+
prepare: prepare.yml
48+
converge: converge.yml
49+
verify: verify.yml

molecule/specific-version/scenario-vars.yml

-22
This file was deleted.

molecule/specific-version/verify.yml

+11-17
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,17 @@
11
---
22
- name: Verify
33
hosts: all
4-
vars_files:
5-
- scenario-vars.yml
4+
vars:
5+
specify_app_protect_signatures_version: true
6+
specify_app_protect_threat_campaigns_version: true
7+
app_protect_signature_version_matrix:
8+
alpine: "=2021.01.20-r1"
9+
debian: "=2019.07.16-1"
10+
redhat: "-2019.07.16"
11+
app_protect_threat_campaigns_version_matrix:
12+
alpine: "=2021.01.03-r1"
13+
debian: "=2020.08.20-1"
14+
redhat: "-2020.08.20"
615
tasks:
716
- name: Check if NGINX Plus is installed
817
package:
@@ -53,21 +62,6 @@
5362
package_facts:
5463
manager: auto
5564

56-
- name: Get installed NAP version
57-
debug:
58-
msg: "{{ ansible_distribution }} {{ ansible_distribution_version }} - App Protect version installed: {{ ansible_facts.packages['app-protect'] | map(attribute='version') | first }}"
59-
when: nginx_app_protect_state != "absent"
60-
61-
- name: Get installed NAP signatures version
62-
debug:
63-
msg: "{{ ansible_distribution }} {{ ansible_distribution_version }} - Signatures version installed: {{ ansible_facts.packages['app-protect-attack-signatures'] | map(attribute='version') | first }}"
64-
when: nginx_app_protect_install_signatures
65-
66-
- name: Get installed NAP threat campaigns version
67-
debug:
68-
msg: "{{ ansible_distribution }} {{ ansible_distribution_version }} Threat Campaigns version installed: {{ ansible_facts.packages['app-protect-threat-campaigns'] | map(attribute='version') | first }}"
69-
when: nginx_app_protect_install_threat_campaigns
70-
7165
- name: Verify installed NAP signatures version matches requested version
7266
assert:
7367
that: (ansible_facts.packages['app-protect-attack-signatures'] | map(attribute='version') | first) == (app_protect_signature_version_matrix[ansible_os_family | lower] | regex_replace('^-|=','') + (ansible_os_family | lower == 'debian') | ternary('~' ~ ansible_distribution_release, ''))

0 commit comments

Comments
 (0)