coop-cloud/backup-bot-two
6
0
Fork 4
Code Issues 13 Pull Requests 1 Packages Projects Releases Wiki Activity

backup -- ls: shows too many paths? #37

New Issue
Closed
opened 2023-10-12 13:41:46 +00:00 by decentral1se · 6 comments
decentral1se commented 2023-10-12 13:41:46 +00:00
Owner
Copy Link

I think I would not expect the timestamps?

And further, the lines with /var, /var/lib etc.

🌻 ./abra app backup ls abra-test-recipe.foo.com
2023-10-12T13:33:57.595887937Z /secrets
2023-10-12T13:33:57.595887937Z /secrets/abra-test-recipe_foo_com_pass_one_v1
2023-10-12T13:33:57.595887937Z /secrets/abra-test-recipe_foo_com_test_pass_two_v1
2023-10-12T13:33:03.084078341Z /var
2023-10-12T13:33:03.084078341Z /var/lib
2023-10-12T13:33:03.088078326Z /var/lib/docker
2023-10-12T13:33:02.108081747Z /var/lib/docker/volumes
2023-10-12T13:24:40.985842291Z /var/lib/docker/volumes/abra-test-recipe_foo_com_test-volume
2023-10-12T13:24:40.985842291Z /var/lib/docker/volumes/abra-test-recipe_foo_com_test-volume/_data

Maybe something like?

🌻 ./abra app backup ls abra-test-recipe.foo.com
/secrets/abra-test-recipe_foo_com_pass_one_v1
/secrets/abra-test-recipe_foo_com_test_pass_two_v1
var/lib/docker/volumes/abra-test-recipe_foo_com_test-volume/_data

Also, it would be good to include the time/date of this backup? Is there also an accompanying snapshot ID or additonal metadata you think would be useful to have on the receiving end of this?

I think I would not expect the timestamps? And further, the lines with `/var`, `/var/lib` etc. > 🌻 ./abra app backup ls abra-test-recipe.foo.com > 2023-10-12T13:33:57.595887937Z /secrets > 2023-10-12T13:33:57.595887937Z /secrets/abra-test-recipe_foo_com_pass_one_v1 > 2023-10-12T13:33:57.595887937Z /secrets/abra-test-recipe_foo_com_test_pass_two_v1 > 2023-10-12T13:33:03.084078341Z /var > 2023-10-12T13:33:03.084078341Z /var/lib > 2023-10-12T13:33:03.088078326Z /var/lib/docker > 2023-10-12T13:33:02.108081747Z /var/lib/docker/volumes > 2023-10-12T13:24:40.985842291Z /var/lib/docker/volumes/abra-test-recipe_foo_com_test-volume > 2023-10-12T13:24:40.985842291Z /var/lib/docker/volumes/abra-test-recipe_foo_com_test-volume/_data Maybe something like? > 🌻 ./abra app backup ls abra-test-recipe.foo.com > /secrets/abra-test-recipe_foo_com_pass_one_v1 > /secrets/abra-test-recipe_foo_com_test_pass_two_v1 > var/lib/docker/volumes/abra-test-recipe_foo_com_test-volume/_data Also, it would be good to include the time/date of this backup? Is there also an accompanying snapshot ID or additonal metadata you think would be useful to have on the receiving end of this?
decentral1se commented 2023-10-12 13:45:39 +00:00
Author
Owner
Copy Link

Related #27

Related https://git.coopcloud.tech/coop-cloud/backup-bot-two/issues/27
decentral1se commented 2023-10-12 13:46:01 +00:00
Author
Owner
Copy Link

@3wordchant any thoughts on output for this which would be useful?

@3wordchant any thoughts on output for this which would be useful?
moritz commented 2023-10-18 13:32:08 +00:00
Owner
Copy Link

I think for finding files with grep it would be nice to list all files recursively. But instead of being the default behavior, it could be activated with a flag like --all / -a . And the default behavior could be to provide a path, like unix ls works. And maybe activating timestamps with a -t flag?

I think for finding files with `grep` it would be nice to list all files recursively. But instead of being the default behavior, it could be activated with a flag like `--all / -a `. And the default behavior could be to provide a path, like unix `ls` works. And maybe activating timestamps with a `-t` flag?
👍 1 🎉 1
decentral1se commented 2023-10-18 14:53:20 +00:00
Author
Owner
Copy Link

That sounds Very Cool @moritz 👍

That sounds Very Cool @moritz 👍
moritz commented 2024-10-24 16:05:09 +00:00
Owner
Copy Link

The usage is:

Usage: backup ls [OPTIONS] [PATH]

Options:
  -s, --snapshot TEXT
  -a, --all

backup ls will now show /var/lib/docker/volumes/ per default, if no PATH is given. The path can also be set per INCLUDE_PATH env.
The --all flag will show all files recursively. This flag can also be set with the env SHOW_ALL.

The usage is: ``` Usage: backup ls [OPTIONS] [PATH] Options: -s, --snapshot TEXT -a, --all ``` `backup ls` will now show `/var/lib/docker/volumes/` per default, if no `PATH` is given. The path can also be set per `INCLUDE_PATH` env. The `--all` flag will show all files recursively. This flag can also be set with the env `SHOW_ALL`.
❤️ 1
moritz commented 2024-10-24 16:09:39 +00:00
Owner
Copy Link

Added -t, --timestamps flag (TIMESTAMPS env) to show timestamps.

Added `-t, --timestamps` flag (`TIMESTAMPS` env) to show timestamps.
🎉 1
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
Something is not working
duplicate
This issue or pull request already exists
enhancement
New feature
help wanted
Need some help
invalid
Something is wrong
question
More information is needed
wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
3wordchant aadil abra-bot ammaratef45 Apfelwurm appletalk arjan basebuilder Brooke carla cas codegod100 coopcloud cyrnel decentral1se fauno flancian Frando iexos javielico jmakdah2 joe-irving kawaiipunk knoflook kolaente lambdabundesverband marlon mayel mirsal moritz nicksellen notplants p4u1 pau pharaohgraphy renovate-bot ripclap rix rscmbbng sef simon sixsmith stevensting tobias trav val virtualboys wolcen wykwit xynosis yksflip
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: coop-cloud/backup-bot-two#37
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?