bump version to 5.4-6

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

d/control: bump version dependency of libpve-access-control

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

api/nodes journal: fix parameter types and add minimum

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

partially revert "api/nodes journal: add and enforce parameter conflicts"

This partially reverts commit f9b08743a5a2322427ebdce78eb84c990bae7e31
as we had some wrong assumptions about lastentries and the other
params, so just note conflicts in the description but let the tool
itself make the checks

This reverts commit f9b08743a5a2322427ebdce78eb84c990bae7e31.

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

fix #2185: add option to change nfs version on gui

this enables us to specify an nfs version while editing/creating an nfs
mount. it used to default to vers=3 without the ability to change it in
gui. now it supports: 3, 4, 4.1 and 4.2

it should also be possible to add further options in the future (rsize,
wsize, timeo, etc.) on this screen.

Co-Authored-by: Dominik Csapak <d.csapak@proxmox.com>
Signed-off-by: Oguz Bektas <o.bektas@proxmox.com>

api/nodes journal: add and enforce parameter conflicts

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

api/nodes: add journal to index

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

vzdump: allow plugins to set guest resume time

This refactors things a bit to avoid having the same two lines in 3
places and allows the plugin to set the "guest was resumed" time
stamp at the point it really was resumed, not only once the backup
completed (see #503). Further, if a plugin prints it's own
"resumed/running after X seconds" message, it can unset the vmstop
time and thus avoid printing the message twice.

related to a fix for #503

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

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

ensures we've the new Journal panel available

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

gui: node: change syslog to JournalView and journal api

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

node: add journal api

this uses the new journalreader instead of journalctl, which is a bit
faster and can read from/to cursor and returns a start/end cursor

also you can give an unix epoch as time parameters

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

fix #2197: ui: editing CDROM doesn't require VM.Config.Disk rights

A user which just has the default role PVEVMUser on a VM is able to
edit the CDROM, either to no media or if storage permissions are
available also to other ISOs, through the API. So reflect this again
for the WebUI. We could add a check for "VM.Config.CDROM", but that's
not really worth, if we keep in mind that the Ext.Caps state is
heuristically only.

fixes commit d35b5b2afbe9287a90129d861cf6fa27b91c1ed3

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

ui: NodeSelector: check status field for offline nodes

the memory stat is not available if the user has no audit privilges on
/nodes/NODENAME but the status is always available

also it makes simple more sense to check if it is online not only if the
memory value is there

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

add new parameters cidr(6) to the network api

to also allow to set it directly and not via address/netmask

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

followup: do not break on mixed, as one with no level could follow

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

followup code cleanup

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

fix #2183: fix blinking subscription and show unknown status

we skip offline nodes, check for nodes without subscription,
and finally check for different levels

if for some reason we do not get any subscription status (should not happen)
we show 'unknown'

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

ui: TFAEdit: use new tfa api call

with the api call to userid/tfa we get the users tfa type as well
as the realm tfa type, so we can replace the call to the realm
with this

to properly show the loadmask, we want to initiate the api call when
the window is already shown, the 'show' event works for this

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

ui: TFAEdit: change tfa_type default to null

If a dependency of a formula returns undefined, it will not get updated,
even if the other parts of the formula would work.

So we change the default to 'null' which gets handled differently,
but serves the same purpose for us.

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

follouwp: reduce duplicate code

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

ui: storage: handle empty response in case of file upload error

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

bump version to 5.4-5

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

api: handle undefined shellcmd parameter better

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

api: ceph/osd: conciser metadata array to hash mapping

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

followup: refactor & code cleanup

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

adapt osd api call for ceph nautilus

ceph nautilus changed the structure of 'pg dump osds'
they moved the data one level below

parse both new and old format, and bail if it returns anything else

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

Fix #482: Add timestamps to backup create task log

Adding timestamps to the log messages facilitates troubleshooting.

We only log this in the task log, as the syslog and the backup log
(stored together with the backup on the target storage) already have
date/time prefixed, so only the task log missed this info in case of
multiple backups tasks for a single job.

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

call cfs_update() before retrieving cluster status

otherwise this potentially returns outdated information (like the
cluster being quorate when corosync has crashed on all nodes 5 minutes
ago).

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

fix #1910: urlencode usernames in UserEdit

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

ui: tfa: handle if TFA setup with no specific type set

if we have no info about TFA in the userview (x as key instead of
x!oath or x!u2f) we disabled the whole window and the only action
was to delete

instead show all options, so the user can overwrite the setting

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

bump version to 5.4-4

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

ui: tfa edit: fix initial state of delete button

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

vzdump: code cleanup empty newlines

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

vzdump: use strftime for backup basename assembly

we already use POSIX strftime a lot in the stack, so nothing new,
also just use the perl built in localtime

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

fix #263: only include node rrd stats if user has Sys.Audit

It makes sense to not give users without Sys.Audit permissions to
much information over a node and this is relatively easy and cheap to
check and enforce at those two points.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Tested-by: Oguz Bektas <o.bektas@proxmox.com>

followup: code cleanup and print error

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

pveceph: install ceph service print sucess message

This patch adds a success message on successful ceph.service
installation. And adds a newline to make a successful ceph package
installation more visible.

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

fix #2177: use getValue() to get otp value

as '.value' isn't always valid

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

tfa: disable, not mask, other cards if tfa is already setup

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

tfa: comment that TOTP secret is meant to be base32

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

make jslint happier

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

ui: tfa: set initial tab based on current tfa type

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

ui: tfa: mask panels for not currently used types

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

ui: user view: show tfa type name

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

www: parser: add parseTfaType helper

The tfa keys can now contain the type as additional
information, so let's parse it out.

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

d/control: bump widget-toolkit version dependency

It's not that nothing would work without it, but ceph cannot be
configured via gui, and knowing that some user want to do strange
things with mixing package versions, we save us some hassle just
recording this to ensure one cannot run into this..

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

fix ceph install wizard not accepting ipv6

changes the vtype to the newly defined IP64CIDRAddress. depends on the
introduction of the IP64CIDRAddress vtype in proxmox-widget-toolkit.

Signed-off-by: Mira Limbeck <m.limbeck@proxmox.com>

d/control: bump doc-generator build dependency version

When building a current version of pve-manager a dependency to the
documentation is missing, as it ships new references used by our
onlineHelp tooling this is required.

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

cleanup Status plugins use statements

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

followup: code cleanup, remove unnecessary type check

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

fix #1326: allow multiple status server definitions per type

we allow an id like storage.cfg but leave it optional (so we do not
break existing configs):

 influxdb: name

so that one can export the data to multiple servers of the same type

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

bump version to 5.4-3

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

ui: dc/options: allow to edit cluster wide migration settings

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

followuop: ui/dc/options: fix default

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

ui: dc/options: allow to edit HA shutdown policy

add some helpers handling this, can be re used for adding the
migration field, maybe we want to move this to widget-toolkits parent
ObjectGrid class, a bit cleaned up.

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

ui: lxc: do not disable complete add resource button if no disk caps

While for now you only have elements which require the add disk
capability in the add sections it makes no sense to hide it, as it
would suggest that one can add no resource at all, and can easily
introduce a bug if we add another type of resource here, independent
of disk add cap.

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

fix #1814: do not enable disk related buttons for audit user

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

ui: u2f: refactor error code map out and use in login window

to avoid showing numbers as error codes to users, even though the
strings are not much more helpful either...

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

remove non-existant property

vmComboSelector does not have such a property

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

ui: Restore: Fix error where the textfield did not default to next free VMID

This was introduced when refactoring our seprated CT & VM ID
selectors into an unified one in
commit ed416f20248d46d95c9d1547a2e1b833aeb097d6

Signed-off-by: Christian Ebner <c.ebner@proxmox.com>

use correct icon for hibernate in cmdmenu

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

fix jslint warning

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

bump version to 5.4-2

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

d/control: bump version dependency to libpve-access-control

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

api: ceph/mon: improve no ip error

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

ui: tfa: add onlineHelp

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

ui: qemu system panel: set onlineHelp

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

Ceph MON - Reword error message on IP config

Reword the error message in find_mon_ip to make it more clear, that
there is no active IP configuration for the ceph public network.

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

ui: tfa login: wording followup

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

ui: login: remove fixed OTP field completely

we now have the TFALoginWindow which pops up if TFA is configured,
so use only this single stream lined approach.

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

ui: use 2-step authentication for non-tfa realms

(Ext.Msg.prompt() fails on firefox...)

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

HTTPServer: verify_ticket now returns a hash

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

Revert "ui: always show otp field"

This reverts commit 1702b529b66307f9aeafc0e9f28c7afe3864ffbb.

ui: ceph install: open ceph.com in new window

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

pvereport: only comput report def if we generate a report.

it's really not a good idea to start doing exec, file exist checks,
... on every module load of this, this may even hang completely
differen parts if something pull us in, and confuse one completely...

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Reported-by: Wolfgang Link <w.link@proxmox.com>

followup: reduce wording, less text has better chance to be read

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

cleanup format.

Supplementary Ceph installation summary

Add installation node for the other participant Ceph Node.

update shipped aplinfo index

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

ui: improve datacenter summary ceph node selection

instead of trying a single random node 3 times, try all nodes
until we either find one with ceph, or until we tried them all

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

ui: make Datacenter subscription info clickable when not valid

to make it easier for the user to buy the subscriptions

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

ui: fix blinking subscription info panel

not having a subscription trumps having mixed subscriptions, so we
check that correctly

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

ui: tfa: also include validity check for password field

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

ui: tfa: use userid directly for totp url generation

as name is now a displayfield, which by default does not
gets submitted, so just use the fixed userid directly

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

ui: tfa: show username as read only display field for info

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

ui: tfa: remove fixed labelwidth

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

ui: tfa: allow apply only on valid inputs

only when the totp form and the challenge is valid, allow pressing the
apply button, default is disabled, as the 'user_tfa' data binding was
not used anywhere else replace it with something more fitting.

change allowBlank for the challenge

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

ui: qemu: change suspend to disk icon to fa-download

it ressembles a state downloaded to disk, so quite fitting for
hibernate/suspend to disk

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

add a filter mask for the TFA Secret

this way a user cannot (easily) enter wrong characters
else if an invalid Character is entered, one can still hit apply
but not all characters will be used for the secret

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

ui: parser use parseBoolean for firewall

so we detect on/ff,true/false,1/0

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

bump version to 5.4-1

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

ui: ct: fix displaying nic firewall if explicitly 0

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

ui: restore: add tooltip for unique checkbox and declare static

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

fix: #1218 Expose unique MAC address generation on restore of VMs and CTs in UI

Adds a checkbox to the restore window, allowing the user to avoid MAC address
collisions when restoring a VM/CT to a different vmid.
When restoring to the same vmid the checkbox is not visible.

Signed-off-by: Christian Ebner <c.ebner@proxmox.com>

ui: followup trailing comma fix

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

ui: tfa: fix secret validator

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

ui: user view: improxe "has TFA" check

detect also our old variant for TFA, so all users where the keys
values is non-empty have TFA

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

ui: user edit: move TFA key IDs field to advanced section

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

LoginWindow: make static

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

ui: tfa: move qrcode creation to controller

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