MichaIng / DietPi

Lightweight justice for your single-board computer!
https://dietpi.com/
GNU General Public License v2.0
4.81k stars 494 forks source link

Cron | journalctl shows CRON job gets repeatedly called once every minute. #1654

Closed Fourdee closed 6 years ago

Fourdee commented 6 years ago

http://dietpi.com/phpbb/viewtopic.php?f=11&t=2982#p11505

MichaIng commented 6 years ago

https://github.com/ayufan-rock64/linux-build/blob/master/package/root/etc/cron.d/make_nas_processes_faster

Part of Rock64 image, ugly implementation, spammy like this, not needed for users that don't use these NAS services, should be enough to call the command ONCE after boot, or am I wrong?

@Fourdee Do we wan't to remove that file on DietPi-Prep? Or mv /etc/cron.d/make_nas_processes_faster /etc/cron.d/.make_nas_processes_faster and activate again on samba/nfs/ftp installation, if this is really the best way to achieve process priorities?

MichaIng commented 6 years ago

Okay also present on NanoPi Neo, we have to check where this comes from, base image or some APT package update and check if this is really helpful for file severs or not.. Pine64 possibly affected as well. Often shows up in combination with openmediavault...

Fourdee commented 6 years ago

@MichaIng

Do we wan't to remove that file on DietPi-Prep? Or mv /etc/cron.d/make_nas_processes_faster

I believe, we should remove the file on all images, let dietpi-process_tool (and the user) handle it.

Okay also present on NanoPi Neo

Interesting. cant see it in openmediavault_0.2.3_all.deb, but there are multiple packages i've not checked.

In the preimage used (nanopi-neo_debian-nas-jessie_4.14.0_20171218) OMV is installed, so it must be a OMV addition.

Either way, lets remove the file from all images PREP + patch?

MichaIng commented 6 years ago

@Fourdee

Either way, lets remove the file from all images PREP + patch?

Jep, I think so, PR is up 😃: https://github.com/Fourdee/DietPi/pull/1665

Fourdee commented 6 years ago

@MichaIng

Many thanks for the fix 👍 . I'll mark this as closed PR should resolve: https://github.com/Fourdee/DietPi/pull/1665