Skip to content

Commit

Permalink
Correct runtime error during cleanup.
Browse files Browse the repository at this point in the history
TASK [ansible-role-nginx-config : Remove NGINX config files] *******************************************************************************************************************************************************
Monday 27 May 2024  11:53:13 +0100 (0:00:00.029)       0:00:14.736 ************
fatal: [localhost]: FAILED! => {"msg": "'dict object' has no attribute 'results'. 'dict object' has no attribute 'results'"}
  • Loading branch information
pecastro committed May 29, 2024
1 parent dcff52f commit 1b72aa5
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,13 +5,15 @@
BUG FIXES:

- Fix incompatibility when using the `listen` directive and setting both the `quic` and `so_keepalive` parameters.
- Correct cleanup error when `nginx_config_cleanup_paths` is not defined.

CI/CD:

- Add Molecule tests covering common NGINX use cases (web server, reverse proxy), enabling the NGINX stub status metrics, and NGINX Plus API and live metrics dashboard.
- Update the platforms used in the various Molecule scenarios.
- Use the local role name (`ansible-role-nginx-config`) instead of the fully qualified role name (`nginxinc.nginx_config`) in Molecule to ensure tests always work as intended in environments where the role has been already installed beforehand.


## 0.7.1 (October 3rd, 2023)

ENHANCEMENTS:
Expand Down
2 changes: 1 addition & 1 deletion tasks/config/cleanup-config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,5 +12,5 @@
ansible.builtin.file:
path: "{{ item }}"
state: absent
loop: "{{ nginx_config_files['results'] | map(attribute='files') | sum(start=[]) | map(attribute='path') | list + nginx_config_cleanup_files | default('') | list }}"
loop: "{{ nginx_config_files['results'] | default('') | map(attribute='files') | sum(start=[]) | map(attribute='path') | list + nginx_config_cleanup_files | default('') | list }}"
notify: (Handler - NGINX Config) Run NGINX

0 comments on commit 1b72aa5

Please sign in to comment.