Make the Snapshot selector an optional parameter in the Clone Window

The snapshot selector was known to confuse first time pve users.
Some wondered that they had no snapshot called 'current' in their
snapshot trees, and other thought that snapshots would be included
in the copied image.

So before displaying the Clone Window, do an API call to verify
if the source VM has snapshots.

Disable the submit button based on whole form validity

Until now we disabled the submit button, based on
the feature API call where we tested if the selected VM, and its snapshot
can be used as the source of a clone.

This had the following problems:
 * the feature test for copy clone / linked clone always returned true,
 since the passed paramaters were already filtered before user input
 (ie we only allowed a linked clone to be selected if the source VM is
 a template, or a snasphot to be selected if existed on source VM)
 * the guest ID input field was not validated

With the current patch, the validation is now made on each of the form
fields. verifyFeature() is still used to populate a list of valid nodes
passed to the Node Selector.

Use correct parameter for GuestIDSelector

Better naming for kv1 comb box, document the two private functions

add sort to jsgettext.pl

the output from find may not be stable (depending on the filesystem),
and is surely not the same on two different systems,
so for consistency in the po files, we sort the output from find with
sort

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

update po files

remove duplicates (see previous commit)

update gettexts to avoid duplicates in po file

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

remove obsolete inline documentation

add type column to storage content view

and move the grouping feature in the class definition while changing
the text to a generic one

this allows us to group by all columns and still have meaningful
grouping headers

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

remove unnecessary emptyText

we already set it statically

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

correct and insert german translations

this patch adds missing and corrects wrong translations for german

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

check general health for ceph monitors, not only time

we did not check the healthservices array for the monitor widget,
this patch does that, and takes the worst health state

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

fix #1247: disregard timecheck for one monitor

if we only have one monitor, set the timecheck to HEALTH_OK
(since we get none)

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

update translation files

fix jslint errors

since the value at the top is now directly a string,
we need the jslint confusion at the number and not for the
string at the bottom anymore

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

fix gettext strings

we expect a gettext on a single line and to not contain any
variable/logic, so we remove the long text from it altogether (since we
do not want to translate that at the moment), and we rearrange the code
so that gettext only contains strings

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

disable submit button in migrate window when not valid

when the form is not valid (e.g. an offline or no host selected),
disable the submit button

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

show only relevant vms in bulk action window

this adds a default filter for bulk start and stop, so that only the
relevant vms are visible (stopped and started respectively)

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

make filtered columns in grid more prominent

this makes the column header not only italic but also bold, so that one
can more easily see that the column is filtered

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

Reload the grid list of backups after the backup task completion

This allows visual feedback for first time users doing a backup.

change Start/Stop/Migrate all to Bulk actions

this uses the new vmselector and the new vmid filter in the backend
to allow starting/stopping/migrating selected vms instead of all

by default all vms are selected to have the same default behaviour

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

add vmselector form field

this is a form field which is a grid for selecting vms

if nodename is given, it will filter the vms only to the given node

you can filter the grid with the column header, and only the selected
and visible items are in the value of the field

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

add BulkAction window class

this is mostly copied from MigrateAll.js, but a more generic way,
to allow startall and stopall to also use it

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

add vmid filter to (start/stop/migrate)all

this is a simple filter which allows us to limit the actions to specific
vmids

this makes it much simpler to start/stop/migrate a range of vms

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

ha: re-add node to service status

On the old HA status we saw where a service was located currently,
this information was lost when we merged the resource and the status
tab.
Add this information again.

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

remove not needed vnc code

since we access novnc always with the NoVncIndex.pm and never in extjs
itself, we can delete this code

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

remove vznet.conf

leftover from openvz - not used anymore...

Reload the Ceph OSDTree afer adding an OSD

Hide the Edit window when we start a background task

We closed the Edit window too early, so listeners set to
the 'close' event would reload the underlying grid before the action
successfully completed on the server side.

fix #1230: add blocksize for ZFSPoolPlugin

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

Clone.js: add onlineHelp

HA: Add a warning/info message for HA setups < 3 votes fix: #1228

Fix a race conditon provoking the error no such file '/pve-storage-content'

(NB: the error was only displayed intermittently in the browser console)

The reload call could try a store reload before the store has
a defined url.
The call here is useless since the initial display of
the storage selector compoment will call setStorage() to pass an url
to the store and load it.

Also remove the test on me.store.proxy.url since it was not working
(ExtJS autodefined an URL based on the model name)

Replace nested ternary operators with if/else statements

perftest2.pl: use -k (keepalive) to show max performance

remove PVE::API2Client class and related examples

We now have all API client code in an extra package:

https://git.proxmox.com/?p=pve-apiclient.git;a=summary

code cleanup

bump version to 4.4-5

add unpriviledged flag to container restore window.

bump version to 4.4

Change remove partition order.

If we remove first the journal the data partition will automatically mounted
and can't destroy the partition.
This is trigger by the udev ceph rule.

Fix parsing block device.

Get the partition num and block device from sysfs.
This ensure different block device types will work.

bump version to 4.4-3

Enable service for ceph monitor.

We do not use the ceph.service what normally start ceph-mon,
so we have to ensure ceph-mon is enabled.

replace SysV init script with our own service

for more information see http://tracker.ceph.com/issues/18305 .

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

Set correct permission for ceph user.

We have to set the correct permission,
because ceph greater than infernalis use ceph as daemon user.

add jewel

rename pve.conf to ceph-after-pve-cluster.conf

add systemd replacement for ceph init script

the old sys V init script does a lot of stuff, most of which
is already replaced with the system units since jewel,
except for the OSD activation which is still done by either
udev or the old init script.

include systemd service as drop in replacement for the init
script when using jewel. the service is not enabled by
default (as this would break hammer), but needs to be copied
to /etc/systemd/system on upgrade to (manual) or install of
jewel (automatic).

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

do not hide ceph on dashboard if the api call fails

with this patch, we do not hide the ceph status anymore if the api call
fails (too slow, e.g.) as long as we showed it at least one time

also, we only stop the updates if we never successfully got the status

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

add filter to dashboard storage graph

with this patch, the storage graph on the dashboard only counts the
storages which are selected in the settings window

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

add a storage selection in the settings window

this will be used to filter the dashboard storage graph to the selected
storages, so that one can choose which storages are relevant

e.g. if you have one nfs server with multiple exports (on the same filesystem),
you only want to count it once

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

load the diffstore on init

if the rstore in the diffstore is already loaded at the time of
the instantiation, load the data instantly

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

add an alias to the diffstore

to use it declarative like this:

store : {
    type: 'diff'
}

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

bump version to 4.4-2

rebalance_lxc_containers: make it work with old style lxc setups

rebalance_lxc_containers: avoid repeated warnings if rebalance fails

Only warn once.

rebalance_lxc_containers: fix hotplug

factor out code to modify cpusets into $modify_cpuset->()

rebalance_lxc_containers: make it work with new lxc/<ID>/ns subgroup

remove debug log statement

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

fix typo in ceph jewel iops parsing

this prevented the write iops performance charts to get any meaningful
data

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

filter the local node in the migrateall window

we do this already for single vm migrations, also do it for migrate all

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

validate nodeselector after store is loaded

this shows the validation status immediatly if no node is selected (for
instance because no node is online or you have only one node and open
the migration window)

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

make date column resizable

this makes the date column in the snapshottree a little bit bigger and
resizable

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

remove obsolete belongsTo

this was not needed

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

snapshottree: remove sorter before saving the state

extjs saves the state including the sorter, but cannot serialize the
sorter function. when restoring the state, it is not a valid sorter,
so we delete it before saving

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

set fixed height on global search window

the buffered renderer does not work with auto height, and was disabled.
to enable it, we set a fixed height

when nothing is found, display 'none' to indicate nothing was found

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

move chart series initialization into initComponent

with extjs 6.2 the charts are initialized differently, so that we can
not do this in the afterrender event, instead we do it in the
initComponent, after calling callParent

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

move hideHeaders into static configuration

in extjs 6.2, this property will not be set in ApplyIf, so we move it
into the static configuration, so that the headers will not be visible

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

Rename filterFn to vmtypeFilter

since we have now two filters defined it makes sense to use
a separate name here

Filter by default the displayed guest backups fix #1161

This add a default filter which is executed on store initial load,
so we display only the backups matching our ID.

reduce scope of jslint warnings to the problematic statement

also remove it from qemu/Clone.js since it validates OK

Convert the VMIDSelector to a generic GuestIDSelector

This allows to display a 'CT', a 'VM', or a 'CT/VM' label
to be displayed, depending on the context.

Currently when restoring a CT backup or creating a CT via the
wizard, we are asked to enter a  *VM* ID, which is confusing.

bump version to 4.4-1

make realmselector non-editable

since most people do not have many login realms to choose from, making
this search/editable makes no sense

but by making this non-editable, you can now click anywhere on the text
to open the options, instead of only the little arrow

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

fix #1176: sort user/group/realm selector

this adds a default sort for the user, group and realmselector
so that the entries are in a consistent order

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

aplinfo.dat: update

fix jslint confusion warning

ExtJS components accept for the items property either a
single component or an array of components which generates
a jslint type confusion warning.

Add online help for start/shutdown for containers

Turn the qemu StartupEdit widget into a generic component

and use this component for Qemu.

bump version to 4.3-14

fix typo in vmtype check

by mistake we checked if me.vtype is 'qemu'
but the property is me.vmtype, so we would always show
restart mode

note that this error was purely cosmetic, behaviour was correct

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

bump version to 4.3-13

adds a help button to the migration window

for now we have to explicitely define the
    onlineHelp: 'blockid'
string, so that the parser picks it up

in the future we should refactor that window, so that we define the
blockid when declaring the component

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

change lxc migration option to restart mode

since online migration does work at the moment, and with ha we already
have the restart mode

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

Remove last extra comma

also whitespace / indent cleanup

Add online help for startup/shutdown order and LXC general settings

Add the possibility to create an unprivileged container at creation time in the GUI.

The setting is afterwards displayed as a read only option in the option time

show in/out/up/down icons in osd overview

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

make ceph status in cluster dashboard clickable

to get faster from the datacenter dashboard to the ceph dashboard

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

add noout button and reorder actions

this patch adds a set/unset noout button (for easy maintenance of your
ceph cluster) and reorders the buttons so that global actions (reload,
add osd, set noout) are left, and osd specific actions are on the right

to reduce confusion, there is now a label left of the osd actions which
displays the selected osd

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

add ceph flags api calls

we add a get/post/delete api call for ceph flags

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

also return the ceph flags in osd api call

we want to set/get the flags in the ceph/osd tab, so we have to
return it there

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

pvereport: tell lsblk to use ascii

The UTF-8 characters it uses otherwise may be not handled correctly
when some users sent the report to us.

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

plug new ClusterConfig API at /cluster/config

Add Windows 2016 as available ostype to select

update changelog for 4.3-12

The actual 4.3-12 release is done in a separate
branch "hotfix-4.3-12", because we cannot release
commit 58b12d605 now (requires updated pve-cluster
package).

fix #1212: allow options for delete in pvesh

we did not allow to use parameters in the delete handler in pvesh,
but we make use of them (e.g. force on snapshot deleting)

as the get/set and delete handler do the same, refactor the if/else
paths of them

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

change ssh-public-keys check in wizard

we changed the ssh public key field to a pvetextfield, which return not
an empty string but null, so the .length check failed with an exception

we do not need to delete the field because it does not exists, when
no key is given

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

use certificate cache from PVE::Cluster package