Produits Entr'ouvert » Publik Installation Développeur

Marche dans le cas où l'utilisateur ne définit pas de user dans son inventaire ou sur la CLI.
Dans le cas contraire erreur :

TASK [common : set user variable] ***********************************************************************
fatal: [localhost]: FAILED! => {"msg": "The task includes an option with an undefined variable. The error was: 'dict object' has no attribute 'stdout'\n\nThe error appears to have been in '/home/cazino/src/publi
k-devinst/roles/common/tasks/main.yml': line 7, 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- name: set user variable\n  ^ here\n\
nexception type: <class 'ansible.errors.AnsibleUndefinedVariable'>\nexception: 'dict object' has no attribute 'stdout'"} 

Tenté de supprimé le premier when dans le patch, l'utilisateur de l'inventaire est quand même surchargé.

Manu, j'ai oublié pourquoi on ne prend pas simplement "ansible_user"? (surchargeable un peu partout, https://docs.ansible.com/ansible/latest/user_guide/playbooks_variables.html)

Déjà essayé mais ça ne marchait pas, en particulier dans les roles qui sont appelés par un import_role: si ma mémoire est bonne.

Mais import_role est une fonctionnalité très récente, ça vaut peut-être le coup de re-essayer avec la dernière version d'ansible de pypi, ça a peut-être été corrigé.

Pour la question "ansible_user", nous n'utilisons pas cette variable parce que la procédure actuelle utilise "sudo" et que "ansible_user" donne alors "root". Dans ce cas il y a bien la varibale SUDO_USER qui est intéressante mais qui n'existe pas dans une utilisation simple.

chris@pad:~$ sudo ansible -m setup localhost | grep ansible_user
 [WARNING]: provided hosts list is empty, only localhost is available. Note that the implicit localhost does not match 'all'
        "ansible_user_dir": "/root", 
        "ansible_user_gecos": "root", 
        "ansible_user_gid": 0, 
        "ansible_user_id": "root", 
        "ansible_user_shell": "/bin/bash", 
        "ansible_user_uid": 0, 
        "ansible_userspace_architecture": "x86_64", 
        "ansible_userspace_bits": "64", 
chris@pad:~$ sudo ansible -m setup localhost | grep chris
 [WARNING]: provided hosts list is empty, only localhost is available. Note that the implicit localhost does not match 'all'
            "PWD": "/home/chris", 
            "SUDO_USER": "chris", 
                "device": "/home/chris/.Private", 
                "mount": "/home/chris",

Pour faire marcher mon patch initial il faudrait déplacer "user" depuis group_vars vers roles/common/defaults. Ceci afin que tu puisse la surcharger avec ton propre inventory:

https://docs.ansible.com/ansible/latest/user_guide/playbooks_variables.html#variable-precedence-where-should-i-put-a-variable

Je regardais un talk de la dernière debconf à Hamburg sur Ansible, et le gars disais qu'il faut privilégier roles/defaults/mail.yaml pour cette raison, voir 7:20 dans https://saimei.ftp.acc.umu.se/pub/debian-meetings/2018/miniconf-hamburg/2018-05-20/ansible_bcp.webm

Christophe Siraut a écrit :

Pour la question "ansible_user", nous n'utilisons pas cette variable parce que la procédure actuelle utilise "sudo" et que "ansible_user" donne alors "root". Dans ce cas il y a bien la varibale SUDO_USER qui est intéressante mais qui n'existe pas dans une utilisation simple.

Je n'utilise jamais sudo, je fais ansible-playbook -K.

Pour faire marcher mon patch initial il faudrait déplacer "user" depuis group_vars vers roles/common/defaults. Ceci afin que tu puisse la surcharger avec ton propre inventory:

Hmmm actuellement le user est dans group_vars/all qui a une priorité plus flaible que inventory host_vars/*
J'ai plutôt l'impression que le problème est que le set_facts que tu utilise dans le patch a une priorité plus haute que inventory host_vars/*

Bref je peux me tromper, je te laisse le soin de proposer et tester un patch :)

Cool cool et j'ai du coup pu simplifier la doc.