bump version to 1.4.1

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

use POSIX::_exit instead of exit in forked child

This avoids calling the END blocks and destructor routines most of
which have nothing to do with the fork, so it cannot be assumed to be
safe to do from the fork.

Move the call to pve_rados_shutdown from the destructor routine,
which will not be called by the child anymore. Make sure the cleanup
is also done when the worker dies.

Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>

reset inherited signal handlers in child worker

For example, if pvestatd is the parent, the inherited signal handlers
from pvestatd, i.e. PVE/Daemon.pm's handlers would be invoked by the
child.

Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>

refactor pverados worker into dedicated function

No functional change intended.

Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>

buildsys: expand clean target

& drop the odd find command execution, just fix your editor to not
sprinkle random files all over the (source) place.

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

d/source/format: document that this is a 3.0 (native) source

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

buildsys: switch upload dist to bookworm

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

bump version to 1.4.0

and switch to native versioning, this perl binding comes from us
(Proxmox) after all

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

d/copyright: update years

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

d/control: define compat level via build-depends and raise to 13

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

bump version to 1.3-1

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

d/triggers: activate a pve-api-updates

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

create new mon_cmd method for new behavior with compat wrapper

Rename the $noerr param to $no_result_catch while changing the method
name, as it was a bit confusing because we only ever adhere to it for
the actual command result, not other die's.

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

avoid one-argument "bless"

can be pretty confusing because if we ever would add a subclass
basing on this module it would get the parent as class and thus the
method calls would not be overridable

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

update POD

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

cleanup use statements and drop unused exporter

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

mon_command: optionally ignore errors

In some situations we do not want to abort if the Ceph API returns an
error. For example if we run the 'osd ok-to-stop' or similar calls, we
are interested in the status message in the error case.

Signed-off-by: Aaron Lauterer <a.lauterer@proxmox.com>

mon_command: refactor to pass all data to perl

By passing back all return values from the Ceph API (RADOS.xs) to Perl
we are more flexible to make more than just the data available further
up the stack. These values are:

* return code
* status message (can contain useful information)
* data

The Ceph API interaction happens in a child process. We need to en- and
decode the returned hash in JSON to pass it between the child and parent
process.

RADOS.pm::mon_command now returns not just the data, but all information
as a hash ref.  Therefore dependent packages (pve-manager, pve-storage)
need to adapt.

Signed-off-by: Aaron Lauterer <a.lauterer@proxmox.com>

radocs mon command: fix indentation style-mix

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

mon command: fix calling free() code style-mix

the error case checked for nullness on outbuf but not outs, while the
normal code path did the exact reverse, which is rather confusing for
any reader, as they could think there's something implied by this.

There isn't and rados' rados_buffer_free is already having its own
non-null check, not that it would matter much to call free with null
anyway.., but always using the rados provided method ensures tracing
shows the free, which could be good on debugging..

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

mon_command: free outs buffer

Signed-off-by: Aaron Lauterer <a.lauterer@proxmox.com>

bump version to 1.2-1

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

buildsys: change upload dist to bullseye

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

buildsys: fix librbd linkage

I really hate such compiler quirks change behavior... -.-

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

buildsys: update CFLAGS

better split to different lines, add comment why we override those in
the first place and enable the -Wstack-protector-strong and -Wformat
checks

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

buildsys: consistent makefile variable parenthesis

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

mon_command: include prefix in error message

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

bump version to 1.1-2

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

buildsys: drop unused CFLAGS line

it got overwritten the line afterwards

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

fix read after string end

outs is not a zero-terminated string but has its length given by
outslen, so use that (with a maximum of the size of msg)

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

buildsys: switch upload dist over to buster

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

bump version to 1.1-1

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

set debian source format to 1.0

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

buildsys: use dpkg-dev makefile helpers for pkg info

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

bump version 1.0-6

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

Add missing Build-Depends

Also use https for the homepage while in there already.

Signed-off-by: Rhonda D'Vine <rhonda@proxmox.com>

build: use a common BUILDSRC target for dsc and deb

And use $(PACKAGE)-$(VERSION) instead of 'build/' as
directory name, as it is also what the source package's
tarball ends up with inside.

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bsys: fix git HEAD parse to reflect actual build commit

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

bsys: add dsc target

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

d/control: use variable depends

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

bsys: move to dpkg-buildpackage

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

fix #1950: use /etc/pve/ceph.conf as default

If there is no symlink between '/etc/ceph/ceph.conf -> /etc/pve/ceph.conf',
executing a RADOS command results in an exception.
For example on a cluster node where no monitor is configured and no
'pveceph init' was performed, this leads to unexpected conditions. On such a
node, if you check the monitor tab in GUI the monitors are correctly displayed,
but they seem to have no quorum, because actually the RADOS command didn't return.

Signed-off-by: Tim Marx <t.marx@proxmox.com>

Make error messages in Rados.pm prettier

Signed-off-by: Alwin Antreich <a.antreich@proxmox.com>

bump version to 1.0-5

allow to specify the userid with rados_create

This allows to connect to a cluster with a different user, besides admin

Signed-off-by: Alwin Antreich <a.antreich@proxmox.com>

Split method pve_rados_connect

To be able to connect through librados2 without a config file, the
method pve_rados_connect is split up into pve_rados_connect and
pve_rados_conf_read_file.

Signed-off-by: Alwin Antreich <a.antreich@proxmox.com>

white space cleanup

Signed-off-by: Alwin Antreich <a.antreich@proxmox.com>

bump version to 1.0-4

buildsys: update make upload target for stretch

buildsys: generate perlapi dependency version

buildsys: use fakeroot for dpkg-deb

buildsys: use gzip -n to disable timestamps

buildsys: get perl path from perl

bump version to 1.0-3

fix perl path

update package dependencies for Debian Jessie

bump version to 1.0-1

Depend on ceph dumpling (instead of emperor), and use our own version schema.

add method to get/set timeout

allow clean shutdown with displaying 'partial read' error.

detect forked connections

Else more than one process can kill our worker!

cleanup

initialize PVE::RPCEnvironment in test.pl

implement timeout

depend on libpve-access-control

We need to be able to close open server sockets.

always for worker process

librados does not work well when we fork after rados_connect().
So we always fork a separate worker and use IPC to send/receive data.

cleanup test code

avoid memory leak, add debug helper DPRINTF

fix compiler flags and warnings

improve examples

avoid unnecessary warning

fix .so install path

use json format by default

code cleanups

add simple test script

allow to set client_mount_timeout

initial import