use HelpButton from widget toolkit

Proxmox.button.Help renamed the css class for the button styling, as
this class is only used rarely and the widget toolkit does not
provides a (shared) css file itself, just rename it here too.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Reviewed-by: Dominik Csapak <d.csapak@proxmox.com>

use window.PasswordEdit from widget toolkit

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Reviewed-by: Dominik Csapak <d.csapak@proxmox.com>

use TaskViewer & TaskProgress window from widget toolkit

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Reviewed-by: Dominik Csapak <d.csapak@proxmox.com>

use windowEdit from widget toolkit

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Reviewed-by: Dominik Csapak <d.csapak@proxmox.com>

use PendingObjectGrid from widget toolkit

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Reviewed-by: Dominik Csapak <d.csapak@proxmox.com>

qemu/Options: cleanup & use new features from ObjectGrid

Ther's a little delay with the revert button on pending changes, but
that was the case previously too, so don't worry for that in this
patch

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Reviewed-by: Dominik Csapak <d.csapak@proxmox.com>

grid/FireWallOptions: cleanup & use new features from ObjectGrid

switch add_boolean and add_integer row to a simple wrapper.
We really use it often and can save quite some lines with it so it
does not make sense to remove it completely yet.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Reviewed-by: Dominik Csapak <d.csapak@proxmox.com>

lxc/Options: cleanup & use a few new features from ObjectGrid

Use also same store update interval as qemu/Options

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Reviewed-by: Dominik Csapak <d.csapak@proxmox.com>

lxc/Resources: cleanup & use new features from ObjectGrid

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Reviewed-by: Dominik Csapak <d.csapak@proxmox.com>

dc/OptionView: cleanup & use new features from ObjectGrid

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Reviewed-by: Dominik Csapak <d.csapak@proxmox.com>

use ObjectGrid from widget toolkit

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Reviewed-by: Dominik Csapak <d.csapak@proxmox.com>

use KVCombobox from widget toolkit

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Reviewed-by: Dominik Csapak <d.csapak@proxmox.com>

use RRDTypeSelector from widget toolkit

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Reviewed-by: Dominik Csapak <d.csapak@proxmox.com>

use Checkbox from widget toolkit

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Reviewed-by: Dominik Csapak <d.csapak@proxmox.com>

use BondModeSelector & BondPolicySelector from widget toolkit

widget name stayed the same, so no need to change

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Reviewed-by: Dominik Csapak <d.csapak@proxmox.com>

use LanguageSelector from widget toolkit

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Reviewed-by: Dominik Csapak <d.csapak@proxmox.com>

remove unneeded SendKeyMenu

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

use Proxmox.Utils instead of PVE.Utils

some function are now in Proxmox.Utils instead, so we have to use that

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

use Button from widget toolkit

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

use Proxmox.button.StdRemoveButton where possible

most 'Remove' buttons we have used the same code pattern over and over,
with the StdRemoveButton we have a component which does all of this for
us

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

create some models

we want to use the getId method of some records, but for this
we need that the records are defined as an instance of a model,
so we create some and use them

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

use DNSView/DNSEdit from widget toolkit

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

add 'service' parameter to syslog api call

so that we can filter the journal by service

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

use ServiceView from widget toolkit

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

use Tasks from widget toolkit

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

change pve-invalid-row class to proxmox-invalid-row

because we will use that in the widget toolkit

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

use UpdateStore from widget toolkit

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

use JsonObject from widget toolkit

same xtype/alias, so there is nothing to do except remove it from
pve-manager

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

use RRDChart and RRDStore from widget toolkit

also remove old unused RRDView

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

add rrd models

they will be used with the rrdstore

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

remove unused MigrateAll window

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

use NodeAPT grid from widget toolkit

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

use NetworkView/NetworkEdit from widget toolkit

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

use TimeView and TimeEdit from widget toolkit

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

use IntegerField from widget toolkit

there we implemented deleteEmpty, which we can use in some cases.
this also fixes a bug in the ControllerSelector where
an empty field would be allowed (but should not)

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

use TextField from proxmox-widget-toolkit

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

use ComboGrid from proxmox-widget-toolkit

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

use pve-i18n package

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

add proxmox-widget-toolkit to pve gui

we will use components from proxmox-widget-toolkit rather than
directly from pve-manager

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

correct the upgradecmd for xtermjs

use the pveupgrade command directly without bash inbetween,
the incorrect quoting led to '--shell' not being passed to
pveupgrade and closing the connection

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Tested-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

bump version to 5.1-43

add spec-ctrl cpu flag checkbox

also make the mechanism for the flag checkboxes generic

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Tested-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

add defaultSize parameter for DiskStorageSelector and set it to 8 for lxc

we had it previously on 8GB for containers, but with the refactoring,
this got lost. this patch changes this back

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

bump version to 5.1-42

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

readability fixup

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

add PCID checkbox to ProcessorEdit

but not in wizard, and default is off
(because we do not know if the cpu supports it)

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Tested-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

fix #1605: reflect backend values of minimum memory limit

The backend's minimum value for container memory is 16M,
not 32.

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump version to 5.1-41

pveupdate - always run check for available updates

prevent double ticket api call

commit:
9d4aa0edbbbd42429d7f46eae72766eb6d87c78e

added the defaultButton property to the loginwindow, but we missed that
pressing enter in the passwordfield/userfield led to a double ticket
api call.

while it did not any harm, remove it, so that we only have a single
ticket api call

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Acked-by: Wolfgang Bumiller <w.bumiller@proxmox.com>
Fixes: 9d4aa0edbbbd ("add defaultButton to loginWindow")

use activate instead of show

we changed this almost everywhere in the transition from extjs4 to
extjs6, but we forgot one

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

Set Console Button icon style at definition time

This will allow the 'Upgrade' button to get an icon too.

Signed-off-by: Emmanuel Kasper <e.kasper@proxmox.com>

Use the new xterm.js icon with the corresponding Menu Item

Signed-off-by: Emmanuel Kasper <e.kasper@proxmox.com>

Add new icon for xtermjs consoles, based on xtermjs logo

Signed-off-by: Emmanuel Kasper <e.kasper@proxmox.com>

jslint: remove trailing comma after last objects property

Signed-off-by: Emmanuel Kasper <e.kasper@proxmox.com>

cleanup

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump version to 5.1-40

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

show storage usage in tree

this adds a little usage indicator in the tree for all storage definitions,
so that one can easily see how much of a storage is used

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Acked-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

remove (near)full status for storages

since this is not a good default, we do not want the user
to have to configure those limits for each storage, and
a warning triangle was confusing for multiple users,
we remove this again

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Acked-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

Send an email when a replication job fails.

A email notification will be send for each job when the job fails.
This message will only send when an error occurs and the fail count is on 1.

Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Acked-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

Indentation cleanup.

Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Acked-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

add xtermjs to consolebutton

also rework the button.
move most things in the static configuration

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

add xtermjs flag to VNCConsole iframe

so that we can open an xtermjs iframe instead of novnc

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

add new xtermjs viewer window to Utils

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

add xtermjs to pveproxy dirs

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

add termproxy api call for nodes

and add dependency for pve-xtermjs

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

api: nodes/subscription: implement reasonable access rights

Allow users which have Sys.Audit on a specific node to get the
subscription status and those with Sys.Modify to set and check
(update) it.

This mirrors the required permissions from other node specific
actions, e.g., APT (package management).

We always showed the Subscription Panel and all its elements in the
WebUI, so no need for change there.

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

bump version to 5.1-39

Fix #1589: auth_handler: raise exception on init_request error

cfs_* methods cann now die (rightfully so) when the IPCC endpoint is
not connected, or another grave IPCC error arised.

As we did not catch those problems in the RPCEnvironments
init_request method, which loads the user config, this got
propagated to the anyevents auth_handler call in its
unshift_read_header method where then all errors where processed in
the same way => with an unauthorized response logging an logged in
user out.

So catch this error and raise an internal server errror exception
instead. Anyevent needs some minor modifiaction in a separate patch
to handle PVE::Exceptions correctly, so this is the partial fix for
bug #1589

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

update catalan translation

Add a proper confirmation message when we detach an used disk

Signed-off-by: Emmanuel Kasper <e.kasper@proxmox.com>

Fix #1526: Use 'detach' instead of 'remove' when the disk is used

Signed-off-by: Emmanuel Kasper <e.kasper@proxmox.com>

Fix JSLint type confusion warnings

The type confusion warnings arose from string and integer concatenation,
and object properties of the name which were alternatively set with
strings and integers.

Signed-off-by: Emmanuel Kasper <e.kasper@proxmox.com>

add versioned dependency on libpve-storage-perl

for API changes needed for new DiskSelector

bump version to 5.1-38

pveceph: add missing use statement

ceph: create mon: fix & improve check if IP is in public net

If a CIDR gets passed to Net::IP it is expected to not be from the
middle of  an subnet, i.e., 192.168.1.12/24 is *not* OK but
192.168.1.0/24 would be OK.

As the Network/interfaces files also accepts CIDR notation for the
'address' param (now also for IPv4) this let to problems in our node
monitor IP detection code, which used the interface file and Net::IP to
find any address from the ceph public network.

So change to our newer helper PVE::Network::get_local_ip_from_cidr to
get all configured and ready (=up) IPs from this network.

Also handle the case where multiple networks where returned, add a
parameter to allow specifying one of those and ask the user to do so.

If no public network is configured and no mon-address parameter was
passed, we fall back to the remote node IP of the node, as was done
previously. We expect that the user only overwrites the mon-address
if he knows what he do and omit checks here.

bump version to 5.1-37

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

ceph: create pool: check for quorum at start

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

Add each host SSL fingerprint to each node status

Signed-off-by: Emmanuel Kasper <e.kasper@proxmox.com>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

fix #1578: api/resource hastate: fallback to resource cfg state

With ignored or still queued services we have no hastate for a
service in the manager status available.

As we use hastate in the web UI to determine if a service is
configured for HA this could lead to confusion there.
For example, the VM/CT 'Manage HA' window thinks tries to add the
service again if its in the 'ignored' state, and then the backend
errors out because it is already configured.

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

Do not translate Unix/Linux specific terms

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

Do not translate PC hardware specific terms

The SKU (Stock Keeping Unit) field was never translated anyway in our po files,
probably because nobody knew what the initials meant.

Signed-off-by: Emmanuel Kasper <e.kasper@proxmox.com>

Do not translate Qemu specific terms

Multiqueue is the proper name as used by Qemu, as seen in
https://www.linux-kvm.org/page/Multiqueue

Signed-off-by: Emmanuel Kasper <e.kasper@proxmox.com>

Do not translate IT specific terms

Signed-off-by: Emmanuel Kasper <e.kasper@proxmox.com>

Do not translate PC hardware specific terms

Signed-off-by: Emmanuel Kasper <e.kasper@proxmox.com>

Do not translate Ceph specific terms

Signed-off-by: Emmanuel Kasper <e.kasper@proxmox.com>

make ssh connection transparent

do not use an escape character for the host shell

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

access log rotate: try to reload not restart services

with the use of systemd the /etc/init.d/pveproxy restart call got
intercepted and redirected to systemd, which did an restart.
But, that is a behvaior change to the intention of the init.d script,
which would not restart but reload the service.
Thus change this to systemds equivalent:
systemctl try-reload-or-restart pveproxy.service

Which tries to reload, if the unit supports it, and only if that does
not worked it restarts.

Same for spiceproxy.

Reported-by: Stefan Priebe <s.priebe@profihost.ag>
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

buildsys: add excludes to dh_fixperms

Instead of fixing up permissions manually as the make
install target is actually responsible for this (and already
doing it).

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

remove unecessary semicolons

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

whitespace cleanup

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

improve tree/grid icons

we improve the icons in the tree and the resource grid by
differentiating between cluster online/offline status and no rrd data

when we have no rrd data from a node/storage, instead of showing a
red x (which is scary) even if the node is reachable by corosync (which
confused quite a bit of people, because we show all nodes as online in
the datacenter summary), we show the node/storage with a '?'

this signals that something is wrong with this node, even if we can
reach it via cluster methods

this rewrite of the logic includes a refactoring of the method
of getting the icon, because we want the same icons in the tree and the
grid, and an optimization on how we use the css classes
(introducing a x-grid-custom-icon class)

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

add new tree and grid classes for the 'unknown' status

this displays a '?' similar to the check/cross, to show something
is wrong, but we do not know if its offline

also cleanup the classes a bit, so we do not have to have so many
different ones

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

add a new status column in the resourcestore

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

add default 'unknown' status for vms and containers

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

include a 'status' field for storages

by default it is unknown, if the storage is >= 60% full, it is
'nearfull', if it is >= 90% full, it is 'full'

if we have any other information, it is 'available'

we can use this information for the tree

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

add status to node stats

by default it is unknown,
if it is a standalone node its online
else it is either 'online' or 'offline' depending on the corosync status

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

reset FileSelector grid when changing the storage

otherwise, when the new storage returns an error, we show the content
of the old storage

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

use DiskStorageSelector in lxc/MPEdit.js

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