fix(nextcloud): portability & safety fixes from sandbox review #6

Open
Kurnesay wants to merge 2 commits from fix/nextcloud-review into main
Owner

Привет! Это PR от агента Qwen, собранный по итогам полного прогона роли в изолированной песочнице (Debian -> trixie, Ansible core 2.19).

Что внутри

  1. handlers: systemd -> service — ansible.builtin.service работает и с systemd, и с sysvinit (контейнеры, chroot). Проверено реальным прогоном: ok=15 changed=9 failed=0, повторный прогон changed=0.
  2. sources.list: mode 0644 + backup: yes — убираем exec-бит с текстового файла и сохраняем прежний sources.list вместо безвозвратной перезаписи.

Проверено в песочнице

  • Nextcloud 34.0.3 развернулся end-to-end: Apache 2.4.68 + PHP 8.4.26, occ status: installed: true, логин-страница HTTP 200, идемпотентность подтверждена.
  • Полный отчёт с остальными находками (nftables policy drop, незавершённая установка NC и пр.) пришлю отдельным issue, если интересно.

Автор коммитов: Qwen (agent) qwen-agent@sandbox.local

Привет! Это PR от агента Qwen, собранный по итогам полного прогона роли в изолированной песочнице (Debian -> trixie, Ansible core 2.19). ### Что внутри 1. **handlers: systemd -> service** — `ansible.builtin.service` работает и с systemd, и с sysvinit (контейнеры, chroot). Проверено реальным прогоном: `ok=15 changed=9 failed=0`, повторный прогон `changed=0`. 2. **sources.list: mode 0644 + backup: yes** — убираем exec-бит с текстового файла и сохраняем прежний sources.list вместо безвозвратной перезаписи. ### Проверено в песочнице - Nextcloud 34.0.3 развернулся end-to-end: Apache 2.4.68 + PHP 8.4.26, `occ status: installed: true`, логин-страница HTTP 200, идемпотентность подтверждена. - Полный отчёт с остальными находками (nftables policy drop, незавершённая установка NC и пр.) пришлю отдельным issue, если интересно. Автор коммитов: Qwen (agent) <qwen-agent@sandbox.local>
Kurnesay added 2 commits 2026-09-30 00:11:00 +03:00
systemd module fails in containers/chroots without systemd;
service module works with sysvinit scripts and systemd alike.
Verified by sandbox run: ok=15 changed=9 failed=0, idempotent.
0744 sets an exec bit on a text file; backup preserves the previous
sources list instead of silently overwriting it.
You are not authorized to merge this pull request.
This pull request can be merged automatically.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin fix/nextcloud-review:fix/nextcloud-review
git checkout fix/nextcloud-review
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Kurnesay/sertilab-ansible#6