10000 AnsibleInventory does not support host_vars or group_vars with an extension · Issue #213 · nornir-automation/nornir · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

AnsibleInventory does not support host_vars or group_vars with an extension #213

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
dmfigol opened this issue Jul 29, 2018 · 1 comment
Closed

AnsibleInventory does not support host_vars or group_vars with an extension #213

dmfigol opened this issue Jul 29, 2018 · 1 comment

Comments

@dmfigol
Copy link
Collaborator
dmfigol commented Jul 29, 2018 •
edited
Loading

Currently AnsibleInventory expects host_vars or groups_vars files only without an extension:

    @staticmethod
    def read_vars_file(element: str, path: str, is_host: bool = True) -> VarsDict:
        subdir = "hosts_vars" if is_host else "group_vars"
        filepath = os.path.join(path, subdir, element)
        if not os.path.exists(filepath):

Even with present files host_vars/*.yml and group_vars/*.yml logs show an error:

2018-07-30 01:47:27,695 -       nornir -    DEBUG - read_vars_file() - AnsibleInventory: var file doesn't exist: host_vars/matrix-2
2018-07-30 01:47:27,695 -       nornir -    DEBUG - read_vars_file() - AnsibleInventory: var file doesn't exist: host_vars/matrix-3
2018-07-30 01:47:27,695 -       nornir -    DEBUG - read_vars_file() - AnsibleInventory: var file doesn't exist: host_vars/matrix-3
2018-07-30 01:47:27,695 -       nornir -    DEBUG - read_vars_file() - AnsibleInventory: var file doesn't exist: host_vars/matrix-4
2018-07-30 01:47:27,695 -       nornir -    DEBUG - read_vars_file() - AnsibleInventory: var file doesn't exist: host_vars/matrix-4

Working on a fix, will submit PR.
dmfigol added a commit to dmfigol/nornir that referenced this issue Jul 30, 2018
@dmfigol
Add AnsibleInventory support for host_vars and group_vars with *.yml …
a1d8494 
…and *.yaml extensions

Fix nornir-automation#213
@dmfigol dmfigol mentioned this issue Jul 30, 2018
Merged
dbarrosop pushed a commit that referenced this issue Jul 30, 2018
@dmfigol @dbarrosop
Add AnsibleInventory support for host_vars and group_vars with *.yml …
2084658 
…and *.yaml extensions (#214)

Fix #213
@dbarrosop
Copy link
Contributor

Fixed via #214

omershtivi pushed a commit to omershtivi/nornir that referenced this issue Dec 6, 2018
@dmfigol @omershtivi1
Add AnsibleInventory support for host_vars and group_vars with *.yml …
3ee1f84 
…and *.yaml extensions (nornir-automation#214)

Fix nornir-automation#213
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dmfigol @dbarrosop
0