Skip to content

Fixes/4851#4965

Draft
Nitin75408 wants to merge 3 commits intoansible:mainfrom
Nitin75408:Fixes/4851
Draft

Fixes/4851#4965
Nitin75408 wants to merge 3 commits intoansible:mainfrom
Nitin75408:Fixes/4851

Conversation

@Nitin75408
Copy link

Fix collections_paths from ansible.cfg being ignored (Issue #4851)
Problem
ansible-lint ignored the collections_paths setting from ansible.cfg, even though other settings were respected. The ansible-compat Runtime class would override ANSIBLE_COLLECTIONS_PATH with hardcoded defaults, ignoring user configuration.

Solution
Read collections_paths from ansible.cfg and set ANSIBLE_COLLECTIONS_PATH environment variable before Runtime initialization, ensuring proper configuration precedence.

Changes
src/ansiblelint/config.py - Added ansible_collections_paths field to Options
src/ansiblelint/ansible_config.py (NEW) - Module to read and parse ansible.cfg collections_paths
src/ansiblelint/app.py - Load ansible.cfg and set ENV VAR before Runtime creation
test/test_ansible_cfg_collections.py (NEW) - 11 test cases validating the fixF

Fixes#4851

@github-actions
Copy link
Contributor

Label error. Requires exactly 1 of: breaking, build, chore, ci, docs, feat, fix, perf, refactor, style, test. Found: . Follow https://www.conventionalcommits.org to get auto-labeling to work correctly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant