ui/Parser: printPropertyString: simplify and always put defaultKey values first

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

ui/Parser: parsePropertyString: log errors to console

allows to catch bugs faster as we get a call trace and may see this
error even when our caller doesn't handle an undefined return value
from this method.

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

ui/Parser: add generic functions property_strings

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

add Directory list/create gui

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

add LVMThin list/create gui

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

add LVM List/Create Gui

only let the user change the name of the vg

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

move DiskSelector to its own file, for generic use

also set the default diskType to undefined and
change the api endpoint to /nodes/NODE/disks/list

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

ui: storage: combine RBD external and hyperconverged add dialog

Combine both dialogues. This reduces storage choices from the
Storage -> Add menu, and thus improves usability.

We make the whole dialog also more intelligent by querying the
monitors, so that we can show them for the hyper converged case
keeping a similar interface for both cases. Further we can use this
to decide if a hyperconverged setup is possible at all.
While this information would also be available through the fact if
the pool selector has entries I wanted to show them anyway and could
make logic a bit easier using it.

fix doubleclick on vm with spice console

allowSpice needs to be a real boolean, because of a
'=== true' check to open a spice console if allowed, so make it one here

this fixes the issue that a doubleclick on a tree item only opened
the novnc console even if spice is the default and the vm is spice
enabled

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

bump version to 5.2-7

pveproxy: add link to extjs for api-viewer

allows us to remove the external loading of extjs from the cloudflare
cdn in pve-docs easier with having the external hosted version in
mind.

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

correctly install PVE/CLI/pvesh.pm

pvesh: add standard_output_properties to set/create/delete

Most of those API calls return simple strings, so it does not really matter.
Nethertheless, some calls return complex data structures and we want to set
output format for them.

pvesh cleanup: use a handler class - PVE/CLI/pvesh.pm

pvenode: remove useless call to query_terminal_options

pvenode: use print_api_result to print acme account information

pvenode: use print_api_result to print certificate information

pvenode: fixup output formatting

pvesh: implement 'ls' command

pvesh: do not use CLIHandler $option parameter, simplify code

ui: permissions: allow to select root path '/' directly

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

set resize to 'off' for the novnc popup

this makes it easier to handle the resize parameter in novnc code

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

improve xtermjs settings button behaviour

we now put a form around the fields/buttons and can track
the valid/dirty state, and use this for setting the button states

now the buttons are correctly enabled/disabled when the form
is dirty/valid/empty

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

nodeconfig acme-domain: allow domains starting with numbers

they are valid

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

pvesh: use standard output options

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

pvesh usage: correctly handle uri paramaeters, cleanups

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

pvesh usage: new option --returns to print result schema

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

use new timestamp renderer for /nodes/<nodename>/time

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

fixup pvesh for changes in pve-common

In pve-common the following changes happened:
  0adee98 ("cli_handler: pass common output options as separate parameter")
  352b7a1 ("PVE::CLIFormatter - implement new output options")
We now get the format passed via an options hash in the
output function instead of seeing it as an api call
parameter.

Similarly, --quiet is now already handled there.

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

pvesh: complete rewrite using PVE::CLIHandler

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

pvesh - remove interactive readline mode

We have good command line completion and history with 'bash', so there is
no real need to duplicate this functionality.

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

pvenode: fix print_api_result usage, avoid print_api_list

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

Submit pool if non-empty in lxc CreateWizard

The lxc CreateWizard ignores the pool setting. That makes creating
containers impossible for users who are restricted to using a pool. This
patch fixes that.

Signed-off-by: Andrew Neitsch <andrew@neitsch.ca>

bump version to 5.2-6

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

debian/control: update pve-common version dependency

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

pvenode: use new CLIFormatter options

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

PVE/API2/Nodes.pm: document return properties for cluster node list.

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

fixup: s/destriction/description/

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

PVE/API2/Cluster.pm - document return properties for cluster resource list.

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

ui: vm hotplug selector: refactor for sane formatting

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

Add cephfs to allowed storages for vzdump backup

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

ui/storage: cifs: reoder fields

username, password and domain name are all related, they are non-pve
options related to accessing the CIFS storage and have all influence
on the pveCIFSScan component.
So group at kleast user and password together. But keep domain on the
right for the sake of balance.
Move the content type selector to the right, as the RBD and other
panels do.

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

ui/storage: cifs: exit early from username change cb on displaying

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

ui/storage: cifs: only allow entering a password if user is set

we ignore the password in the backend if no user is passed,
which can be very confusing for the user - or more important, me,
when testing cifs/storage stuff.

So change the mechanics slightly. Previously we changed the
allowBlank property of the password field if username was changed.
I set it to always required, but disable the whole field if we do not
have a username.

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

trivial: ui: cifs: cleanup unnecessary lines

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

whitespace fix

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

add task list/log/status to pvenode

and use the newly created print_api_* helpers for this

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

task log: set default in the api call

we have the defaults documented here, so set them here too
otherwise if the default change in PVE::Tools, we probably forget to
update the api description

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

node task list: set the defaults better

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

add more api schema information for node task list and log

this makes for a better documentation

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

fix correct call to 'usage_str'

we forgot to give usage_str also the param_mapping sub,
else it fails on calling 'help'

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

bump version to 5.2-5

d/control: update pve-common version dependency

as we use the new param_mapping functionallity from PVE::CLIHandler

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

use PVE::RESTHandler explicitly

since we use it in the code

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

replace read_password with param_mapping in pvesh

and use get_standard_mapping from CLIHandler

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

add star/stop/migrateall commands to pvenode

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

startall: refactor and fix error message

for vms, it showed:
Starting VM 100 failed: status

instead of showing the actual status

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

bump version to 5.2-4

ACMEv2 order "ready" status update

since letsencrypt updates their implementation to the ACMEv2 spec [1],
we should correctly parse the order status

1: https://community.letsencrypt.org/t/acmev2-order-ready-status/62866

note that we (for now) try to be compatbile to both versions,
with and without ready state, this can be changed when all letsencrypt
apis have changed

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

buildsys: remove excess directory creations

/usr/share/$(PACKAGE) is used for country.dat, installed
with -D
/usr/share/man/man1 is defined as $(MAN1DIR) in
defines.mk and already created in bin/Makefile where it is
actually used.

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

cleanup: move bin/test to toplevel

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

cleanup: add network-hooks directory and move respective files

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

cleanup: add configs directory and move respective files

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

cleanup: rename bin/init.d to services

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

cleanup: replace repoid.pl script by simple git commands

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

buildsys: remove wildcard target from Makefile

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

api/tasks: explicit import raise_param_exc

we inherited the import from PVE::RESTHandler but may want to get rid
of it there. So explicitly import it here.

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

ui/ct: add autostart-after-create checkbox to wizard

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

bump version to 5.2-3

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

add ifupdown2 as alternative to ifenslave.

ifupdown2 conflict with ifenslave, as it's replace it

ui: add auostart after create checkbox

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

Remove swap/memory limit in the Manager, fixes #1799

add ebtables option to cluster firewallsettings

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

bump version to 5.2-2

let all users retrieve the directories and tos for acme

else all non-root users get an empty dropdown box for the directories
and get no feedback why that is

with this, they can select it, but ultimately get an api error if the
permissions are not sufficient

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

api/acme: fallback to default for account name parameter

We defined 'default' as fallback default value for the optional
pve-acme-account-name standard option but did not honored that.

Thus we got a perl error ($account_name not defined) if we did not
passed a name. Fix that by actually falling back to 'default' in this
case.

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

fix notes gear tool

me was used before it was defined

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

indentation fixes

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

fix #1499 Implement a combobox for Permissions paths

Signed-off-by: René Jochum <r.jochum@proxmox.com>

Modernize PVE.form.GroupSelector,

as example for future Uses.

Signed-off-by: René Jochum <r.jochum@proxmox.com>

fix #427: add notes panel to node

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

accept type 'node' for NotesView and refactor it

so that we can use it for the notes of a node,
also changes the configuration to a static declaration,
this adds an 'edit' button for the node panel

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

let NotesView load itself

instead of needing to load from the outside, let NotesView handle the
loading itself

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

correctly update qemu ip information

we updated the status from the status store too often, so that
if the guest agent was not running, the message would
change between 'no information' and 'not running'

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

only show 'more' buttons for ips when there are actually ips to show

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

check for VM.Montior privileges on qemu AgentIPView

so that the user does not get the useless 'no information' or
'not running' message, but the real cause

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

ui: use cbind to make usb input panel static

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

correctly parse iscsi content

we parse the content as a list, so it is always an array

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

ui: dc/storage: allow to renenable iscsi storages again

The dc/storage refactoring done in
commit f28ec3e4b3e6cca1f45630e75f9858c558680979
introduced a regression where an once deactivated iSCSI storage could
not be activated again through UI, as we did not called the parent
setValue function correctly, thus the Enable checkbox was always set
to true. A uncheck -> check cycle couldn't also be done as the dirty
changes check disabled the submit button in this case.

Enable/Disable through API or CLI still worked.

Reported-by: Bastian Sebode <b.sebode@linet-services.de>
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

ui: vm: allow to add socket backed serial devices

We show and can remove serial devices but couldn't add new ones
through the WebUI.
Add a simple component to allow adding serial ports backed by a
socket, which can be especially useful now with xterm.js

Passing through serial devices from /dev isn't possible with this, as
it is normally a root only operation and not that often used.

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

fixup: jslint errors

fix some small problems where jslint complains, introduced by
commit: 675e71d37a07f522e065dbefb6a20dde3119fbc3

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

bump version to 5.2-1

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

test: replication: adapt test data

otherwise the checks in write_config fail

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

test: replication: mock cfs_(write|lock)_file

which is needed since the HA recovery series

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

add missing onlineHelp links

for:
* dc/Cluster
* node/Certificates
* qemu/CloudInit

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

correct node free calculation

avail - used is wrong
total - used = free

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

update included aplinfo.dat

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

ui/UserEdit: fixup grammar in password match error message

Passwords is plural, so it needs "do", not "does".

We already have the "do" version translated as it is used by the PMG
so this adds no new string.

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

hide irrelevant buttons and tabs for container templates

like we do with vm templates

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

trailing whitespace fixup

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