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>

fixes #1503 Add role CRUD to the GUI.

As given in the subject this implements role create/update/delete over
the manager.

There's currently no coler highlightning for "special" roles.

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

add _blank target for optional url

so that they open in a new tab/window

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

bump version to 5.1-53

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

add container move volume to gui

this reuses the vm HDMove window to move volumes

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

make HDMove.js also usable for lxc

this makes it possible to reuse the window for container move_volume

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

close HDMove on taskDone

this triggers the 'destroy' event of the window at the correct time

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

d/control: update build and install dependencies

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

ui/acme: remove 'Renew Cert' button for now

You can normally do this with the Order Cert button, with almost the
same result, and it isn't something which will be needed often, if
the UI is somehow inaccessible (due to cert problem) then this is
useless anyway, and the CLI must be used anyhow.

Can be easily reverted if we deem this necessary later on.

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

add node/Certificates.js and use it

this adds the grid for showing the custom/builtin Certificates
and the means to upload and delete custom ones

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

add node/ACME.js

this provides the grid for editing domains for letsencrypt,
order/renew the certificates, and the window for creating an
ACME account

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

add render functions for Certificates

this adds render_optional_url for a field which may contain a link
and render_san which simply displays the different values
in each line

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

Extend the .gitignore(s)

Signed-off-by: René Jochum <rene@jochums.at>

pveupdate: add ACME certificate renewal

renew certificate if an acme config entry and a custom certificate
exists on the local node and the certificate expires soon.

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

add pvenode CLIHandler

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

add certificates API endpoints

to allow retrieval of certificate information, and uploading or removing
of custom certificate files.

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

add ACME certificate API endpoints

for creating/ordering a new certificate and renewing respectively
revoking an existing one.

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

add ACME account API endpoints

for registering, updating, refreshing and deactiving a PVE-managed ACME
account, as well as for retrieving the (optional, but required if
available) terms of service of the ACME API provider / CA.

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

add node configuration file and API

this currently only contains a description and the node-specific ACME
configuration, but I am sure we can find other goodies to put there.

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

add CertHelpers utility

PVE-specific certificate helper functions

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

disable the mountpoint selector and path field by default

since in the wizard, the validity check happens before the
viewmodel can set the visible/enabled state so this is invalid there,
even if the remaining panel is valid

setting it to disabled by default, makes the fields 'valid' at the
beginning but retain their behavior

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