buildsys: add excludes to dh_fixperms

Instead of fixing up permissions manually as the make
install target is actually responsible for this (and already
doing it).

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

remove unecessary semicolons

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

whitespace cleanup

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

improve tree/grid icons

we improve the icons in the tree and the resource grid by
differentiating between cluster online/offline status and no rrd data

when we have no rrd data from a node/storage, instead of showing a
red x (which is scary) even if the node is reachable by corosync (which
confused quite a bit of people, because we show all nodes as online in
the datacenter summary), we show the node/storage with a '?'

this signals that something is wrong with this node, even if we can
reach it via cluster methods

this rewrite of the logic includes a refactoring of the method
of getting the icon, because we want the same icons in the tree and the
grid, and an optimization on how we use the css classes
(introducing a x-grid-custom-icon class)

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

add new tree and grid classes for the 'unknown' status

this displays a '?' similar to the check/cross, to show something
is wrong, but we do not know if its offline

also cleanup the classes a bit, so we do not have to have so many
different ones

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

add a new status column in the resourcestore

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

add default 'unknown' status for vms and containers

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

include a 'status' field for storages

by default it is unknown, if the storage is >= 60% full, it is
'nearfull', if it is >= 90% full, it is 'full'

if we have any other information, it is 'available'

we can use this information for the tree

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

add status to node stats

by default it is unknown,
if it is a standalone node its online
else it is either 'online' or 'offline' depending on the corosync status

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

reset FileSelector grid when changing the storage

otherwise, when the new storage returns an error, we show the content
of the old storage

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

use DiskStorageSelector in lxc/MPEdit.js

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

use DiskStorageSelector in qemu/HDMove.js

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

use DiskStorageSelector in qemu/Clone.js

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

use DiskStorageSelector in qemu/HDEfi.js

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

use DiskStorageSelector in qemu/HDEdit.js

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

move lxc mountpoint edit from ResourceEdit.js to MPEdit.js

to follow qemu code

also fix the jslint errors in MountPointInputPanel and MountPointEdit

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

add new DiskStorageSelector.js

this is a wrapper for selecting a storage/disk image

it is a simple container with 4 form fields inside
which can be reused, whenever we need to select a storage for a disk,
image etc.

we have code similar to this four times already
(qemu image creation, lxc mp creation, qemu cloning, qemu efidisk
creation)

so it was time to refactor this and use the storage information from the
backend instead of hardcoding values in the frontend

for this we need to pass the format=1 parameter to the storage api call,
and to not load the fileselector when it is initially disabled

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

Fix #1523: Display the Max Backups field only if relevant

Signed-off-by: Emmanuel Kasper <e.kasper@proxmox.com>

Add Storage Controller class

This controller will display the 'Max Backups' field only if the storage can
hold backups.

Signed-off-by: Emmanuel Kasper <e.kasper@proxmox.com>

add ver get parameter for js/css files to improve cache behaviour

we do not set an Expire header for our js files, and certain browsers
do not even make the request then (on which we would answer
with a correct 304 not modified)

so to force the browser to load a new version of the gui when we change
something, we add the package version as a get parameter

when doing this, the browsers still cache the file, until the get
parameter changes, which is exactly what we want

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

do not use PVE::API2 in spiceproxy.pm

we do not need it there and withouth this we save ~30MB memory for
this daemon and its workers

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

prevent visual flashing of button when changing element in the tree

when changing between elements in the tree, either the
start or shutdown buttons were visually flashing (meaning they are
enabled/disabled for a short time and change state shortly after)

this patch fixes that

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Acked-by: Emmanuel Kasper <e.kasper@proxmox.com>

fix #1539: add missing shellquote for proxied commands of pvesh

if we do not do this, passing arguments with spaces
(e.g., pvesh set YYY --param "one word")
leads to ssh calls like this:
ssh REMOTE pvesh set YYY --param one word

which cannot be parsed correctly

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

Add %-used and min_size to "pveceph lspools" command

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

Fix: #1542, use percent_used key instead of calculation

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

FirewallOptions: fix DHCP default value

we always had dhcp enabled by default in the backend, but
showed 'Disabled' on the gui, this patch show 'Enabled' by default

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

bump version to 5.1-36

bump version to 5.1-35

bump version to 5.0-34

make bluestore the default on the api

with this we also have to send '0' to from the frontend, when the
bluestore checkbox is not checked

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

Render full name of VirtIO Block controller

This will hopefully reduce confusion with SCSI VirtIO controller

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

Update VM SCSI controller type after OS selection in Wizard

This was done previously without any kind of user visible feedback, so users
would not know that they had a Virtio SCSI controller as default controller
for Linux VMs.

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

Update VM SCSI type based on iothread value when inside the wizard

We keep the same logic as before: once the iothread option is selected,
the SCSI controller is switched to virtio-scsi-single, until iothread
is unselected again.

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

Add a field to show which SCSI controller type we are currently using

We use the field label 'SCSI Controller'. 'Type' is redundant, otherwise
we would have Network Card Type, Display Type, Network Card type, etc ...

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

Do not use the noVirtIO property to build the controller list

This property is only used for CDEdit, so we can deduce this from
the fact that autoselect is set to 'cdrom'

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

Save the components we found via down() in a variable for later reuse.

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

Remove unused noScsi parameter

SCSI is a valid bus for Disks and CD drives, so it should always
be appear in the bus list.

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

Do not use 'autoselect' as a boolean when preselecting a bus

The bus selector is displayed when we add a Hard Disk or CD Drive.
When it is displayed, we *always* preselect the next available
slot on the controller of our choice.
So this test is not needed.

We keep the test on the string value of 'autoselect' to select
a bus position when adding a CD Drive.

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

fix #1518: ui: lxc: add 'more' menu with HA manage item to panel

Move the remove button into the more button and add a 'Manage HA'
button there, mirroring the changes from the qemu panel.

Allows to add an unmanaged CT to HA or to edit the HA settings of a
managed one.

Fixes: #1518
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Acked-By: Dominik Csapak <d.csapak@proxmox.com>

ui: lxc: remove obsolete unmount button

This button was hidden by default and was intended to show up when a
CT is mounted only.

The check if mounted, was done for OpenVZ and may not become true
currently also we only have CLI helpers for mount/unmount but no API
– so just remove it.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Acked-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

fix #1518: ui: qemu: add HA manage menu item to panel

Allows to add an unmanaged VM to HA or to edit the HA settings of a
managed one.

Fixes: #1518
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Acked-by: Dominik Csapak <d.csapak@proxmox.com>

ui: qemu: combine system changing buttons

add a 'More' Button to the VM and CT Config Panel (its base panel)
and move all buttons which may result in config changes there.
After that we have separated power/migrate actions from system
affecting actions (clone, remove, convert to template).

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

ui: add PVE.menu.Item

Extend 'Ext.menu.Item' with a simplified handler logic also used in
'PVE.button.Button'.

If 'confirmMsg' config is set we wrap the defined handler in a
confirm dialog, useful if the menu item just makes an API call and
does not has an own (edit) window shown.

In contrast to the 'pveButton' we do not have a selection model,
enable function and the respective logic here.

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

ui: rename global 'Help' button to 'Documentation'

To make it more clear that this will always end you on the
Documentation index.

Also, multiple Help buttons which all may or may not do something
different are a bit weird/confusing - thus distinction can here be
really helpful, IMO.

We target HD ready (1280x720) as minimal target resolution where this
still fits nicely. This (or bigger) resolution is also used by our
screenshot framework for the documentation - so it also fits there.

Adapt the icon displayed to `fa-book` which resembles the one often
used for manuals and is clearer than a '?', IMO.

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

ui: ceph: add some help buttons

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

ui: pveWindowEdit: allow sane onlineHelp specification for edit windows

instead of heuristically choosing the onlineHelp entry from our first
item prefer the one we have in our config, if noone the fallback to
the old heuristic.

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

Use run_cli_handler instead of deprecated run_cli

test: mock PVE::Cluster::cfs_update

Because the replication's purge_old_state() function now
fails if cfs_update() fails, and since it tries to access
the actual local pmxcfs it should never have been called by
tests anyway.

ui: ha: make vote warning shorter

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

ui: ha: use vmComboSelector when adding new resource

Make adding resources more comfortable by showing the user also the
name, type, node, ... in a nice combo box grid when adding a
resource instead of just a integer spinner.

Fixes #1517
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Acked-by: Dominik Csapak <d.csapak@proxmox.com>

ui: add VM ComboSelector

this is a mixture of the multi select VMSelector and the single
select pveGuestIDSelector.
Useful when a single VM must be selected but also complementary
information should be shown to the user.

Most of the times the VMID is not really useful for an user as
numbers are harder to remember and to associate as names.
So a dialog which need a VMID (e.g. HA resource addition) forced the
user to lookup the Guest Name -> Guest VMID mapping and then enter it
correctly in the respective input field.

This can be improved by using a combo grid based selector which shows
additional information in the edit window itself, can be sorted and
filtered to quickly select a guest.

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

ui: GuestStatusView: simplify HA state display

simplify HA Information in VM/CT Summary panel.
Use the already translated  'HA State' instead of 'Managed by HA'
(which now becomes obsolete as it was the single appearance).
Simplify PVE.Utils.format_ha respectively to mach the new name,
as here is the only use of this function we have no side effects.

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

ui: re-align global search field to now shorter version

commit 2ddb072661a9230dac16ec91c2f4ba91b8144dba removed showing the
short SHA1 commit hash in the version info field but did not
realigned the search field so that it got a asymmetric space.

Adapt the Version info minimal width so that this is again the case.

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

ui: inline novnc: always focus console panel after render

Enforce focus on the Inline CT/VM/Host Console panel backed by noVNC.

Else, if a Console is already active on VM A and we select VM B the
console there does not gets the focus as its now in the three and
regaining focus to the console is not fully intuitive - i.e. just
clicking on the console won't do the trick, you have to click on
specific spots (e.g. console panel header) or re open the console.

The only possible drawback of the focusing is that if one has the
console open, selects another VM he cannot navigate the tree down
with the arrow keys (as long as the console is open) - but thats
hardly a situation where one gets into and can be worked around.

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

fix #1531: match nodename exactly for vmidselector

else we would get vms for wrong hosts if the nodename is a pre/postfix
of another nodename (e.g. 'abc' and 'abc123')

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

fix updateStore destroy

the store never fires/gets a destroy event, thus it
never stops the load_task and never unqueues itself in case
the store gets destroyed but not explicitely stopped

with overriding the destroy method, this works now as intended

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

ha/resource: add ignored state to web gui

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

bump version to 5.0-33

and add versioned dependencies:
- libpve-common-perl: for PVE::Subscription
- libpve-access-control: for new VM.Snapshot.Rollback privilege

ceph: disable OSD unit in destroyosd

while OSDs units should only be runtime enable and disappear on reboots,
this serves as an additional safeguard to ensure no leftover units can
exist.

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

ceph: correctly parse 'pg dump' output

Reported-By: Waschbüsch IT-Services GmbH <service@waschbuesch.it>
Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

fix #1516: fix mouse-scrolling of overflowhandlers in firefox

this is a workaround for scrolling in toolbars, etc. in firefox with the
mouse

while the result is not very "pretty", it maintains the old behaviour
for all other browsers and makes it work in firefox

we can drop this when we update to a new extjs release

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

doc: spice-example-sh: support special characters in passwords

doc: spice-example-sh: fix ticket/csrf extraction

We extract them from the json data by first deleting all
quotes, then deleting everything up to their entries, then
everything after the next comma. If the entry is at the end
there's no comma, instead there are closing braces, so we
need to strip away these as well.

doc: spice-example-sh: add quotes everywhere

build: reformat debian/control

using wrap-and-sort -abt

fix #1510: do not scroll back to focus when view gets refreshed

this adds a workaround, so that we restore the scroll position
when we restore the focus

with the flag 'jumpToFocus', we can enable the old behaviour

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

PVE/API2/Subscription.pm - use new PVE::Subcription class

Subscription.js - use enableTextSelection instead of 'selectable' feature

The 'selectable' feature does not work anymore.

Clean up unnecessary code.

The jobid is already parsed by the file parser.

remove obsolete Posix use clause

besides the cleanup purpose this fixes an actual problem, perls POSIX
module has the following caveat:

`Everything is exported by default (with a handful of exceptions).
This is an unfortunate backwards compatibility feature and its use is
strongly discouraged. You should either prevent the exporting (by
saying "use POSIX ();", as usual) and then use fully qualified names
(e.g.  "POSIX::SEEK_END"), or give an explicit import list. If you do
neither and opt for the default (as in "use POSIX;"), you will import
hundreds and hundreds of symbols into your namespace.'

see `perldoc POSIX`

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

window.Wizard: increase default size

The current size was choosen with the older, more compact, classic
theme. The Crisp theme, now in use, has a bit more spaces and bigger
font sizes to adapt to the increased pixel density of current
displays.
Increase the size a bit (~13%) but try to keep the aspect ratio.

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

PVE.qemu.OSTypeEdit: declare static

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

wizard: merge CD/DVD and OS panels

We had a single column for the CD/DVD panel, fill the second column
with the now in size reduced OS type selector, as ISO selection and
OS type selection are tied together.

Reduces steps needed for creating a VM.

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

OSType edit: switch to combobox

Move from the radiogroup to two comboboxes, where the base OS and
then a specific Version can be choosen.

Avoids multiple occurences of strings like "Microsoft Windows" and
saves a lot of space - which will be used to merge this panel with
the ISO selector panel in the create wizard.

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

kvm_ostype: move to store-like format

move from the key => value format to one where all versions from a
base OS type can be directly used as data for a store.

We will rely on this in the next patch to allow a easy transition to
combobox UI for the OS type selector.

We also avoid having the base type multiple times.

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

refactor OSDefaults, OSTypeEdit: add and use getDefaults

encapuslation wise it makes sense to have a helper which assembles a
os specific setting object in the OSDefaults class, where those
settings are defined.

Else we always need to check if the current OSType on the user side
of this class, which is against encapsulation.

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

ui: reuse ResourceStores default handling for guest description

First, there was an additional space between "'" and ")" which lead
me to this.
I could never trigger the case where vmname is undefined, and saw
that we have a almost same default handling in data/ResourceStore.js

So, just use that one instead here - it misses the single-quotes
inside the parentheses, but as there are already said parentheses
this is visually still very OK, IMO.

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

use log method from base VZDump::Plugin

we copied our log method over there to resolve a cyclic dependency,
now use it here to reduce code duplication.
As we are below pve-guest-common in the dpendency hierachy we may use
it, but it does not may (or at least should) use us.

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

VM.Snapshot.Rollback privilege added

VM.Snapshot.Rollback privilege added

Signed-off-by: Matthias Urban <matthias.urban@pure-systems.com>

vzdump/pigz: use ProcFSTools for core count, not sysconf

A "sysconf(84)" is not really descripitve. There is no POSIX const
entry for _SC_NPROCESSORS_ONLN (number of processors online) as its
not standard but an adoption of glibc.
Better just use our proven ProcFSTool's cpuinfo method for getting
the desired information.

This is kept only for backwards compatibillity as we guaranteed that
we use #cpus/2 if 'pigz' is set to 1, else I would have removed this
alltogether and just directly passed 'pigz' to the command...

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

Turn the offline/online checkbox of the Migrate Windows into a displayfield

This checkbox had not effect whatsoever:
 * if the VM was online and offline was selected, the migration would fail
 with the message that the --online flag is needed for running VMs
 * if the the VM was offline and online was selected, the migration would
 happen offline anyway

Rework Migrate Window with declarative synthax

Also group all qemu vs LXC tests in a single place.

Allow to set the onlineHelp via a config Object

For cases when we need to set the online help block id of a modal window after the help
button has been configured.

bump version to 5.0-32

Disallow quotas for unprivileged containers

Disables the quota checkbox for unprivileged containers in the creation
wizard, as well as when editing or adding mountpoints.

Whitespace and typo fix

init: shorter pve-guests.service Description

rename pve-manager.service to pve-guests

the name 'pve-manager' collides with our pve-manager package name,
which - from the user point of view - provides mainly the API and
WebUI.
An user could thus think that restarting 'pve-manager' would restart
the WebUIs server, which is relatable.
But, the pve-manager.service does not controls the WebUI or its
server but is responsible for starting all guest with 'onboot=1' in
their config on system boot and to stop all remaining running guests
on system shutdown.

Thus rename it to pve-guests and adapt its description. This may not
seem as ideal name at first glance, but its better than the current
option. Further it leads to log messages like:
 > Starting PVE guests (Service providing start-on-boot and stop-all-on-shutdown)
 > [...]
 > Started PVE guests (Service providing start-on-boot and stop-all-on-shutdown)
 > [...]
 > Stopping PVE guests (Service providing start-on-boot and stop-all-on-shutdown)

which makes it clearer what happens, or what this service is for.

Alias the new service to the old pve-manager.service for legacy
reasons. While our services do not depend on it an user could have
made an own service which used pve-manager.service as synchronisation
point.

Linitian then complains about init.d/pve-manager not having a related
systemd service file. Instead of renmaning it just drop it.

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

pve-manager.service: move open-iscsi ordering to pve-storage.target

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

pve-manager.service: allow logging STDERR and STDOUT

The API calls are not to chatty but may give helpfull hints about
what gets tried to be done.

This may help an Admin to figure out which guest delays his host
shutdown.

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

pve-manager.service: refuse manual start/stop

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

pve-manager.service: do not allow stop to timeout

This services is responsible for bringing up 'onboot' marked guests
on host power-on and stop _all_ guests gracefully on host shutdown
(be it reboot, shutdow, halt, ..)

It's type is set to 'oneshot', which implies a TimeoutStartSec of
'infinity', by default. With Jessies Version of Systemd the
TimeoutStopSec defaulted to TimeoutStartSec, if not set – so also
'infinity'.
But, Debian Stretchs Version of Systemd makes TimeoutStopSec defaults
to 'DefaultTimeoutStopSec' if it was not set, which is by default 90
seconds – much less than infinity.

This may cause non-gracefull shutdowns of guests, as after the 90
seconds systemd sends a SIGKILL to the pvesh 'stopall' process.
This may end in a bad guest state then. But besides that it can also
lead to a hanging shutdown in some cirumstancesm, as some guest still
operated on storages, so systemd-shutdown - the binary which gets
exec'ed by systemd to become the new PID 1 cannot finish its
sync/umount/shutdown procedure. It has a watchdog armed on sync, if
that triggers you may even get a fully shut down system.
Else it can possibly hang forever, at least until the power plug gets
pulled or similar actions are taken.

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

show 'RBD' instead of 'RBD (external)' on storage summary

we cannot differentiate between external and pve-managed there,
so we just want to show 'RBD'

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

drop unnecessary check for 'local'

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

make bluestore the default choice when creating osds

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

add new "RBD (PVE)" storage choice

this is for adding a pve managed ceph rbd storage, so that the user
just has to select the pool, and does not need to write the monitor
hosts and copy the keyring

the old "RBD" is renamed to "RBD (external)"

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

add CephPoolSelector

this allows us to give the user a list of pve managed ceph pools

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

ceph: do not automatically use wal if only journal is given

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

rename 'Remove' ceph pool button to 'Destroy'

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

enable showProgress for create/destroy pool

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