clean up some error handling

Does not change functionality, makes some code a bit more readable
and lessens code reuse.

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

prefer 'name' subkey over 'ring0_addr' for nodename

Use the name subkey from the cmap keys by default, if not set
fallback to the ring0_addr.
This fixes some issues when we move the corosync communication to
a different network and use an specific address or an new hostname
for that. Withouth this patch the nodename in the .members special
file changes together with ring0_addr, which can result in quite a
few problems (e.g.: in the ha-manager).
This allows also to separate the webinterface traffic from corosync.
IP adresses can be used for ring0 addresses directly now also,
without making problems.

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

bump version to 4.0-22

check_vmid_unused: fix error message

bump version to 4.0-21

add function to lock a domain

This can be used to execute code on an 'action domain' basis.
E.g.: if there are actions that cannot be run simultaneously even if
they, for example, don't access a common file and maybe also spread
across different packages we can now secure the consistence of said
actions on an 'action domain' basis.

The need to use a dirty hack like cfs_lock_storage with some
arbitrary storage name becomes obsolete. Also the code behaviour
and meaning becomes clearer.

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

bump version to 4.0-20

install bash completion configuration file

convert pvecm into a PVE::CLI class

bump version to 4.0-19

check_vmid_unused: simplify code

check_vmid_unused message change

changed message if vmid used on a different cluster node

bump version to 4.0-18

new helper: complete_vmid

bump version to 4.0-17

new bash completion helper: complete_migration_target

moved from PVE::LXC

add some bash completion helpers

bump version to 4.0-16

do not pass undefined value to PVE::JSONSchema::parse_config

bump version to 4.0-15

use simple files for LXC configuration

cluster typo bug #588 fix

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

bump version to 4.0-14

add observed file status.cfg

We want to use that for new status plugins (graphite server).

bump version to 4.0-13

pvecm use of uninitialized value $ARGV[0]

Just a small bug fix for the pvecm script which did not verify
the argument existence during the $ARGV[0] check (when argument not used).

Script run error -> Use of uninitialized value $ARGV[0] in
string ne at /usr/bin/pvecm line 26.

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

path_is_lockdir: optimize away a strlen call

bump version to 4.0-12

use activate-noawait triggers

improve parser for /etc/ssh/ssh_host_rsa_key.pub

fix filename in error message

use systemctl instead of service

bump version to 4.0-11

allow sshd root login when we setup a PVE cluster

pvecm create: put brackets around hostnames for rsync

ssh and friends differ with respect to ipv6notations
 * ssh: 'user@host'
 * ssh-copy-id: 'user@host' as it uses ssh
 * scp: 'user@[host]:file'
 * rsync --rsh=ssh: '[user@host]'

rsync accepts brackets for all of ipv4, ipv6 or named hosts, so simply
defaulting to always using them works

pvecm create: add corosync.conf parameters for ipv6

pvecm create now adds the following additional corosync.conf parameters:
 * totem.ip_version
 * totem.interface.ringnumber
 * totem.interface.bindnetaddr

For ipv6 corosync needs a 'totem.interface' list with at least one entry
containing a bindnetaddr setting. Additionally 'totem.ip_version' needs to
explicitly be set to ipv6 (or an 'mcastaddr' set, which corosync can
choose automatically, though, so we let it do just that).

include ::1 in a certificate's subjectAltName list

cache address families for known nodes

remote_node_ip: option to include the packet family

If an array is requested, the function now returns ($ip, $family),
otherweise just the IP alone.
Several ipv6 related changes in other packages need to pass the packet
family to functions and will make use of this functionality.

bump version to 4.0-10

whitespace cleanups

use freeaddrinfo on getaddrinfo results when done

ipv6 support for lookup_node_ip

bump version to 4.0-9

trigger pve-api-updates event

system() is declared to warn on an unused result, replacing this with umount2(2) is more efficient anyway

With _FORTIFY_SOURCE and optimization, glibc defines a fortify_function for open(2) which checks whether the mode parameter was provided when O_CREAT was used.

make it compile as user: don't try to resolve the node IP when the 'printmanpod' or 'verifyapi' options are used

replace hardcoded perl paths with variables coming from perl's Config and ExtUtils::Embed modules

pve-cluster : fix typo error in mkdir /etc/corosync

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

new helper PVE::Cluster::check_vmid_unused

bump version to 4.0-8

allow unpriviledged users to read lxc configuration

We need this to run unprivileged lxc containers.

bump version to 4.0-7

track configuration changes inside lxc subdirectories

remove vm_is_ha_managed method

bump version to 4.0-6

add more 'observed' files.

bump version to 4.0-5

also track totem.config_version

correctly track corosync config changes

clear changes flag after read (to avoid confusion)

bump version to 4.0-4

fix corosync cmap reader

depend on corosync-pve

fix path for corosync config

use systemctl to start/stop services, bump version to 4.0-3

bump version to 4.0-2

correctly depend on perlapi-5.20.0

fix service configuration

add method vm_is_ha_managed

always return false for now.

implement addnode/delnode for cororsnc 2

depend on systemd (instead of insserv)

add parser for corosync.conf

use corosync-quorumtool to set expected votes

remove stale docs

cpgtest: updates for corosync 2

remove cman, and use corosync.conf instead of cluster.conf

use external corosync-keygen

avoid lintian hardening-no-relro warning

by adding -Wl,-z,relro compiler flag

add Pre-Depends to avoid warnings

avoid race conditions when starting threads

Use Mutex/Cond to wait until server threads are fully functional.

add missing debian/pve-cluster.service

add dh-systemd build dependency

add systemd support

do not create /etc/pve with debian/dirs

Because that can fail with chrashed pmxcfs (mountpoint not connected).

update to debian jessie

confdb.c: this is broken now

add ability to update cfs locks, bump version to 3.0-17

bump version to 3.0-16, fix lintian warnings

enable debug package for pve-cluster

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

bump version to 3.0-15

datacenter.cfg: validate email_from option

Signed-off-by: Stanislav German-Evtushenko <ginermail@gmail.com>

datacenter.cfg: add new email_from option

Signed-off-by: Stanislav German-Evtushenko <ginermail@gmail.com>

bump version to 3.0-14, add new value (html5) for console settings

bump version to 3.0-13

fix bug 204: use DERIVE instead of COUNTER

make sure we read latest versions inside locked code

File locks are used for read/modify/write cycles, so we need to make
sure that we always read latest versions.

bump version to 3.0-12, update libfuse2 dependency

fix bug #489

bump version to 3.0-11

stop after ceph