docs: mention support subscription plans

and change enterprise repository section to present tense.

Signed-off-by: Dylan Whyte <d.whyte@proxmox.com>

tools/zip: only add zip64 field when necessary

if neither offset nor size exceeds 32bit, do not add the
zip64 extension field

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

ui: tape/DriveStatus: open task window on click on state

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

ui: tape: convert slot selection on transfer to combogrid

this is much handier than number field, and the user can instantly
see which one is an import/export slot

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

ui: tape/Restore: let the user choose an owner

so that the tape backup can be restored as any user, given
the current logged in user has the correct permission.

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

proxmox-tape: add missing notify-user to 'proxmox-tape restore'

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

tape: use MB/s for backup speed (to match drive speed specification)

tape: speedup backup by doing read/write in parallel

tools/zip: add missing start_disk field for zip64 extension

it is not optional, even though we give the size explicitely

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

backup info: run rustfmt

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

update to pxar 0.10

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

fix feature flag logic in pxar create

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

tape: write_chunk_archive - do not consume partially written chunk at EOT

So that it is re-written to the next tape.

bump version to 1.0.10-1

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

d/postinst: fixup tape permissions if existing with wrong permissions

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

d/postinst: only check for broken task index on upgrade

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

d/postinst: tell user that we restart when updating from older version

d/postinst: restart when updating from older version

Else one has quite a terrible UX when installing from 1.0 ISO and
then upgrading to latest release..

commit 0ec79339f7aebf9 for the fix and some other details

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

server/worker_task: improve endtime for unknown tasks

instead of always using the starttime, use the last timestamp from the log
this way, one can see when the task was aborted without having to read
the log

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

tape/send_load_media_email: move to server/email_notifications

and reuse 'send_job_status_mail' there so that we get consistent
formatted mails from pbs (e.g. html part and author)

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

tape: improve backup task log

typo fixes all over the place

found and semi-manually replaced by using:
 codespell -L mut -L crate -i 3 -w

Mostly in comments, but also email notification and two occurrences
of misspelled  'reserved' struct member, which where not used and
cargo build did not complain about the change, soo ...

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

ui: fix typo in comment

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

ui: fix typo in options

Signed-off-by: Oguz Bektas <o.bektas@proxmox.com>

d/postinst: fix typo in comment

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

tape: fix scsi volume_statistics and cartridge_memory for quantum drives

tape: documentation language fixup

Signed-off-by: Dylan Whyte <d.whyte@proxmox.com>

ui: tape: increase tapestore interval

from 2 to 60 seconds. To retain the response time of the gui
when adding/editing/removing, trigger a manual reload on these actions

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

ui: NavigationTree: do not modify list while iterating

iterating over a nodeinterfaces children while removing them
will lead to 'child' being undefined

instead collect the children to remove in a separate list
and iterate over them

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

tape: improve backup task log (show percentage)

tape: improve backup task log format

tape: read_tape_mam - pass correct allocation len

tape: cleanup - remove wrong inline comment

api2/tape/backup: continue on vanishing snapshots

when we do a prune during a tape backup, do not cancel the tape backup,
but continue with a warning

the task still fails and prompts the user to check the log

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

tape: lock artificial "__UNASSIGNED__" pool to avoid races

tape: improve MediaPool allocation by sorting tapes by ctime and label_text

bump version to 1.0.9-1

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

ui: navigation: adapt tape icons also here

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

ui: tape drive icon: invert

Less anti-aliasing to do, so it looks better when small.

Note, I use a mask as else we could not cut out something
transparent, but only over paint it filled with white, which is not
the same and not how font awesome does it.

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

ui: tape panel: add/use icons

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

ui: tape: code style: xtype coms first

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

ui: tape/window/Tape{Backup, Restore}: allow empty notify user field

we have to set skipEmptyText on comboGrids

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

tape/linux_tape: remove debug prints

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

tape/backup: add success/error notifications

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

ui: remove cartridge memory panel (show window after butten press instead)

ui: fix date rendering in drive status

ui: tape/Drive{Config, Status} move actions to status panel

so that the 'DriveConfig' panel is purely for configuration

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

ui: tape/DriveStatus: add cartridge memory grid

that the user can load when a tape is inserted

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

docs: fix note about global media pool namespace

docs: alabaster theme: better contrast for background color of topics

The old value just was way to heavy, and notes/warnings/...
admonitions did not stick out anymore.

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

ui: add online help to drive/changer status

tape: add docu link targets for online help

tape: clean - fix if cleaning tape is already loaded

tape: test tape alert flags after drive clean

ui: images: simplify tape icon

1. Use same height as width
2. drop the top/bottom "frame" lines
3. instead of two "bearings" side by side, like VHS or music-compact
   tapes have, use a single bearing and an outer circle to denote the
   magnetic tape edge

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

pmt: add tape-alert-flags command

docs: use code-block inside tape-backup.rst

docs: document tape backup jobs

add proxmox-tape bash-completion

api2/tape: add notify_user to backup(-jobs) and restore api calls

so that a user can be given that will be notified for
manual intervention (e.g. inserting a tape)

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

ui: tape: add notify-user fields

to backup jobs, backups and restores

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

config/tape_job: add notify-user config to tape jobs

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

tape: avoid Updatable for now

tape: finish api permission checks

acl: use /tape/device instead of /tape/drive and /tape/changer

tape: improve PoolWriter logging

Log reason why we allocate a new media set.

cleanup: rename token_user into auth_id_filter

ui: use shorter datetime format for encryption key creation time

ui: do not show MediaSet Uuid in backup content grid

tape: fix media_expire_time

tfa docs: language fixup

Simplifies the introduction a bit and makes it more readable.
Also some other minor language fixes throughout the section.

Signed-off-by: Dylan Whyte <d.whyte@proxmox.com>

tape: fix wrong media set expire time

tape: improve media status in list_media

ui: add acl paths for tape permission

ui: tape: add missing monStoreErrors calls for the grids

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

tape/drive: improve error and email handling for requesting a tape load

Try once first to load the correct tape before sending an email to
insert the correct one. This way, the admin does not get a mail
if the correct tape is already inserted.

Also include the error we got that prompted the email to insert the
tape. This means that if the admin gets prompted to insert e.g.
"FOO" but inserts "BAR", he'll get an email that the wrong
tape is inserted.

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

ui: tape: only add tapestore and nav element once

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

add proxmox-tape zsh-completions

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

ui: tape/DriveStatus: fix handler name for eject

We call 'ejectMedia' not 'eject', so rename the function.

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

ui: webauthn registration: code cleanup long variable name

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

ui: webauthn registration: adapt error messages

There are variuos hints that 'InvalidStateError' can be assumed to
come from an already registered token[0][1].

So indicate that more strongly, also omit the extra line and warning
triangle, there's already an error icon present.

[0]: step 2: https://w3c.github.io/webauthn/#sctn-registering-a-new-credential
[1]: step 3: https://w3c.github.io/webauthn/#sctn-op-make-cred

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

ui: LoginView: show webauthn errors in window

instead of silently discarding the error, else the user might be
confused because nothing happened

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

config/tfa: webauthn: disallow registering a token twice

by adding the existing credential id to the 'excludeCredentials' list

this prevents the browser from registering a token twice, which
lets authentication fail on some browser/token combinations
(e.g. onlykey/solokey+chromium)
while is seems this is currently a bug in chromium, in a future spec
update the underlying behaviour should be better defined, making this
an authenticator bug

also explicitly catch registering errors and show appropriate error messages

0: https://bugs.chromium.org/p/chromium/issues/detail?id=1087642

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

Revert "ui: window/Settings / WebAuthn: add browser setting for userVerificationo"

even if the options *could* be set in the frontend, the backend
actually has to do validation of those settings, thus we should not
make that a browser setting

additionally, having the value 'preferred' does not actually make sense,
since it does not add any security (the backend skips the
userverification check then)

This reverts commit aca4c2b5a9de23f7cafab92da0f88123f4ca5d8c.

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

config/tfa: set UserVerificationPolicy to Discouraged

the current default is 'Preferred', which is not really useful, as the
(web) client can simply change this to discouraged, since the
webauthn_rs crate does not verify the 'user_verified' bit of the
response in that case

setting this to 'Required' is not really useful either at the moment,
since a user can have a mix of different authenticators that may or
may not support user verification

there is ongoing discussion in the crate how to handle that[0]

we could probably expose this setting(discouraged/required) to the user/admin
and save it to the credential and allow only registering credentials
of the same type or filter them out on login (i.e. if there is an
authenticator that can handle userVerification, require it)

in any case, the current default is not helpful for security, but
makes loggin in harder, since the key will by default want to verify
the user

0: https://github.com/kanidm/webauthn-rs/pull/49

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

zpool parsers: small code cleanup

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

zpool parsers: fixup test value

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

tape: impl access permission for media content list

tape: impl access permissions for tape jobs

tape: add acl path /tape/job/{id}

tape: impl access permissions for encryption key config

tape: impl api permissions for drive/changer/pool configuration

complete_acl_path: add more paths

add roles to implement tape backup permissions

docs: mark tape backup as technology preview

Fix 3321: fix interactive restore command explanation

The previous description in the docs was false. The restore command
with the pattern parameter will search the entire backup archive,
regardless of pwd.

Signed-off-by: Dylan Whyte <d.whyte@proxmox.com>

install proxmox-tape binary

ui: enable experimental tape UI if tape.cfg exists

dashboard: fix datastore full estimate for not changing usage

If the usage of a datastore did not change, we did not
return an estimate. The ui interpreted this as 'not enough data', but
it should actually be 'never'.

Fixing this by always setting the estimate first to 0 and overwriting
if we successfully calculated one, and checking for 'undefined' in the ui.

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

ui: render medium-wearout as percentage