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>

ui/Parser: printPropertyString: sort non-defaultKey parts for stability

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

ui: dc/options: use proxmoxNetworkSelector for network

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

pveproxy: allow to request debug sources directly over GET parameter

Avoids the need to stop the current pveproxy and restart it in debug
mode, if one wants to get the debug un-minified sources for ExtJS,
for example.

For a local PVE instance the following URL would ensure you load the
debug sources:
https://localhost:8006/?debug#v1:0:18:4:25:19::8:20:28

For convenience allow to omit the parameters value, and default to
true, iow, ?debug and ?debug=1 are the same, one could force it off -
even if the server was started in debug mode - by using ?debug=0

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

followup: code cleanup to arrow functions

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

gui: show 0 for max_relocate/restart correctly

0 || '1' will always return '1'

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

fix #2267: delete address(6) and netmas(6) with cidr(6)

otherwise a user cannot delete an ip from an interface

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

build: bump versioned dependency on pve-cluster

needed for PVE::Corosync::resolve_hostname_like_corosync

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

5to6: make corosync totem checks more verbose

to avoid just printing the subheader with no results

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

5to6: add more corosync subheaders

to improve readability

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

5to6: fail if a corosync node has neither ring0 nor ring1 defined

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

5to6: reword/-structure corosync message

and fix a typo as well

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

5to6: attempt to resolve corosync rings

and only fail if unable to

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

update shipped aplinfo index

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

followup: balance columns

having this new selector on the left column makes it a better fit for
both possible edit situations, rootfs is now fully balanced and both
mp/rootfs kip the "skip replication" setting on the same place - in
sync with VMs

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

fix #1451: allow some extra mount options for container

This allows the following non-problematic mount options for lxc
container mountpoints:
* noexec - Do not permit execution of binaries on the mounted FS
* noatime - Do not update inode access times on this filesystem
* nosuid - Do not allow suid or sgid bits to take effect
* nodev - Do not interpret character or block devices on the FS

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

followup: hide mask, level and RO by default

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

gui: ceph: show config database under the config

the column titles are the same as the 'ceph config dump' cli

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

ceph: add api call for config database

simply returns the database content

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

use new url

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

bump version to 6.0-1

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

ceph: remove obsolete ceph.service and do not distribute it anymore

in nautilus there is no ceph-disk anymore and osd activation
does not use udev anymore so this service is not needed anymore

remove it and do not copy it when installing a new ceph cluster

in pve-storage.target we replace ceph.service with ceph.target

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

pveceph install: set repo URL to download.proxmox.com nautilus repo

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

gui: ceph: parse the whole message for monitor ids

sometimes a monitor id is only mentioned in a later part of the message
not always the first

for example, if more than one monitor is missing the msgr2 protocol
each monitor gets its own line

before this patch the whole error would only be attributed to the first
monitor mentioned, now it is attributed to all

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

5to6: only tell to solve problems if fails are present

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

5to6: improve final note on errors and/or warning a bit

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

5to6: fiy typo: s/detailled/detailed/

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

5to6: check common services pveproxy pvedaemon pvestatd

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

5to6: add log_systemd_unit_state and use for pve-cluster and corosync

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

5to6: import run_command

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

5to6: clarify "cluster quorate" message a bit

As this is the state from the pmxcfs

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

5to6: fixup: really set $total_votes to 0 if not defined

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

5to6: drop detailed quorumtool output

since we already print most of that anyway, and it is rather long.

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

5to6: more quorum / vote checking

handle expected votes set to non-standard value, and try to adjust
calculations for qdevice setups.

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

5to6: quote some Ceph flags/options

to improve readability

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

5to6: check for ceph global keyring config

which causes issues after upgrading to Nautilus.

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

5to6: followup: also detect ceph conf keys separated with -

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

5to6: ceph mon host check: switch config check from defined to hash values exist

we can get here if a empty (or one with just comments) ceph.conf
exists

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

5to6: ceph global mon host check: also detect mon-host

and adapt message a bit

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

pve5to6: check ipv6/ipv4 settings in ceph.conf

having ipv6 enabled while not disabling ipv4 prevents
nautilus osds to start if no ipv4 network is given (because they
are trying to bind to both ip families and die if one of them
is not found)

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

pve5to6: check ceph config for mon_host line

this already works on luminous, so it does not harm to add it already,
and is recommended when both msgr1 and msgr2 is activated in nautilus

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

ceph: mon create: add known monitor ips to mon_host if it is empty

this fixes an issue where only one monitor is in mon_host, which is
offline, prevents a client connection

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

ceph: services: improve addr selection

we map '$type addr' to '$type_addr' anyway in the ceph.conf parser,
so this is not necessary

also use 'public_addr' if it is set

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

ceph: osd: use get-or-create to create a bootstrap-osd key on demand

if for some reason the cluster does not have this key, generate it

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

ceph: osd create: check for auth before getting bootstrap key

we do not need it if auth is 'none'

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

5to6: invert check for noout for nautilus

mainly because it looks strange to get a warning after the upgrade is
finished and noout has been removed again

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

5to6: followup: still include nodename in IP check

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

5to6: reuse $nodename

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

5to6: improve some log messages

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

5to6: followup sheepdog message

do not suggest that we ever supported it as first class storage, it
was always just a experimental support and upstream is
EOL/unmaintained now.

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

5to6: add check for configured Sheepdog storages

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

followup: do not use string comparision for integers

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

api: ceph: automatically create manager after the first monitor

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

ceph: init: fix undef var access

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

5to6: improve ceph warning on non OK state

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

ceph: init: only handle keyring if auth is cephx

if auth is 'none' there is no client keyring, so do not generate it and
do not write it into the config

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

gui: ceph: fix syslog resize

makes the logview inside resize with the window

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

pve5to6: be a bit more verbose for misc. checks

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

pve5to6: followup: improve coding style and outputs a bit

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

pve5to6: list vms with active vmx/svm flag

list all vms with either max/host cputype or vmx/svm explicitely set
(this can only happen in the args)

give a general message if none is found at the moment (and do not warn)

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

gui: ceph pools: fix pool % used display

nautilus uses values from 0 to 1 instead of 0 to 100

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

api: subscription: return https backed link

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

bump version to 6.0-0+4

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

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

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

api: backup pool: enforce VM.Backup permissions on pool also on create

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

api2tools: refactor get_resource_pool_guest_members and declare missing use

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

api: backup pool: enforce VM.Backup permissions on pool

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

ui: pool backup: adapt labels a bit

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

fix #1278 gui: backup: add backup mode pool

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