IBims1NicerTobi
  • Joined on 2023-08-14
IBims1NicerTobi commented on pull request borgmatic-collective/borgmatic#895 2024-07-08 07:59:30 +00:00
WIP: Add container volume backups with podman cli

Yeah I wouldn't mind working that out too. Kinda seems like wasted effort to try and hack in support for this one hook only to change the API afterwards.

IBims1NicerTobi commented on pull request borgmatic-collective/borgmatic#895 2024-07-07 11:26:24 +00:00
WIP: Add container volume backups with podman cli

I would seriously propose writing a clear api for the database dumps since I would assume there a a lot more coming soon. If you were up for that change I would propose writing something like an…

IBims1NicerTobi commented on pull request borgmatic-collective/borgmatic#895 2024-07-07 11:01:56 +00:00
WIP: Add container volume backups with podman cli

Yeah but tbh that just feels hacky. I really dislike the use of magical names here and I think it would be much clearer if every hook just provided the name it used for the database dump directory…

IBims1NicerTobi commented on pull request borgmatic-collective/borgmatic#895 2024-07-06 17:57:21 +00:00
WIP: Add container volume backups with podman cli

Also it looks like I cannot just call the hook podman because the _databases part of the path is hardcoded in borgmatic/actions/restore.py in line 166.

IBims1NicerTobi commented on pull request borgmatic-collective/borgmatic#895 2024-07-06 17:32:36 +00:00
WIP: Add container volume backups with podman cli

@witten Any opinion on this?

IBims1NicerTobi pushed to main at IBims1NicerTobi/borgmatic 2024-07-05 13:07:33 +00:00
7bcc35c816 Fixed naming
IBims1NicerTobi commented on pull request borgmatic-collective/borgmatic#895 2024-07-05 13:02:39 +00:00
WIP: Add container volume backups with podman cli

Can we maybe add a function to every database hook to the name of the configured databases instead of assuming their position in the schema?

IBims1NicerTobi commented on pull request borgmatic-collective/borgmatic#895 2024-07-05 13:01:21 +00:00
WIP: Add container volume backups with podman cli

Ok we maybe need some docs saying that the hook dump and restore layout of the schema is related to the restore code. The code in actions/restore.py checks for the database name on a specified…

IBims1NicerTobi pushed to main at IBims1NicerTobi/borgmatic 2024-07-05 12:23:01 +00:00
7729911fe8 Renamed hook in schema
IBims1NicerTobi commented on pull request borgmatic-collective/borgmatic#895 2024-07-05 12:21:24 +00:00
WIP: Add container volume backups with podman cli

Yeah other than volumes I would also like to support labels.

IBims1NicerTobi commented on pull request borgmatic-collective/borgmatic#895 2024-07-04 21:32:50 +00:00
WIP: Add container volume backups with podman cli

So as of right now I have no Idea how exactly the restore function would have to look like and I would really appreciate your input. The restore command should look like this: `borg extract…

IBims1NicerTobi created pull request borgmatic-collective/borgmatic#895 2024-07-04 21:24:42 +00:00
WIP: Add container volume backups with podman cli
IBims1NicerTobi pushed to main at IBims1NicerTobi/borgmatic 2024-07-04 21:23:42 +00:00
dea11fc393 Fix import order
IBims1NicerTobi pushed to main at IBims1NicerTobi/borgmatic 2024-07-04 21:20:18 +00:00
3cc7484784 Inital version for podman volume backups
IBims1NicerTobi commented on issue borgmatic-collective/borgmatic#671 2024-07-04 16:57:19 +00:00
New hook: podman volume export

Yeah but podman volume import is kinda odd. You import a volume and the contents get merged with the current content of the volume. So either restore deletes the podman volume, creates it again…

IBims1NicerTobi commented on issue borgmatic-collective/borgmatic#671 2024-07-04 16:38:43 +00:00
New hook: podman volume export

Yeah the plan is to store it as a tarball since we can just dump that to a named pipe and save storage that way

IBims1NicerTobi commented on issue borgmatic-collective/borgmatic#671 2024-07-04 16:35:39 +00:00
New hook: podman volume export

Basically I would like the restore command to just act exactly like podman volume export {volume_name}

IBims1NicerTobi commented on issue borgmatic-collective/borgmatic#671 2024-07-04 16:30:27 +00:00
New hook: podman volume export

Thats not what I meant. Podman volume export generates the entire volume contents as a tar archive.

IBims1NicerTobi commented on issue borgmatic-collective/borgmatic#671 2024-07-04 16:17:11 +00:00
New hook: podman volume export

@witten would you mind if the restore command would just dump the volume to stdout? Podman volumes are just normal tar archives and I think it would be very flexibe to just run `borgmatic ...