GithubHelp home page GithubHelp logo

Comments (7)

fboender avatar fboender commented on July 4, 2024

Hey Dale,

I have confirmed this bug. I'll have to give some thought to a good solution. Thank you for reporting it and I'll try to have a solution as soon as possible.

Regards

Ferry

from ansible-cmdb.

fboender avatar fboender commented on July 4, 2024

Hey Dale,

I can't find in the Ansible documentation where it says this is allowed in hosts files. It seems fine for playbooks and variable files (http://docs.ansible.com/ansible/intro_inventory.html#splitting-out-host-and-group-specific-data), but it doesn't seem to be allowed officially in hosts files?

from ansible-cmdb.

fboender avatar fboender commented on July 4, 2024

It seems this is explicitly not allowed / discouraged in the manual:

If you need to store lists or hash data, or prefer to keep host and group specific variables separate from the inventory file, see the next section.

The next section is http://docs.ansible.com/ansible/intro_inventory.html#splitting-out-host-and-group-specific-data

Perhaps it works right now in Ansible, but I am apprehensive about adding support for undocumented features. Especially since it would require rewriting a large part of the hosts file parsing code ;-)

I suggest you split the variables out in separate files as suggested by the manual.

from ansible-cmdb.

dlbewley avatar dlbewley commented on July 4, 2024

I hear what you are saying, but FWIW This format comes from https://github.com/openshift/openshift-ansible which is from RH who are the new Ansible benefactors.

from ansible-cmdb.

fboender avatar fboender commented on July 4, 2024

You're right, and it does seem to work. I'll re-open this issue and take another look to see if ansible-cmdb can support this.

from ansible-cmdb.

fboender avatar fboender commented on July 4, 2024

Hey Dale!

I've committed a fix that implements support for this. I had to make some guesses about how it's supposed to work, since it's not really documented.

Thank you for reporting the bug!

Regards

from ansible-cmdb.

dlbewley avatar dlbewley commented on July 4, 2024

Thanks for accommodating it. Cheers!

from ansible-cmdb.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.