Configure maximum size and timespan of journald logs #594

2025-05-21T09:38:00Z

greg commented

2025-05-21 09:38:00 +00:00

This has been run of mastodon-3, still need to do it on the rest of the nodes

Ready to review

Closes #506

This has been run of mastodon-3, still need to do it on the rest of the nodes Ready to review Closes #506

greg added the kredits-1 ops bug labels 2025-05-21 09:38:00 +00:00

greg added 2 commits 2025-05-21 09:38:00 +00:00

Configure maximum size and timespan of journald logs 3e2ee30334

Closes #506

Update mastodon-3 node file after Chef run 49b636305e

greg requested review from raucao 2025-05-21 09:38:00 +00:00

greg added this to the 2025 project 2025-05-21 09:38:00 +00:00

raucao commented

2025-05-21 10:50:37 +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].

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 on 2025-05-22 12:43:02 +00:00

Sign in to join this conversation.