Configure maximum size and timespan of journald logs #594

Let's put those values in default attributes, so we can override them per role or environment? I think we could also define a `service` without having to create a custom service, so we can just notify `:restart, service[systemd-journald]`.

👍 1

raucao added

feature

and removed

bug

labels 2025-05-21 10:53:51 +00:00

greg added 1 commit 2025-05-21 11:41:38 +00:00

Use the systemd unit instead of an execute resource 7dc4f674a0

Also extract the attributes so it is possible to override them.

greg commented

2025-05-21 11:43:47 +00:00

Let's put those values in default attributes, so we can override them per role or environment?

I think we could also define a service without having to create a custom service, so we can just notify :restart, service[systemd-journald].

It does look better with these changes

> Let's put those values in default attributes, so we can override them per role or environment? > > I think we could also define a `service` without having to create a custom service, so we can just notify `:restart, service[systemd-journald]`. It does look better with these changes

🎉 1

raucao approved these changes 2025-05-21 12:11:15 +00:00

raucao left a comment

Confirmed to work by testing on strfry-1. 👍

Confirmed to work by testing on `strfry-1`. 👍

raucao commented

2025-05-21 12:12:42 +00:00

I think we don't need to run this everywhere immediately. For most nodes, it should be enough as a drive-by improvement next time they're converged. And the ones that had issues with disk space in the past already have it configured manually.

👍 1

raucao merged commit 98be234a4f into master

2025-05-21 12:12:58 +00:00

raucao deleted branch feature/506-journald_logs_config

2025-05-21 12:12:59 +00:00

raucao referenced this issue from a commit

2025-05-21 12:13:00 +00:00

Merge pull request 'Configure maximum size and timespan of journald logs' (#594) from feature/506-journald_logs_config into master

raucao moved this to Done in 2025/Q2 on 2025-05-22 12:43:02 +00:00

Sign in to join this conversation.