refactor logging about crun HOT 10 CLOSED

would this be a good time to bring up logging to syslog?

from crun.

@dmolik if you need that, I can try to add it in the current mess :-)

Is it anything runc already does and we need to keep some CLI compatibility? If not, would something like --log=syslog:IDENTITY be good for you?

from crun.

@dmolik I've opened https://github.com/giuseppe/crun/pull/28

It is still work in progress, but you can start using syslog, like crun --log=syslog: and journald as crun --log=journald:. If not specified, they both use the container name as the identifier, otherwise they use whatever is specified after :. So for example if you want to override an identifier in the logging, you can do: crun --log=syslog:FOO.

What do you think?

from crun.

Setting the facility name would be cool.

from crun.

I've implemented that functionality in the PR.

You can use --log=syslog:IDENTITY to override the identifier, otherwise it defaults to the container name. Similarly for --log=journald:

from crun.

is that enough for you? :-) Can I merge the PR?

from crun.

LGTM. Thanks! Sorry for the slow response.

from crun.

@giuseppe Should this issue be closed?

from crun.

@giuseppe Can we close?

from crun.

yes, let's close it as there is no work going on and it is not really a specific issue

from crun.