datakollektivet/systemer
7
0
Fork 0
Code Issues 1 Pull requests 4 Projects Releases Packages Wiki Activity Actions

Opprette systemd timers for å sende e-poster osv #8

Open
gingermusketeer wants to merge 5 commits from setup-timers into main
pull from: setup-timers
merge into: datakollektivet:main
Conversation 4 Commits 5 Files changed 6 +52 -2
gingermusketeer commented 2026-01-16 10:29:05 +00:00
Owner
Copy link

Dokumentasjonen finnes på loomio-deploy.

Dokumentasjonen finnes på [loomio-deploy](https://github.com/loomio/loomio-deploy?tab=readme-ov-file#install-crontab).
gingermusketeer reviewed 2026-01-16 10:48:35 +00:00
bin/lint
@ -0,0 +1,3 @@
#!/bin/bash
set -e
uv run ansible-lint **/playbooks/*.yaml --exclude .ansible "$@"
gingermusketeer commented 2026-01-16 10:48:35 +00:00
Author
Owner
Copy link

Lettere å kjøre dette via et skript enn å husker hvordan man gjør det!

Lettere å kjøre dette via et skript enn å husker hvordan man gjør det!
papiris commented 2026-01-21 20:21:05 +00:00
Owner
Copy link

Kan være greit å sette RandomizedDelaySec=10m, FixedRandomDelay=10m, eller RandomizedOffsetSec=10m for å unngå potensielt høy arbeidslast på et lite system. Om fordeling av systemlast

Hvis oppgaven kan ta over 1 time, bør vi sette DeferReactivation=true, og muligens øke TimeOutStartSec. DeferReactivation og TimeOutStartSec ved Type=Oneshot

Kan være greit å sette OOMPolicy=stop i service-fila, slik at vedlikeholdsoppgaven avsluttes ved for høy minnebruk på systemet, heller enn at systemet prioriterer å kjøre vedlikeholdsoppgaven over selve Loomio-instansen. OOMPolicy

Kan være greit å sette `RandomizedDelaySec=10m`, `FixedRandomDelay=10m`, eller `RandomizedOffsetSec=10m` for å unngå potensielt høy arbeidslast på et lite system. [Om fordeling av systemlast](https://www.freedesktop.org/software/systemd/man/latest/systemd.timer.html#AccuracySec=) Hvis oppgaven kan ta over 1 time, bør vi sette `DeferReactivation=true`, og muligens øke `TimeOutStartSec`. [DeferReactivation](https://www.freedesktop.org/software/systemd/man/latest/systemd.timer.html#DeferReactivation=) og [TimeOutStartSec ved Type=Oneshot](https://www.freedesktop.org/software/systemd/man/latest/systemd.service.html#Type=) Kan være greit å sette `OOMPolicy=stop` i service-fila, slik at vedlikeholdsoppgaven avsluttes ved for høy minnebruk på systemet, heller enn at systemet prioriterer å kjøre vedlikeholdsoppgaven over selve Loomio-instansen. [OOMPolicy](https://www.freedesktop.org/software/systemd/man/latest/systemd.service.html#OOMPolicy=)
gingermusketeer commented 2026-01-23 14:22:53 +00:00
Author
Owner
Copy link

@papiris Takk. Jeg har lagt til noen av endringene du foreslo.

Oppgaven bør ikke ta mer enn 10-20 sekunder, tenker jeg. Den starter noen jobber og sletter ting fra databasen.

@papiris Takk. Jeg har lagt til noen av endringene du foreslo. Oppgaven bør ikke ta mer enn 10-20 sekunder, tenker jeg. Den [starter noen jobber og sletter ting fra databasen](https://github.com/loomio/loomio/blob/master/lib/tasks/loomio.rake#L222).
stigo reviewed 2026-01-23 14:28:27 +00:00
stigo left a comment
Owner
Copy link

Ville vurdert å prefikse units med "loomio-" eller noe slikt, slik at det blir mer obvious at det er den applikasjonen den interacter med.

F.eks ved å endre filnavn til:

  • loomio-hourly-task.service
  • loomio-hourly-task.timer

Dersom man har flere relaterte units kan man da enkelt liste opp units via prefix, og f.eks se journal for de med:

$ journalctl -eu 'loomio-*'

$ systemctl list-timers 'loomio-*'
Ville vurdert å prefikse units med "loomio-" eller noe slikt, slik at det blir mer obvious at det er den applikasjonen den interacter med. F.eks ved å endre filnavn til: - loomio-hourly-task.service - loomio-hourly-task.timer Dersom man har flere relaterte units kan man da enkelt liste opp units via prefix, og f.eks se journal for de med: ``` $ journalctl -eu 'loomio-*' $ systemctl list-timers 'loomio-*' ```
❤️ 1
alexanrf scheduled this pull request to auto merge when all checks succeed 2026-02-07 15:25:36 +00:00
This pull request doesn't have enough approvals yet. 0 of 1 approvals granted.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin setup-timers:setup-timers
git switch setup-timers
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
datakollektivet/systemer!8
No description provided.