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>

fix #1278 api: add pool backup option

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

whitespace cleanup

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

ui: migrate: add storage and size information to preconditions

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

ceph: disallow ipv4 in new cluster when ipv6 is enabled

ms_bind_ipv4 is default true and osds look for both
ipv6 and ipv4 addresses in cluster network/public network

since we only allow for one network each (which must be either
ipv4 or ipv6) we disallow ipv4 if ipv6 is detected

this fixes not starting osds on an ipv6 enabled, newly-setup cluster

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

ceph: mon create: fix not starting monitor if no public_addr is in conf

when there is no 'public_network' in the config, the monitor
can only find an ip if it is given explicitly, either via commandline
(not possible with systemd) or via the ceph.conf

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

silent some lintian warnings

"privacy-breach-generic" is  a false positive here (lintian bug
907957)

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

d/maintscript: really remove the old apt hook for kernel

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

buildsys: fixup: fully remove pve-apt-hook mentionings

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

config: remove pve-apt.conf

Instead of marking all pve-kernel-.* packages as NeverAutoRemove we compile
a list of specific versions to keep.
This functionality is added in the proxmox-ve package (along with the config
for booting kernels with systemd-boot).

5to6: get_pkg: cache version list

this is a short running script, so the version list can be re-used,
the chance that there where updates in between are slim and racy
anyway. IF getting the versions did not succeeded, we still retry on
every call though, simpler and ensures a warning is printed in the
caller check vicinity.

Makes script noticeable faster.

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

5to6: followup kernel version checks, make compatible with PVE 5

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

5to6: add check for scanned simple OSDs

only as a warning, since this will also trigger a Ceph health warning
and is easily recoverable.

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

5to6: add check for running kernel version

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

ui: ceph wizard: drop local setAllowBlank overwrite

all our combogrids now have this

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

ui: cluster: use new network selector

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

ui: vm migrate: rename icon to severity and uses renderer

and some other minor style changes

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

pve5to6: improve is-node-IP active check

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

pve5to6: add total count in summary

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

pve5to6: align summary counters

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

pve5to6: list packages with updates in new line

with two spaces indentation, like apt does

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

pve5to6: versions: detect if already upgraded

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

pve5to6: add and use print_header

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

pve5to6: make this a simple CLI command

with this it's enough to call `pve5to6`, no (sub)command needed

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

move check_kvm_nested call to check_misc

Signed-off-by: Mira Limbeck <m.limbeck@proxmox.com>

pve5to6: add IP check to misc

Adds the same check we run in pve-cluster before joining a node to make
sure the hostname resolves to a configured IP.

Signed-off-by: Mira Limbeck <m.limbeck@proxmox.com>

Revert "d/control: fix mail-transport-agent-dependency-does-not-specify-default-mta"

This reverts commit 1cda58da007f1e372de81573c1cfddb2e5a01afa.
As else Debian installations always get exim as default MTA, just
lets keep postfix for now

pve5to6: add check for nested kvm

this warns the user that he cannot live migrate VMs with svm/vmx to PVE6 when
the nested parameter of the kvm module is on

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

5to6 add color support

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

5to6: fix ssh config check

in case the config file does not exist at all.

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

use https links to our sites

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

update sources.list to buster

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

switch over default console viewer to xterm.js

at least where possible, this affects mostly the node shell button.

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

bump version to 6.0-0+3

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

gui: cluster: initial work to switch ringX to linkX params

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

backup jobs: Aquire lock before modifying vzdump.cron

Signed-off-by: Christian Ebner <c.ebner@proxmox.com>

ui: workspace: cope better with upgrade related false positive 401 HTTP codes

While we nowadays can work much better with package upgrades relating
the cluster stack it still happens that a pve-cluster upgrade can
produce a false-positive 401 (auth failure) code for a currently
valid ticket, e.g., because a pmxcfs lock was requested but the
pmxcfs was currently not mounted due an upgrade triggered restart.

A frequent case for a few false positive 401 is also a cluster
creation, especially if not done over the web GUI.

Thus add a counter, which gets set to 0 on each successful login or
ticket renewal and gets increased on each 401 error. Only show the
logged out window if we get five or more 401 responses. While 5 may
sound a bit much one needs to remember that we always have quite a
few API call in flight (resource update store, stores from current
panel ...) and thus, if one got really auth denied it will still show
quite fast (1 to 5 seconds, depending on which panel is currently
opened). Further, the backend naturally does not allows to do
anything during this time, this has no security implications
whatsoever.

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

pve5to6: avoid Use of uninitialized value $osd_flags in pattern match

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