@garbas said in 2018:

I think having it false by default is ok, we can have a different discussion about it if we want to enable this or not and when.

Source: #36772 (comment)

So here we are.

For me, it's OK to have it just enabled when needed (e.g. for netdata). I havn't used it outside that.

But since Ubuntu, Fedora and Arch Linux have it enabled by default, we could enable it too by default for everyone. Or just desktop users.

Source: https://docs.netdata.cloud/collectors/cgroups.plugin/#monitoring-systemd-services

And since systemd now enables some options by default, we should have at least enabled them. And why not all to provide the full feature to our users.

Source: #29095 (comment)

Like said earlier, i had no performance issues, but i didn't measure the difference.

@GrahamcOfBorg test systemd

@flokli looks like you have added this test section as well?

machine# sysctl: cannot stat /proc/sys/net/core/default_qdisc: No such file or directory
machine: exit status 1```

@mmahut I rebased on latest master. The diff in the tests should look clean now.

There's still another failure in the systemd test:

error: command `dumpe2fs /dev/vdb | grep -q "^Last mount time: *n/a"' unexpectedly succeeded at /nix/store/39fb7m572abixj1qi7ddzx5lp3k64lw2-nixos-test-driver/lib/perl5/site_perl/Mach
ine.pm line 376, <__ANONIO__> line 194.                                                                                                                                               
(12.12 seconds)  

But that's unrelated to this PR - and the cgroup accounting specific tests pass.

@flokli thank you, have a conflict in the release-notes, do you mind to rebase, please?

@mmahut done - fixed the indentation of the previous changelog entry too.

@GrahamcOfBorg test systemd

@mmahut the systemd test is still broken in master (as written in the comment above), but the newly added subtest does work.

If we're fine with this PR, we should merge it.

I opened #67555 to track the systemd vm test breakage on master.