ui: qemu/HardwareView: check EFI Disk button permissions

Make sure that the `Add EFI Disk` button is disabled if the user does
not have the needed permissions.

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

ui: qemu/HardwareView: change heuristic perms to const

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

ui: qemu/HardwareView: eslint: enforce "no-mixed-operators" rule

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

ui: dc/MetricServerView: add onlineHelp to edit windows

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

ui: add necessary fields for influxdb http api

and en/disable them accordingly to the selected mode

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

status/influxdb: remove unnecessary comment

we already have that information in the reference docs, no need to
have it here as well

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

status/influxdb: implement influxdb 2.x http api

needs an organization/bucket (previously db) and an optional token
the http client does not fit exactly in the connect/send/disconnect
scheme, so it simply creates a request in 'connect',
does the actual http connection in 'send' and nothing in 'disconnect'

max-body-size is set to 25.000.000 bytes by default (the influxdb default)
and the timeout to 1 second (same as default graphite tcp timeout)

the token (if given) gets saved in /etc/pve/priv/metricserver/$ID.pw
it is optional, because the 1.8.x compatibility api does not need
authentication (in contrast to influxdb 2.x)

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

status/plugin: extend with add/update/delete hooks

like we do in it for the storage section configs

we will need this to store the token for influxdbs http api

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

status/plugin: extend send/_connect/_disconnect/test_connection

by providing the id or cfg to have better context in those methods
we will need that for influxdb http api

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

api: cluster/metricserver: prevent simultaneosly setting and deleting of property

like we do in other apis of section configs (e.g. storage)

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

pvereport: also add .sources files

while this format is probably not much in use currently, the plan is to make it
the default format in Debian, see 'man 5 sources.list'.

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

vzdump: avoid parsing already parsed option

When a job is updated, verify_vzdump_parameters() is called twice. This led to
parse_property_string being called with the already parsed option.

Reported on the pve-user mailing list:
https://lists.proxmox.com/pipermail/pve-user/2021-January/172258.html

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

pvereport: add apt sources

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

ui: ceph: fix keeping selection after operation

Found by having the console open and getting a message that 'me' (the
controller) does not have a method getRootNode()

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

ui: eslint: some more trailing comma fixes

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

ui: eslint: enforce "dot-notation" rule

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

ui: eslint: enforce "no-extra-boolean-cast" rule

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

ui: eslint: enforce "no-extra-parens" rule

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

ui: eslint: enforce "no-unneeded-ternary" rule

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

ui: eslint: fix various spacing related issues

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

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>