README.dev: remove lvm.git

README.dev: add extjs.git and pve-guest-common.git

README.dev: add missing build depends for pve-cluster

add missing build dependencies

README.dev: adopt package list for stretch

add keeplocale parameter to run_command

since the "lang" param has not worked, introduce a "keeplocale"
parameter instead.

the default behaviour is the same (set LC_ALL to 'C'), but we can use
the parameter to keep the locale from the host (eg. for the vncshell)

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

Makefile: upload to --product pve,pmg

We use the same library for new proxmox mail gateway (pmg).

bump version to 5.0-1

Makefile: use "--product pve --dist jessie" for upload

bump version to 4.0-93

SectionConfig::check_value: return bool/numbers as numeric value

This is important when we convert the result to JSON.
Else the GUI receives booleans as "0", which evaluate to true
in JS!

NOTE: "0" evaluates to false with perl.

JSONSchema::check_type: return success (not value)

RESTEnvironment->init_request: reset result_attributes

SectionConfig: add new parameter $skip_type to createSchema()

This is useful when there is only a single type of entries.

SectionConfig:write_config: sort options by key name

Else we can get different config digest...

SectionConfig: allow to get class specific updateSchema()

Useful when you have different section types, and you want to have
separate update API methods.

(maybe) fixes #1229: fix port reservation

when reserving ports, we use lock_file to lock the
reservation file, but then use file_set_content which
writes a new file and renames it, making the lock invalid
and different processes waiting for the lock get inconsistent
data

instead we use a designated lock file for the lock, so that we don't
lose the lock when writing the reservation file

this should fix the problem that sometimes multiple vms get the
same vnc/spice port

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

fix trailing whitespaces

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

Fix example to latest RestHandler

looking at the handle() sub from RestHandler.pm

    my $result = &$func($param);

this parameters are not passed anymore

bump version to 4.0-92

deps: moving skiplock breaks qemu-server << 4.0-109

JSONSchema: adding skiplock option here

Network: stop using ifconfig

We shouldn't mix different tool sets on the one hand, and on
the other hand net-tools is an optional package in stretch
and there's no real need for us to depend on it.

bump version to 4.0-91

add Breaks for upgrade path

buildsys: missing build-dependencies

cleanup previous commit

Fix interface writer for bridge_ports and slaves.

The pve-iface-list allow to separate interfaces with [;,\s].
But in the interface file are only whitespace allowed to separate interfaces.

bump version to 4.0-90

RESTEnvironment: let check_worker take a $self

bump version to 4.0-89

remove AbstractConfig and AbstractMigrate

they have their own repo / package now (pve-guest-common)

set compat level to 9

To avoid warnings on debian stretch.

runtest.pl: use lib '.' - because newer perl does not have that by default

To make tests work again on debian stretch.

fix #1232: cleanup bridges on veth_delete

When a container stops or hotplug changes are applied we
do a veth_delete() which does not cleanup the firewall
bridges or OVS ports. This is problematic at the next
startup. When creating a network device we usually want to
copy the MTU of the bridge we intend to put it on, however,
with OVS still having the old port lying around the
recreated device gets associated with the bridge before we
read its MTU, potentially reducing it to that of the newly
created device.

This cleanup also gets rid of stale fwbr/fwln devices from
stopped containers.

bump version to 4.0-88

remove libmime-base32-perl - we do not need that

PVE::Ticket - new helper class to create auth tickets

bump version to 4.0-87

JSONSchema: remove formatter option

We do no longer store that information in the method info.

RESTHandler.pm: remove register_page_formatter (no longer required)

PVE::RESTEnvironment: new base class for PVE::RPCEnvironment

Copied most code from PVE::RPCEnvironment.

bump version to 4.0-86

CLIHandler.pm: remove run as root restriction

This should be checked somewhere else.

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

CLIHandler.pm: add setup_environment hook

Most PVE command line tools will simply call

 PVE::RPCEnvironment->setup_default_cli_env();

there.

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

remove calls to PVE::RPCEnvironment

This introduces a cyclic dependency, so we need to remove this.

PVE::CLI::* classes should instead add ad call to

 PVE::RPCEnvironment->setup_default_cli_env();

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

bump version 4.0-85

refactor delete_snapshot for readability

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

set 'snapshot-delete' lock when deleting snapshot

unless we are in cleanup mode from a failed snapshot_create,
in which case the config already has a 'snapshot' lock.

this is necessary to prevent concurrent starting of or other
actions on the guest in the windows inbetween holding the
flock, e.g. when removing the volume snapshot (which can
take a bit of time).

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

bump version to 4.0-84

and remove RELEASE

CpuSet write_to_cgroup: catch errors from close()

CpuSet short_string - avoid ranges like 0-0, 1-1, 2-2, ...

bump version to 4.0-83

safe_read_from: add filename parameter to improve error messages

bump version to 4.0-82

cpuset: fix short_string

don't drop the next current cpu after finishing a range

bump version to 4.0-81

use better regex for remote ip check

while the input should be correctly generated by the other site its
still better to be on the safe side, so check if the IP is really an
valid one.

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

schema_get_type_text: do not always expand enums

We try to keep the text short by default.

schema_get_type_text: render generic types as <type>

bump version to 4.0-80

use the pvecm mtunnel command to get remote migration ip

Ask the pvecm mtunnel command (was earlier in qm) if the remote side
has an IP configured in a given migration_network denoted by a CIDR.

If such a IP is available reset the nodeip and rem_ssh variables
and check if we can connect to the other side with ssh public key
authentication.

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

tempfile: use /tmp for fallback-tempfiles

At this point we know it's not a tmpfs (as tmpfs definitely
supports O_TMPFILE), so /tmp makes more sense than /run as
default path.

tempfile: unliked-file fallback

some file systems (eg. ZFS) don't support O_TMPFILE

Fix #1188: tempfile: use /run by default

as /tmp is not a tmpfs by default and some file systems
(like ZFS) don't support O_TMPFILE

JSONSchema: named formats in parse_property_string

While our code currently passes the format hashes directly,
some code gets them via the format property of a
configuration description which verifies successfully via
check_format() for named property string formats, so this
should be allowed.

bump version to 4.0-79

CpuSet: read $MAX_CPUID from /proc/cpuinfo

add get_local_ip_from_cidr

The get_local_ip_from_cidr method can be used to determine addresses
configured on interfaces from the callers node which are located
in the subnet. The subnet can be passed in CIDR notation.

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

fix typo

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

bump version to 4.0-78

CpuSet: add helper to format/print cpu sets.

bump version to 4.0-77

CpuSet: Simply class to handle cpu sets

Add check for infiniband to write functions as well

Signed-off-by: Michael Rasmussen <mir@datanom.net>

Add support for infiniband nics

Signed-off-by: Michael Rasmussen <mir@datanom.net>

bump version to 4.0-76

Update to reflect current status

Signed-off-by: Michael Rasmussen <mir@datanom.net>

SectionConfig: always write out explicitly set booleans

Because when only writing the ones which are true we're
breaking the ones which default to true (like the mkdir
option on directory storages, where we need a false value
to be written out explicitly).

bump version to 4.0-75

correct spelling of synopsis

bump version to 4.0-74

Network: add disable_ipv6 and use it

Many interfaces used to get an ipv6 link-local address which
was usually unusable and therefore pointless.

In order to ensure consistency this is called in various
places:
* $bridge_add_interface() and $ovs_bridge_add_port() because
  it's generally a good choice for bridge ports.
* tap_create() and veth_create() because the activate the
  interfaces and we want to avoid the link local address to
  exist temporarily between bringing the interface up and
  adding it to a bridge.
* create_firewall_bridge_*() because firewall bridges aren't
  meant to have addresses either.
* activate_bridge_vlan() - if vlan_filtering is disabled we
  create vlan-bridges and neither them nor their physical
  ports should have link local addresses.

fix doc generator (do not convert efidisk0 to efidisk[N])

Also fix spacing in typetext generator.

bump version to 4.0-73

schema_get_type_text: always access values as number

Else PVE::RESTHandler::api_dump prints values as strings.

add typetext property for more complex types

bump version to 4.0-72

harden file_set_contents against symlink attacks

allow Regexp objects for strings in the schema

The 'pattern' property has type string and format regex, so
it makes sense to allow Regexp objects to be used for it.

While check_type() doesn't know the format, Regexp objects
can be treated like strings anyway, including compared via
'eq' or matched via '=~', so we allow strings to generally
come from a Regexp object.

tools: optional prefix for random_ether_addr

bump version to 4.0-71

fix mac address generation limitation

Commit de9a267 introduced vec() to optimize the generation
by using binary operations instead of converting back and
forth between hex and strings, but forgot to switch over to
the binary sha1 method. This resulted in only the first 6
hex digits of the output string making up the address.

fix 1046: add non-snapshotted disks as unused

df: untaint the result

use new repoman toolkit

bump version to 4.0-70

Fix #132: hold a lock while setting up vlan bridges

It's possible for two simultaneous VM starts to try to
create vlan bridges on non-vlan-aware bridges
simultaneously, which causes one of them to fail with the
error "can't add bridge ...".

bump version to 4.0-69

vlan fixups: remove existing vlan ids

bridge_add_interface should remove the existing vids from
the interface (which is usually 1, which was previously left
on the interface)