-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
journald logging support #9
Comments
@orzFly sorry for my miss, I miss this issue. Double check that all of our supported OS could compile with So it is now newly added, see alvistack/containers-podman@d9f05cc#diff-889c5495b68d2a0c0d1f238818716a71939918499efbc0fc6d2994319624fc67R5-R23 After OBS complete package build you could fetch it with asked feature. Thank you for reporting. |
@orzFly sorry again, the change doesn't benefit as expected, it break all of my docker image packaging by GitLab Runner + Libvirt + Vagrant + Podman. Looks like this could only benefit if (see containers/conmon#67):
So I need to revert the changes. Please kindly understand... |
Never mind. Thank you again for this. Sorry to hear the breakings! I regret to not saying before, that podman defaults to journald logging if its compiled with. So there should be some changes of the default config for non-breaking upgrades. I was planning to upgrade my podmans this weekend and try your new version, but it seemed to be late. I do run podman as root since I use podman as a replacement for Docker. Not sure if it's possible to have crun and conmon all compiled with systemd-journald support in the future. |
Seems kubic repo also try to avoid this support from my point of view. Moreover people always have more wish to replace Docker with Podman, especially in rootless mode, so root-only default compile and setup seems not a good start with good price and performance ratio :-( |
@bverhagen from https://build.opensuse.org/project/show/home:alvistack#comment-1715355
I had try that before, but no luck and not such useful: #9 |
needs build tag
systemd
https://github.com/alvistack/containers-podman/blob/alvistack/v4.1.1/Makefile#L314
https://github.com/containers/podman/blob/10569c988f8ad3bfa796e52c61d7f4ef5266c193/libpod/container_log_linux.go#L2
https://github.com/containers/podman/blob/10569c988f8ad3bfa796e52c61d7f4ef5266c193/libpod/container_log_unsupported.go
The text was updated successfully, but these errors were encountered: