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>

ui: node/command menu: eslint fixes and code cleanup/refactoring/modernize

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

ui: node certs: use new loadFile helper to avoid confusion

we did not handle those cert related files like SSH keys, so do not
suggest so.

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

ui: utils: split out general "load local file" helper

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

ui: node/certs: eslint fixes and code cleanup/refactoring/modernize

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

ui: node certs: make delete button safer, add confirmation

IMO the behavior was really dangerous prev., possibly causing
downtime of API accessing stuff with a single mistaken click.

Follow our general pattern and ask for confirmation

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

ui: lxc/context cmd: factor out confirmed CT command

unifying CT and VM command menu would be pretty easy now, but I need
to wrap the eslint stuff up, so cannot chase all tails..

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

fixup! ui: qemu/context cmd: code cleanup/refactoring/modernize

ui: lxc/context cmd: code cleanup/refactoring/modernize

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

ui: menu/item: eslint fixes and code cleanup/refactoring/modernize

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

ui: menu/template: hide clone/migrate button if not possible

Depending on permissions and cluster state those action may not make
sense to be shown.

This streamlines behavior with the one we already have for
non-templates.

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

ui: menu/template: eslint fixes and code cleanup/refactoring/modernize

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

ui: qemu/context cmd: factor out confirmed VM command

could be also a generator (higher order fn), but it's easier to
grasp the scope and the like this way

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

ui: qemu/context cmd: code cleanup/refactoring/modernize

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

ui: lxc/Network: eslint fixes and code cleanup/refactoring/modernize

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

ui: lxc/Mountpoints: eslint fixes and code cleanup/refactoring/modernize

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

ui: parser: fixup printLxcMountPoint object iteration

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

ui: lxc/CreateWizard: eslint fixes and code cleanup/refactoring/modernize

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

ui: ha/Resources: eslint fixes and code cleanup/refactoring/modernize

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

ui: ha/GroupView: eslint fixes and code cleanup/refactoring

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

ui: ha/Group Edit: use simple map+join to serialize nodes-prio

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

ui: ha/Group Edit: eslint fixes and code cleanup/refactoring

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

ui: grid/Resources: eslint fixes and code cleanup/refactoring

this is a major cleanup, using the current ExtJS and ES2018 features
allows to do a lot more concise and nicer.

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

ui: grid/Replication: eslint fixes and code cleanup/refactoring

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

ui: grid/Firewall: eslint fixes and code cleanup/refactoring

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

ui: qemu/OS defaults: code cleanup/refactoring

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

ui: form/Guest Selector: eslint fixes and code cleanup/refactoring

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

ui: form/View Selector: eslint fixes and code cleanup/refactoring

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

ui: form/USB Selector: eslint fixes and code cleanup/refactoring

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

ui: form/API Token Selector: eslint fixes and code cleanup/refactoring

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

ui: form/Priv.Selector: eslint fixes and code cleanup/refactoring

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

ui: form/Node Selector: eslint fixes and code cleanup/refactoring

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

ui: form/Global Search: eslint fixes and code cleanup/refactoring

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

ui: form/Controller Selector: eslint fixes and code cleanup/refactoring

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

ui: parser: fixup printing LXC network

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

ui: eslint auto-fixes

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

ui: eslint: Miscellaneous eslint fixes

Function syntax, raw words, string templates, no prototype builtins

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

ui: eslint: State equalities more precisely

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

ui: eslint: Remove useless escapes

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

ui: eslint: Functions may not be completely empty

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

ui: eslint: No unused or duplicate variables

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

ui: eslint: fix usage of 'me'

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

ui: dc: log, options, support: eslint fixes and code cleanup/refactoring

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

ui: dc/StorageView: eslint fixes and code cleanup/refactoring

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

ui: dc/UserEdit: eslint fixes and code cleanup/refactoring

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

ui: dc/Tasks: eslint fixes and code cleanup/refactoring

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

ui: dc/TFA Edit: eslint fixes and code cleanup/refactoring

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

ui: dc/Summary: eslint fixes and code cleanup/refactoring

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

ui: dc/FW Security Group: eslint fixes and code cleanup/refactoring

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

ui: dc/RoleView: eslint fixes and code cleanup/refactoring

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

ui: dc/NodeView: mark only usage over 75% with a warning

As 60% is pretty low, and normally still OK and even wanted to have
(no need for having an over-dimensioned setup with lots of unused
resources).

Note, this is not a off-by-one, I want to compare real bigger > 75%

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

ui: dc/NodeView: eslint fixes and code cleanup/refactoring

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

ui: dc: health, config, guests; eslint fixes and code cleanup/refactoring

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

ui: dc/corosync-link: eslint fixes and code cleanup/refactoring

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

ui: dc/cluster: eslint fixes and code cleanup/refactoring

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

ui: dc/Backup: eslint fixes and code cleanup/refactoring

this one would deserve much more, especially the newer backup job
detail and not-backed-up view should be split out and done in a
declarative way (view view-controller)

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

ui: dc/ACMEPlugins: eslint fixes and code cleanup/refactoring

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

ui: dc/ACLView: eslint fixes and code cleanup/refactoring

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

ui: data/ResourceStore: eslint fixes and code cleanup/refactoring

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

ui: buttons: eslint fixes and code cleanup/refactoring

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

ui: workspace: eslint fixes and code cleanup/refactoring

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

ui: utils: eslint fixes and code cleanup/refactoring

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

ui: state provider: try to find string encoded values in dictionary

My browser here is pretty fixated on the history, and due to some
unknown reason I got a history fragment string like:
`#v1:=0:=18:=4:::::::` so the "hashes" got into strings, and I was
not able to move them back.

Adding a match here to always try reverse-mapping the hash to the
original input helps to fix that here.

We'd only run into issues if we'd use a integer-string as id, but we
really don't (IIRC not even allowed in extjs) so this is safe to do.

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

ui: state provider: eslint fixes and code cleanup/refactoring

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

state provider: drop confirm-query when going back over initial history

eslint complains about the window.confirm call, and neither I nor
some other colleagues here did see that dialogue ever in practice.

The confirm here does not adds any real value, as all other back
navigation are unprotected, but have similar potential in loosing
state.

One can always navigate forward via their user agent and for console
(noVNC, xterm.js) we have now a separate check.

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

ui: toolkit: eslint fixes

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

ui: parser: eslint fixes and refactoring

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

ui: parser: drop old openVZ parser

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

fix #3426: ui: parser: adapt parseSSHKey to accept "sk-" prefix

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

ui: cloudinit: use PVE.Parser.parseSSHKey

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

fix #3430: handle hook script paths better

if passing the hook script command as string, it might get interpreted
as shell command with side-effects. this is pretty harmless, since only
root is allowed to set the script parameter anyway, but making it more
robust and future-proof does not hurt.

tested with a reproducer of "/bin/echo $(touch $(whoami))" as script
parameter, with a file with that name existing, being executable and
having the following contents:

----8<----
echo "hello from hook script"
---->8----

without this change, the hookscript itself is not executed, but
'/bin/sh -c "/bin/echo $(touch $(whoami)) job"' and similar calls are,
which cause the file 'root' to be touched in the current working
directory of the vzdump process (or task worker).

with this change, the file is executed as is without any side-effects of
shell commands in the file name, and the 'hello from hook script' lines
are printed whenever the hook script is called by vzdump.

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

report: top: also print command line arguments

at least the first 512 characters, which is the biggest width one can
instruct `top` too.

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

report: top: list more processes and order them by CPU time spent

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

report: re-organize: add systemd-load, unify bios/pci in HW one

Also rename disks to blockdevices, as especially with the iSCSI stuff
the listed devices do not necessarily need to be real disks.

Grouping current(ish) load info in it's own system-load section seems
to be a better fit in general too.

bios/pci stuff is all hardware, so group it there to avoid adding to
much sections.

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

report: dir2text: print what we process on STDERR

to avoid those "lonely" "OK" prints for those

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

report: avoid extra oder array, define it inline

Makes all report-def entries hashes which allows to drop some
handling for the ARRAY ref case.

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

report: avoid adding to report by side-effects, be more explicit

And move the helper methods up and scope them to module-local only

Uses the fact that perl methods return the last statement, so the
dir2text sub closures in the command list do not need to be changed

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

report: set timeout and PATH env only locally

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

report: whitespace cleanup

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

pvereport: get pressure stall information

PSI can be queried in /proc/pressure/{cpu,io,memory} for the
corresponding resources. this helps us track down disruptions caused
by resource overcommitment.

Signed-off-by: Oguz Bektas <o.bektas@proxmox.com>

api: pools: fix typo/syntax error

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

api: pools: whitespace, indentation and code cleanups

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

api: pools: whitespace, indentation and code cleanups

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

bump version to 6.4-6

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