WHATS_NEW: update

tests: use less demanding ext4

tests: lower demand for disk resources

Convert test to use a single skeleton and only different pieces
keep in separate tests.

Lower raid disk usage to smaller size and switch to ext4
as way less demanding fileystem.

vdo: use fixed size vdopool wrapper

Instead of using size of 'empty header' in vdopool use fixed size 4K
for a 'wrappeing' vdo-pool device.

This fixes the issue when user tried to activate vdo-pool after
a conversion from vdo managed device with 'vgchange -ay' - where
this command activated all LVs with 'vdo-pool' wrapping device as well,
but this converted pool uses  0-length header.

This 4k size should usually prevent other tools like 'blkid' recognize
such device as anything - so it shouldn't cause any problems with
duplicate indentification of devices.

cleanup: rename function

Rename _thin_lv_has_device_id and use common function naming
with _lv_has_thin_device_id().

cleanup: simplify

configure: update

autoreconf

configure.ac: print locking dir

Correcting output results from configure script when
printing locking dir.

Shuffle dmfilemapd close to dmeventd option and tracing.

Couple indent changes.

integrity: hide creation of internal LVs

Creation of 'hidden' local LVs should be visible only in
verbose mode (-v) when creating a raid LV with integrity.

raidintegrity: allow writecache and cache

Allow writecache|cache over raid+integrity LV.

tests: integrity fixes

wait_recalc was mistakenly being called on raid without integrity
wait_recalc loop was exiting without error

lvmdbusd: Correct seg. fault on s390x ELN

syscall 186 is specific to x86 64bit.  As this is different from arch
to arch and between same arch different arch size we will only grab
thread ID using built-in python support if it is supported.

Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=2166931

lvmdbusd: Correct locking for _common_log

Reduce the lock time and include the flush in the lock.

Reported by: Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>

Fix import of utils from lvmdbusd.cfg

Signed-off-by: Adam Williamson <awilliam@redhat.com>

post-release

pre-release

make: generate

tests: nomda-restoremissing

This test was broken by commit 69ea2e3f9
"vg_read: keep MISSING_PV when device with no mda reappears"
Change the test to match the new behavior.

lvmdbusd: Add a new 'Shared' property to the Vg interface

Signed-off-by: Vojtech Trefny <vtrefny@redhat.com>

lvmdbustest: Correct env. logic

Signed-off-by: Tony Asleson <tasleson@redhat.com>

lvmdbustest: Add notes on error injection testing

Not doing this by default for automated build testing as it generates a
lot of output and slows things down by another ~10%.

lvmdbustest: Add env. variable to skip signal testing

This is currently need if we are collecting code coverage data as we don't
handle this unit test correctly.

lvmdbustest: Check for needed env. variables

lvmdbustest: Don't use hard coded /tmp for error inject log

lvmdbusd: Add a retries during initial load

When the daemon is starting we do an initial fetch of lvm state.  If we
happened to get some type of failure with lvm during this time we would
exit.  During error injection testing this happened enough that
the unit tests were unable to finish.  Add retries to ensure we can get
started during error injection testing.

lvmdbustest: Shutdown cleanly with "exit"

Make the error injection wrapper handle the exit case, so that we can
clean up gracefully when instructed to do so.

lvmdbustest: Only inject 1 missing key error

Previously we were injecting a missing key in the lv, vg, and pv.
Given the order of processing in lvmdbusd, this prevented us from
exercising all the error paths.  Change to returning just 1 instead.

lvmdbustest: Correctly set up env. for error injection

lvmdbustest: open log file with "a"

So we retain the log across fork & exec and lvm shell testing.

lvmdbusd: Handle missing key in get_key

When we sort the LVs, we can stumble on a missing key, protect against
this as well.

Seen in error injection testing:
Traceback (most recent call last):
  File "/home/tasleson/projects/lvm2/daemons/lvmdbusd/fetch.py", line 198, in update_thread
    num_changes = load(*_load_args(queued_requests))
  File "/home/tasleson/projects/lvm2/daemons/lvmdbusd/fetch.py", line 83, in load
    rc = MThreadRunner(_main_thread_load, refresh, emit_signal).done()
  File "/home/tasleson/projects/lvm2/daemons/lvmdbusd/utils.py", line 726, in done
    raise self.exception
  File "/home/tasleson/projects/lvm2/daemons/lvmdbusd/utils.py", line 732, in _run
    self.rc = self.f(*self.args)
  File "/home/tasleson/projects/lvm2/daemons/lvmdbusd/fetch.py", line 40, in _main_thread_load
    (lv_changes, remove) = load_lvs(
  File "/home/tasleson/projects/lvm2/daemons/lvmdbusd/lv.py", line 148, in load_lvs
    return common(
  File "/home/tasleson/projects/lvm2/daemons/lvmdbusd/loader.py", line 37, in common
    objects = retrieve(search_keys, cache_refresh=False)
  File "/home/tasleson/projects/lvm2/daemons/lvmdbusd/lv.py", line 72, in lvs_state_retrieve
    lvs = sorted(cfg.db.fetch_lvs(selection), key=get_key)
  File "/home/tasleson/projects/lvm2/daemons/lvmdbusd/lv.py", line 35, in get_key
    pool = i['pool_lv']
KeyError: 'pool_lv'

lvmdbusd: Replace assert with exception

Instead of using an assert we will raise an LvmBug exception

tests: update missing-pv

add more checks and expand comments explaining behavior

vg_read: keep MISSING_PV when device with no mda reappears

Remove old code that became incorrect at some point.
It's probably a fragment of an old condition that was left
behind because it wasn't understood.  We don't want to drop
the MISSING_PV flag just because the PV has no mda in use.
The device that was missing may have stale data, so the user
needs to decide if the device should be removed or restored.

tests: vg-raid-takeover

Different sequences of steps that could be used to handle raid LVs
after VG takeover (what would happen in cluster failover) combined
with the loss of a disk.

vgchange: allow changing system ID with majority of PVs

when used with --majoritypvs.  This allows the fail-over
of a VG between systems by changing the VG system ID when
a PV is missing.

lvmlockd: clean up get_local_nodeid

Hard to see if fclose calls were correct, and coverity couldn't figure
it out either, so make it clear.

toollib: fix segfault if using -S|--select with log/report_command_log=1 setting

When we are using -S|--select for non-reporting tools while using command log
reporting (log/report_command_log=1 setting), we need to create an internal
processing handle to handle the selection itself. In this case, the internal
processing handle to execute the selection (to process the -S|--select) has
a parent handle (that is processing the actual non-reporting command).

When this parent handle exists, we can't destroy the command log report
in destroy_processing_handle as there's still the parent processing to
finish. The parent processing may still generate logs which need to be
reported in the command log report. If the command log report was
destroyed prematurely together with destroying the internal processing
handle for -S|--select, then any subsequent log request from processing
the actual command (and hence an attermpt to access the command log report)
ended up with a segfault.

See also: https://bugzilla.redhat.com/show_bug.cgi?id=2175220

lvmdbusd: Add needed env. variables

When running lvmdb.py by itself for testing we need these.

lvmdbusd: Move cfg.lvmdebug data init

Move this to the cfg file itself, so that initialization runs when it
gets processed.

lvmdbustest: Test duplicate VG rename

lvmdbustest: Add more info on asserts

lvmdbustest: Don't hard code to /tmp

During lvm testing they use an alternative tmp directory.

lvmdbustest: Add more output for test_log_file_option

lvmdbustest.py: Increase udev timeout

lvmdbusd: Remove unused import

lvmdbusd: Add lock to prevent concurrent lvm shell access

There is a window of time where the following can occur.

1. An API request is in process to the lvm shell, we have written some
   command to the lvm shell and we are blocked on that thread waiting
2. A signal arrives to the daemon which causes us to exit.  The signal
   handling code path goes directly to the lvm shell and writes
   "exit\n".  This causes the lvm shell to simply exit.
3. The thread that was waiting for a response gets an EIO as the child
   process has exited.  This bubbles up a failure.

This is addressed by placing a lock in the lvm shell to prevent
concurrent access to the shell.  We also gather additional debug data
when we get an error in the lvm shell read path.  This should help if
the lvm shell exits/crashes on its own.

lvmdbustest.py: Remove use of root_dir in glob

This feature has only been in python since 10/2021.

lvresize: fix check for mounted and renamed LV to handle spaces

Replace spaces with \040 in directory paths from getmntent (mtab).

The recent commit 5374a44c5712 compares mount point directory paths
from /etc/mtab and /proc/mounts, in order to detect when a mounted
LV has been renamed.  The directory path comparison does not work
correctly when the path contains spaces because getmntent uses
ascii space chars and proc replaces spaces with \040.

configure: update

WHATS_NEW: updates

configure.ac: use pkg-config to detect static libs

Add some Gentoo based patches for better support of static linking.
This are not tested nor supported by upstream developers.
Usage requires presence of several libraries in their static form
which is however not commonly available.

Selinux modified by zkabelac to still work on older sofrware which
did not provided libselinux.pc at a time - see keep the old check
present and use pkg-config only when possible.

configure.ac: add STATIC_LDFLAGS

Add support for specif STATIC_LDFLAGS when linking static binaries.

configure.ac: misc

configure.ac: fail configure for missing libraries

When user enables lockd libraris, code needs to fail, when then are
missing. Also when notify-dbus support if enabled, and libsystemd is
missing, abort configuration.

configure.ac: update dlm check

Check for pkg-config --libs libdlm_lt  and test if the returned value
contains word 'pthread' - if so, it's likely a buggy result from
incorrect config file and use directly  -ldlm_lt for this case.

makefiles: add pthread to libdevmapper pkgconfig

Add pthread to pkg_config file.

makefiles: avoid grep

Avoid getting annoyting grep warning and use just awk to get the
value.

aux: lvmdbus checks for JSONDecodeError

Skip lvmdbus testing with too old python.

tests: add missing quotes

Add quotes around paths and also add fixme for checking
system is actually usable for a test with all running
services.

tests: use aux mdadm wrapper

Wrapper takes care of proper MD device name around its various
incarnations.

tests: remove option chunk for mdadm raid1

Not sure why, but --chunk specification for mdadm raid1 is no longer
'wanted' ???

tests: add fixme

tests: some skips with 6.2 kernel

Fix did not managed to get to final release so skip test on this kernel.

vgimport: invalidate hints

There may be some unconventional methods of sharing disks with vgimport
where hints could cause confusion.  Hints should be disabled when
sharing disks, or pvscan --cache used to regenerate hints as needed,
but invalidating hints from vgimport might help reduce confusion.

post-release

pre-release

lvmdbustest: Check and remove lvm debug collection files

In setup/teardown ensure we check for and remove any lvm debug
collection, to prevent FS from filling up.

lvmdbusd: Disable collecting lvm debug data by default

lvmdbusd: Fix grammar

configure: update

tests: safety first...

tests: lvmdbus testing use --debug optinally now

To shorten down logging output of lvmdbus - use for now lvmdbus
with --debug option only when LVM_DEBUG_LVMDBUS=1 is set.

tests: check failing vdo conversion

When we detect not usable vdo on a machine, check lvconvert fails.

makefiles: move include earlier

configure.ac: various minor updates

configure.ac: remove unneded vars and settings

configure.ac: fix incorrect check for valid configuration

configure.ac: use standardized LIBS and CFLAGS

Convert lvmlockd to use configure _LIBS and _CFLAGS for
discovered libraries.

TODO: ATM we ignore discovered libdlm and use libdlm_lt instead.
Also libseagate_ilm is hard to find unicorn for testing.

configure.ac: use LIBSYSTEMD

Convert naming SYSTEMD_CFLAGS/LIB -> LIBSYSTEMD_CFLAGS/LIBS
to better fit library check for libsystemd.

Build lvmlockd with SD_NOTIFY when we have defined LIBSYSTEMD_LIBS.

debug: missing backtraces

cov: avoid using strcpy

Coverity is complaining about unchecked strcpy here, which is
irelevant as we preallocate buffer to fit in copied string,
however we could actually reuse these size and use just memcpy().
So lets make some simple conversions.

lvconvert: use log_print_unless_silent

For better -qq.

lvconvert: fix return codes

Several inverted return code resulted the command returned
a success as exit code, while the function errored internally.

pvscan: remove warning about excluded root dev

This was trying to warn about a misconfig where the root PV
was excluded by the devices file, but it wasn't covering all
cases.  Also, it's not very useful because the root LV is
usually activated directly in the initrd.

pvscan: filter does not need to be checked for symlink names

With the recent use of DEVLINKS, there is no longer any real
point in checking the filter for symlink names.  Removing
this check should not change behavior with or without symlinks
in the filter.

configure: update

configure.ac: more cleanup

Some variable simplification,
correct messages with PATH_SBIN.

configure.ac: remove UDEV_SYSTEMD_BACKGROUND_JOBS

Missed to go with 042fbd43d25f4cc69ceec1aea4d0658bec54b6e3.

configure.ac: fix support for LOCKDIDM

Commit 7a8b7b4adde5c31c79aee8b0792cd8369652afc6 introducing lockidm
support missed to use AC_SUBST() for a variable and provide it only
in prebuilt configure, so with the next autoreconf this variable
was lost and IDM support was no longer compiled.

Fixes: https://github.com/lvmteam/lvm2/issues/98
Reported-by: Tom Prohofsky

acinclude.m4: add local copy of PKG_CHECK_VAR

Just in case an old distro tries to autoreconf and doesn't
come with newer pkg_ macros.

tests: use PWD for dbus testing

lvmdbusd ATM is leaking too many log files.
At least try to keep them removed with test failure.

tests: skip test without mkfs.ext4

cov: drop unneeded header

man: update lvmcache cache setting

Correct setting of migration_threshold and add clarify
how the user can restore default cache settings for cache policies.
Also document mq aliasing with smq for newer kernels.

makefiles: update doc

Document using of target useful for checking variable content.

makefiles: allow using configure cppflags

Configure defines 'CPPFLAGS' and we should use defined value
for compilation.

Reported-by: debian

makefiles: use configured systemd library paths

Use discovered/selected systemd library from configure.

WHATS_NEW_DM: update

libdm: match reactivation of sibling for snapshot

Apply same code for libdm as in device_mapper dir from
commit c8a5948a71b041ae2cce03e1b3a9843f0fab023f.

libdm: improve parallel create of control node

When two parallel commands would have tried to create our
/dev/mapper/control node at the same time, one of them could
actually fail even if the 2nd. command actually mknod()
this control node correctly.

So for EEXIST case add detection if the control node is ok.

This may possibly help with some race case in early boot.