GitLab

[__user] fix user delete on freebsd

See merge request ungleich-public/cdist!882

Signed-off-by: Jaak Ristioja <jaak@ristioja.ee>

[__group] fix --gid on freebsd

See merge request ungleich-public/cdist!881

[__motd] debian|ubuntu|devuan use /etc/motd

See merge request ungleich-public/cdist!880

[__ssh_authorized_keys] Fix bug where --option was not multiple

See merge request ungleich-public/cdist!878

This went against both documentation and intent.

[__ssh_authorizedkey{,s}] Improve documentation.

See merge request ungleich-public/cdist!877

[explorer/machine_type] simplify linux add openbsd

See merge request ungleich-public/cdist!876

[explorer/machine_type] Add support for FreeBSD.

See merge request ungleich-public/cdist!875

Fix some type manpages

See merge request ungleich-public/cdist!874

By abstracting away vendor-dependent pattern matching for the linux code, we can
re-use that and be reasonably sure about OpenBSD machines being virtualised when
we can identify the undelrying virtualisation technology.
It remains to be solved how to tell if an OpenBSD machine is physical; in that
case previous cdist behaviour ("unknown") remains.

For NetBSD something similar to OpenBSD could be done, with different sysctls:
hw.machine and hw.model wary of adding those without testing though, so for
NetBSD previous cdist behaviour ("unknown") remains.
https://netbsd.gw.com/cgi-bin/man-cgi?sysctl+7.i386+NetBSD-9.0

More research is needed for {Net,Open}BSD support.

Indentation is left as-is for the linux code as I intend to simplify it in a
future MR, this way the diff is minimal.

Because at least the --mode default value may not be expected, the
manual lists the "default values". As they are not explicitly set in the
`__file` type, it is a bit more difficult, but should resolve to
following:
  - the mode comes from the umask set in the cdist code to protect file
    creation and uses the strongest umask possible.
  - the owner and group comes from the ssh user, which should always be
    the root user. (I think this can be swaped, too, but who will?)

At the owner and group parameter, it could be replaced with something
like "the ssh user and group", which would be more correct, but less
understandable.

This should resolve some misunderstanding when leave a time-related
parameter with it's default value.

[logging] Mute warning on return_output=True when running scripts.

Closes #806

See merge request ungleich-public/cdist!872

This fixes #806 which contains more information about the issue.

The TL;DR: this warning is not being useful and hinders debugging types because
it creates an innecessary line for each explorer.

An alternative proposal was #807 but was abandoned in favour of just dropping
the warning.

[explorers] Improve *BSD support.

See merge request ungleich-public/cdist!869

[docs] Improve cdist.cfg.skeleton

See merge request ungleich-public/cdist!868

update README

See merge request ungleich-public/cdist!870

cpu_cores and memory did lacked support for other BSDs.