use login instead of bash

since bash does not play well with utf8 encoded input,
eg. pressing ALTGR + M on an english intl keyboard layout int novnc
does not enter the mu symbol but makes bash think you pressed alt+5,
we should use login without authorization when logging in as root

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

keep locale for vncshell and spiceshell

use the new keeplocale parameter from run_command and do not delete the
LANG and LANGUAGE variable for the vncshell

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

ZFSEdit.js: correctly set 'nowritecache' on zfs plugin

fix #1299: sort pools in poolselector by default

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

add css fix for chrome >=56

with chrome/chromium version 56, they changed the way how they calculate
height of text in input fields (now it is correct according to the
standard) but extjs up to 6.2.1 has a fix for this in css

this patch reverses the fix in extjs and can be reverted when they
remove the fix upstream

the bug (EXTJS-23628) and the fix are described here:
https://www.sencha.com/forum/showthread.php?335045

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

migrateall: use get_filtered_vmlist

instead of using get_start_stop_list in a rather hacky way use the
new get_filtered_vmlist method and adapt the loop, as now only one
level is required.

most changes  are just an indent shift left as we lost an unnecessary
loop level

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

refactor get_start_stop_list

we used get_start_stop_list for assembling a list of VMIDs for
stopall, startall and migrateall.

While get_start_stop_list did already some filtering we did some
more in the specific startall and stopall calls.
Add a new private helper which does all the filtering and just
assemble the startup order in the get_start_stop_list method.

Move the template and ha managed filtering in the new
get_filtered_vmlist method, this allows us to remove the filtering
from the API calls them self and we do not load the vms config twice
(it was cached, but nonetheless unnecessary)

get_filtered_vmlist is also intended to be used by migrateall in a
next patch

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

Do not translate IT terms which should be used 'as is' in every language

for 'Initialize GPT' we use the alternate 'Initialize Disk with GPT' string
which was already existing

Rename 'create' to 'isCreate' in pveWindowEdit and subclasses

This fixes around 20 jslint type confusion warnings.
Also reduce the scope and document jslint warnings along the way.
No functional changes.

Indentation fix

InfluxDB plugins: send nodename when updating CT/VM status

This allows filtering by node in InfluxDB queries, so the statistics
of all virtual guests on a specific nodes can be queried.

While for InfluxDB this is only a tag which does changes where the
data is stored, Graphite - our other status plugin - has no such
mechanics available. If we would add it to the object hierarchy,
e.g.: "qemu.$vmid.$nodename" a migration of a VM would result in two
different datasets.
So avoid breaking setups and omit it for Graphite for now.

Suggested-by: Daniel1108 <danielgallegosanchez@gmail.com>
CC: Daniel1108 <danielgallegosanchez@gmail.com>
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

Make explicit the meaning of an empty storage field

By default when no target storage is selected in the clone panel, the clone
will use the same storage*s* as the source.

This was not obvious to guess that from the panel.

Add a clone button in the VM toolbar

Up to now the Migrate function had a first class button in the toolbar,
but the Clone function didn't.
With this button the toolbar still fit correctly in a smaller
1280 pixel wide display.

Move the snapshot API test to a static function

This will allow code reuse in a following patch.

No functional change.

Fix handling of failed API call verifyFeature()

This call can for instance fails when a source VM has references
to non existing storages.

pveperf: remove outated POD replaced by pve-docs

Remove the POD content and the overwriting makefile rule
'pveperf.1.pod' so that the rule from pve-doc-generator.mk matches

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

Use ip from iproute2 package instead of net-tools

iproute2 is a direct depency from pve-qemu-kvm and should
be always available

use -statistics so we get the RX/TX traffic sum like ifconfig
use -details to get extra info like current vlan_protocol

Fix #1266: use a renderer when displaying Unix timestamps.

The timestamp is displayed read only in a large window, and not sortable,
hence we use the human readable renderer.

Add a new renderer for displaying dates in a human readable way

render_timestamp returns a Date as
"2017-01-31 16:17:33"
which is good for computers and column sorting but unfit
for human consumption

the new renderer added by this patch renders a Date as
"Tuesday 31 January 2017 16:18:08"

Day-month-year order is the most used ordering
in the world and also commonly used in the USA for
official documents.
source:
https://en.wikipedia.org/wiki/Date_and_time_notation_in_the_United_States
https://en.wikipedia.org/wiki/Date_format_by_country

PVE/API2/Tasks.pm: corretly return true after loading

bump version to 4.4-12

REST/RPCEnvironment's check_worker is a method now

spiceproxy: import PVE::SafeSyslog

bump version to 4.4-11

depend on libjs-extjs

delete extjs source files

bump version to 4.4-10

move VZDump base Plugin

moved to pve-common-guest

do no translate ha states

because they have a special meaning, do not translate them

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

Use the enabled/disabled toggler to render grid options

Allows getting rid of multiple "Enable foobar" gettext strings in
LXC and Firewall option panels.

Add new renderer function for toggling enabled/disabled options

Remove technical terms from content to be translated

bump version to 4.4-9

use new PVE::Storage::check_volume_access()

pvestatd.pm: corretly use new RPCEnvironment

Call $rpcenv->active_workers()

pveproxy: do not expose /pve2/js/ (no longer required)

Remove redundant helper for login window

the information is already provided by the label, the helper does not add
meaningfull information here.

Do not try to translate domain specific terms

No one is translating DHCP in his/her own language.

Remove duplicate strings

remove reference to PVE.ConsoleWorkspace

We already removed that code (unused).

bump version to 4.4-8

PVE/HTTPServer: do not return cookie_name from auth_handler

This is no longer required.

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

HTTPServer.pm: rewrite rest_handler (simplify code)

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

remove base_handler_class - directly use PVE::API2 instead

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

HTTPServer.pm: call rpcenv->init_request earlier.

Because we access file from /etc/pve/ in auth_handler.

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

HTTPServer.pm: call $rpcenv-init() when necessary.

This code was removed from the generic server, so we need
to add it here. cleanup follows ...

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

pveproxy.pm: set title to 'Proxmox VE API'

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

remove www/bootstrap completely (now in pve-http-server)

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

pveproxy.pm: use PVE::APIServer::AnyEvent::add_dirs()

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

use new libpve-http-server-perl package

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

bump version to 4.4-7

fix CSRF token generation

this broke with the recent refactoring

Disable autoSelect of target storage

autoSelect: true  was surreptitiously commited with the refactoring
of 406ac585db7fc285a63f8638a14fecdb92e28282
( Use component references via lookupReference() )

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)