pvmove.c: code cleanup

polldaemon: remove get_copy_vg and get_copy_lv wrappers

These wrappers have been replaced by direct calls
to vg_read() and find_lv() in previous commits.

This commit should have no functional impact since
all bits were already unreachable.

polldaemon.c: call find_lv directly

lvconvert.c: call find_lv directly

polldaemon.c: call vg_read directly

replace calls in wait_for_single_lv() and report_progress() fns

lvconvert.c: call vg_read directly

pvmove.c: call vg_read directly

polldaemon: move dev_close_all out of poll_get_copy_vg

let's call dev_close_all() only before we're about to 'sleep'
for at least one second during the polling.

(it's questionable whether to call dev_close_all() at all in
polldaemon code. Natural extension would be to drop it completely)

tests: add test for duplicate pvs

configure: update localedir

Previous patch incorrectly skipped replace of @LOCALEDIR@.

The standard option is --localedir  so use --with-localedir
as backward compatible option and set localedir if it's not
yet been set (if the could ever happen).

Use double-eval to translate $datarootdir to $prefix to real dir.

configure: LOCALEDIR needs evaluated value

Also fix typo.

tests: fix calcucaltion

Code works properly.

tests: thin_restore  not needed

Few more test which could live without thin_restore.

makefiles: drop LVM_SHARED_PATH

We already have LVM_PATH define used everywhere else
to access LVM binary so stay with one name.

makefiles: better clean

More exact clean of library exported symbols files.

Also use $(firstword) test to check for empty string
so 'make clean' has now cleaner condensed look.

Clean also created include links.

makefiles: use := for shell calls

makefiles: use single target

Possibly  easier to follow - to have just a single dependency line
and use  if() within rule.

Also replace $(words) with $(firstword) which is more commonly used.

configure: move DEFS to configure.h

tests: no warn if test does not need thin_repair

Set LVM_TEST_THIN_REPAIR_CMD to /bin/false for test which
doesn't need it.

This way - even if on the system there is no such tool present,
test will not result with warning about missing tool.

Also remove from Makefile settings of TEST vars which are set in
through /lib/paths  - this also allows to override them in test.

post-release

pre-release

mirror: use proper 64bit constants

ed2a08bf25e3b4e6a88c55402ab5dc11061ad68e missed to use 64bit
constants.

spec: new man page

config: remove UNDEFINED from cache_pool_cachemode

Replace UNDEFINED with COMMENTED because the code
requires a value to be returned from the config.

config: remove UNDEFINED from thin_pool_discards and thin_pool_zero

Replace UNDEFINED with COMMENTED for these two since
undefined seems to break things.

lvmpolld.8.in: man page rewrite

- add client functionality
- fill in long option variants

lvmpolld: don't return success on invalid option

lvmpolld: add long option variants for all short ones

lvmpolld: introduce client functionality

as of now lvmpolld works as client utility for
querying running instance of lvmpolld server
on metadata, state, etc.

Currently the only request implemented is the '--dump'.
It prints out full lvmpolld state (mimics lvmdump -p command).

pvmove.c: relocate id components extraction

we don't want to fail properly set pvmove after metadata
update. failure to copy id components could end with dangling
mirror moving PV segments but no monitoring from lvmpolld or
classical polldaemon.

lvpoll.c: harden the checks for proper LV name

lvpoll now process passed LV name properly. It respects
LVM_VG_NAME env. variable and is able to process LV name
passed in various formats:

- VG/LV
- LV name only (with LVM_VG_NAME set)
- /dev/mapper/VG-LV
- /dev/VG/LV

lvpoll.c: replace arg_count with arg_is_set

didn't need to count the occurence but check if set

example.conf.in: apply previous improvements to example.conf

config_settings.h: improve lvmpolld config description

lvm.8.in: add reference to lvpoll built-in command

lvmdump.8.in: describe lvmpolld related option

commands.h: reorder lvpoll options

lvm-lvpoll.8.in: man page for built-in command

lvmpolld.8.in: hide origin of lvmpolld man page

:)

lvmpolld-client.c: explain known return codes

lvmpolld returns few well known return codes. Explain
these to users and suggest reading lvmpolld log files
for more details

lvmpolld: label known return codes

so that lvmpolld-client can decode and describe these
return codes properly

libdaemon: fprintf(stderr...) -> ERROR()

log data structures are ready. so why not pass
error messages through ERROR()

libdaemon: suggest daemon already running

when dm_creat_lockfile fails, it's probably due to
another instance is holding the same pid file...

config: thin_pool_chunk_size_policy should be commented

It is commented in existing example.conf, so leave
that as it was.

config: thin_pool_chunk_size_policy is not undefined

The default policy setting does not depend on any
system/kernel values.

man: missed y|n for wipesignatures

config: add comments to match current example.conf

Use CFG_DEFAULT_COMMENTED and CFG_DEFAULT_UNDEFINED to
replicate the existing comments in example.conf.

Fix host_list to be cfg_array.

UNDEFINED is only used if the value depends on other
system/kernel values outside of lvm.  The most common
case is when dm-thin or dm-cache have built-in default
settings in the kernel, and lvm will use those built-in
default values unless the corresponding lvm config setting
is set.

COMMENTED is used to comment out the default setting in
lvm.conf.  The effect is that if the LVM version is
upgraded, and the new version of LVM has new built-in
default values, the new defaults are used by LVM unless
the previous default value was set (uncommented) in lvm.conf.

libdm: new dm_task_get_info with internal_suspend

Introduce new implmentation of dm_task_get_info() function
with support for reading internal_suspend.
.
This time it is done in a 'versioned' way.

We keep the old fashion dm_task_get_info(Base) to implement
the old behavior of 1.02.95 libdm code.

libdm version 1.02.96 introduced 'macro' wrapper
dm_task_get_info_with_deferred_remove with new implementation
of dm_task_get_info() - we cannot do anything else then to
provide compatible version of this symbol.

Now in version 1.02.97 we add new versioned implementation of
dm_task_get_info(DM_1_02_97) symbol.

This has the effect that i.e. rpm build will finaly resolve proper
dependency on a new symbol - so it will be no longer possible,
to build a new binary and use old library
(rpm -q --provides will show libdevmapper.so.1.02(DM_1_02_97)(64bit))

Also the history is now tracked. If a new function is added (or
reimplemented), it needs to be placed in proper file,
so it could be exported with right versioning symbol.
File .exported_symbols.Base should and any existing older DM
should be treated as read-only after a release.

Also - only libdm has been currently enhanced with versioned .Base
file, as soon as other libs (liblvm, libdevmapper-event) needs changes
they should also get their exported symbol files - meanwhile
make.tmpl handles both cases.

lvm.conf: add more information to the comment header

config: description can refer to etc location

config: avoid configurable description text

It's just simpler to avoid configurable values in
the description text by rewording to avoid them.

nix: socat for f18, nc for f17

Use common f17_f18 and add extras for f17 and f18

makefiles: disable po file targes

po targets are not correct - so disable them for now.

makefiles: use srcdir

Use -I$(srcdir) as that's what we really want here.
For %.pot use all includes.

nix: drop util-linux

Doesn't appear on centos???

nix: add more packages

spec: package old sysv initscript for lvmpolld

nix: add pkgconfig

Install for better config

tests: respect @CONFDIR@/machine-id

Obtain location of machine-id.

NOTE: lvmconfig cannot dump 'life' value - should be fixed.

tests: do not use |&

Bash 3.2  doesn't understand this syntax (i.e. RHEL5).
And it's even better - reports syntax error and return success.

nix: socat is quite new package

Install 'nc' on older systems.
Test suite handles both.

conf: system_id_source is referencing CONFDIR

Since we allow to configure /etc in configure and compile-in
dir for /etc we need to properly advertise this location later.

makefiles: protect CFLAGS

When CFLAGS and LDFLAGS are passed into - protect them,
and avoid even recursive subdir 'extension' of them.

makefiles: assign vars before include

Before we include, set INCLUDE and TARGETS.
Extend CFLAGS after include.

makefiles: drop DEBUG mangling

DEBUG is already set through make.tmpl.

configure: set optimize flags when CFLAGS is unset

If we are given CFLAGS - preserve user's request and avoid placing
just -O2 there.

configure: preserve CXX/CFLAGS

AC_PROG_CC
AC_PROG_CXX

Does not preserve CFLAGS CXXFLAGS

nix: fix socat package name

makefiles: use fullpath when in assign

We need to put full path right in the assign moment,
otherwise command:

make rpm rpmbuild=/my/tmp/dir

cannot work as one would have expected.

makefiles: testclient is not valid target

makefiles: drop way too generic deps

libdaemon: Fix socket reuse error paths.

Invert S_ISSOCK validation.
Fail instead of replacing a symlink with a new socket.
After failure, skip calling fcntl with invalid socket_fd.

configure: start to use AS_IF

nix: more base packages

spec: packaging polld

Package lvmpolld

Drop legacy SysV init subpackage.

lvmpolld: dump cleanup

lvmpolld-client.c: use lvmpolld debug class where appropriate

tests: remove forgotten -vvvv option

libdm-common.c: remove trailing whitespace

lvmpolld-client.c: be more specific about fallback on error

if client fails to contact lvmpolld it fallbacks to using classical
polldaemon. Be more specific and give some hints to users what went
possibly wrong

new debug class for lvmpolld client code

lvmpolld.8.in: clarify the timeout parameter

Remove references to systemd native service. Now
the libdaemon supports shutdown on idle no matter
the init process implementation installed.

libdaemon: shutdown on idle also in non-systemd environment

tests: ndev by default

configure: spec.inc is generated at build

Collect all needed info at runtime for spec.inc

makefiles: use LN_S

makefiles: runtime spec.inc

Support  CLEAN_DIRS
Var for rpmbuilddir
Use LN_S
Generate spec.inc at runtime for 'make rpm'.

tests: remove forgotten set -v in aux

lvmpolld: by default spawn lvpoll cmd with -An

lvmpolld: set use counters properly in lvmpolld_store

set active_polling_count to zero in pdst_init fn

lib/polldaemon.h: remove trailing whitespace

lvmdump.sh: print out lvmpolld service status

tests: split flavours

Remove duplicate flavour-udev-lvmetad-lvmpolld
and put them 1-per-line.

nix: now some files are generate so keep them

more tweaks ahead

vgimport: fall back when lvmetad is not running

If lvmetad is configured, but not running,
vgimport would not fall back and run without
lvmetad, but would report an error about
requiring lvmetad.

tests: typo in aux

Ooops editor grabbed q in test.

makefiles: move clean

Move clean: target below, so it's not a default target to execute.

makefile: cleanup after build

Provide cleaning rules for build dir and release tgz.

tests: disable lvmetad and lvmpolld

Since now we enable those by default when compiled with those daemons,
explicitely disable them in tests when needed.

Alphabetically sort configurables.

spec: now generated