PVE:Daemon start/restart with systemd

Added some lines which make it possible to stop/start/restart daemon services with pve scripts,
avoiding the problem of systemd not recognizing the new status of the (re)started service processes.

The scripts now rely on systemctl.

Signed-off-by: Alen Grizonic <a.grizonic@proxmox.com>

bump version to 4.0-7

new helper dump_journal to view systemd journal

remove superfluous substitution

regexps are greedy so the first substitution eats up
multiple groups of double-colons up to the last one already.
So unless there's some intended hidden side effect this
statement has no obvious purpose.

bump version to 4.0-6

add trigger for pve-api-updates

bump version to 4.0-5

defer some daemon setup routines

A first step towards untangling some of the intermingled data and
functionality setup tasks for the daemons:

Daemon::new now only validates and untaints arguments, but doesn't
perform any actions such as setuid/setgid until the new Daemon::setup
method which is now executed from Daemon::start right before entering
Daemon::$server_run.

CLIHandler::handle_cmd now takes an optional $preparefunc which is
called after handling 'printmanpod' and 'verifyapi'.

fix a regex typo in run_command

m/|/ is always true as it effectively matches 'nothing or nothing
anywhere in a string'
looks like it was supposed to be m/\|/

bump version to 4.0-4

spice-proxy format now uses new address format

spice-proxy used dns-name before which matches a combination of letters,
digits and dots, which happens to include ipv4 addresses but not ipv6
ones.
Since 'dns-name' sounds like it's only for names, I'm adding an address
format which is either a dns-name or an ip (including ipv6).

prevent the use of AI_ADDRCONFIG

perl's IO::Socket::IP passes AI_ADDRCONFIG if no GetAddrInfoFlags are passed,
which is often useful but also causes it to error when explicitly trying to
bind to 127.0.0.1 when there are no _other_ IPv4 addresses present.

update changelog

new helper: getaddrinfo_all

As it's commonly used in ipv6 support code a getaddrinfo wrapper passing
default flags and dealing with the (err,result) tuple was added.

add a socket family argument to next_*_port functions

Instead of assuming a local address of 0.0.0.0, the next_*_port family
of functions now takes an optional packet family argument (AF_INET/AF_INET6),
used for ipv6 support.

bump version to 4.0-3

add utility to fetch the socket family for a hostname

optional socket family parameter for create_reusable_socket

provide Tools::unpack_sockaddr_in46

Use IO::Socket::IP instead of INET

fix typo

add additional methods for IP address verification

cleanup: remove unnecessary prefix

bump version to 4.0-2

register new standard option pve-startup-order

We can use this for lxc and qemu.

improve error message

use libspice-protocol-dev (instead of debian package)

use wiki style headers for README.dev

update installation instructions

rename data to src

bump version for Debian Jessie

bump version to 3.0-24

next_migrate_port: higher port range - all other ranges are 100 ports

Signed-off-by: Stefan Priebe <s.priebe@profihost.ag>

Fix next_unused_port already bin detection

unsecure migration does not use localhost to bind to. Instead it uses a specific IP on the node.
As we don't know that IP in this part of the code the only chance to check for a possible already
existing bind is to bind to all IPs.

Signed-off-by: Stefan Priebe <s.priebe@profihost.ag>

bump version to 3.0-23

read_etc_network_interfaces : don't always add vmbr0

Currently it's impossible to remove vmbr0 from network gui,

if we really want to remove it, or replace it with an OVS vmbr0

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

bump version to 3.0-22

Daemon: change option name 'run_dir' to 'pidfile'

Daemon: new option to change user/group (setuid/setgid)

Daemon: make sure we kill all child processes on TERM

Daemon: add helper to create sockets

Those sockets are not closed and reopened at restart.

Daemon: catch finished worker earlier

Else we get zobies for 5 seconds at restart.

Daemon: also terminate old_workers on stop

Daemon: call terminate_server if we get unexpected errors

Daemon: implement new option leave_children_open_on_reload

We want to be able restart daemons without closing existing connections.

Daemon: correctly set got_hup_signal flag (before we call terminate)

Daemon: raise error if we cannot parse lockfd

Daemon: remove useless parameter

Daemon: untaint value read from ENV

Daemon: log error and exit if something fails inside constructor

RESTHandler: define empty DESTROY to avoid useless warings

We oserverd the following warning sometimes:

  (in cleanup) no such method 'main=HASH(0x38b2d48)::DESTROY'

Defining an empty DESTROY seems to avoid it.

Daemon: do not wait for lock on restart

Daemon: cleanup logging

Daemon: keep lock file open during restart

So that nobody else can grab the lock

Daemon: move ENV manipulation into new()

after_fork_cleanup: move code into separate sub

Daemon: try harder (up to 5 times) to get lock

code cleanup

Daemon: implement pre-fork server (max_wrokers property)

improve doc

implement new Daemon features

- allow to specify run_dir (directory to store pid file)

- make hup optional

- correctly untaint pid when reading from file

- restart daemon by HUP or stop/start

- allow to register reload command

Daemon: call setsid at startup

bump version to 3.0-21

add abstract class to implement daemons

cleanup error message

set RELEASE to 3.3

bump version to 3.0-20

create_firewall_bridge_ovs : activate ovsint interface

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

avoid warning when parsing long hex values with hex()

improve debug message

add FVN digest

Userful for short strings.

add additional check for duplicate method definitions

If we have a SUBCLASS, other methods are not allowed.

fix typo

new helper read_proc_net_route

add 'noerr' parameter to parse_tap_devive_name()

And simply ignore devices with random names in cleanup_firewall_bridge().

add additional comment

rename link to fwln && link peer to fwpr

also plug fwpr to vmbr, and fwln to fwbr

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

we need to match link+ rule from iptables rules, and need to have a name different
than link(\d+)i(\d+), for distinguished bridge/ovs interface unplug

Based on patch from Alexandre, but I prefer "link${vmid}o${devid}"

add firewall bridge support for openvswitch

    eth0-->vmbr0--vlan--fwintXiY-->fwbr-->tapXiY (firewalled tap)
                --vlan-->tapXiY (non firewall tap)

This is based on patches from Alexandre, but tries to factor out
common code into privat methods.

add firewall bridge support for linux bridge

    eth0----->vmbr0
    eth0.94-->vmbr0v94<--tapXiY (non firewalled tap)
                      <--linkXiY-->linkXpY-->fwbrXiY-->tapXiY (firewalled tap)

This is based on patches from Alexandre, but tries to factor out
common code into privat methods.

tap_unplug: auto-detect bridge, improve error messages

Based on patch sent by Alexandre.

map_path_to_methods: add code to return paths with uri patterns.

A previous commit removed that code, but we need it in pvesh to
display nice help messages.

bump version to 3.0-19

fix typo

new helper register_page_formater

map_path_to_methods: do not skip folders which evaluate to false

bump version to 3.0-18

speedup find_handler

Avoid to scan the same path multiple times. That way we can speedup things
by factor 5 (depends on patch length and method count).

bump version to 3.0-17

dump_logfile: add filter regex

bump version to 3.0-16

register new standard option pve-config-digest

move assert_if_modified to PVE::Tools

bump version to 3.0-15

dont use undefined var $ifcount

Delayed vlan interface creation until all checks are done.

Signed-off-by: Pablo Ruiz Garcia <pablo.ruiz@gmail.com>

Handle cases where the vlan bridge has already been setup previouslly.

Signed-off-by: Pablo Ruiz Garcia <pablo.ruiz@gmail.com>

Add support for bridges with more than one physical link (ie. two bonds each connected to a different switch using MSTP/PVSTP to load balance vlans across links)

Signed-off-by: Pablo Ruiz Garcia <pablo.ruiz@gmail.com>

use full name for PVE::ProcFSTools::write_proc_entry()

bump version to 3.0-14