small nit follouwp: use $var->[0] to acccess array ref element

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

vzdump: test: add first tests to the guest include logic

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

vzdump: make guest include logic testable

As a first step to make the whole guest include logic more testable the
part from the API endpoint has been moved to its own method with as
little changes as possible.

Everything concerning `all` and `exclude` logic is still in the
PVE::VZDump->exec_backup() method.

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

vzdump: format size: future proof against huge VM disks

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

add model information to ZFS create window

and improve the layout by adding flex values and increasing the width.

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

ui: migrate: sort migrations hints by text column

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

Allow setting no target storage and make it default

so the current disk locations can be preserved even if
there are multiple local disks. And users don't have to
manually select the current storage if there is only one
local disk.

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

Don't show empty parentheses when size is not known

The size of VM state files and the size of unused disks not
referenced by any snapshot is not saved in the VM configuration,
so it's not available here either.

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

drop jslint lines

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

ui: ceph/status: cleanup legacy version handling

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

bump version to 6.2-6

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

ui: migration: add maxHeight to migration window

to prevent indefinite growth in case of e.g. many local disks

Signed-off-by: Tim Marx <t.marx@proxmox.com>

ui: backup grid: use boxLabel type and VMID

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

ui: add checkbox for vmid filter for backupview

instead of hardcoding the text 'type-id-' into the searchbar
to accomodate for the additional size, add an overflowHandler
to the toolbar (for very small display sizes)

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

ui: ceph/pool: allow autoscale on create

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

ceph: extend pveceph pool ls

to present more data on pools and a nicer formated output on the command
line.

Signed-off-by: Alwin Antreich <a.antreich@proxmox.com>

ceph: extend the pool view

to add the pg_autoscale_mode since its activated in Ceph Octopus by
default and emmits a waring (ceph status) if a pool has too many PGs.

Signed-off-by: Alwin Antreich <a.antreich@proxmox.com>

ceph: avoid fork/exec overhead for every ceph_cluster_status call

detect heuristically if we need to get the full mon and mgr map extra
for Octopus

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

ui: ceph/StatusDetail: cleanups

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

Make PVE6 compatible with supported ceph versions

Luminous, Nautilus and Octopus. In Octopus the mon_status was dropped.
Also the ceph status was cleaned up and doesn't provide the mgrmap and
monmap.

The rados queries used in the ceph status API endpoints (cluster / node)
were factored out and merged to one place.

Signed-off-by: Alwin Antreich <a.antreich@proxmox.com>

ui: followup: use title case

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

sdn: vlan : add mtu field

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

sdn: add vlan aware option to vnet

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

sdn: use new helper methods to update config

and do a breaks on older network package as we do not depend on it
due to it being an optional/experimental feature, so reverse the
depends with the breaks.

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

PVE/VZDump.pm: use new --quiet option for proxmox-backup-client prune

ui: fix HotplugFeatureSelector

we recently changed the setValue behaviour of the inputpanel and editwindow
(we now set all fields with the same names), which leads to wrong
behaviour here

use a different name for the internal checkboxes to avoid this

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

ui: fix missing change from 'pve-' to 'pmx-' models

we forgot to change these

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

ui: dc/summary: sync height of first two panels with others

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

d/control: bump version of proxmox-widget-toolkit dependency

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

css: remove icon colors

they are now in the widget-toolkit

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

ui: Replication: use render_duration from widget-toolkit

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

api: improve node index with missing/broken cert

since this API endpoint is used for the node selector in the GUI, which
causes quite widespread breakage.

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

pveceph: make installing luminous on PVE 6 experimental

It's only available for the upgrade from 5.4 anyway

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

pveceph: allow installing octopus as experimental option

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

bump version to 6.2-5

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

ui: grammar followup

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

ui: node/subscription: eslint fixes

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

ui: fix system report window indentation + cleanups

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

ui: order remove button later and add separator to sys report

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

subscription delete: followup fixes

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

api, ui: allow to remove subscription

Signed-off-by: Moayad Almalat <m.almalat@proxmox.com>
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

ui: vm/options: increase localtime edit window width

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

Fix 2718: GUI: handle localtime option correctly

The backend treats an undefined value and 0 differently. If the option
is undefined, it will still be set for Windows in config_to_command.
Replace the checkbox with a combobox covering all options.

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

fix #2574: ui: permission-path selector: push each unique path only once

In clusters a storage was pushed as many times as there are nodes
using it. Ensure we only push a path once, do this with a memory
oriented solution, i.e., build a extra object to remember what we
already pushed. This is preferred over a me.findExact call as it's a
o(n) vs o(n^2) solution compute wise and we make the user wait if
this would need long, so reducing compute time over memory use is
here a good choice - even if we will seldom run into situations where
this actually will make a difference with such small data counts.

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

ui: some eslint fixes

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

ceph: get_local_version: allow optional v prefix for version number

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

use RealmCombobox from widget-toolkit

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

ui: use RoleSelector from widget-toolkit

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

ui: qemu: stop pending changes in correct scope

I saw some cases where the pending changes just continued to get
triggered for some VMs, even if they shouldn't anymore - effectively
resulting in 3 - 4 concurrent pending changes call at a time.

It could be this, or not, but it's cleaner this way so go for it

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

ui: refactor pool edit window

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

ui: QinQ: indentation and stricter limits

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

qinq: add vlan-protocol option

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

sdn: restrict zone && vnet name to 8 characters

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

vnetview: remove mtu field

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

ui: TokenEdit: change default focus to tokenide field

Avoids that the user selector always pops up and tokenid is normally
more likely to change as the userid gets pre-selected to the one from
the logged in user.

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

fix #2747: ui: API token create: always reset base submit URL

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

d/control: bump versioned to proxmox-widget-toolkit

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

ui: remove Realm model and RealmComboBox

and use it from widget-toolkit

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

ui: use PMX.image.Logo from widget-toolkit

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

acme: account: hide TOS checkbox during load and reuse field references

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

ui: acme: label account name field as such

As else some people could be confused and put in their name, as below
is the email field, and name+email are quite often pairs.

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

ui: ACMEAccountCreate: improve layout of account dialog

by increasing the space for the link, and using a boxLabel instead
of fieldLabel, which has better spacing for longer text

also move the e-mail before the directory to have all textboxes together

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

ui: dc/ACME: fix not refreshing api column

When using a diffstore, we have to specify all fields that are
displayed, otherwise the store does not know which fields to check for
change

for the acme plugin view, 'api' was missing

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

d/control: versioned dependency for pve-doc-generator

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

bump version to 6.2-4

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

ui: sdn: move anycast addresses to advanced items

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

sdn: add onlineHelp references and various nits

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

api: apt/versions: track libpve-network-perl as optional package

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

re-bump do 6.2-3

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

ui: memory field: third time's a charm...

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

bump version to 6.2-3

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

update aplinfo

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

ext. metric server: workaround stupid MTU problem..

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

metric server: improve flush on big data updates

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

ui: cluster join: hide all fields in assited mode until we got some info

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

ui: clusterjoin: fix cluster network field visibility

Signed-off-by: Tim Marx <t.marx@proxmox.com>

ui: cluster: code cleanups

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

dc/Realms: fix adding of new ldap realm

we cannot pass 'delete' on create api call, and we have to make
sure that 'default_opts' and 'sync_attributes' are alwyas available,
since they are used in onGetValues
(they were only created during setValues, which is not called
when adding a new realm)

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

ui: also set minimum memory's maxValue to higher default initially

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

update aplinfo

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

bump version to 6.2-2

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

api acme: reduce after validation-request delay, print status on err

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

d/control: bump versioned build-dependency to doc-generator

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

ui: acme view: use "Challenge Plugins" name

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

ui: acme plugin edit: allow to overwrite label and add a hint

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

ui: acme dns plugin edit: wider labels, = as separator

Some API endpoints hvae very long variable names..

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

ui: acme dns plugin edit: add online help

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

fix #2708: ui: increase default memory in VM create wizard to 2GiB

as nowadays a lot of VMs won't even start the installer with 512 MB.

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

ui: dc/ACMEClusterView: show TaskProgress on account removal

this is not a synchronous api call, so open a taskprogress window
and reload after the task is done

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

ui: dc/ACMEClusterView: load the correct store on reload

if we use a diff/update store combo, we have to load the updatestore
not the diff store, else we get spurious empty grids

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

ui: ACMEAccountCreate: make name only optional if no default exists

if the 'default' account exists, make the name field required and
remove the emptytext

get the information by querying the grid store. this may be not
up-to-date, but it is less intrusive that an extra api call that
blocks the window

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

ACME: add comment for implicit standalone plugin fallback

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

ACME: fix fallback to implicit standalone plugin

we need to parse the config even if it does not exist - it will return
the 'standalone' entry that's needed to be backwards compatible with
existing setups.

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

ui: node/{ACME, Certificates}: add stopUpdate on destruction

else the stores never stop updating

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

ui: node/ACME: only enable order button when it should work

to order a ceritificate, we need at least one configured domain, and
the configured account (or default) must exist

so track the domaincount in the viewmodel and introduce a
'canOrder' formula which is only true when domaincount > 0 and
account is set (if the configured account does not exist, or no account
exists at all we set 'account' to 'null')

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

ui: node/ACME: use accountselector for verification

instead of using API2Request manually, just reload the store of
the accountselector and check if the configured account is in it

this should fix the spurious loading mask of the panel when
loading the accounts

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

ui: node/ACME: fix some eslint errors/warnings

* unneeded brackets for arrow function
* unused variables
* me not defined
* trailing commas

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

ui: dc/ACMEClusterView: use a diff/update store combo for the grids

so that they are get automatically reloaded with the default interval (3seconds)

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

ui: ACME: add emptyText and add minHeight

without the minHeight, the panel does resize weirdly on the first load

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

NodeConfig/get_acme_conf: make domains always a hash

on all call sites, we assume $cfg->{domains} is a hash, but if we do not
have any domains configured, that fails with
'Can't use an undefined value as a HASH reference at ...'

so always make domains a hash to avoid this

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