AWS to Backblaze B2 provider switch #6

Closed
Ghost wants to merge 8 commits from aws-to-backblaze into master
There is no content yet.
Ghost added 3 commits 2021-12-14 07:39:34 +02:00
inex requested changes 2021-12-16 20:14:59 +02:00
inex left a comment
Owner

Нужен rclone.args=serve restic --stdio в аргументы для restic.

К тому же, как насчёт назвать сервис просто restic-backup?
Может будет неплохой идеей вызывать этот сервис и вручную из API,
получая логи из системного логгера и предотвращая случайное накладывание
ручного бекапа на автоматический.

Нужен `rclone.args=serve restic --stdio` в аргументы для restic. К тому же, как насчёт назвать сервис просто `restic-backup`? Может будет неплохой идеей вызывать этот сервис и вручную из API, получая логи из системного логгера и предотвращая случайное накладывание ручного бекапа на автоматический.
Ghost added 1 commit 2021-12-17 07:13:43 +02:00
Ghost requested review from inex 2021-12-17 07:14:26 +02:00
Ghost added 1 commit 2021-12-17 07:16:25 +02:00
Ghost added 1 commit 2021-12-17 07:17:49 +02:00
inex requested changes 2021-12-17 09:53:20 +02:00
inex left a comment
Owner

Соррян, там похоже нужны кавычки в аргументе. Копировал из питона, где они не нужны были.

ExecStart = "${pkgs.restic}/bin/restic -o rclone.args="serve restic --stdio" -r rclone:backblaze:${cfg.backblaze.bucket}:/sfbackup --verbose --json backup /var";

Ещё надо разрешить новый конфликт с files.nix: я успел добавить фикс для плеромы там.

И если ты ещё не, прогони на коде nixpkgs-fmt

Соррян, там похоже нужны кавычки в аргументе. Копировал из питона, где они не нужны были. ```nix ExecStart = "${pkgs.restic}/bin/restic -o rclone.args="serve restic --stdio" -r rclone:backblaze:${cfg.backblaze.bucket}:/sfbackup --verbose --json backup /var"; ``` Ещё надо разрешить новый конфликт с files.nix: я успел добавить фикс для плеромы там. И если ты ещё не, прогони на коде `nixpkgs-fmt`
Ghost added 2 commits 2021-12-17 18:55:25 +02:00
inex approved these changes 2021-12-20 07:00:37 +02:00
inex requested changes 2021-12-20 09:42:13 +02:00
inex left a comment
Owner

Ой, перейдя на прямой вызов команды мы забыли про ротацию. Быстро засорим b2 пользователей, если сейчас сольём.

А почему мы перешли с использования доступных в Nix параметров?

Мы вполне можем задать services.restic.backups.<name>.extraOptions = ["rclone.args='serve restic --stdio'"]

Ой, перейдя на прямой вызов команды мы забыли про ротацию. Быстро засорим b2 пользователей, если сейчас сольём. А почему мы перешли с использования доступных в Nix параметров? Мы вполне можем задать `services.restic.backups.<name>.extraOptions = ["rclone.args='serve restic --stdio'"]`

Закрыто в пользу #9

Закрыто в пользу #9
inex closed this pull request 2021-12-22 18:44:09 +02:00

Pull request closed

Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 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: SelfPrivacy/selfprivacy-nixos-config#6
There is no content yet.