Add systemd timers to run the fixes periodically #3

There's no success in controlling the systemd start/stop for user timers, though they work perfectly fine on the user level of the systemd (Thanks a lot for the hint @rokosun !)

To switch to the timers though we need to modify the fix scripts, since they store a state in the loop and to remove the loop, we need to store previous values somewhere else (for example, in the /tmp files) for them to survive between the timer ticks.

There's no success in controlling the systemd start/stop for user timers, though they work perfectly fine on the user level of the systemd (Thanks a lot for the hint @rokosun !) To switch to the timers though we need to modify the fix scripts, since they store a state in the loop and to remove the loop, we need to store previous values somewhere else (for example, in the /tmp files) for them to survive between the timer ticks.

eSh force-pushed systemd-timers from a9a7ce9bad to 90f98c64cc

2022-09-18 16:22:42 +00:00

Compare

rokosun commented

2022-09-18 18:17:00 +00:00

To switch to the timers though we need to modify the fix scripts, since they store a state in the loop and to remove the loop, we need to store previous values somewhere else (for example, in the /tmp files) for them to survive between the timer ticks.

Yes, but do we need to use timers for the scripts that store values between loops? For this purpose the while loop we currently use would be enough, this also means that the entire script don't have to be rerun, only the part inside the while loop needs to be rerun which makes it more efficient.

> To switch to the timers though we need to modify the fix scripts, since they store a state in the loop and to remove the loop, we need to store previous values somewhere else (for example, in the /tmp files) for them to survive between the timer ticks. Yes, but do we need to use timers for the scripts that store values between loops? For this purpose the while loop we currently use would be enough, this also means that the entire script don't have to be rerun, only the part inside the while loop needs to be rerun which makes it more efficient.

eSh commented

2022-09-18 21:53:36 +00:00

Yes, but do we need to use timers for the scripts that store values between loops? For this purpose the while loop we currently use would be enough, this also means that the entire script don't have to be rerun, only the part inside the while loop needs to be rerun which makes it more efficient.

True, especially after an introduction of the fix you've made for subscribing on the settings changes, the timers have no relevancy anymore. I will close this PR after the new approaches are tested enough 😉

> Yes, but do we need to use timers for the scripts that store values between loops? For this purpose the while loop we currently use would be enough, this also means that the entire script don't have to be rerun, only the part inside the while loop needs to be rerun which makes it more efficient. True, especially after an introduction of the fix you've made for subscribing on the settings changes, the timers have no relevancy anymore. I will close this PR after the new approaches are tested enough 😉

👍 1

eSh closed this pull request

2022-09-18 22:32:54 +00:00

eSh deleted branch systemd-timers

2022-09-18 22:33:01 +00:00