Revert the ansible collections path to ansible/collections so we don't inadvertently break any existing checkouts.

Direct ansible-lint to use .ansible/collections so downloads are excluded from linting by our .ansible-lint.yml

Author: maxstack

.github/workflows/lint.yml

@@ -1,5 +1,5 @@
----
-name: Lint
+	---
+	name: Lint
 
 on:  # yamllint disable-line rule:truthy
   workflow_call:
@@ -30,6 +30,9 @@ jobs:
 
       - name: Run ansible-lint
         uses: ansible/[email protected]
+        if: always()
+        env:
+          ANSIBLE_COLLECTIONS_PATH: .ansible/collections
 
       - name: Load super-linter configuration
         # Use grep inverse matching to exclude eventual comments in the .env file

README.md

@@ -172,5 +172,9 @@ docker run --rm \
 ```
 
 ```shell
-ansible-lint -c .ansible-lint.yml ansible/
+ANSIBLE_COLLECTIONS_PATH=.ansible/collections \
+    ansible-lint -c .ansible-lint.yml ansible/
 ```
+
+Specifying `ANSIBLE_COLLECTIONS_PATH` ensures `ansible-lint` downloads collections and roles under the `.ansible` directory, separating them from our own roles under the `ansible` directory.
+We exclude these downloaded files from linting by listing `.ansible` under `exclude_paths` in `.ansible-lint.yml`.

ansible.cfg

@@ -5,7 +5,7 @@ gathering = smart
 forks = 30
 host_key_checking = False
 remote_tmp = /tmp
-collections_path = .ansible/collections
+collections_path = ansible/collections
 roles_path = ansible/roles
 filter_plugins = ansible/filter_plugins
 callbacks_enabled = ansible.posix.profile_tasks