fix #2314: remove GZIP env var

...and replace instead with command line argument.
Avoids a deprecation warning.

Signed-off-by: Stefan Reiter <s.reiter@proxmox.com>

gui: ceph wizard: fix typo

s/Coniguration/Configuration/

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

Fix #582: Add default delay for backups

A default of 5 seems like a reasonable delay for removing backups from
the GUI.

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

ui login: followup: add some hints to the 595 login error message

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

ui: login: do not show connection failure as login failure

In case pvedaemon, pveproxy is not running or we get somehow else a
595 response (connection failure), we want to tell this to the user,
since "Login failed" doesn't help.

Handle all connection failures, leave the rest the same. We could go
into more detail (CONNECT_FAILURE, CLIENT_INVALID, LOAD_FAILURE and
SERVER_INVALID), but i didn't see any reason to make all these
distinctions now.

Signed-off-by: Oguz Bektas <o.bektas@proxmox.com>

ui: vm/Hardware: do not depent mapping of cdrom, cloudinit, .. on the CSS class

This was pretty brittle to begin with, on should not attach logic to
CSS class names or the like, if possible.

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

ui: vm/Hardware: convert more icons from bitmap to font awesome

since commit 03558174ef4f15f542a61584013a195481188e7f we can use the
Font Awesome icons without them being shown in the remove dialog, so
convert more to them - they are vectors and look a bit better.

Also ensure that all icons are rendered as fixed width, with fa-fw

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

(re)bump version to 6.0-5

The other commit was pushed out by slight mistake but not shipped,
so re-bumped the date, but with same version (commid hash is shipped
anyway)

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

ui: vm/HW: use more font awesome icons and fix color

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

pveAudioInputPanel: comment out onlineHelp doc reference for now

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

Add audio device to HardwareView

Adds the following:

* Menu entry to create a new audio device
  only clickable if:
    * permissions are there
    * no audio device exists
* listing in the hardwarew view

Signed-off-by: Aaron Lauterer <a.lauterer@proxmox.com>

Add edit dialog for audio device

Signed-off-by: Aaron Lauterer <a.lauterer@proxmox.com>

bump version to 6.0-5

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

Include Windows Server 2019 in OS-Type list

Signed-off-by: Stefan Reiter <s.reiter@proxmox.com>

Add hv-tlbflush and hv-evmcs as optional flags

since they have both been removed as automatic hyper-v flags.

hv-evmcs description intentionally does not contain a mention of Windows, since
it works for Linux guests as well.

Signed-off-by: Stefan Reiter <s.reiter@proxmox.com>

ceph: add perl flag GET and PUT call

use PUT for setting or unsetting, as POST/DELETE (like the old node
specific API used) makes no sense. One does not creates or deletes
the flag, they are always here. One just updates their value

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

ceph: move flag API endpoints to node-agnostic /cluster/ceph path

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

ceph: move possible_flags to Ceph::Tools for intra-module reuse

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

api/cluster: move ceph calls into sub- directory/module

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

cluster api: group and sort module use

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

remove unused ext-pve.css

not used since commit 5783c7f4ddbeeddb7038f40c94908b0f3a6b5fa7 from
2016...

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

ceph flags: rename flags2 to flag before public release

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

ui: ceph/osd: add a real fix for the window-not-centered issue

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

ui: ceph/flags: ensure window is vertically centered with a hack

and some small changes of col width and missing semicolon

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

ui: ceph/flags: commit changes into flag store to avoid having dirty rows

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

ui: ceph/flags: rename 'Set Flags' to 'Manage Global Flags'

as one can also unset, not only set them, further the scope should be
made a bit more clear, "Cluster could have worked too"

Additionally specify that we mean the "OSD Flags" in the window
title (it's a bit to long for the button)

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

api/ceph: followup: refactor out getting the current set flags

Add a new private helper which just does that for us, to avoid having
the same, or multiple implementations doing the same thing.

Allow to pass the $rados for sharing the same connection but also
just create it else for convenience

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

gui: ceph: convert 'set noout' button to 'set flags'

by using the new CephSetFlags EditWindow

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

gui: ceph: add a ceph flag EditWindow

this simply shows a grid with the available flags of the backend,
with a checkbox indicating the status

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

ceph: set/unset flag: remove unused variable

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

ceph: add PUT 'flags' api call

this api call can set multiple flags at once, but does this in a
worker since this can take quiet some time

also we only set/unset flags that are not already set/unset (respectively)

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

ceph: add GET 'flags2'

we want a GET api call where we return all available flags with
description, but the GET 'flags' api call already returns a string
and not an array, so we cannot use that

instead, we add a new api call that returns that feature and
with e.g. pve 7 we can remove the 'flags' api call and
rename 'flags2' to 'flags'

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

5to6: cleanup cert check and make more general

If we need to add other types or increases the min size this makes it
easier as it's just a schematic definition.

Also just do a pass/fail on each cert, so drop the last "summary
pass".

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

pve5to6: cert: check certificates key size

Debian Buster raised the default security level (1 -> 2) for TLS
connections.

This moves from the 80 bit security level to the 112 bit security level
and will require 2048 bit or larger RSA and DHE keys, 224 bit or larger
ECC keys, and SHA-2.

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

pvenode: add public key size & type to cert info

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

gui: cert: add public key type & size to certs view

by default the fields are hidden

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

fix #2265: enable discard for virtio-blk

QEMU 4.0 supports this. Guest needs to have at least Kernel 5.0 for it
to be used.

SSD emulation is still not supported AFAICT, so no change here.

Signed-off-by: Stefan Reiter <s.reiter@proxmox.com>

ceph: add flag hash

this is used to generate the paramters for the set/unset api call
and will be used for the parameters of the upcoming 'set multiple flags' call

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

ceph tools: factor out frequent keyring and config init check

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

ui: ceph: update text for outdated versions a bit

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

api/osd: opinionated code cleanup of list

among others: reduce use of sub-hash as index for another hash

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

gui: ceph dashboard: improve version hints

show the difference between a service where the host has outdated
packages, and where only the service runs an outdated version and only
needs to restart

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

gui: ceph osd: improve version hints

we now traverse the whole tree for the versions and save not only
the maxversion but if we have mixed versions at all

the traversal contains code to inject the host versions into the treenodes

this way we can omit any icon in case everything is on the same level
and if not we:

* show a check at update hosts and services
* show a 'restart' symbol for osds that only need a restart
* show an 'upgrade' symbol (fa-upload) for osds/hosts that need
  a package upgrade

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

gui: ceph: add HEALTH_UPGRADE state

to have 2 different states for
* packages are outdated
* packages are up to date but the service runs with the old version

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

ceph: osd list: add hostversions to the host nodes

we want to improve the version hints in the osd tree gui and need
the version at the host nodes

we could (and want to) workaround it in the gui to have that
info for both versions of the api call

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

pveam update: also print success

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

update shipped aplinfo index

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

aplinfo: make apltest a bit more flexible

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

aplinfo: update from aplinfo-pve-6.dat

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

aplinfo: remove 4.x and older key, add 6.x key

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

aplinfo: use aplinfo signed with 6.x release key

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

aplinfo: re-work and allow different filenames per apl source

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

aplinfo: code cleanup

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

fix #2292: ceph osd create: use size parameter for db/wal

commit 970f96fdbb20d96127ae2d738344f2c5c61b18fc did not account for
getting the correct size parameter from the api call, so we ignored
it always resulting in uses not be able to set an explicit db/wal
size

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

ui: qemu/HWView: cloudinit: use the font-awesome provided cloud symbol

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

ui: qemu/HWView: small code cleanup for remove dialog

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

ui: qemu/HWView: only render icon when in grid

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

gui: fix debug-mode error

setData on an 'empty' store does not work in extjs debug mode
setting store to '{}' here does create a store on component creation
so the store is not 'empty'

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

ceph destroymon: actually die on errors

instead of silently ignoring them. since we are in a task worker here
this is especially important - otherwise the task status/result is also
wrong!

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

ceph init: actually die on errors

instead of silently ignoring them

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

ui: window/migrate: use space separator between type and VMID

Like previous to commit 9706707d2f87edf123a12951865b667b4b2ce881
which broke this, we have some assumptions in our gui-test/screenshot
automation which lays assumptions on certain window titles, so try to
keep them stable.

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

bump version to 6.0-4

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

pve5to6: check for ceph-volume osds

do not warn in case there exist nautilus osds
since the upgrade has to be done by the time a user should add new
osds, it does not make sense to warn in that case

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

gui: ceph installer: fix typo

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

followup code cleanup

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

ceph: mon list: show only as running when monitor is quorate

nautilus puts non running monitors also in the monmap, so only show
as running when it has quorum

this is also not 100% correct, but the only 'correct' alternative is
to try and get/parse the systemd status of the units and broadcast it
to the pmxcfs

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

ui: cephfs: also add new tooltip like pool create

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

ui: dc/Cluster: rename ring columns to link

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

followup coide cleanup

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

ui: improve console fallback logic if default is not available

instead of instantly defaulting to novnc when the chosen
default is not available, try the other (better variant)
before falling back to novnc

e.g. having no default set is equivalent to having it set
to xtermjs, and having a vm with spice then previously
defaulted to novnc, now to spice (the reverse is also true)

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

ui: fix wrong defaultConsole fallback in tree

having spice does not mean it can also have xtermjs, and
we get the information in the status call already
so give the correct parameters to openDefaultConsoleWindow

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

bump version to 6.0-3

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

filter unusable lxc mountoptions for rootfs

disable nodev and noexec per thomas' suggestion[0].

[0]: https://pve.proxmox.com/pipermail/pve-devel/2019-July/037994.html

Signed-off-by: Oguz Bektas <o.bektas@proxmox.com>

Revert "add beta text with link to bugtracker"

This reverts commit 1e12b3179148b67ec6f94654c963f33a1a300bb0.

ui: node/disks: retune column width and hide vendor by defauöt

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

ui: node/disks: reuse Utils.format_boolean for GPT column

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

pveam: add mail to sections

it includes the PMG

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

bump version to 6.0-2

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

ui: dc/cluster: enforce second link if in join info

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

ui: dc/cluster: join: s/ring/link/

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

ui: dc/cluster: allow to create cluster with second link

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

ui: dc/cluster: add onlineHelp reference to create call

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

gui: ceph configdb: fix store load

instead of using API2Request, use the proper grid store load

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

ui: ceph/pool: enable add_storages by default and add tooltip

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

ui: ceph/service list: re-do column widths with flex

this way we see thew full status of a MDS, and for our minimal
resolution of 720p and targeted resolution of 1080p it works good.

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

ui: ceph/pool: rework status grid a bit

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

ui: remove notify calls on viewmodel

They should not be required and result in exceptions thrown if this
component is used in combination with ExtJS debug sources

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

ui: ceph/osd: merge apply + commit latency columns

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

ui: ceph/osd: split out used% and total avail column

as "used -> total" is just confusing

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

ui: ceph/osd: hide Type column by default

this column shows what type the current row is (host, osd, root),
nowadays that's quite visible through the tree view + icons, so just
hide it by default to make this interface a bit less noisier.

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

ui: ceph/osd: only show at max 5 digits after decimal point for weight

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

ui: ceph/osd: rename Action to More

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

ui: ceph OSD: tweak default column width a bit

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

ui: ceph/osd: reintroduce Used column

previous commit 76235ae132c69898048c1f7579e7da40adbd7a6e accidentally
removed this column during refactoring, let's reintroduce it again.

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

gui: ceph osd: rewrite osd panel and add scrub button

this rewrite of the osd panel converts it to a static declaration of
the layout with a controller/viewmodel logic

while doing this, add icons to buttons, and add a scrub/deep scrub entry

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

api/ceph: code cleanup

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

api/ceph: add osd scrub api call

can be called to (deep) scrub a specific osd

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

keyring: use ckeyring_path variable in chown cmd

A fixed path for the ceph.client.admin.keyring was used in the chown
command. This patch uses the ckeyring_path variable instead, to minimize
changes should the path change.

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

Fix: typo in ckeyring_path

pveceph init failed, as the copy used a wrong path with typo as
target and then the chown tried to operate on the correct keyring
path, which was then non-existing.

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

ui: dc/options: minor followup cleanups

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