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 8f60ee4ccacc14f1bbe025874522c151cc375f83
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: 9265782932cdbcf8e6c7781aec437d06a22048d4

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>

give vms with 'io-error' an error icon

so that one can see that the vm has an error, even when not
directly investigating the status of that vm

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

show qmpstatus if available

this patch shows the qmpstatus if it is available on the summary in
parenthesis

this shows things like 'suspended' and 'io-error'

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

implement multiField renderer for infopanels

we can use this for fields which use more than one value of the
guest status

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

dc/ClusterEdit: join: fix assisted entry chechbox change listener

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

jslint fixes

those found their way in during a rebase of my patches

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

bump version to 5.1-52

use correct multiKey and getObjectValue calls for memory line

also show shares and if ballooning is off

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

add xterm.js settings

this adds font-settings to the 'My Settings'
window and makes that bigger

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

set the default console in the gui when setting the option

until now, we only set the default console once on page load,
this enables to set it everytime the dc->options panel loads,
so also after setting an option

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

remove the novnc panels for node/container

so that by default the users get xtermjs
not for vms though, since most will still not use a serial
terminal as the display

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

allow xtermjs to be the default console

if not available for a vm, we fallback to novnc like with spice

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

fix #1691: use asynchronous API call for HD create

It should use an asynchronous API call to create an image,
because some storages may need longer to alloc an image.

If a storage needs more than 30 sec
the proxy will timeout and we have an orphan image.

Also, it is possible to increase the timeout for some
storages if an asynchronous call is used

fix #1732: actually post values in IScsiEdit

this prevented creating/editing an iscsi storage

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

bump version to 5.1-51

overwrite the built-in 'run_editor' function

this is needed for the 'enter' handler

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

bump version to 5.1-50

better focus of lxc/MPEdit

this gives the focus to either the 'path' field
or to the 'X' tool (if it is a rootfs)

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

merge cores, cpulimit and cpuunits for containers

like we do for qemu vms

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

lxc/MPEdit: rework for advanced options

this is a complete rework of the inputpanel (long overdue)
it uses a viewModel and viewcontroller to avoid the
multiple is zfs/root/bind checks and concentrate them
in one place

also some features get optimized (e.g. the noreplication checkbox)

adds a setNodename to the DiskStorageSelector so that we can bind
the nodename there

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

Utils: add forEachMP

in the style of forEachBus, but for containers, so that we
only have to save the count once

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

lxc/ResoureEdit: move some cpu fields to advanced options

and show them now also in the wizard

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

qemu/CreateWizard: add advanced options on general tab

add start on boot, guest agent and boot order fields

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

qemu/NetworkEdit: rework panel for advanced options

move some fields into advanced options, and remove nat mode

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

qemu/MemoryEdit: rework panel and move most things in advanced options

this is a major rework of the memory inputpanel
previously we had to select fixed/variable and
could set shares/ballooning, depending on what was selected

now the panel is much simpler:

non advanced:

memory: the target maximum memory

advanced:

min-memory: the minimum memory, if different from the memory field
we have 'dynamic' memory

shares, only if we have 'dynamic' memory

ballooning-device: better naming, deactivates the ballooning device

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