ui: eslint: fix trailing comma and comma related whitespaces errors

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

ui: eslint: fix trailing spaces

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

ui: tfa/totp: fix setting issuer in secret URL

it's recommended to set the issuer for both, the get parameter and
the initial issuer label prefix[0].

[0]:
https://github.com/google/google-authenticator/wiki/Key-Uri-Format#label

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

ui: qemu/audio: improve displayed text for backend drivers

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

ui: qemu/audio: submitValue is default true for combobox

this was previously a display field, where submitValue defaults to
false, so we required to enable it explicitly. As it changed to a
combo box we can drop that.

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

ui: qemu: add the none audio driver as an option

Signed-off-by: Gilles Pietri <contact+dev@gilouweb.com>

ui: add netdevice: fix #3203: Use OS default model

In the VM create wizard we automatically set e1000 for Windows and virtio for
Linux.  We should also do this when adding a network device in the hardware
view.

OSDefaults.generic.networkCard (=e1000) is always available.  Hence, leave this
as default value for the field and then try to get the ostype via API and
overwrite the default e1000.

Signed-off-by: Dominic Jäger <d.jaeger@proxmox.com>

ui: acme: Remove false gettexts

Translating these is not useful; e.g. API would be translated to API in German.

Signed-off-by: Dominic Jäger <d.jaeger@proxmox.com>

report: fix separation of commands over multiple lines

once we get over the allowed line length we must move each atom
(e.g., argument or other value) to a separate line, not pairing a
few.

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

pvereport: code cleanup, line length

Signed-off-by: Aaron Lauterer <a.lauterer@proxmox.com>

pvereport: rework report contents

add:
* HA status
* ceph osd df tree
* ceph conf file and conf db
* ceph versions

removed:
* ceph status, as pveceph status is now printing the same information

Signed-off-by: Aaron Lauterer <a.lauterer@proxmox.com>

pvereport: fix multipath inclusion

we now push it to the correct hash if it is installed

Signed-off-by: Aaron Lauterer <a.lauterer@proxmox.com>

vzdump: fix error format for errors coming from storage_info

Errors from storage_info() are newline-terminated, because perl would append
the line number otherwise. Chomp those errors, because sendmail() relies
on the presence of a newline to decide if it's multiple problems or only one.

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

vzdump: sendmail: fix html by closing the tags

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

vzdump: defaults: convert to prune-backups early enough

Fixes the case where reading from /etc/vzdump.conf fails.

Also convert the options read from /etc/vzdump.conf before the loop. That
avoids showing a wrong warning when 'prune-backups' is configured in
/etc/vzdump.conf, and maxfiles isn't. Previously, because 'maxfiles' from the
schema defaults was automatically set, the call to parse_prune_backups_maxfiles
after the loop threw the warning that both options are defined.

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

pveceph: add timeout to status command

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

pveceph: change status from long JSON to ceph -s

Printing a lot of very detailed JSON output on the CLI is not very
useful.

Printing the `ceph -s` overview is much more suited to give an overview
of the ceph cluster status.

Signed-off-by: Aaron Lauterer <a.lauterer@proxmox.com>

safe destroy: Add HA to purge tooltip

This makes the enumeration complete.

Signed-off-by: Dominic Jäger <d.jaeger@proxmox.com>

ui: improve description of guest-fstrim checkbox

The phrasing left some room to speculation when this would actually be
run, e.g. after cloning a full VM?

Currently the only instances where it is used is after moving a disk or
migrating a VM.

Signed-off-by: Aaron Lauterer <a.lauterer@proxmox.com>

ceph: extend error for wrong config-file link

Since I had to look up the cause of the error-message in our source
explaining why exactly ceph-operations fail, because
/etc/ceph/ceph.conf exists.

reported via our community forum:
https://forum.proxmox.com/threads/osd-ersetzen-neu-erstellen.80793/

quickly tested on a virtual ceph cluster

Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>

update aplinfo

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

status/plugin: do not test connection if disabled

so that if one disables the plugin (e.g. because it is offline),
it will work even when the server is not reachable

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

vzdump: use debugmsg instead of warn

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

vzdump: warn when both storage and dumpdir are defined in vzdump.conf

and prefer storage, because the storage configuration might contain more
settings. Warning is preferable over dying, because all backups would be
affected (even if they don't use the vzdump.conf parameters) and the settings
could've been compatible (i.e. dumpdir being the storage's dump dir). Previously
one of the two options would randomly be chosen in the loop in new(), because of
perl hash iteration.

Reported here: https://forum.proxmox.com/threads/vzdump-times-out-very-often-on-zfs-storage-pool.80035/post-354277

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

bump version to 6.3-3

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

pvestatd: fix container cpuset scheduling

Since pve-container commit

c48a25452dccca37b3915e49b7618f6880aeafb1

the code to get the cpuset controller path lives in pve-commons PVE::CGroup.
Use that and improve the logging in case some error happens in the future.
Such an error will only be logged once per pvestatd run,
so it does not spam the log.

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

fix #3182 #3183: change backup retention mask logic

instead of relying on the contentTypeField (which does not need to
exists, e.g. for iscsi), explicitely write it into the panel/icon
mapping and check that

better would be if we query the backend about storage capabilities,
but such an api call does not exist yet, so this should be ok for now

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

test: allow running replication tests in parallel

running multiple workers reduces built time from >3s to ~0.6s here.

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

test: replication: cleanup use statements

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

test: buildsys: allow to execute vzdump tests in parallel

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

test: buildsys: rework target names, prefix with test

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

vzdump: defaults: correctly parse prune-backups and convert maxfiles

Also simplify handling in new(), now that we never have maxfiles there anymore.

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

vzdump: convert maxfiles CLI parameter to prune-backups

in preparation to clean up handling in new().

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

vzdump: storage_info: adapt to new maxfiles backend behavior

It's automatically converted to prune-backups when using storage_config() now.

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

test: add tests for retention parameters for vzdump's new()

To get a more complete picture, instead of mocking storage_config,
PVE::Cluster's get_config is mocked. This ensures that the prune-backups
validation and the maxfiles conversion are also called.

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

api: fixup: commit change to storage module

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

ui: USB selector: use new API endpoint

so that we can sunset the usbscan from pve-storage's scan API, which
was never fitting there anyway.

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

api: add usb list in hardware endpoint

As envisioned in[0][1], better late than never.

[0]: commit 523d5f486ba2d3830d1575327b550ddc1b2ef131
[1]: https://lists.proxmox.com/pipermail/pve-devel/2018-November/034694.html

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

api: use storage scan module from pve-storage

Signed-off-by: Thomas Lamprecht <t.lamprecht@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>

ui: storage: find the correct storage record

by default, findRecord only anchors at the beginning, e.g.
findRecord('id', 'foo');
could find either an entry with id 'foo', 'foobar', 'foobaz', etc.

extend the call to set 'exactMatch' to true, else we sometimes
used the content types of storage e.g., 'local-lvm' for showing
the panels for 'local'

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

ui: don't refer to enc key as 'master key'

the master key is something entirely different, this might confuse
users..

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

bump version to 6.3-2

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

ui: pbs: set onlineHelp (again)

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

ui: bootorder: add onlineHelp link

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

api: metrics/server: test connection on add/update

just a basic check, but better than not checking at all

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

ui: storage edit: ensure to set keep-all by default

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

ui: pbs key show: improve labelWidth for 'Key' in translations

it is not that short in other languages, e.g. German

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

ui: storage: pbsedit: Fix spelling

No e in unusable.

to loose means to free from restraint, or to make less tight or strict [1].
to lose means to fail to keep, or to miss something from one's possession [2].
So "losing" (instead of "loosing") would be bad.

[0] https://www.merriam-webster.com/dictionary/unusable
[1] https://www.merriam-webster.com/dictionary/loose
[2] https://www.merriam-webster.com/dictionary/lose

Signed-off-by: Dominic Jäger <d.jaeger@proxmox.com>

ui: storage edit: avoid returning prune values if unsupported

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

ad #3140: allow interface suffix in dns entries

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

ui: metrics: Link help button to correct chapter

... instead of generic PVE admin guide.

Signed-off-by: Dominic Jäger <d.jaeger@proxmox.com>

api: metrics/server: add minimum and maximum to port schema

we just added the api, so it would be good to only accept valid ports
(they were wrapped before)

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

bump version to 6.3-1

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

api2: osd destroy: untaint device before pvremove

We get the device list from ceph-volume lvm list, and decode the json
output, which at that point is tainted (perlsec (1)).
Untaint it here before calling, because it is currently the only
call-site using the information in a problematic way (run_command).
(the only other call-site being in pve5to6)

Alternatively we could untaint while reading the information, but then
should only return a small subset of the ceph-volume output.

The issue is most likely due to
cb9db10c1a9855cf40ff13e81f9dd97d6a9b2698 in pve-common ('run_command:
improve performance for logging and long lines'),

Tested on a virtual testsetup by creating OSDs with second DB disk,
and destroying it via GUI (did not manage to get the error without the
DB disk)

Reported via our community forum:
https://forum.proxmox.com/threads/insecure-dependency-in-exec-during-osd-destroy.79574/

Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>

api2: osd destroy: fix error function

Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>

ui: storage content: add encryption and verification columns for PBS

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

ui: factor out backup verification renderer

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

ui: guest backup view: add encrypted column for PBS storages

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

ui: storage content: allow to specify extraColumns (again)

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

ui: storage/browser: pass storage plugin type to content panels

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

ui: storage: pbs paperkey: print short fingerprint in heading

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

ui: storage: pbs: move datastore field to right column for balance

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

ui: storage: pbs: improve encryption key handling

Move encryption key handling in its own tab.

Allow uploads, warn more visible on edit/remove

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

ui: utils: add more generic loadTextFromFile upload helper

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

ui: storage: base: allow adding extra tab panels

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

api: resources: return plugin type

The storage one for now.

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

ui: dc/storage: set onlineHelp for prune tab in edit window

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

bump version to 6.2-17

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

API2/Ceph: fix createpools 'add_storages' parameter

by deleteing it from $ceph_param we deleted it also from
$param since it was only a reference

fix it by extracting it beforehand

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

ui: storage: show no-keeps hint exactly when needed

extracting the logic from the previous checkbox listener into a function, which
is also called on field changes and once in afterrender. Calling it initially
makes sure the hint is also displayed at the beginning when editing a storage
with no retention options configured, and the initial disabling of the input
fields for the isCreate case now also happens through that call.

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

ui: guest backups: add separator, fix left-over Comments title

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

ui: storage backup view: add prune window

adapted from PBS. Main differences are:
    * API has GET/DELETE distinction instead of 'dry-run'
    * API expects a single property string for the prune options

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

ui: add ability to show and edit comments for backups

for now we filter out pbs storages

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

ui: change comment column to notes

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

followup code cleanup

actually cleaning up the initial state, the last patch was OK as is..

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

storage: base edit: Don't send delete during creation

... as this is now allowed by the API (createSchema() in PVE::SectionConfig).
It is only allowed by the update API call (updateSchema()).

Signed-off-by: Dominic Jäger <d.jaeger@proxmox.com>

UI: Ceph Wizard: language fixup

Improve the language found in the ceph install Wizard.

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

ui: wizard: fix ceph docs link

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

ui: avoid ||=

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

bump version to 6.2-16

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

ui: storage: pbs: add hint for backup retention

Normally it's better to configure pruning (backup retention) on the
backup server only, as then multiple client settings may not
interfere with each other due to outdated or to strict prune
settings (remember, the strictest prune setting always wins).

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

storage: drop max backup fields, replaced by pruning

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

storage: base edit: add Backup Retention tab

Support all prune keep- settings we have.

masked if the storage cannot support backups.

We do not care if it is disabled as content type for now, a user can
just set them nonetheless (could show a hint though).

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

storage: base edit: move main inputpanel into tab panel

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

ui: storage base panel: cope with undefined column2

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

d/control: bump versioned dependency of libpve-storage-perl 6.2-11

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

cleanup keep-all handling

If keep-all is set to 0, adding it doesn't make a difference,
if the key is not in the hash, it won't show up in the 'values'.

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

vzdump: adapt to new keep-all prune option

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

ui: snapshot tree: add separator between rollback and edit button

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

close #3114: ui: snapshots: switch Edit and Remove Button & add warning logo

Signed-off-by: Moayad Almalat <m.almalat@proxmox.com>

pveceph: update octopus repository

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

ui: ceph install wizard: fix indentation

Hi to the people from the future which came here due to git blame or
the like. `git show -w` should proof that you got the wrong one.

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

ui: make remaining content views not stateful

Commit 2dcc87c798406e74b9f28a894663937967b2c57c removed the statefulness from
the other storage content views, so remove it from the remaining ones too.

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

ui: storage: get content types from resources

to avoid waiting for a status API call.

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

cluster resources: include content type for storages

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

ui: ceph install mask: avoid showing version

we do not know yet which version will get installed for real, so just
drop that part.

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

ui: ceph: allow selecting which version to install

Try to find out the newest version in the cluster and select that (if
in the known OK list).

Slightly hacky, but nothing really out of the ordinary, so should be
OK.

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

ui: ceph wizard: move out info panel to separate component

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

ceph: do not require allow-experimental flag for octopus anymore

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