inventory files with groups of groups causes an error about ansible-cmdb

[fboender]

Hey Jeff!

Thanks for the bug report!

Groups of groups are supported in the latest version of the Git repo. There is no package with that fix yet. Your use-case will probably still fail though, since I did not anticipate that a group "lb:children" would also count as a group called "lb". I will fix this as soon as possible.

[coryrwagner]

I'm also seeing KeyError errors when trying to parse groups of groups, with the latest code from the repo.

[n2aws]

Ferry,

Thanks for the fast reply.

I'm using the version from master, and the groups of groups work as
advertised, however it was the particular use-case I mentioned that was
failing. I should have clarified in the issue report that the bug was
being filed against master, and not the 0.3.deb version.

Thanks!

On Wed, Aug 19, 2015 at 5:26 PM, Ferry Boender [email protected]
wrote:

Hey Jeff!

Thanks for the bug report!

Groups of groups are supported in the latest version of the Git repo.
There is no package with that fix yet. Your use-case will probably still
fail though, since I did not anticipate that a group "lb:children" would
also count as a group called "lb". I will fix this as soon as possible.

—
Reply to this email directly or view it on GitHub
#16 (comment)
.

Jeff Palmer
https://PalmerIT.net

[coryrwagner]

I'm still seeing KeyError errors when trying to parse groups of groups, with the latest trunk code from the repo.

Example:

[sftp-all:children]
sftp-product1
sftp-product2

[sftp-product1:children]
host1
host3

[sftp-product1:children]
hosta
hostd

It will complain about sftp-product1.

[fboender]

Hey Cory!

Yes, I'm aware of this problem! The problem is with the fact that "sftp-product1:children" is itself also a group called "sftp-product1". I did not anticipate this. It means that children groups are basically recursive and I have to do a rewrite of the way I'm handling children groups. I'm planning on fixing this tomorrow.

[fboender]

Hi Jeff!

I've rewritten the inventory parsing logic and committed it. (It took a bit to find a clean way of doing it). Could you check to see if it's handling your inventory properly now?

Thanks!

[coryrwagner]

I can confirm that this change fixed the issues that I was having with nested groups.

[fboender]

That's great to hear Cory! Thank you for verifying the fix. I'm going to release a new version with packages soon, as this is a rather important bugfix.

[n2aws]

Ferry,

I was just able to confirm it works properly on my unmodified inventory directory too.