From a4afcec939953f2e3239b5f44e2b6a38b86ec950 Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Wed, 2 Feb 2022 19:01:38 +0100 Subject: [PATCH] Bump ansible-lint from 5.3.1 to 5.3.2 in /.github/workflows/requirements (#481) --- .../requirements/requirements_molecule.txt | 2 +- CHANGELOG.md | 10 ++++++++++ README.md | 2 -- molecule/default/converge.yml | 6 +++--- molecule/default/molecule.yml | 7 ------- molecule/default/verify.yml | 2 +- molecule/downgrade/molecule.yml | 7 ------- molecule/module/molecule.yml | 7 ------- molecule/plus/molecule.yml | 7 ------- molecule/source/molecule.yml | 7 ------- molecule/uninstall/molecule.yml | 7 ------- molecule/uninstall_plus/molecule.yml | 7 ------- molecule/upgrade/molecule.yml | 7 ------- molecule/upgrade/prepare.yml | 6 +++--- molecule/upgrade/verify.yml | 13 ++++++++++++- 15 files changed, 30 insertions(+), 67 deletions(-) diff --git a/.github/workflows/requirements/requirements_molecule.txt b/.github/workflows/requirements/requirements_molecule.txt index cf5f0fa..2461b22 100644 --- a/.github/workflows/requirements/requirements_molecule.txt +++ b/.github/workflows/requirements/requirements_molecule.txt @@ -1,6 +1,6 @@ ansible-core==2.12.1 jinja2==3.0.3 -ansible-lint==5.3.1 +ansible-lint==5.3.2 yamllint==1.26.3 molecule[docker]==3.5.2 docker==5.0.3 diff --git a/CHANGELOG.md b/CHANGELOG.md index 5307825..9642a6d 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,5 +1,15 @@ # Changelog +## 0.23.0 (Unreleased) + +BREAKING CHANGES: + +CentOS 8 has reached EoL and has thus been removed from the list of supported platforms. + +BUG FIXES: + +The Molecule `upgrade` scenario verification test no longer has to be updated on each new NGINX OSS release. + ## 0.22.0 (December 9, 2021) BREAKING CHANGES: diff --git a/README.md b/README.md index 95db16e..4555b4b 100644 --- a/README.md +++ b/README.md @@ -81,7 +81,6 @@ Amazon Linux: - 2 CentOS: - 7.4+ - - 8 Debian: - buster (10) - bullseye (11) @@ -109,7 +108,6 @@ Amazon Linux 2: - any CentOS: - 7.4+ - - 8 Debian: - buster (10) - bullseye (11) diff --git a/molecule/default/converge.yml b/molecule/default/converge.yml index 17427dd..a7e2b6f 100644 --- a/molecule/default/converge.yml +++ b/molecule/default/converge.yml @@ -4,15 +4,15 @@ pre_tasks: - name: Set repo if Alpine set_fact: - version: "=1.21.4-r1" + version: "=1.21.5-r1" when: ansible_facts['os_family'] == "Alpine" - name: Set repo if Debian set_fact: - version: "=1.21.4-1~{{ ansible_facts['distribution_release'] }}" + version: "=1.21.5-1~{{ ansible_facts['distribution_release'] }}" when: ansible_facts['os_family'] == "Debian" - name: Set repo if Red Hat set_fact: - version: "-1.21.4-1.{{ (ansible_facts['distribution']=='Amazon') | ternary('amzn2', ('el' + ansible_facts['distribution_major_version'] | string)) }}.ngx" + version: "-1.21.5-1.{{ (ansible_facts['distribution']=='Amazon') | ternary('amzn2', ('el' + ansible_facts['distribution_major_version'] | string)) }}.ngx" when: ansible_facts['os_family'] == "RedHat" - name: Enable NGINX @CentOS-AppStream dnf modules shell: diff --git a/molecule/default/molecule.yml b/molecule/default/molecule.yml index 8a25143..e7c2750 100644 --- a/molecule/default/molecule.yml +++ b/molecule/default/molecule.yml @@ -48,13 +48,6 @@ platforms: volumes: - "/sys/fs/cgroup:/sys/fs/cgroup:rw" command: "/usr/sbin/init" - - name: centos-8 - image: centos:8 - dockerfile: ../common/Dockerfile.j2 - privileged: true - volumes: - - "/sys/fs/cgroup:/sys/fs/cgroup:rw" - command: "/usr/sbin/init" - name: debian-buster image: debian:buster-slim dockerfile: ../common/Dockerfile.j2 diff --git a/molecule/default/verify.yml b/molecule/default/verify.yml index d93a4e9..fa5e230 100644 --- a/molecule/default/verify.yml +++ b/molecule/default/verify.yml @@ -30,4 +30,4 @@ chdir: "{{ ((ansible_facts['system'] | lower is not search('bsd')) | ternary('/etc/nginx', '/usr/local/sbin')) }}" changed_when: false register: version - failed_when: version is not search('1.21.4') + failed_when: version is not search('1.21.5') diff --git a/molecule/downgrade/molecule.yml b/molecule/downgrade/molecule.yml index ae42aab..2e7323f 100644 --- a/molecule/downgrade/molecule.yml +++ b/molecule/downgrade/molecule.yml @@ -41,13 +41,6 @@ platforms: volumes: - "/sys/fs/cgroup:/sys/fs/cgroup:rw" command: "/usr/sbin/init" - - name: centos-8 - image: centos:8 - dockerfile: ../common/Dockerfile.j2 - privileged: true - volumes: - - "/sys/fs/cgroup:/sys/fs/cgroup:rw" - command: "/usr/sbin/init" - name: debian-buster image: debian:buster-slim dockerfile: ../common/Dockerfile.j2 diff --git a/molecule/module/molecule.yml b/molecule/module/molecule.yml index 8a25143..e7c2750 100644 --- a/molecule/module/molecule.yml +++ b/molecule/module/molecule.yml @@ -48,13 +48,6 @@ platforms: volumes: - "/sys/fs/cgroup:/sys/fs/cgroup:rw" command: "/usr/sbin/init" - - name: centos-8 - image: centos:8 - dockerfile: ../common/Dockerfile.j2 - privileged: true - volumes: - - "/sys/fs/cgroup:/sys/fs/cgroup:rw" - command: "/usr/sbin/init" - name: debian-buster image: debian:buster-slim dockerfile: ../common/Dockerfile.j2 diff --git a/molecule/plus/molecule.yml b/molecule/plus/molecule.yml index 15dbff0..8871df8 100644 --- a/molecule/plus/molecule.yml +++ b/molecule/plus/molecule.yml @@ -48,13 +48,6 @@ platforms: volumes: - "/sys/fs/cgroup:/sys/fs/cgroup:rw" command: "/usr/sbin/init" - - name: centos-8 - image: centos:8 - dockerfile: ../common/Dockerfile.j2 - privileged: true - volumes: - - "/sys/fs/cgroup:/sys/fs/cgroup:rw" - command: "/usr/sbin/init" - name: debian-buster image: debian:buster-slim dockerfile: ../common/Dockerfile.j2 diff --git a/molecule/source/molecule.yml b/molecule/source/molecule.yml index 8a25143..e7c2750 100644 --- a/molecule/source/molecule.yml +++ b/molecule/source/molecule.yml @@ -48,13 +48,6 @@ platforms: volumes: - "/sys/fs/cgroup:/sys/fs/cgroup:rw" command: "/usr/sbin/init" - - name: centos-8 - image: centos:8 - dockerfile: ../common/Dockerfile.j2 - privileged: true - volumes: - - "/sys/fs/cgroup:/sys/fs/cgroup:rw" - command: "/usr/sbin/init" - name: debian-buster image: debian:buster-slim dockerfile: ../common/Dockerfile.j2 diff --git a/molecule/uninstall/molecule.yml b/molecule/uninstall/molecule.yml index 46d6c10..dbfb493 100644 --- a/molecule/uninstall/molecule.yml +++ b/molecule/uninstall/molecule.yml @@ -48,13 +48,6 @@ platforms: volumes: - "/sys/fs/cgroup:/sys/fs/cgroup:rw" command: "/usr/sbin/init" - - name: centos-8 - image: centos:8 - dockerfile: ../common/Dockerfile.j2 - privileged: true - volumes: - - "/sys/fs/cgroup:/sys/fs/cgroup:rw" - command: "/usr/sbin/init" - name: debian-buster image: debian:buster-slim dockerfile: ../common/Dockerfile.j2 diff --git a/molecule/uninstall_plus/molecule.yml b/molecule/uninstall_plus/molecule.yml index 15dbff0..8871df8 100644 --- a/molecule/uninstall_plus/molecule.yml +++ b/molecule/uninstall_plus/molecule.yml @@ -48,13 +48,6 @@ platforms: volumes: - "/sys/fs/cgroup:/sys/fs/cgroup:rw" command: "/usr/sbin/init" - - name: centos-8 - image: centos:8 - dockerfile: ../common/Dockerfile.j2 - privileged: true - volumes: - - "/sys/fs/cgroup:/sys/fs/cgroup:rw" - command: "/usr/sbin/init" - name: debian-buster image: debian:buster-slim dockerfile: ../common/Dockerfile.j2 diff --git a/molecule/upgrade/molecule.yml b/molecule/upgrade/molecule.yml index 8ae258c..fdb4c6a 100644 --- a/molecule/upgrade/molecule.yml +++ b/molecule/upgrade/molecule.yml @@ -20,13 +20,6 @@ platforms: volumes: - "/sys/fs/cgroup:/sys/fs/cgroup:rw" command: "/usr/sbin/init" - - name: centos-8 - image: centos:8 - dockerfile: ../common/Dockerfile.j2 - privileged: true - volumes: - - "/sys/fs/cgroup:/sys/fs/cgroup:rw" - command: "/usr/sbin/init" - name: debian-buster image: debian:buster-slim dockerfile: ../common/Dockerfile.j2 diff --git a/molecule/upgrade/prepare.yml b/molecule/upgrade/prepare.yml index 8d06299..729377a 100644 --- a/molecule/upgrade/prepare.yml +++ b/molecule/upgrade/prepare.yml @@ -4,15 +4,15 @@ pre_tasks: - name: Set repo if Alpine set_fact: - version: "=1.21.3-r1" + version: "=1.21.5-r1" when: ansible_facts['os_family'] == "Alpine" - name: Set repo if Debian set_fact: - version: "=1.21.3-1~{{ ansible_facts['distribution_release'] }}" + version: "=1.21.5-1~{{ ansible_facts['distribution_release'] }}" when: ansible_facts['os_family'] == "Debian" - name: Set repo if Red Hat set_fact: - version: "-1.21.3-1.{{ (ansible_facts['distribution']=='Amazon') | ternary('amzn2', ('el' + ansible_facts['distribution_major_version'] | string)) }}.ngx" + version: "-1.21.5-1.{{ (ansible_facts['distribution']=='Amazon') | ternary('amzn2', ('el' + ansible_facts['distribution_major_version'] | string)) }}.ngx" when: ansible_facts['os_family'] == "RedHat" - name: Enable NGINX @CentOS-AppStream dnf modules shell: diff --git a/molecule/upgrade/verify.yml b/molecule/upgrade/verify.yml index 4664d05..b327871 100644 --- a/molecule/upgrade/verify.yml +++ b/molecule/upgrade/verify.yml @@ -24,10 +24,21 @@ url: http://localhost status_code: 200 + - name: Fetch NGINX version + uri: + url: https://version.nginx.com/nginx/mainline + return_content: true + check_mode: false + register: nginx_versions + + - name: Set NGINX version + set_fact: + nginx_version: "{{ nginx_versions.content | regex_search('([0-9]+\\.){2}[0-9]+') }}" + - name: Verify NGINX has been upgraded command: nginx -v args: chdir: "{{ ((ansible_facts['system'] | lower is not search('bsd')) | ternary('/etc/nginx', '/usr/local/sbin')) }}" changed_when: false register: version - failed_when: version is not search('1.21.4') + failed_when: version is not search(nginx_version)