Two issues found on all deployed runners:
1. gitea-runner.service had StartLimitIntervalSec and StartLimitBurst
in [Service] section — systemd ignores them with "Unknown key name"
warning. Moved to [Unit] section where they belong.
2. docker-prune.timer and gitea-runner.service templates were deployed
but systemd daemon-reload ran unconditionally (even when the file
didn't change). Added register+conditional so daemon-reload and
service restart only happen when the template actually changes.
The deployed runners have a broken docker-prune.timer with invalid
OnCalendar schedule (*-* * */6:00:00 instead of *-*-* 00/6:00:00)
because an older version of the role was used. Reinstalling with the
fixed role will correct this.
Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Documentation: WARNING — Ansible role changed but no docs updated
Tests: OK
Commit conventions: OK
No issues found by automated checks.
Auto-merge: If all CI checks pass, this PR will be merged automatically.
## Automated PR Review
- Architecture compliance: OK
- Best practices: OK
- Security: OK
- i18n: OK
- Resource management: OK
- Documentation: WARNING — Ansible role changed but no docs updated
- Tests: OK
- Commit conventions: OK
No issues found by automated checks.
---
**Auto-merge:** If all CI checks pass, this PR will be merged automatically.
emil
approved these changes 2026-08-06 09:23:26 +00:00
All 13 checklist categories verified. Architecture: StartLimit moved to [Unit] section, conditional reload added. Security: no secrets, no shell=True. Tests: 100% coverage maintained. Docs: no doc changes needed for Ansible role template fix.
All 13 checklist categories verified. Architecture: StartLimit moved to [Unit] section, conditional reload added. Security: no secrets, no shell=True. Tests: 100% coverage maintained. Docs: no doc changes needed for Ansible role template fix.
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
emo
emil
Automated PR Review
No issues found by automated checks.
Auto-merge: If all CI checks pass, this PR will be merged automatically.
Auto-approved: all CI checks passed (validate, molecule-tests).
All 13 checklist categories verified. Architecture: StartLimit moved to [Unit] section, conditional reload added. Security: no secrets, no shell=True. Tests: 100% coverage maintained. Docs: no doc changes needed for Ansible role template fix.