[BUG] Since version 2.1.0p5 modules not working anymore about ansible-collection-checkmk.general HOT 7 CLOSED

Sure this only happens since p5? I did change the path handling in p4. I also just checked the tests I added and found that I generate paths in a loop. So to get "/Blub/Baa" I first create "/Blub" and then "/Blub/Baa". The error also says that the REST API only wants to create the last folder given. Or in other words. This behaves like mkdir and not mkdir -p. So my assumption is this has happened before.

from ansible-collection-checkmk.general.

Yea I have tested it with p4 and there it worked. And yes it creates like „mkdir“ because „mkdir -p“ was not working in the beginning.. but I don’t understand what this has to do with the problem?!

from ansible-collection-checkmk.general.

The error message you have written says that the parent folder /blub/blub/bla does not exist. Since the rest endpoint does not create parent folders this message is expected. We can fix this in the rest api or let the ansible module deal with that.

from ansible-collection-checkmk.general.

Ok.. I have copied the module and fixed it locally by myself anyway. So just take it as notice and do what you think is ok..

The parent folder is already existing and its folder ID (not title!) is: ~BLUB~blub~BLA
The module does a .lower() and then the the ID of the folder is like in the output and it searches for the id ~bla~blub~bla and does not find it.

The .lower() is since march in the module so anything real strange was done by the API which did not throw this error and it was working like expected and folders got created. But since p5 ist was not working anymore and I debugged it several hours before I created this issue.

from ansible-collection-checkmk.general.

Btw.. and just fyi - I also fixed the host module which was not working anymore. The API Maybe was sending not the first / before and now it does.. so to fix the host module just remove the slash which is set by the module:

OLD:

current_folder = "/%s" % extensions.get("folder", "")

NEW:

current_folder = "%s" % extensions.get("folder", "")

from ansible-collection-checkmk.general.

@elwood218 your issue should be fixed with #147. It will be included in version 0.9.0 which will be released today.
Can you verify that the fix works for you too and close this issue if so?

from ansible-collection-checkmk.general.

Closing this due to staleness, as there is another fix in 993b4ce.

If there is anything else, please open a new issue.

from ansible-collection-checkmk.general.