clippy: remove unnecessary mut

curiously this one is also not caught by rustc

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

clippy: this case I disagree with

clippy suggests a way-too-long
for (i, index) in index.iter_mut().enumerate().take(index_count) {
    *index = ...
}

which is just massive in comparison, and not at all more
readable...

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

clippy lints

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

readability cleanup

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

fix #2866: invalidate bitmap on crypt_mode change

signed and plain backups share chunks, so bitmap reusal is okay for
those combinations. switching from encrypted to not encrypted or
vice-versa could have pretty fatal consequences - either referencing
plain-text chunks in 'encrypted' backups, or referencing encrypted
chunks in 'unencrypted' backups without still having the corresponding
keys..

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

add archive_name helper

since we're about to introduce a third duplicate of this line..

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

refactor incremental checks

in preparation for adding more checks to check_incremental

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

bump version to 0.6.3-1

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

build: fix make -j

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

bump proxmox-backup to 0.8.10

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

Cargo.toml: sort dependencies

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

adapt to chunk reader changes

for verification of chunk crypt mode <=> index crypt mode match

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

relay proxmox dependency to 0.2

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

bump version to 0.6.2-1

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

update proxmox and proxmox-backup crates

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

read_image_at: iterate until buffer is filled

QEMU will always assume EOF when less bytes than requested are returned
by a block drivers 'read' interface, so we need to fill the buffer up to
'size' if possible.

Signed-off-by: Stefan Reiter <s.reiter@proxmox.com>

use AsyncIndexReader for read_image_at

Replaces the removed BufferedFixedReader and makes the API fully async
(not block_in_place or block_on which could break with many requests at
once).

A tokio::sync::Mutex needs to be used for exclusive access, since a
regular one would not work with async/await calls.

Signed-off-by: Stefan Reiter <s.reiter@proxmox.com>

bump version to 0.6.1-1

ignore missing or bad previous index

This is not a fatal condition, it just means that we cannot do
incremental backups client-side. This can happen if a new disk was added
the a VM since the last backup for example.

Signed-off-by: Stefan Reiter <s.reiter@proxmox.com>

bump version to 0.6.0-1

produce correct manifest

add compress and encrypt paramaters

This breaks binary API again!

bump version to 0.5.1-1

cleanup: also improve code resuse and parameter checks for restore

bump version to 0.5.0-1

check for null parameters

cleanup: allways call _async functions instead of duplicating code

proxmox_backup_check_incremental: check if device_name is NULL

proxmox_backup_abort: check for reason == NULL

bump version to 0.4.0-1

use new get_runtime_with_builder

bump version to 0.3.3-1

src/commands.rs: avoid useless digest conversions

src/commands.rs: avoid useless lock

src/registry.rs: split out Registry code

rename worker_task.rs -> backup.rs

rename ProxmoxRestore into RestoreTask

remove pub attribute where not required

src/restore.rs: implement with_runtime

src/restore.rs: share remote chunk reader

src/restore.rs: use backup_type from setup

bump version to 0.3.2-1

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

src/restore.rs: use OnceCell to avoid locking

src/worker_task.rs: use OnceCell to avoid locking

current-api.h: update to include changes from previous commit

add proxmox_backup_check_incremental() api

current-api.h: update to include changes from previous commit

proxmox_restore_new: use snapshot string (again) instead of (type, Id, time).

And provide a new helper to create such strings: proxmox_backup_snapshot_string()

add restore data reading API

For accessing existing backups in a VM.

make Registry generic and add ImageAccessInfo

Signed-off-by: Stefan Reiter <s.reiter@proxmox.com>

bump version to 0.3.1-1

write_data: return 0 for reused chunks

bump version to 0.3.0-1

log abort error to stderr

cleanup: remove stale comments

make restore api thread safe

cleanup (use better name)

current-api.h: update docs

make backup C api thread safe

Protect everything with a Mutex, so it is now allowed to make parallel
calls from different threads.

Revert "avoid mutexes"

This reverts commit 96afa3d02c06e30f5fa9a0edd5ed203ac05ad7b8.

Turns out we cannot guarantee correct locking at caller site ...

improve error message

add_config: use Vec<u8> instead of DataPointer

avoid mutexes

simpletest.c: fix error check

avoid extra worker thread and command channel

bump version to 0.2.0-1

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

add tools module

with:
* utf8_c_string(ptr) -> Result<Option<String>>
* utf8_c_string_lossy(ptr) -> Option<String>
* unsafe utf8_c_string_lossy_non_null(ptr) -> String

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

add api check

commit the current generated proxmox-backup-qemu.h as
'current-api.h' and run `diff -up` between the two in build
steps, so any changes to the C api are caught at build time.

cbindgen seems to be using a BTreeMap for the data, so it
should always come out sorted, so this should be a
stable-enough check.

Link: https://github.com/eqrion/cbindgen/issues/12
Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

depend on  proxmox-backup v0.5.0

also upload _dbgsym_ .deb

bump version to 0.1.8-1

proxmox_backup_connect: return info about previous backup

use bool type for incremental flag

simpletest.c: fix compile error

bump version to 0.1.7-1

simplify code

add incremental backup support

Tell the server we're doing an incremental backup if QEMU notifies us
with "is_incremental" in register_image. We do this by using the
'reuse-csum' parameter when registering each archive's index, thus
switching the server to incremental mode (where it only expects changed
chunks and verifies the previous backup's checksum).

We use the newly changed API download_previous_fixed_index() to replace
known_chunks generation and also give us the verified index file to
calculate the new checksum with. The manifest for verfication is
downloaded during "Connect".

To initialize the session cache for checksums, lazy_static is used.

Signed-off-by: Stefan Reiter <s.reiter@proxmox.com>

make compatible with newest proxmox-backup crate

Signed-off-by: Stefan Reiter <s.reiter@proxmox.com>

bump version to 0.1.6-1

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

update proxmox-backup crate to 0.2.3

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

fix backing up images that are not multiples of 'chunk_size'

by removing the client side checks

since the backup server has to check those things anyway, there is
little benefit of checking it here, but poses a risk that there
are two diverging checks for the client and server

especially the next to last check (end_offset > device_size) was
wrong, since we do not actually check the size of the actual chunk
(which would be 'size'), but the generic set chunk size (defaults to 4M)
which is obviously wrong (the last chunk is probably smaller)

since it is not needed anymore, we can omit to get the 'device_size'

fixes an issue where we could not backup an image which was not
4M aligned (or smaller than 4M altogether)

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

update proxmox-backup crate to 0.2.2

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

bump version to 0.1.5-1

src/restore.rs: hold reference to the runtime

avoid compiler warning

bump version to 0.1.4-1

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

restore: reuse the clients download_fixed_index

it's the same code

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

Cargo.toml: update versions

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

switch from failure to anyhow

Similarly to how we did in proxmox-backup in mid april

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

bump version to 0.1.3-1

call C-callback if send command fail

And fix previous commit.

cleanup: use macro raise_error_int

correctly convert error messages to CString

remove debug statements

And print unrecoverable errors to stderr.

bump version to 0.1.2-1

build.rs: .with_include_guard("PROXMOX_BACKUP_QEMU_H")

Makefile: upload to pve instead of pbs product

Makefile: add upload target

depend on specific proxmox-backup git version

fix compile errors