ui: fix Guest Summary Notes height

fixes 'infinite' height of the guest notes, by setting the height on the
container a level up

the notes view itself is already scrollable, so if it is too large,
there'll be a scrollbar

this does not change anything for templates, as there we show nothing
under the notes and thus there the infinite height is ok

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

vzdump: log the count of backups we pruned#

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

vzdump: make log for stop-mode guest-type agnostic

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

node config: limit description/comment length to 64 KiB

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

pve6to7: use file_get_contents

instead of File::Slurp

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

pve6to7: improve user.cfg parser

make it a bit more like the actual one - remove whitespace padding, use
same regex/split calls.

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

pve6to7: add check for pool permissions

the two checks make sure that:
* no user defined role 'PVEPoolUser' exists
* the user gets a hint for roles only containing Pool.Allocate and
    not Pool.Audit

a very simple parser for user.cfg was implemented to be able to
parse the (in pve 6 invalid) Pool.Audit permission

Signed-off-by: Lorenz Stechauner <l.stechauner@proxmox.com>

vdzump: log with what retention settings we prune

may not be obvious from the task log alone if we're relying on the
retention setting configured on a storage or the ones from local
node's vzdump.conf

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

ui: backup prune: add label for storage prune settings

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

ui: backup window: also reset removal fields when defaults API call fails

to avoid showing settings for the wrong storage.

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

fix #2745: ui: backup window: allow specifying remove parameter for manual backup

and also show the retention options that will be used for a given storage. A
user with Datastore.AllocateSpace and VM.Backup can already remove backups from
the GUI manually, so it shouldn't be a problem if they can set the remove flag
when starting a manual backup in the GUI.

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

ui: backup window: switch to two-column layout

in preparation to add more items in the future, e.g. removal settings, note
container.

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

ui: backup window: switch to proxmox input panel

It's not an Ext.form.Panel, so there is no fieldDefaults.

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

ui: backup window: set loading mask early enough

but not too early. Because of an ExtJS bug/limitation, it can only happen after
the window is rendered, so use an afterrender listener. Without setting the
mask there, the window will be active already before the storage selectors
change listener triggers, which can only happen after the storage selectors
store is loaded.

Made noticable by the new "filling in defaults" behavior, but the issue was
already present earlier, where the compression selector for PBS storages would
be disabled late, after the window was already active.

Also move the setValue call into the afterrender listener, so ordering is easy
to verify/more stable.

Reported-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

ui: backup window: also set initialDefaults variable in failure case

so that mailto and mode are not overwritten if the first /vzdump/defaults call
failed, but a later one succeeds.

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

ui: backup window: avoid issuing API call with null/empty parameter

could be triggered when there are no backup storages at all configured or if
the 'Backup now' button is clicked before the storage selector from the guests
'Backup' tab could load its store.

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

ui: parser: make comment single-line

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

api: nodes: fix typo

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

ui: storage: update hint about prune-backups default

and also remove the keepLastEmptyText, which too suggested the old default.

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

pve6to7: add check for CIFS credentials

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

pve6to7: improve vzdump.cron check

handle errors, and fix filename referenced in the warning.

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

pve6to7: add checks for backup retention options

Note that it's not possible to use read_vzdump_defaults() and storage_config(),
because they auto-converts maxfiles already.

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

ui: eslint fixes

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

ui: ceph: select Pacific as default release

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

add beta text with link to bugtracker

ui: TFA: use correct user-id value when creating TOTP QR code

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

cli: pveceph: remove outdated comment

Already removed by commit 00fa70bfd9acbcd4ff20d426c1c5bf9fb82d9ee9.

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

api: ceph: osd: create: rename size parameters

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

api: ceph: remove moved 'flags' calls

living below /cluster/ceph now. Frontend was adapted as part of the move in
commit 1225095df1c326ce55b71bac994a91a1a71168fb

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

api: ceph: remove obsoleted disks call

replaced by /nodes/{node}/disks/list

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

api: move cpu path into qemu/capabilities

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

api: nodes: remove deprecated upgrade parameter for vncshell

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

api: nodes: remove deprecated upgrade parameter for spiceshell

The change not to pass the 'upgrade' parameter in the frontend was made in
953f6e9bb3ab31ad074f1040d7143ecfa79e11e8 (the commit doesn't talk about it, it's
likely an accidental squash of two changes)

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

api: nodes: remove deprecated upgrade parameter for xtermjs

The switch to 'cmd' was made by commit af39a6f09651e15d1c83536e25493a2212efd7d3
in the pve-xtermjs repo and is included in 4.7.0

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

Revert "VZDump: add TARFILE to environment for hookscripts"

This reverts commit 5d0d14f179a740a13359d002f7e030665ee0f21e.

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

test: vzdump: adapt to new default

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

configs: vzdump: use prune-backups instead of the deprecated maxfiles

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

vzdump: remove deprecated size parameter

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

bump version to 7.0-2

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

ui: disk list: enable wipe disk button

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

ui: add task description for wipe disk

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

d/control: bump versioned dependencies on pve-access-control

to -2 containing 'Pool.Audit'

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

d/control: add libpod-parser-perl to B-D

'podselect' was moved there in bullseye.

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

d/control: add pve-eslint to B-D

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

fix #3402: add Pool.Audit permission

everywhere where Pool.Allocate was unnecessarly used it was replaced
with Pool.Audit.

`/cluster/resources` now returns pool infomation for guests only if
the requesting user has the Pool.Audit permission on the pool.

`/pool/` now returns only pools where the requesting user has the
Pool.Audit permission.

Signed-off-by: Lorenz Stechauner <l.stechauner@proxmox.com>

actually install pve6to7 tool

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

buildsys: www: touch incremental-lint in check target to avoid triggering twice

Without this the check and the, through the 'install' target
triggered, incremental lint target triggered a full eslint run.

Makes it similar to what PBS did from the beginning of eslint
inclusion..

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

buildsys: www: make 'all' target a no-op

preparatory steps for better make dependencies when executing the
lint check.

People can still just make the .js targets directly or use the
install target.

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

ui: ceph/Status: fix icon in status grid

there was a spurious leftover single quote("'"), that prevented
the correct css class to be inserted
(e.g., fa-exclamation' instead of fa-exclamation)

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

ui: form/CalendarEvent: remove displayField from config

in extjs 7.0, setting the displayField overwrites the displayTpl, which
we use here

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

ui: tree/SnapshotTree: use beforedestroy instead of destroy

on extjs 7.0, this triggers when the handler is not available anymore,
so use the beforedestroy event instead

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

ui: panel/ConfigPanel: return early on no selection

treelist technically allows deselection and this happens on destruction
in extjs 7.0 so simply return early to avoid errors accessing
components that do not exist anymore

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

ui: panel/ConfigPanel: change ui class to pve-nav

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

ui: ResourceGrid: move first update to afterrender

else with extjs 7.0, the first render does not happen yet, because
the grid is not visible at that time

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

ui: form/VMCPUFlagSelector: mark radio-buttons as 'not form fields'

else they might get picked up individually

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

influxdb: rework comment

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

fix #3440: influxdb: remove duplicate vmid tag

remove vmid from data part, it is already contained in object part.
this is accomplished by adding the parameter $excluded to
build_influxdb_payload().

Signed-off-by: Lorenz Stechauner <l.stechauner@proxmox.com>

bump version to 7.0-1

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

ui: nav: move bg-color override to CSS class

colors overrides really must not happen inline...

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

ui: ResourceGrid: correctly remove ResourceStore listener

while the function in '.on' and '.un' are identical, they are not the
*same* function, thus the '.un' does not really remove the listener,
and we have leftover references to the grid which means it will never
really garbage-collected

instead, use '.mon' on the grid, which automatically cleans up the
listeners

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

ui: dc/RoleView: add variableRowHeight

otherwise the grid is confused about the row height (since it is
variable here) and renders the scrollbars weirdly

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

ui: Parser: fix bind and dev mounts for lxc

match returns 'null' if the regex does not match, which is not
destructurable. so we have to save the match and check if it valid

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

pveceph: allow to select test repository on installation

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

enable ceph 16.2 pacific

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

buildsys: change upload/config dist to bullseye

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

Revert "buildsys: change upload/repo dist to bullseye"

pushed by mistake, please ignore this and the previous one

This reverts commit 4816c96931fb52993dac6ba966303ebf25d30a12.

buildsys: change upload/repo dist to bullseye

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

d/control: order ifupdown2 first so that it gets prefered

Lets still allow the old on for backwards compatibility, we may want
to phase that out with Bookworm or so

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

d/copyright: update years

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

d/rules: catch all dh rules goes to top

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

d/control: update dh compat level to 12

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

pve5to6: preparatory steps for when 7 is released

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

add pve6to7 WIP

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

pveceph install: fix fallback for default version

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

bump version to 6.4-7

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

enable eslint on build

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

ui: form/storage selector: re-validate after storage list was reloaded

two style fixes thrown in

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

ui: window/Migrate: avoid triggering another info request if already in-progress

use the node as key, so that a target switch would still trigger a
new one - else there's a slight chance that a user could get the
check out-of-sync (from another node).

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

ui: window/Migrate: eslint fixes and code cleanup/refactoring/modernize

Use the Proxmox.Async.api2 helper to avoid deep nesting

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

ui: window/Browser Settings: eslint fixes and code cleanup/refactoring/modernize

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

ui: window/FW Log Rate: eslint fixes and code cleanup/refactoring/modernize

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

ui: window/wizard: eslint fixes and code cleanup/refactoring/modernize

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

ui: window/ceph install: eslint fixes and code cleanup/refactoring/modernize

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

ui: window/bulkaction: eslint fixes and code cleanup/refactoring/modernize

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

ui: window/startup: eslint fixes and code cleanup/refactoring/modernize

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

ui: snapshot tree: eslint fixes and code cleanup/refactoring/modernize

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

ui: panel/StatusPanel: eslint fixes

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

ui: panel/RunningChart: eslint fixes

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

ui: panel/NotesView: eslint fixes

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

ui: panel/IPSet: eslint fixes

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

ui: panel/ConfigPanel: eslint fixes

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

ui: resource tree: eslint fixes and code cleanup/refactoring/modernize

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

ui: form/pci: add simple IOMMU group renderer

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

ui: qemu/PCI Edit: code cleanup/refactoring/modernize

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

ui: storage/PBS edit: eslint fixes

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

ui: qemu: eslint fixes

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

ui: node/ZFS-storage: eslint fixes and code cleanup/refactoring/modernize

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

ui: node/LVM-Thin-storage: eslint fixes and code cleanup/refactoring/modernize

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

ui: node/LVM-storage: eslint fixes and code cleanup/refactoring/modernize

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

ui: node/directory-storage: eslint fixes and code cleanup/refactoring/modernize

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