feat: select volumes to download #48

New Issue

moritz · 2024-01-11T18:03:19Z

moritz commented

2024-01-11 18:03:19 +00:00

Following: #44

select the volumes via VOLUMES env or --volumes parameter.

i.E.

--volumes all
--volumes db,data

Following: https://git.coopcloud.tech/coop-cloud/backup-bot-two/issues/44 select the volumes via `VOLUMES` env or `--volumes` parameter. i.E. ``` --volumes all --volumes db,data ```

👍 1

decentral1se commented

2024-10-28 16:00:54 +00:00

@moritz

I do see now the following:

		backupbot.py
		Line 465 in 1214f59c79
	
				@click.option('volumes', '--volumes', '-v', envvar='VOLUMES')

But AFAIU this is for a boolean? vs. something like this in restore

		backupbot.py
		Line 114 in 1214f59c79
	
				@click.option('container', '--container', '-c', envvar='CONTAINER', multiple=True)

@moritz I do see now the following: https://git.coopcloud.tech/coop-cloud/backup-bot-two/src/commit/1214f59c79b87539146d131eb8771a2220850da7/backupbot.py#L465 But AFAIU this is for a boolean? vs. something like this in `restore` https://git.coopcloud.tech/coop-cloud/backup-bot-two/src/commit/1214f59c79b87539146d131eb8771a2220850da7/backupbot.py#L114

moritz commented

2024-10-28 16:53:25 +00:00

For the restore command it's already implemented, but not for download command. For download, VOLUMES is only a flag, if the volumes should be downloaded or not, but the selection is not implemented.

For the `restore` command it's already implemented, but not for `download` command. For download, `VOLUMES` is only a flag, if the volumes should be downloaded or not, but the selection is not implemented.

👍 1

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: coop-cloud/backup-bot-two#48