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>

ui: console: add beforeLoad callback

we sometimes need to adapt parameters, but in a lot contexts the
component is already created, and the activate event order of
components may make this hard to do in a deterministic way.

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

api: nodes: allow root to pass arbitrary arguments to commands

If the command itself allows it, which normally means it has good
verification of passed arguments.

We may want to re-evaluate security here if we allow execution for a
group of non-root users.

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

api: cluster/ceph: allow version metadata call on non-ceph nodes

the information comes only from the key value store in the pmxcfs, so
we do not actually require ceph to be installed.

So only check if ceph is locally initialized and create the rados
connection after the early return when only versions scope is set.

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

ceph services: cleanup variable names

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

ui: lxc: code cleanup ssh key upload

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

ui: metrics: set default for port

while it may not be always correct, we can only win by setting a
reasonable default, taken from

* graphite:
  https://graphite.readthedocs.io/en/stable/feeding-carbon.html#the-plaintext-protocol
* influxdb, udp port:
  https://docs.influxdata.com/influxdb/v1.8/administration/ports/

Also, changing that is easy for the admin and can always be done
afterwards.

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

ui: metrics: move server and port on same line

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

ui: metrics: style cleanup: defined xtype first, name second

those are both the most important for what the field is, so they
should come first.

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

ui: metrics: advancedColumn1 may now be empty

we have now improved detection in widget toolkit, not yet packaged
though

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

ui: metrics: use icons for add server

we could use the official influxdata one, they provide it in their
branding guide - and the use here would seem to comply with their
rules, and would fall under fair-user, probably.

Still, not 100% clear for me and Graphite is not easy to find a good
one with clear rules, so just reuse the bar-graph Font Awesome one,
better than nothing (looks empty-wrong)

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

ui/api: metrics: move server to subpath, post to path with ID

We actually wanted to use that scheme for more new API paths, lets
see if it is really fitting starting with this.

Use the new widget-toolkit submitUrl helper to add the ID on create.

And unify the edit/create window creation, which may fit better in a
separate commit, it's quite small and was to cumbersome to untangle,
so just go against my one rules here... :(

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

ui: metrics: add server base edit window

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

ui: add MetricServerView to Datacenter

adds ui support for configuring the External Metric servers.
for now, influxdb and graphite edit windows do not share code,
it would be ideal to do that once we add more plugins

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

api: drop Config from MetricServer module name

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

api2/cluster: add 'metricserver' api endpoints

modeled after our typical api endpoints for sectionschema configs

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

influxdb: avoid three line comment if one is enough

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

ui: manager: use eslint for lint target

not yet linting by default, we need to fix a few thousands linter
errors before that (most are trivial though)

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