gcc: initialize variables

tests: better slowdown

CI: Add timeout

Increase timeout from default 1h to 2h.

tests: skip some tests with valgrind

Valgrind tends to be too slow for any kind of slowing down,
thus skip some tests that are time sensitive.

debug: common error handling

cleanup: move allocation code

Allocate new memory after validation passed.

lvconvert: run error path code only for shared VG

Error path for locking only for share VG.

lvconvert: move _pmspare handling before conversion

pool: use predefined name for metadata LV

While create new LV for pool volume, use name from 'pool_metadata%d' naming
sequence. This LV is later on renamed to  pool_t/cmeta, but if there
is any error in the middle, we may evenutally leave some 'volume',
With this name it can be slightly more obvious how it got there,
but also when we handle _pmspare name - we get slightly more predictible
name used there for it.

However for a standard usage this commit shall no visible impact as the
name is used temporarily just for cleaning LV.

lvmlockd: fix lvconvert to thin-pool

lvmlockd: let lockd_init_lv_args set lock_args

Set the lock_args string in addition to doing initialization.
lvconvert calls lockd_init_lv_args() directly, skipping
the normal lockd_init_lv() which usually sets lock_args.

lvmlockd: fix thick to thin lv conversion

tests: update pool uuid handling

Pools now preserve meta LV UUID with swapmetadata operation
to make the usage of lvmlockd easier.

Update test to reflect this change.

gcc: cleanup warnings

Correcting signess comparation.
Also use standalone /* fall through */ comment as that's
the only one supported by gcc to quiet warning.

cleanup: no longer used

Previous commit made this var unused.

make: make install_libexec a target of install_lvm2

This makes sure libexec scripts are available for lvm2 commands.

lvconvert: use vg mempool

Keep allocation for a VG within vg memory pool.

lvconvert: fix regresion from integrity check

Testing code sliped into commit and cause regression in testing for
a raid with integrity.

lvconvert swapmetadata: update lvmlockd locking for uuid exchange

lvmlockd locking for swapmetadata adjusted for commit
ac36153e9943 lvconvert: preserve UUID for swapped metadata

Now that the LV uuid is swapped between LVs, the lvmlockd lock can
simply be moved between them, and the same lock can continue to be
used for the LV outside of the pool.

WHATS_NEW: update

debug: add missing log_error

Report error with failure.

debug: use just single label

debug: use common error exit path

lvconvert: vdo needs writable LV

commands: add lv_is_writable

lvconvert: fix ret values fro integrity remove

Fix return value from _lvconvert_integrity_remove()
as it is expected to match _add() and be 0/1.

Also add some missing log_error() messages.

lvconvert: preserve UUID on pool repair

Once lvm2 repairs pool's metadata LV and preserves the original metadata LV
with unmodified metadata, for such LV in VG use newly created UUID for new
_pmspare and actually preserve UUID for this hidden _pmspare (if it
exists).

lvconvert: preserve UUID for swapped metadata

When swapping metadata LV for a pool volume, preserve UUID for metadata
LV during the swap - this allows to better utilise locks with lvmlockd.

lvconvert swapmetadata: fix lvmlockd locking

The lockd lock needs to be freed for the LV that is becoming
the new metadata LV, and a new lockd lock needs to be created
for the old metadata LV that is becoming an independent LV.
Fixes b3e45219c2f4

device_id: rewrite validation of devname entries

The old approach was too complicated and didn't work correctly
in some cases.

CI: Add .gitlab-ci.yml

post-release

pre-release

WHATS_NEW: update

make: generate

report: fix pv_major and pv_minor report field type

We incorrectly marked pv_major and pv_minor fields as being of string
type, even though the values were already correctly handled as integers
internally. This confused -S|--select that tried to compare string
values instead of integers.

Reported here: https://github.com/lvmteam/lvm2/issues/122

lvmdevices: add --delnotfound to delete entries for missing devices

Used with update: lvmdevices --update --delnotfound.
Removes all devices file entries for which no matching device is found.

vgmknodes: skipping thin pool is not an error

tests: catch all output

Previous fix was invalid (after some in-place shuffling)
'dd' copied goes to 'stderr' so we need to catch all output.
Grep needs to check output of tee tool.
Ensure 'C' locales are in use with 'dd'.

vdo: use long --verbose

vdo tools needs long option.

tests: new logic for repair

meta name deduced from cachepool name.

WHATS_NEW: update

tests: use zero device

Change the test to use 'zero' backend, so we could use larger size
and get 'larger' buffer of dirty blocks for fast CPUs...

tests: check for loaded brd

Skip the test if the brd is already loaded in the system.

tests: mark test as warning

Until more universal solution for thin-pool corruption gets in place.

debug: add backtrace

cleanup: remove double //

dev_dir ends with /.

cleanup: remove duplicated code

Commit cac4a9743acb826d785c0e51e9a752d8959ced80 moved setting
if layer structed to the function front, but the old code
was still left in original place.

lvconvert: code shuffle

Use cachepool name for create name for metadata backup LV.
(so we do not generate 2 'sequences' of metadata filenames.)
Move path preparion before handling _pmspare.
Also drop extra call to sync_local_dev_names() as it's
already got in sync with call of exec_cmd().

lvconvert: add support --setactivationskip

When repair thinpool or cachepool, lvm2 leaves original metadata
volume backup. To avoid potential damage of those data, mark such
volume as 'read-only' and also allow user to use --setactivationskip
option for this volume.

TODO: likely better default would be to automatically skip, but
that might need some more thinking about recovery reporting doc.

lvconvert: use public LVs for recovery

Replace the use of internal /dev/mapper names with the use of
public LV names /dev/vg/lv for use with repair tools.
For this make the activation of _pmspare LV to be handled as
a component activation with public name.
Metadata is already atomatically activated this way (as readonly).

So if there is any 'error' happening, we leave public LVs in
system.

vdo: refactor code

Shuffle code to use prepare_exec_args().
Simplify generation of options string within a single char buffer.

lvm-exec: refactor code

Add prepare_exec_args() for reading option list for
thin/cache_repair, thin/cache_check.

lv_manip: refactor code

Simplify initialization of exec buffer.
Check for misconfigured 'fsadm' executable path.

active: device name limit is NAME_LEN

Use existing NAME_LEN instead of 257 value.
Add missing backtrace.

WHATS_NEW: update

tests: updates

Pass more args with some 'aux' commands:
wipefs_a, enable_dev, disable_dev
(so it's a bit more efficient using single udev_wait call).
Use prepare_vg instead of prepare_pvs.

tests: avoid using /test dir

Keep using test directory for created files.
Trap errors and remove brd in this case.
Use some shell builtins to reduce fork count.
Use "$VAR".
Run 'pvs' with devlist (so not acceing other system devices).

tests: add fsck before resize2fs

tests: faster check for flushing

Try minimizing delay in detecting 'cleaner' policy precence,
so we can fast interrupt running lvconvert.

tests: ensure single leg is broken

Make sure only the first raid5 leg, that contains searched string
is being modified, so we do not 'damage'  2legs at once.

aux: fix version checking

New dmpd tools return version string in different format,
so update code to understand both variant.

Also hide some shell var setting to local functions.

debug: print WARNING:

Add WARNING: in front of log_warn() messages.

lvconvert: hanle metadataswap with lvmlockd

mm: add libzstd to ignore list

No need to lock-in-RAM libzstd (linked from systemd).

tests: check --test

Check lvconvert --thin is not adding more LVs.
Enable testing with lvmlockd.

TODO: lvmlockd is missing support for --test.

tests: more checks with active thin lv

Check --usepolicy works primarily when thin-pool -tpool is there.
Also let the test run with lvmlockd.

tests: skip with lvmpolld

Not needed with lvmpolld pass.

cleanup: indent

lvresize: improve usepolices

lvresize --usepolicy requires resized LVs to be active.
(So it's not only required for shared VG).

The test for active pool needs to use lv_info to query 'layer'
otherwise the pool is considered inactive if it was not activated
explicitely - thun 'implicit' activation with VDO or ThinLV was
not managed by --usepolicy option.

lvmlockd: enable vdo conversion

As we turn 'already' active/locked LV into a data LV of vdopool,
we just keep active lock for such volume (uuid is preserved).

lvmlockd: query the simplest check first

Check for visibility of the LV first.

TODO:

Short check list and test only for those that can't be locked,
but they are visible LVs.

lvconvert: thin convert locking

Fix locking when converting thick to thin LV, ensure
we take pool lock for shared VG.

lvconvert: no metadata for test mode

Avoid trying to build real metadata in --test mode.

man-generator: ensure p is defined

In error path, p pointer could have been undefined.

debug: missed properly modify pointer type

tests: improve LD_LIBRARY_PATH setting

Some linkers do need libdevmapper-event-lvm2.so.2.03,
so add also this symlink to the tests  /lib dir.
This fixes the need to use previous LD_LIBRARY_PATH complex
setting and now works with much shorter list.

tests: update

Just drop debug log in case of error happenig later,
so we do not print uneeded info...

tests: use conf=fdatasync

Since we write here in 1k blocksize - we can't use direct.

tests: skip test if daemon is running

Skip the test, to avoid making all test failed when there is
'left' daemon running.

makefiles: install lvmdbusd executable

Use a different mechanism to ensure, lvmdbudd is executable at
install destination.

makefiles: remove autom4te.cache

For 'make distclean' remove also autom4te.cache
(from autoreconf).

gcc: correct signess comparation

Compare int to int and unsigned to unsigned...

lvchange: add log_error for failure path

When change of monitoring fails, ensure there is log_error() printed.

debug: fix sscanf type

Last commit c38b668fc3ead664a35130c1acec63518630d1dc was pushed
with type 'scanf' instead of 'sscanf' needed for buffer reading.
Interestingly this caused scanning from 'stdin' descriptor and
thus failures in lvm shell usage.

Update README

WHATS_NEW: update

tests: some fixes for lvmlockd testing

tests: reduce paths added to LD_LIBRARY_PATH

Add only stricly needed paths to compiled .so objects.

tests: check for writecache presence

Test is using writecache.

tests: makefile build proper tools list

Add compiled components to the list.

debug: use display_lvname

debug: fix parsing of /proc/self/stat

The code in init_log_file relies on the process name (COMM) to not
contain whitespaces. This change fixes it by looking up the right-most
parenthesis to safely jump past COMM.

For more context see:
https://www.openwall.com/lists/oss-security/2022/12/21/6

Code is only used with testing, so it should have no impact on regular
users.

Reported-by: Hugues Evrard <hevrard@google.com>
mm

cov: fix memleak in vgchange

Release allocated  vgname from get_single_vgname_cmd_arg()
when it is not returned via *vgname_ret.

cov: hide some messages

Living with these...

autoconf: update py-compile

Use newer version of this script from:
https://raw.githubusercontent.com/autotools-mirror/automake/4bd4640e03670e267d9763d377f107c09b4f2202/lib/py-compile

Reported-by: Adam Willamson

lvconvert: correction for thin conversion

Updates for 39457234db1d92f89d4234e2f2f5b703206281ae.
Off-by-one for option array - need 20 fields.
Missed to clear lock_args for cachepool.

README: Move to gitlab

WHATS_NEW: updates

lv: followup seg_monitor reporting

With 3596558861e62ca0c6f37bfba6a4e80e46dcca47 it's been introduced
a more fine grained description.
However 'disabled' might be actually more confusing then empty field,
so keep only the info about 'not enabled'aka dmevend is not allowed
to monitor LV which otherwise could be monitored.