@fnux, sorry for the n00b question: how do I update a PR from my fork? I've pushed some changes, but I don't see them here.
There are no guaranties that $__remote_copy supports --delete (e.g. scp).
That is true. Oh well, if it's not wanted, we can close this.
I find it very useful to push a bunch of data to a…
IMO, forcing something other than the default is not my style, but if that is standard cdist style, so be it.
IIUC, that would mean that cdist team be responsible for shipping working tested config for fail2ban.
Do you think that is out of scope? Have you seen how many jails are in the default…
Is that typically what we do?
Or do we expect the user to use __start_on_boot?
Maybe I should rather fail when there is no fail2ban package installed. Then this type would be only for…
I'll need to make a testing set up for this. It's most likely that other OSes can use the same code, but I just haven't tested it.
There already is a
__mail_aliastype in cdist-contrib: https://code.ungleich.ch/ungleich-public/cdist-contrib/src/branch/master/type/__mail_alias
Ah yes. I remember now wondering why there…
FIXME: What does cdist use for unit testing?
We have unit tests for cdist's python code, but no integration tests for the types.
OK. I don't have anything set up for this yet, but if…
I would rather see this type in cdist-contrib as in core cdist for now. You would also need to move the logic from gencode-remote to the manifest: that's still quite a bit of work.
That makes…
