Commit Graph

4311 Commits

Author SHA1 Message Date
Thomas Lamprecht
cfb103131f 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>
2019-07-11 14:16:11 +02:00
Thomas Lamprecht
413ada0fbb 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>
2019-07-11 14:16:11 +02:00
Thomas Lamprecht
a336875216 ui: ceph/osd: only show at max 5 digits after decimal point for weight
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2019-07-11 14:16:11 +02:00
Thomas Lamprecht
3b0ae6b2f0 ui: ceph/osd: rename Action to More
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2019-07-11 14:16:11 +02:00
Thomas Lamprecht
a3f146fcc1 ui: ceph OSD: tweak default column width a bit
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2019-07-11 14:16:11 +02:00
Thomas Lamprecht
35c9c3d6da 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>
2019-07-11 14:16:11 +02:00
Dominik Csapak
3982a21441 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>
2019-07-11 14:16:11 +02:00
Thomas Lamprecht
9cc5ac9e75 api/ceph: code cleanup
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2019-07-11 14:16:11 +02:00
Dominik Csapak
b7701301a8 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>
2019-07-11 14:16:06 +02:00
Alwin Antreich
ea60e3b72e 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>
2019-07-10 15:42:49 +02:00
Alwin Antreich
177095661b 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>
2019-07-10 15:41:20 +02:00
Thomas Lamprecht
8144a0ddc7 ui: dc/options: minor followup cleanups
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2019-07-10 14:58:19 +02:00
Thomas Lamprecht
6de2baee56 ui/Parser: printPropertyString: sort non-defaultKey parts for stability
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2019-07-10 14:57:40 +02:00
Thomas Lamprecht
6619af8d4f ui: dc/options: use proxmoxNetworkSelector for network
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2019-07-10 14:38:49 +02:00
Thomas Lamprecht
c195a3c2d6 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>
2019-07-09 15:07:41 +02:00
Thomas Lamprecht
77a8d952e0 followup: code cleanup to arrow functions
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2019-07-08 19:26:13 +02:00
Dominik Csapak
088fff5fa4 gui: show 0 for max_relocate/restart correctly
0 || '1' will always return '1'

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
2019-07-08 19:21:53 +02:00
Dominik Csapak
47d13c02b0 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>
2019-07-08 19:19:15 +02:00
Fabian Grünbichler
aa43978d16 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>
2019-07-08 10:44:41 +02:00
Fabian Grünbichler
82662dc83a 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>
2019-07-08 10:44:41 +02:00
Fabian Grünbichler
a2e5aa4e3e 5to6: add more corosync subheaders
to improve readability

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
2019-07-08 10:44:41 +02:00
Fabian Grünbichler
d1dcb3e96e 5to6: fail if a corosync node has neither ring0 nor ring1 defined
Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
2019-07-08 10:44:41 +02:00
Fabian Grünbichler
b481d354c8 5to6: reword/-structure corosync message
and fix a typo as well

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
2019-07-08 10:44:41 +02:00
Fabian Grünbichler
0e78b5157e 5to6: attempt to resolve corosync rings
and only fail if unable to

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
2019-07-08 10:44:41 +02:00
Thomas Lamprecht
cc6bb2ec3e update shipped aplinfo index
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2019-07-08 07:52:34 +02:00
Thomas Lamprecht
7b6c78818c 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>
2019-07-05 18:55:09 +02:00
Oguz Bektas
c115473dae 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>
2019-07-05 18:50:04 +02:00
Thomas Lamprecht
f9bbe5feac followup: hide mask, level and RO by default
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2019-07-05 12:30:28 +02:00
Dominik Csapak
3af21bae7e 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>
2019-07-05 09:52:58 +02:00
Dominik Csapak
21e413dc01 ceph: add api call for config database
simply returns the database content

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
2019-07-05 09:52:58 +02:00
Thomas Lamprecht
93580fec68 use new url
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2019-07-04 17:57:27 +02:00
Thomas Lamprecht
36ee2702f8 bump version to 6.0-1
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2019-07-04 17:10:23 +02:00
Dominik Csapak
0b50996da5 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>
2019-07-04 17:09:48 +02:00
Thomas Lamprecht
cf06393a7a pveceph install: set repo URL to download.proxmox.com nautilus repo
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2019-07-04 15:52:51 +02:00
Dominik Csapak
9e5460013f 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>
2019-07-04 15:38:12 +02:00
Thomas Lamprecht
99181ca860 5to6: only tell to solve problems if fails are present
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2019-07-04 15:32:35 +02:00
Thomas Lamprecht
c50590ef72 5to6: improve final note on errors and/or warning a bit
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2019-07-04 15:30:54 +02:00
Thomas Lamprecht
0e788b3e07 5to6: fiy typo: s/detailled/detailed/
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2019-07-04 15:23:19 +02:00
Thomas Lamprecht
3724703551 5to6: check common services pveproxy pvedaemon pvestatd
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2019-07-04 15:20:28 +02:00
Thomas Lamprecht
14886691a2 5to6: add log_systemd_unit_state and use for pve-cluster and corosync
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2019-07-04 15:20:28 +02:00
Thomas Lamprecht
bd6a59c2d3 5to6: import run_command
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2019-07-04 14:56:50 +02:00
Thomas Lamprecht
5c420879bf 5to6: clarify "cluster quorate" message a bit
As this is the state from the pmxcfs

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2019-07-04 14:21:10 +02:00
Thomas Lamprecht
23a60185fb 5to6: fixup: really set $total_votes to 0 if not defined
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2019-07-04 14:20:41 +02:00
Fabian Grünbichler
6ab8927e85 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>
2019-07-04 14:06:13 +02:00
Fabian Grünbichler
82fd088c0d 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>
2019-07-04 14:06:13 +02:00
Fabian Grünbichler
5d54e2ee38 5to6: quote some Ceph flags/options
to improve readability

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
2019-07-04 11:37:17 +02:00
Fabian Grünbichler
880563233c 5to6: check for ceph global keyring config
which causes issues after upgrading to Nautilus.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
2019-07-04 11:37:17 +02:00
Thomas Lamprecht
60b81f16cf 5to6: followup: also detect ceph conf keys separated with -
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2019-07-04 10:37:35 +02:00
Thomas Lamprecht
f9f8cd7d01 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>
2019-07-04 10:31:08 +02:00
Thomas Lamprecht
780a57b6d1 5to6: ceph global mon host check: also detect mon-host
and adapt message a bit

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2019-07-04 10:30:39 +02:00