Ansible-role-nginx-config: la procédure de nettoyage ne nettoie pas

Bonne prise. Les tests n'ont pris en compte rien d'autre que la suppression de default.conf . Le PR entrant devrait résoudre ce bogue et ajouter un cas de test approprié pour un répertoire entier en cours de nettoyage.

Après git pull, il ne nettoie toujours pas pour moi. Ne nettoie que default.conf. Récursivité essayée définie sur true et false.

(item=/etc/nginx/conf.d/default.conf) => {"ansible_loop_var": "item", "changed": false, "item": "/etc/nginx/conf.d/default.conf", "path": "/etc/nginx/conf.d/default.conf", "state": "absent"}

Après la mise à jour de Jinja2, le nettoyage ne fonctionne toujours pas. On rouvre ça ?

TASK [nginxinc.nginx_config : Find NGINX config files] ************************************************************************************************************************************************************************************************************************
ok: [testing-snap05.MYDOMAIN.TLD] => (item={'directory': ['/etc/nginx/conf.d'], 'recurse': True}) => {"ansible_loop_var": "item", "changed": false, "examined": 7, "files": [{"atime": 1616759901.1814494, "ctime": 1616759896.8093383, "dev": 64780, "gid": 0, "gr_name": "root", "inode": 652060, "isblk": false, "ischr": false, "isdir": false, "isfifo": false, "isgid": false, "islnk": false, "isreg": true, "issock": false, "isuid": false, "mode": "0644", "mtime": 1616759896.5933328, "nlink": 1, "path": "/etc/nginx/conf.d/testing-snap05.staging.MYDOMAIN.TLD_adhoc.conf", "pw_name": "root", "rgrp": true, "roth": true, "rusr": true, "size": 2, "uid": 0, "wgrp": false, "woth": false, "wusr": true, "xgrp": false, "xoth": false, "xusr": false}, {"atime": 1616777440.7914264, "ctime": 1616777440.3514154, "dev": 64780, "gid": 0, "gr_name": "root", "inode": 652158, "isblk": false, "ischr": false, "isdir": false, "isfifo": false, "isgid": false, "islnk": false, "isreg": true, "issock": false, "isuid": false, "mode": "0644", "mtime": 1616777440.13941, "nlink": 1, "path": "/etc/nginx/conf.d/testing-snap05.staging.MYDOMAIN.TLD.conf", "pw_name": "root", "rgrp": true, "roth": true, "rusr": true, "size": 1255, "uid": 0, "wgrp": false, "woth": false, "wusr": true, "xgrp": false, "xoth": false, "xusr": false}], "item": {"directory": ["/etc/nginx/conf.d"], "recurse": true}, "matched": 2, "msg": ""}                                                                                                                                                                                                                                        

TASK [nginxinc.nginx_config : Remove NGINX config files] **********************************************************************************************************************************************************************************************************************

Les nouveaux tests passent, donc je ne suis pas tout à fait sûr qu'il s'agisse d'un bogue dans le rôle. J'ai également essayé d'utiliser le même nom de fichier que dans votre exemple et il a été correctement supprimé.

TASK [ansible-role-nginx-config : Find NGINX config files] *********************
ok: [ubuntu-bionic] => (item={'directory': ['/etc/nginx/conf.d'], 'recurse': False})
ok: [centos-7] => (item={'directory': ['/etc/nginx/conf.d'], 'recurse': False})
ok: [debian-buster] => (item={'directory': ['/etc/nginx/conf.d'], 'recurse': False})
ok: [alpine-3.10] => (item={'directory': ['/etc/nginx/conf.d'], 'recurse': False})

TASK [ansible-role-nginx-config : Remove NGINX config files] *******************
changed: [centos-7] => (item=/etc/nginx/conf.d/default.conf)
changed: [debian-buster] => (item=/etc/nginx/conf.d/default.conf)
changed: [alpine-3.10] => (item=/etc/nginx/conf.d/default.conf)
changed: [ubuntu-bionic] => (item=/etc/nginx/conf.d/default.conf)
changed: [centos-7] => (item=/etc/nginx/conf.d/mock.conf)
changed: [debian-buster] => (item=/etc/nginx/conf.d/mock.conf)
changed: [alpine-3.10] => (item=/etc/nginx/conf.d/mock.conf)
changed: [ubuntu-bionic] => (item=/etc/nginx/conf.d/mock.conf)
changed: [centos-7] => (item=/etc/nginx/conf.d/testing-snap05.staging.MYDOMAIN.TLD.conf)
changed: [ubuntu-bionic] => (item=/etc/nginx/conf.d/testing-snap05.staging.MYDOMAIN.TLD.conf)
changed: [debian-buster] => (item=/etc/nginx/conf.d/testing-snap05.staging.MYDOMAIN.TLD.conf)
changed: [alpine-3.10] => (item=/etc/nginx/conf.d/testing-snap05.staging.MYDOMAIN.TLD.conf)
ok: [centos-7] => (item=/etc/nginx/conf.d/default.conf)
ok: [ubuntu-bionic] => (item=/etc/nginx/conf.d/default.conf)
ok: [debian-buster] => (item=/etc/nginx/conf.d/default.conf)
ok: [alpine-3.10] => (item=/etc/nginx/conf.d/default.conf)

Avez-vous essayé de mettre à jour vers la dernière version d'Ansible ? Les modules utilisés en coulisses ont peut-être été mis à jour depuis Ansible 2.9.6 . (La version minimale absolue d'Ansible que vous devriez utiliser ces jours-ci est de 2.9.10 raison de sa prise en charge native des collections Ansible.)

J'ai Ansible 2.9.6 qui est le dernier disponible pour le dernier Ubuntu LTS, qui est Ubuntu 20.04.

Vous pouvez installer la dernière version en utilisant pip ou le référentiel de packages d'Ansible https://docs.ansible.com/ansible/latest/installation_guide/intro_installation.html#installing -ansible-on-ubuntu. Sinon, vous êtes bloqué avec la dernière version disponible sur le référentiel de packages d'Ubuntu, ce qui n'est généralement pas idéal.

J'ai aussi cette erreur sur le mien. Dernier ansible disponible à partir de brew :

 > ansible --version
ansible [core 2.11.3]

avec débogage

  register: nginx_config_files

- debug:
    msg: "{{ nginx_config_files.files  }}"

- name: Remove NGINX config files

Je suis en train

TASK [nginxinc.nginx_config : Find NGINX config files] *********************************************************************************
ok: [ansible-test] => (item={'directory': ['/etc/nginx/conf.d', '/etc/nginx/snippets'], 'recurse': True})

TASK [nginxinc.nginx_config : debug] ***************************************************************************************************
fatal: [ansible-test]: FAILED! => {"msg": "The task includes an option with an undefined variable. The error was: 'dict object' has no attribute 'files'\n\nThe error appears to be in '...../roles/nginxinc.nginx_config/tasks/config/cleanup-config.yml': line 11, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n- debug:\n  ^ here\n"}

Comme je l'ai mentionné dans un commentaire précédent, il existe maintenant des tests explicites pour ce cas d'utilisation, et ils réussissent tous (https://github.com/nginxinc/ansible-role-nginx-config/blob/main/molecule/cleanup_module/ converge.yml). Il peut y avoir des cas extrêmes qui provoquent toujours l'échec du rôle, mais j'aurais besoin de plus de détails pour pouvoir le reproduire moi-même. D'après ce que je peux dire compte tenu de la sortie de votre tâche, le problème pourrait être que le mot-clé recurse appartient à chaque élément du dictionnaire de répertoire, pas en dehors de ces éléments (consultez le test Molecule que j'ai lié plus tôt pour plus de détails).

PS : nginx_config_files.files n'est pas utilisé (https://github.com/nginxinc/ansible-role-nginx-config/blob/main/tasks/config/cleanup-config.yml#L11-L17) 😄

@alessfg Oui, je viens juste de trouver ça. Venez ici pour écrire sur ma solution :)
Le problème est que la dernière version publiée dans Galaxy (que j'utilise pour mes playbooks) contient la version 0.3.3. Ces changements que vous écrivez sont absents de cette balise et ne sont présents qu'à main . J'ai donc basculé la source vers le référentiel git pour requirements.yml.