Looks good - I also ran a quick test that worked as expected. I'll probably roll this out to real-world infrastructure in the coming weeks. Thanks!
This type is quite neat - I'm merging it and will support it as needed. @sparrowhawk changed job and I don't believe he is still using cdist.
Why not submit this to the main cdist repository? It would make more sense to have package managers up there.
Go ahead, the pull request is already 2 years old. I'd also like to see a better comment than 'Fix IPv6' on line 108 - there's no bug, we just (re)format IPv6 addresses as needed.
FIXME: What does cdist use for unit testing?
We have unit tests for cdist's python code, but no integration tests for the types.
FIXME: Output to stderr is no longer visible by default.…
There are a few edge cases (do we want to replace the remote content, to merge the local and remote directories, apply/conserve specific permissions, etc.) that will be hard to handle in this…