Skip to content

journalctl: allow running vacuum on remote journals, too#13924

Merged
poettering merged 1 commit intosystemd:masterfrom
jktjkt:fix-journalctl-vacuum-on-remote
Nov 4, 2019
Merged

journalctl: allow running vacuum on remote journals, too#13924
poettering merged 1 commit intosystemd:masterfrom
jktjkt:fix-journalctl-vacuum-on-remote

Conversation

@jktjkt
Copy link
Copy Markdown
Contributor

@jktjkt jktjkt commented Nov 2, 2019

Right now the systemd-journal-remote service does not constrain its resource usage (I just run out of space on my 100GB partition, for example). This patch does not change that, but it at least makes it possible to run something like:

  journalctl --directory /var/log/journal/remote --rotate --vacuum-size=90G

fixes #2376

Tested on a Fedora 31: built the code, and then running something like SYSTEMD_LOG_LEVEL=debug /pth/to/my/systemd/build/journalctl --directory /var/log/journal/remote --rotate --vacuum-size=2G says that it is removing files, and I now have more free space. Thanks to @ikelos for the initial idea.

Right now the `systemd-journal-remote` service does not constrain its
resource usage (I just run out of space on my 100GB partition, for
example). This patch does not change that, but it at least makes it
possible to run something like:

  journalctl --directory /var/log/journal/remote --rotate --vacuum-size=90G

fixes systemd#2376

Co-authored-by: Mike Auty <[email protected]>
@poettering poettering added the good-to-merge/waiting-for-ci 👍 PR is good to merge, but CI hasn't passed at time of review. Please merge if you see CI has passed label Nov 4, 2019
@poettering poettering merged commit c488660 into systemd:master Nov 4, 2019
@keszybz keszybz removed the good-to-merge/waiting-for-ci 👍 PR is good to merge, but CI hasn't passed at time of review. Please merge if you see CI has passed label Nov 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

journalctl --vacuum-* --directory… does not remove anything in remote journal storage

4 participants