From: paul via Guix-patches via <guix-patches@gnu.org>
To: 71639@debbugs.gnu.org
Cc: Richard Sent <richard@freakingpenguin.com>
Subject: [bug#71639] [PATCHv2 0/5] Improve on restic-backup-service
Date: Tue, 25 Jun 2024 00:49:07 +0200 [thread overview]
Message-ID: <c4483a64-abf2-62d4-78a2-05c6bf349857@autistici.org> (raw)
In-Reply-To: <e6e444a7e2a4ec32fca27a5098070947974abcd4.1718854920.git.richard@freakingpenguin.com>
Hi Richard,
thank you for this work, it looks really useful and in general I think
the patches are ok but I have some points I'd like to discuss.
I think it would be nice to have an init action for the restic-guix
command shipped with the service and move there the logic for
initializing repositories. The service then could be adapted in a way
that it would call the restic-guix init before calling restic-guix
backup. Would you be interested in implementing this?
At last, my use case for having a restic package field for each job is
to have some critical jobs that I don't want to touch running with
Guix's bootstrapped restic package and some personal jobs that I run
with a restic 0.16 binary package I have in my personal channel . I'm
not sure this warrants a field in each single job, but they are optional
anyway. Anyway I wouldn't consider this a blocker and if the Guix
project has some guidelines in this sense I'd say follow them.
Cheers,
giacomo
next prev parent reply other threads:[~2024-06-24 22:50 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-06-20 3:44 [bug#71660] [PATCH v2 0/5] Improve on restic-backup-service Richard Sent
2024-06-20 3:44 ` [bug#71661] [PATCH v2 1/5] services: backup: Support bootstrapping an initial restic backup Richard Sent
2024-06-20 3:44 ` [bug#71662] [PATCH v2 2/5] services: backup: Add password-command support to restic-service Richard Sent
2024-06-20 3:44 ` [bug#71663] [PATCH v2 3/5] services: backup: Add extra-packages field to restic-backup-job Richard Sent
2024-06-20 3:44 ` [bug#71639] [PATCH v2 4/5] services: backup: Move restic package to restic-configuration Richard Sent
2024-06-20 3:44 ` [bug#71665] [PATCH v2 5/5] tests: Add restic system test Richard Sent
2024-06-24 22:49 ` paul via Guix-patches via [this message]
2024-06-27 3:56 ` [bug#71639] [PATCHv2 0/5] Improve on restic-backup-service Richard Sent
2024-07-07 20:40 ` paul via Guix-patches via
-- strict thread matches above, loose matches on Subject: below --
2024-06-18 22:06 [bug#71639] [PATCH WIP " Richard Sent
2024-06-18 22:08 ` [bug#71639] [PATCH WIP 1/5] services: backup: Support bootstrapping an initial restic backup Richard Sent
2024-06-18 22:08 ` [bug#71639] [PATCH WIP 2/5] services: backup: Add password-command support to restic-service Richard Sent
2024-06-18 22:08 ` [bug#71639] [PATCH WIP 3/5] services: backup: Add extra-packages field to restic-backup-job Richard Sent
2024-06-18 22:08 ` [bug#71639] [PATCH WIP 4/5] services: backup: Move restic package to restic-configuration Richard Sent
2024-06-18 22:08 ` [bug#71639] [PATCH WIP 5/5] tests: Add restic system test Richard Sent
2024-08-22 17:43 ` [bug#71639] [PATCHv2 0/5] Improve on restic-backup-service Fabio Natali via Guix-patches via
2024-08-25 14:12 ` paul via Guix-patches via
2024-09-03 16:43 ` [bug#71639] [PATCH WIP " Fabio Natali via Guix-patches via
2024-09-04 15:49 ` Richard Sent
2024-09-05 10:06 ` Fabio Natali via Guix-patches via
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=c4483a64-abf2-62d4-78a2-05c6bf349857@autistici.org \
--to=guix-patches@gnu.org \
--cc=71639@debbugs.gnu.org \
--cc=goodoldpaul@autistici.org \
--cc=richard@freakingpenguin.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).