bump version to 4.4-6

HTTPServer.pm: factor out code which refers to PVE::Cluster

This will make it easier to move HTTPServer to a separate package.

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

add setup_environment hooks to CLIHandler classes

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

Use component references via lookupReference()

This allows to add the components in each column in the order they
appear in the Clone Window.
References from child components are kept in an object in the parent
component so it should be the same performance-wise (contrary to to Ext.ComponentQuery
which is doing DOM parsing)

HTTPServer.pm: make split_abs_uri() private

and rename baseuri to base_uri to be consistent

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

HTTPServer.pm: improve baseuri matching

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

remove obsolete PVE::REST class

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

bin/Makefile: export PERLLIB=.. so that doc generator works correctly

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

PVE/API2/Formatter/HTML.pm: avoid use of PVE::REST

Instead, pass the HTTP server as last argument to the page formater,
so that we can call $server->create_auth_cookie().

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

PVE/Service/pveproxy.pm: avoid using PVE::REST

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

HTTPServer.pm: add missing 'use' statements

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

HTTPServer.pm: add rest_handler method

copied from PVE::REST. We do not need that class anymore.

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

HTTPServer.pm: allow to set baseuri when creating the class

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

HTTPServer.pm: add auth_handler

copied from PVE::REST (slightly adopted)

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

HTTPServer.pm: add cookie handling methods

Copied from PVE::REST (I want to get rid of this PVE::REST class).

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

Use qcow2 as default disk format for clones fix: #1144

When using a file based storage, qcow2 has more features as raw,
this is why we recommend it for clones, similarly to when adding
new disk images.

Do not show a Clone Mode selection Window for non-templates clones

As Non-templates clones are always a full clone.

Group the Pool selector combobox to the VM properties

The pool membership relates here to the VM membership, not
the storage membership, so it makes sense to move it closer
to the VM ID / VM name form fields.

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

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>

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>

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