- Add ansible-lint config fro remove some warnings

- Add playbook to demonstrate how to use the role
- Update default, template, tests, tests_tasks to remove dev_ prefix variables

Author: LucasJanin

.ansible-lint

@@ -0,0 +1,3 @@
+# .ansible-lint
+skip_list:
+  - run-once  # Ignorer les avertissements liés à run_once

defaults/main.yml

@@ -1,16 +1,9 @@
 ---
 # defaults file for ansible-role-hosts-aliases
 
-# Default empty lists for host information
-dev_hostnames: []                           # List of hostnames to create aliases for
-dev_ips: []                                 # List of IPs corresponding to hostnames
-dev_alias: []                               # List of short aliases for each host
-dev_vspaths: []                             # List of paths to open in VSCode for each host
-dev_ansible_users: []                       # List of SSH users for each host
-
 # File paths
 bash: ".bashrc"                             # Bash profile file to modify
-bash_hosts_aliases: ".bash_hosts_aliases"   # File to store the aliases
+bash_hosts_aliases: ".bashrc_hosts_aliases" # File to store the aliases
 
 # Editor configuration
 editor_path: "code"                         # Path to the VSCode executable

meta/main.yml

@@ -13,6 +13,14 @@ galaxy_info:
   license: MIT
   min_ansible_version: "2.12"
 
+  platforms:
+    - name: GenericLinux
+      versions:
+        - all
+    - name: "MacOSX"
+      versions:
+        - all
+
   galaxy_tags:
     - development
     - system

playbook.yml

@@ -0,0 +1,33 @@
+# File: playbook.yml
+# Description: Playbook to setup hosts aliases
+# Author: Lucas Janin
+# Version: 1.0
+---
+- name: Auto setup hosts on Unifi, PiHole and fastroom
+  hosts: "{{ target_hosts | default('all') }}"
+  gather_facts: true
+  become: true
+  become_method: ansible.builtin.sudo
+  strategy: linear
+
+  tasks:
+    - name: Display discovered host information individually
+      ansible.builtin.debug:
+        msg:
+          - "Host: {{ item }}"
+          - "  IP           : {{ hostvars[item].ansible_host | default('') }}"
+          - "  Ansible User : {{ hostvars[item].ansible_user | default('') }}"
+          - "  Alias        : {{ hostvars[item].alias | default('') }}"
+          - "  VS Path      : {{ hostvars[item].vspath | default('') }}"
+      loop: "{{ ansible_play_hosts_all }}"
+      run_once: true
+
+    # Include hosts aliases role to ansible
+    - name: Include hosts aliases role to ansible
+      ansible.builtin.include_role:
+        name: ansible-role-hosts-aliases
+      vars:
+        user: "ansible"
+        host: "ansible"
+        home: "/home/ansible/"
+      run_once: true

templates/hosts_aliases.j2

@@ -1,11 +1,11 @@
 # {{ bash_hosts_aliases }}
 # {{ comment }}
 
-{% for hostname, ip, alias, vspath, ssh_user in dev_hostnames | zip(dev_ips, dev_alias, dev_vspaths, dev_ansible_ssh_users) %}
-# {{ hostname }} ----- {{ ip }}
-alias ssh{{ alias }}='ssh {{ ssh_user }}@{{ hostname }}'
+{% for host in ansible_play_hosts_all %}
+# {{ host }} ----- {{ hostvars[host].ansible_host | default('') }}
+alias ssh{{ hostvars[host].alias | default(host) }}='ssh {{ hostvars[host].ansible_user | default("ansible") }}@{{ host }}'
 {% if create_editor_aliases | default(true) | bool %}
-alias vs{{ alias }}='{{ editor_path }} --folder-uri "vscode-remote://ssh-remote+{{ ssh_user }}@{{ hostname }}{{ vspath }}"'
+alias vs{{ hostvars[host].alias | default(host) }}='{{ editor_path }} --folder-uri "vscode-remote://ssh-remote+{{ hostvars[host].ansible_user | default("ansible") }}@{{ host }}{{ hostvars[host].vspath | default("") }}"'
 
 {% else %}
 

tests/test.yml

@@ -1,15 +1,8 @@
 ---
 - name: Test hosts aliases role
   hosts: localhost
-  gather_facts: false
+  gather_facts: true
   vars:
-    # Use actual hosts from inventory
-    dev_hostnames: "{{ groups['all'] | difference(['localhost']) }}"
-    dev_ips: []
-    dev_alias: []
-    dev_vspaths: []
-    dev_ansible_ssh_users: []
-
     # Management host information (using /tmp for testing)
     user: "{{ ansible_user_id }}"
     host: "localhost"
@@ -28,26 +21,17 @@
         state: touch
         mode: '0644'
 
-    # Extract IPs from inventory
-    - name: Set IPs from inventory
-      ansible.builtin.set_fact:
-        dev_ips: "{{ dev_ips + [hostvars[item].ansible_host | default('127.0.0.1')] }}"
-      loop: "{{ dev_hostnames }}"
-
-    - name: Set aliases from inventory
-      ansible.builtin.set_fact:
-        dev_alias: "{{ dev_alias + [hostvars[item].alias | default(item)] }}"
-      loop: "{{ dev_hostnames }}"
-
-    - name: Set vspaths from inventory
-      ansible.builtin.set_fact:
-        dev_vspaths: "{{ dev_vspaths + [hostvars[item].vspath | default('/home/' + hostvars[item].ansible_user + '/')] }}"
-      loop: "{{ dev_hostnames }}"
-
-    - name: Set SSH users from inventory
-      ansible.builtin.set_fact:
-        dev_ansible_ssh_users: "{{ dev_ansible_ssh_users + [hostvars[item].ansible_user | default('ansible')] }}"
-      loop: "{{ dev_hostnames }}"
+    # Display discovered host information for testing
+    - name: Display discovered host information
+      ansible.builtin.debug:
+        msg:
+          - "Host: {{ item }}"
+          - "  IP           : {{ hostvars[item].ansible_host | default('') }}"
+          - "  Ansible User : {{ hostvars[item].ansible_user | default('') }}"
+          - "  Alias        : {{ hostvars[item].alias | default('') }}"
+          - "  VS Path      : {{ hostvars[item].vspath | default('') }}"
+      loop: "{{ groups['all'] | difference(['localhost']) }}"
+      run_once: true
 
   tasks:
     - name: Include test tasks

tests/test_tasks.yml

@@ -33,4 +33,5 @@
     src: "../templates/hosts_aliases.j2"  # noqa: no-relative-paths
     dest: "{{ home }}{{ bash_hosts_aliases }}"
     mode: '0644'
+    owner: "{{ user }}"
   register: aliases_created