fix build: add missing file

tape: implement read-label command

tape: implement label command

tape: add media pool handling

tape: add media pool cli

tape: add media pool config api

tape: correctly sort drive api subdir

tape: implement option changer-drive-id

tape: cli cleanup - avoid api redefinition

tape: implement delete property for drive update command

tape: cleanup load-slot api

tape: cli cleanup - rename scana-for-* into scan

tape: implement load-media command

tape: implement eject

tape: implement rewind

tape: implement erase media

tape: add command line interface proxmox-tape

tape: do not remove changer while still used

tape: add changer property to drive create api

tape: set protected flag for configuration change api methods

client: move connect_to_localhost into client module

tape: add media state database

update control file

remove unused descriptions from api macros

these are now a hard error in the api macro

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

tape: expose basic tape/changer functionality at api2/tape/

tape: check digest on config update

tape: add changer configuration API

tape: add tape drive configuration API

tape: add tape device driver

tape: add BlockeReader/BlockedWriter streams

This is the basic format used to write data to tapes.

tape: add BlockHeader impl

tape: add helpers to emulate tape read/write behavior

fix bug #3189: fix change_password permission checks, run protected

depend on mtx (tape changer control)

A very small package with no additional dependencies.

tape: add tape changer support using 'mtx' command

tape: add media invenotry

tape: define useful constants

tape: add media pool config

tape: add tape read trait

tape: add tape write trait

tape: add file format definitions

commit missing file: tape api type definitions

tape support: add drive configuration

start experimental tape management GUI

You need to set the environment TEST_TAPE_GUI=1 to enable this.
The current GUI is only a placeholder.

backup/datastore: move manifest locking to /run

this fixes the issue that on some filesystems, you cannot recursively
remove a directory when you hold a lock on a file inside (e.g. nfs/cifs)

it is not really backwards compatible (so during an upgrade, there
could be two daemons have the lock), but since the locking was
broken before (see previous patch) it should not really matter
(also it seems very unlikely that someone will trigger this)

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

backup/datastore: really lock manifest on delete

'lock_manifest' returns a Result<File, Error> so we always got the result,
even when we did not get the lock, but we acted like we had.

bubble the locking error up

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

client/pull: log how many groups to pull were found

if no groups were found, the task log was very confusing as it
contained no real information why nothing was synced, e.g.:

 Starting datastore sync job 'remote:datastore:local-datastore:s-79412799-e6ee'
 Sync datastore 'local-datastore' from 'remote/datastore'
 sync job 'remote:datastore:local-datastore:s-79412799-e6ee' end
 TASK OK

this patch simply logs how many groups were found and are about to be synced:

 Starting datastore sync job 'remote:datastore:local-datastore:s-79412799-e6ee'
 Sync datastore 'local-datastore' from 'remote/datastore'
 found 0 groups to sync
 sync job 'remote:datastore:local-datastore:s-79412799-e6ee' end
 TASK OK

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

docs: add more thoughts about chunk size

doc: add some thoughts about large chunk sizes

cleanup: use extra file for StoreProgress

cleanup: remove unnecessary StoreProgress clone()

verify: use same progress as pull

percentage of verified groups, interpolating based on snapshot count
within the group. in most cases, this will also be closer to 'real'
progress since added snapshots (those which will be verified) in active
backup groups will be roughly evenly distributed, while number of total
snapshots per group will be heavily skewed towards those groups which
have existed the longest, even though most of those old snapshots will
only be re-verified very infrequently.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

progress: add format variants

for iterating over a single group, or iterating just on the group level

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

pull: factor out interpolated progress

and add group/snapshot count info.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

remove BackupGroup::list_groups

BackupInfo::list_backup_groups is identical code-wise, and makes more
sense as entry point for listing groups.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

gc: don't limit index listing to same filesystem

WalkDir does not follow symlinks by default anyway, and this behaviour
is not documented anywhere. e.g., if a sysadmin mounts 'extra storage'
for some backup group or type (not knowing that only metadata is stored
in those directories), GC will ignore all the indices contained within
and happily garbage collect their chunks..

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

gc: remove duplicate variable

list_images already returns absolute paths, we don't need to prepend
anything.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

gc: log index files found outside of expected scheme

for safety reason, GC finds and marks all index files below the
datastore base path. as a result of regular operations, only index files
within the expected scheme of <TYPE>/<ID>/<TIMESTAMP> should exist.

add a small check + warning if the index list contains index files out
side of this expected scheme, so that an admin with shell access can
investigate.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

gc: shorten progress messages

we have messages starting the phases anyway, and limit the number of
progress updates so that context remains available at all times.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

d/control bump

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

file logger: remove test.log after test as well

and a doc formatting fixup

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

derive PartialEq for Userid

the manual implementation is equivalent

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

tiny clippy hint

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

don't enforce Vec and String in tools::join

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

minor cleanup

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

tools avoid unnecessary copying of parameters/properties

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

minor cleanups

whitespace, formatting and superfluous lifetime annotations

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

network helpers: fix fd leak in get_network_interfaces

This one always leaked.

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

use fd_change_cloexec helper

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

minor cleanup

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

pxar: stricter file descriptor guards

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

client: use tools::pipe instead of nix

nix::unistd::pipe returns unguarded RawFds which should be
avoided

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

changes for proxmox 0.8

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump proxmox dep to 0.8.0

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

ui: fix usage of findRecord

findRecord does not match exactly, but only at the beginning and
case insensitive, by default. Change all calls to be case sensitive
and an exactmatch (we never want the default behaviour afaics).

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

bump version to 1.0.5-1

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

restore: print to STDERR

else restoring to STDOUT is broken..

Reported-by: Dominic Jäger <d.jaeger@proxmox.com>
Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

control: update versions

bump version to 1.0.4-1

docs: fix renamed commands

cli: rename command "upload-log" to "snapshot upload-log"

cli: rename command "forget" to "snapshot forget"

cli: rename command "files" to "snapshot files"

fingerprint: add new() method

fingerprint: add bytes() accessor

needed for libproxmox-backup-qemu0

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

ui: fix broken gettext use

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

cli: use new alias feature for "snapshots"

Now maps to "snapshot list".

client: add 'snapshot notes show/update' command

to show and update snapshot notes from the cli

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

bump version to 1.0.3-1

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

gui: add snapshot/file fingerprint tooltip

display short key ID, like backend's Display trait.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

list_snapshots: return manifest fingerprint

for display in clients.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

refactor BackupInfo -> SnapshotListItem helper

before adding more fields to the tuple, let's just create the struct
inside the match arms to improve readability.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

expose previous backup time in backup env

and use this information to add more information to client backup log
and guide the download manifest decision.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

bump version to 1.0.2-1

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

verification: fix message in notification mail

the errors Vec can contain failed groups as well (e.g., if a group has
no or an invalid owner).

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

verification job: log failed dirs

else users have to manually search through a potentially very long task
log to find the entries that are different.. this is the same summary
printed at the end of a manual verify task.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

cleanup: remove dead code

paperkey: refactor common code

from formatting functions to main function, and pass along the key data
lines instead of the full string.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

client: check fingerprint after downloading manifest

this is stricter than the check that happened on manifest load, as it
also fails if the manifest is signed but we don't have a key available.

add some additional output at the start of a backup to indicate whether
a previous manifest is available to base the backup on.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

manifest: check fingerprint when loading with key

otherwise loading will run into the signature mismatch which is
technically true, but not the complete picture in this case.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>