Commit Graph

3204 Commits

Author SHA1 Message Date
Dominik Csapak
d6f5549e58 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>
2017-09-07 12:35:14 +02:00
Dominik Csapak
1fb090840d ceph: do not automatically use wal if only journal is given
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
2017-09-07 12:35:14 +02:00
Fabian Grünbichler
4358abfdf4 rename 'Remove' ceph pool button to 'Destroy'
Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
2017-09-07 11:38:20 +02:00
Fabian Grünbichler
7244f4bfb6 enable showProgress for create/destroy pool
Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
2017-09-07 11:38:20 +02:00
Fabian Grünbichler
707c28ce80 add showProgress to SafeDestroy
modeled after the mechanism used in window/Edit.js

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
2017-09-07 11:38:20 +02:00
Dominik Csapak
cb852a4dcd add remove_storages parameter to the pool destruction
automatically remove the pveceph managed storages when deleting the
pool on the gui

this is ok because we cannot delete the pool anyway when we have images
on it, and recreating them when creating a pool is now trivial

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
2017-09-07 11:38:20 +02:00
Dominik Csapak
5947248fac add add storages checkbox to ceph pool creation
to be able to automatically generate the ceph storages when creating a
pool

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
2017-09-07 11:38:20 +02:00
Dominik Csapak
1f8a29e804 add a params object to the safedestroy window
we sometimes want to give the api call a parameter,
with this, we don't have to encode it into the url
everytime ourselves, but just give a 'params' object

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
2017-09-07 11:38:20 +02:00
Fabian Grünbichler
fe15d330c4 add task description for cephcreatepool
Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
2017-09-07 11:38:20 +02:00
Fabian Grünbichler
0a521b6634 ceph/destroypool: move image check into worker
vdisk_list can potentially take very long, and we don't want
the API request to time out.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
2017-09-07 11:38:20 +02:00
Fabian Grünbichler
8c42620460 ceph: make create/destroypool API paths async
in order to get task log entries and easily accessible
task/error logs.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
2017-09-07 11:38:20 +02:00
Fabian Grünbichler
c75f019a55 ceph/destroypool: refactor to use get_storages
Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
2017-09-07 11:38:20 +02:00
Fabian Grünbichler
f6144f3493 ceph/destroypool: optionally remove storages
only storages which don't have the 'monhost' option set are removed

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
2017-09-07 11:38:20 +02:00
Fabian Grünbichler
60811ad7eb ceph/createpool: optionally add storages
introduce new API parameter 'add_storages'. if set, one
storage each is configured using the created pool:
- for containers using KRBD
- for VMs using librbd

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
2017-09-07 11:38:20 +02:00
Fabian Grünbichler
0c0d43a7cc ceph: create/destroypool: shorten variable name
Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
2017-09-07 11:38:20 +02:00
Fabian Grünbichler
f4aae93bbf ceph: add get_storages helper
Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
2017-09-07 11:38:20 +02:00
Fabian Grünbichler
ae672a64ce ceph: add add_storage helper
Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
2017-09-07 11:38:20 +02:00
Fabian Grünbichler
a0946ccba8 bump version to 5.0-31 2017-09-01 12:05:14 +02:00
Thomas Lamprecht
3695863919 disable discard for non-scsi disks
Adapted from: Dominik Csapak <d.csapak@proxmox.com> version

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2017-09-01 11:07:43 +02:00
Thomas Lamprecht
ed1ac8f4b3 create: clear iothread on disable
A user may set the `iothread` option and then change the bus type to
something which does not support it (sata, ide), while we disable
it the previously set value remains, which is confusing.
An user could think he coulkd trick the Wizard into using iothreads
for SATA/IDE but we just ignore the value in this case.

Set iothread to false in the case it gets disabled.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2017-09-01 11:07:43 +02:00
Emmanuel Kasper
2a2dce7037 Change the SCSI controller type when editing a harddisk only when creating a VM.
If not, the following could happen:
 * user has a VM with two disks attached to a LSI controller
 * adding a third disk, he notices the iothread option, and clicks it
 * the scsihw type for all disks is changed now to virtio-scsi-single
 * on next cold restart, STOP 0x0000007B INACCESSABLE_BOOT_DEVICE for a Windows VM

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2017-09-01 11:07:43 +02:00
Thomas Lamprecht
8722af4a58 remove unused variables
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2017-09-01 11:07:43 +02:00
Thomas Lamprecht
2d06e72392 HDEdit: push more items in the same call
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2017-09-01 11:07:43 +02:00
Thomas Lamprecht
a1fe04b65d disk/controllerselector: adapt flex
The bus tye selector gets now 2/3 of the space and the bus number
spinner gets 1/3.

It looked a bit strange with both of them getting half of the space,
as on a lot of GUIs the spinner get less space (its just a number
(max 13) vs a wider text).

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2017-09-01 11:07:43 +02:00
Thomas Lamprecht
79a14fbbd8 CreateWizard: remove title from confirm tabpanel
It isn't needed as its obvious that the table below show the
configuration. It just takes away space.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2017-09-01 11:07:43 +02:00
Dominik Csapak
8ad3421180 hide migrate in contextmenu when no cluster or no rights
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2017-09-01 11:07:42 +02:00
Dominik Csapak
3dc5644c9e hide migrate button in lxc for non cluster setups
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2017-09-01 09:59:48 +02:00
Dominik Csapak
6ecdb34077 hide not usable buttons for qemu
this hides not usuable buttons for qemu
e.g. start/shutdown for templates
migrate for non-cluster setups

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2017-09-01 09:59:48 +02:00
Fabian Grünbichler
e16cd81fb5 pveceph/init: improve version check error message 2017-08-25 14:16:09 +02:00
Alwin Antreich
c64c04dd39 implement checks for ceph version & binaries
add version check to ceph init to require luminous or higher and
fix #1481: check existence of ceph binaries before use

Signed-off-by: Alwin Antreich <a.antreich@proxmox.com>
2017-08-25 13:46:42 +02:00
Dominik Csapak
b01e7f7a20 ceph dashboard: better format the warning detail window
this adds the summary as first line,
and the long warning message as monospaced text (like on the console)

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
2017-08-25 13:46:42 +02:00
Dominik Csapak
37f01f69da ceph dashboard: add missing summary/message access level
in luminous, the output of the status/health has changed (again),
so we have to access the correct properties

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
2017-08-25 13:27:09 +02:00
Thomas Lamprecht
9a8a573568 qemu/create: use 'virtio-scsi-single' if 'iothread' is set
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Reported-by: Nils <gismo3303@gmail.com>
2017-08-22 10:49:57 +02:00
Thomas Lamprecht
7a6e0635cf qemu/HDEdit: whitespace cleanup
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2017-08-22 10:42:42 +02:00
Dominik Csapak
134b4017eb fix nodeselector when no element in the tree is selected
it can happen that nothing in the tree is selected,
then PVE.curSelectedNode is undefined, generating an exception there.
so we check if this is available

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
2017-08-22 10:27:03 +02:00
Dominik Csapak
bf6e58d2d1 make the ceph warning window modal
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
2017-08-10 09:38:29 +02:00
Dominik Csapak
d0b1e57003 fix ceph gui icons
this adds a new css class for fontawesome which we need under certain
circumstances (it is copied from fontawesome minus a font-size:inherited)

and it changes the icons in the osd tree from x-fa to fa
(which is the correct one there)

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
2017-08-10 09:38:29 +02:00
Dominik Csapak
6c11e92154 ceph: add application parameter to createpool
and use 'rbd' by default, since we cannot create a cephfs or
rgw from the gui currently

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
2017-08-10 09:38:29 +02:00
Dominik Csapak
6747b0a903 ceph: make size 3/min 2 default with createpool
in the gui this is already the default, so make it also the default
in the backend (also 2/1 is really bad as a default)

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
2017-08-10 09:38:29 +02:00
Fabian Grünbichler
2944eea8a6 build: ensure REPOID has a length of 8
because unlike the git-rev-parse man page says, the default
length is actually calculated based on the number of packed
objects in the clone of the repository.

we don't want this to depend on the build environment and
how the repository was cloned there, so fix a length of 8
for now.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
2017-08-09 13:29:31 +02:00
Fabian Grünbichler
ad98a36fc1 bump version to 5.0-30 2017-08-08 11:08:24 +02:00
Dominik Csapak
6dd90c403e add information about bluestore in the gui disklist
this adds if an osd uses bluestore in the disklist,
and shows if a disk is used for osd journal/db/wal

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
2017-08-07 14:52:02 +02:00
Dominik Csapak
45ea97f209 fix jslint error
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
2017-08-07 14:52:02 +02:00
Dominik Csapak
7d406f1805 show bluestore info in the osd tree
namely the osd type (filestore/bluestore), bluestore device,
db device and wal device

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
2017-08-07 14:52:02 +02:00
Dominik Csapak
4d422ffc3e add bluestore information to osd api call
this adds information about bluestore (which devices and if
bluestore/filestore) to show in the gui

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
2017-08-07 14:52:02 +02:00
Wolfgang Bumiller
db9770dfd7 bump version to 5.0-29 2017-07-31 15:47:17 +02:00
Fabian Grünbichler
040f1aeb6c ceph: ensure mgr starts after pve-cluster
Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
2017-07-31 15:45:32 +02:00
Fabian Grünbichler
74911546dd bump version to 5.0-28 2017-07-31 15:31:33 +02:00
Dominik Csapak
7566b294b6 also mention manager in create monitor title
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
2017-07-31 15:25:34 +02:00
Dominik Csapak
9ad28182db indentation and jslint fix
indentation was wrong on those lines, and js lint complains about
alias not being an array later, so make thoses lines not an array

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
2017-07-31 15:25:34 +02:00