post-release

pre-release 2.03.37

WHATS_NEW: update

make: generate

lvscan: blockdevice option is unused

test: drop usage of LVM_BINARY

Actually we don't want ot define this variable to better
test real installed lvm2-testsuite package

spec: build rpm with timestamp of current date

lvmpersist: remove unregister error message

If PR was already stopped, stopping PR again would
generate an "sg_persist unregiser error" message
when each sg_persist command returned an error.
The actual result of the lvmpersist stop command
is determined by checking that the key is removed,
not by the sg_persist exit code.  If the key is not
removed from any device, then the command fails with
another higher level error message.

make: generate

revert "debug dmevent"

Unwanted.

debug dmevent

test: umounting by dmeventd

Correct and simplify test.
Use LVM_BINARY for path to lvm binary used by script.

test: aux updates

Ssend eval stderr to null as there is no need for extra trace
of eval values in teardown.

Drop passing args to lvmpoll_dump.

test: show lvmlockd debug info in testing

lvmlockd: reduce require of two SIGTERM signals in shutdown

When lvmlockd received SIGTERM, for_each_lockspace() would signal
lockspace threads to stop and then immediately check if they were
done. Due to scheduling, threads often hadn't finished yet, causing
for_each_lockspace() to return a non-zero count, which reset
daemon_quit to 0. This required a second SIGTERM to actually exit,
adding delays to daemon shutdown in tests.

Two issues were fixed:

1. Fixed mutex synchronization for thread_done flag:
   The lockspace thread was setting ls->thread_done while holding
   only lockspaces_mutex, but for_each_lockspace() was reading it
   while holding only ls->mutex, creating a race condition. Now
   thread_done is set while holding both mutexes in the correct
   order (lockspaces_mutex first, then ls->mutex).

2. Added wait loop during daemon shutdown:
   When daemon_quit is set and threads are signaled to stop,
   for_each_lockspace() now waits up to 100ms (checking every 1ms)
   for threads to actually finish before checking their status.
   This wait only activates during daemon shutdown to avoid adding
   delays to normal lockspace operations.

In practice, threads finish in 1-2ms, so shutdown completes quickly
with a single SIGTERM.

Co-Authored-By: Claude <noreply@anthropic.com>

deviced_ids: extended refresh period

If an automatic device ID refresh is triggered, and does not find
some device IDs, then extend the refresh behavior for a number of
seconds, configured in lvm.conf:

device_ids_refresh=<seconds>  accepts 10-600 seconds

Previous settings remain the same:

device_ids_refresh=0  disables automatic refresh
device_ids_refresh=1  enables one refresh

While the refresh period is extended, a new line appears in
system.devices: REFRESH_UNTIL=<YYYYMMDDHHMMSS>

The timestamp is written by the first lvm command to trigger the
refresh, and is set to current time + device_ids_refresh seconds.
REFRESH_UNTIL is removed when all devices are found, or when the
time expires.

test: tune slowdown

Here we have some tricky case - ATM we are using suspend with
flushing when changing DM tables for cache volume - but this
can take too much time with our delayed device.
So trying to minimize chance to hit this race on very fast machines.

test: improve lvmpolld_talk error messages

Refactor lvmpolld_talk() to provide clearer diagnostics when connection
fails. Previously misreported socket failures as "nc not available".
Now distinguishes between missing tools, missing socket, and connection
errors with actual error details.

Co-Authored-By: Claude <noreply@anthropic.com>

test: enhance _check_size

Grab lv_size as the first thing.
Make tests easier and drop '_not' variants.

cache: add debug message

libdaemon: log which signal triggered daemon shutdown

Capture and log the signal number that triggers daemon shutdown.
This helps diagnose unexpected daemon terminations during testing.

test: add more debug for abort case

Improve logging to better capture lvconvert problem.

dmeventd: drop unused var

fdd no longer needed.

signals: rename interruptible_usleep to sigint_usleep

The interruptible_usleep() function returned 1 on interruption (error)
and 0 on success, opposite to LVM2 convention where 1=success, 0=failure.

Changes:
- Rename to sigint_usleep() to align with sigint_*() function family
- Invert return: 1 if sleep completed, 0 if interrupted
- Update all call sites to use ! operator for error checking

This makes the API more intuitive as ! clearly indicates error path
and return values match standard LVM2 conventions.

Co-Authored-By: Claude <noreply@anthropic.com>

WHATS_NEW

make: generate

lvmconfig: make it possible to print various config types in list mode

For example:

❯  lvmconfig --type diff
report {
compact_output=1
headings=2
}
activation {
polling_interval=1
}

❯  lvmconfig --type diff --list
report/compact_output=1
report/headings=2
activation/polling_interval=1

config: add support for list mode for all cfg_def_tree_t types

Before, we printed only CFG_DEF_TREE_LIST as a list instead of
structured tree. But we can also print all the other types as lists
if needed which might be more suitable.

This patch adds a 'listmode' switch to struct config_def_tree_spec
so that the config printing functions (_out_prefix_fn, _out_line_fn,
_out_suffix_fn) can then decide on the actual format.

test: use 4k blocks for fs

test: correcting wait shell vars

test: just skip test if there are suspended devices

test: use should for a while

test: aux type

Removed extra '/'.

test: missed removal of clvmd testing

dmfilemap: stray close code reuse

Reuse header file from libdaemon directory here.
It has better code to close stray descriptors - since there
are systems with very high numbers of supported opened descriptors.

In case project would ever split header file would need to be copied.

Also use >=  check for valid file descriptor.

WHATS_NEW: update

test: aux rename wait variable

test: rework bare wait commands

Replace bare 'wait' commands with specific PID waiting to prevent
test hangs when base shell runs process in the background.

When tests use 'wait' without arguments, it waits for ALL background
processes. This may causes tests to hang indefinitely.
Fix by capturing background job PIDs and waiting
only for those specific processes.

Pattern changed from:
  some_command &
  wait

To:
  some_command &
  SOME_PID=$!
  wait "$SOME_PID" || true

The '|| true' handles cases where the process has already exited.

Co-Authored-By: Claude <noreply@anthropic.com>

test: aux kill_sleep_kill

test: remove clvmd testing infrastructure

Remove all clvmd-related test code since clvmd was removed from lvm2.
This includes prepare_clvmd(), SKIP_WITH/WITHOUT_CLVMD variables,
LOCAL_CLVMD conditional blocks, and outdated comments. Tests now run
unconditionally with non-cluster behavior.

Co-Authored-By: Claude <noreply@anthropic.com>

test: simplify corrupt_dev() function

Replace strings+grep with grep -oba for byte pattern search. This is
more direct and efficient - grep finds the byte offset, we extract it
with cut, and use dd to overwrite at that position. Also add
conv=notrunc to prevent truncation.

test: aux easier code for building config

For older bash use more simple code.

Co-Authored-By: Claude <noreply@anthropic.com>

daemonize: fix file descriptor leaks on error paths

Fix resource leaks in daemonization code across multiple daemons by
properly closing /dev/null file descriptor on all error paths.

Changes across all daemonizing code:
- Skip dup2() when fd already equals target (no-op optimization)
- Close fd on error paths before exit (prevents leak)
- Add Coverity annotations on both error and success paths
- Use consistent == -1 pattern for dup2() error checks
- Remove explicit close(0/1/2) in cmirrord (dup2 handles atomically)

This eliminates resource leak warnings from static analyzers while
maintaining correct daemonization behavior.

lvmpolld: signal child processes before pthread_cancel

Fix lvmpolld shutdown to properly terminate child lvpoll processes
before cancelling monitoring threads.

Previously, pthread_cancel was called on monitoring threads while
their child processes were still running. This could leave orphaned
lvpoll processes and cause shutdown delays.

Now send SIGTERM to child lvpoll processes first, then immediately
cancel the monitoring threads which will reap the children during
their cleanup.

This significantly reduces shutdown time in tests with lvmpolld.

Co-Authored-By: Claude <noreply@anthropic.com>

libdaemon: improve graceful shutdown of client threads

Optimize daemon shutdown to allow client threads to exit quickly
without blocking on socket operations, and refactor code to improve
maintainability.

Functional improvements:

1. Close listening socket BEFORE waiting for client threads
   - Prevents new connections during shutdown
   - Unlinks socket file earlier in shutdown sequence

2. Shutdown all client sockets with SHUT_RDWR
   - Interrupts blocking read() calls in client threads
   - Allows threads to exit immediately instead of hanging
   - Uses shutdown() to wake blocked threads, not just close()

3. Add early exit check in client thread loop
   - Check _shutdown_requested at start of while loop
   - Allows threads to exit gracefully when shutdown is initiated

Code organization improvements:

4. Eliminate goto statement
   - Replace "goto out" with direct function calls
   - Improves code readability and control flow clarity
   - Both daemon_main and daemon_init paths now follow same pattern

5. Extract _shutdown_sockets() helper function
   - Centralizes all socket shutdown logic in one place
   - Handles both listening socket close and client socket shutdown
   - Respects _systemd_activation flag for socket file unlinking
   - Includes comprehensive logging for shutdown operations

6. Extract _daemon_cleanup() helper function
   - Common cleanup path for both daemon_main and normal execution
   - Handles daemon_fini callback, logging, pidfile removal
   - Eliminates code duplication between execution paths

This allows daemon to shutdown cleanly without client threads
blocking on socket operations, enabling faster test execution and
cleaner shutdown sequences.

Co-Authored-By: Claude <noreply@anthropic.com>

cov: missing free

With recent commit f7a308d2e71e332a9cbc01d917182ba562943b77
we missed to relase allocated idname.

dmeventd: fix missing lock when reading _timeout_thread_id

Acquire _timeout_mutex before reading _timeout_thread_id to match
the locking pattern used when writing this variable. While the
variable is only written once and never changed, static analyzers
flag the inconsistent lock usage.

Release mutex before pthread_join() to avoid holding locks during
blocking operations.

make: generate

test: we need fs on origin as well

Last commit made fs only on snapshot - but it's needed also on origin.

test: bash clenaups

Various improvemetns to make shellcheck less complaining.

test: check simplified

We do not need to use 5 second sleep here for checking
opened device cannot be removed - just check with mounted
snapshot - which will also be seen as opened device - but
with known holder - thus 5sec retry loop is avoided.

test: require newer target

man: update pvmove for pvmove_max_segment_size_mb

dmeventd: cleanup

Show help options in alphabetic order.

vgchange: limit persist stop in lockstop

"vgchange --lockstop --persist stop" should skip the
persist_stop() on a VG without PR required, otherwise
it hits errors attempting to stop PR on VGs where it's
not used.

test: select-report: fix checking of failure scenarios

man: update lvmdevices refresh info

test: fix lots of not usage

Added lots of FIXME - since these tests
are not working properly - but ATM it's unclear
what they are supposed to test.

test: fix use of not mimages|mirrorlog....

test: fix use of not is_lv_opened_

Correct the test and wait till dmeventd really umounts directory.

test: fix typo group to grep

We actually want to use 'not grep'.

test: fix use of not sel

Looks like we need to fix 101%.

test: fix use of not test

Fixing 'not ' and making it more readable.
Handle blkid error code 2.

test: fix use of not fill

make: generate

config: stop logging that a default value is used

default values are not interesting to see

lvmlockd: fix lock_args flags usage with dlm

lockd_lockargs_get_meta_flags() only applies to sanlock, and
would cause dlm lockspaces to fail beause it doesn't recognize
the cluster name in a dlm VG lock_args string.

lvmlockd: fix to allow for no vg_args

man: update lvmdevices

lvmdevices: update force option

When lvmdevices --update doesn't see any reason to update
system.devices, it doesn't update.  Let the user force an
update in case they see a reason to do it.

device_id: prevent incorrect dm uuid idtype

If a dm-uuid-based device was added to system.devices, and
the wrong dm-based type was specified, then the system.devices
entry would have the correct idname but the wrong idtype. e.g.

lvmdevices --adddev /dev/mapper/mpatha --deviceidtype crypt_uuid
would add an entry with: IDTYPE=crypt_uuid IDNAME=mpath-<uuid>
rather than the correct: IDTYPE=mpath_uuid IDNAME=mpath-<uuid>

This mistaken type would not affect lvm, and the device would
be entirely usable.  The dm-based types (mpath, crypt, lvmlv)
were interchangable because they all read the device id from
the same dm/uuid sysfs file.

Fix this by adding a missing check that the dm uuid prefix is
correct for the specified idtype.  Also, accept an existing
system.devices file which contains this mistake.

lvmdevices: new option --listids shows id types and names for dev

lvmdevices: new option --delid to delete device by device_id

lvmdevices: new option --addid to add device by device_id

test: aux improve daemon killing

When daemon takes multiple pids, handle them via array.
(as recently made with lvmlockd)

makefile: indent

gcc: fix fp complain about use of unit

Just quiting warning.

vdo: add missing synchronization

After wiping, formating and deactivating vdopool volume,
we should wait on udev before we continue and activate
same volume again with different UUID.

revert "test: update for new vg attr field"

This reverts commit 2855526824fd815bb0394eda3973d34cc4b985f7
for incorrect merging.
We already have 85bef39ed00dd4f657cb7781f2754f5bb5c5ac86.

test: use should for race-prone RAID unrepairable check

The test attempts to verify that RAID6 repair fails when too many devices
are unavailable. However, due to timing issues, the RAID array may
occasionally sync or partially repair before the test disables devices,
making the repair operation succeed when we expect it to fail.

Use 'should' to tolerate race conditions where:
- Fast RAID sync completes before devices are disabled
- Partial recovery happens despite disabled devices
- Background repair processes interfere with test expectations

This test typically fails as expected (unrepairable RAID), but can
legitimately pass in race conditions.

TODO: Implement more reliable control over RAID sync/repair timing to
ensure consistent test behavior.

test: use should for race-prone cache dirty state check

It's currently difficult to guarantee the cache remains dirty long enough
to trigger the expected filesystem corruption check. In rare race conditions,
the cache may flush and become clean before the uncache operation completes,
resulting in a clean filesystem when we expect corruption.

Use 'should' allow occasional test passes when the race condition results
in a clean cache.

TODO: Find a more reliable way to ensure cache stays dirty during test.

test: add some missing local keywords

spec: fix ldconfig_scriptlets macro for old RPM versions

Fix the ldconfig_scriptlets macro to work with RPM < 4.13 (Fedora < 24,
RHEL < 7.3) which don't support the %{!-option:...} conditional syntax.

Changes:
- Replace %{!-n:%{1}} with %%{?1} for positional parameter
- Add proper %% escaping: %{-n:-n} → %%{-n:-n %%{-n*}}
- Use %%{?1} instead of %{!-n:%{1}} to handle optional positional arg

The %{!-n:...} syntax (inverse conditional) was introduced in RPM 4.13.
Older versions like Fedora 19-23 (RPM 4.11-4.12) don't support it and
fail with "Package does not exist" error.

The new syntax works on both old and new RPM:
- %ldconfig_scriptlets libs
  → expands to: %post libs -p /sbin/ldconfig
- %ldconfig_scriptlets -n device-mapper-libs
  → expands to: %post -n device-mapper-libs -p /sbin/ldconfig

Fixes regression from commit d6d13f1c9 which introduced the macro but
used syntax incompatible with old RPM versions.

test: check for presence of seagate_ilm command

test: use exit 1 call in stacktrace

test: remove unused --silent option from aux device functions

Remove the unused --silent parameter from aux device management functions:
  - disable_dev()
  - enable_dev()
  - restore_from_devtable()

The --silent option was declared but is never actually used in the function
implementations - no code checked or acted upon this flag.

Updated callers in test scripts:
  - lvchange-raid-transient-failures.sh
  - lvconvert-repair-transient.sh

Both scripts called disable_dev/enable_dev with --silent flag, which is
now removed since it had no effect.

test: add declarative test requirement options

Add new inittest.sh options to declare test requirements declaratively:

New options:
  --with-multi-host: Test requires LVM_TEST_MULTI_HOST to be set
  --with-idm-failure: Test requires LVM_TEST_FAILURE to be set
  --with-lock-type-idm: Test requires LVM_TEST_LOCK_TYPE_IDM to be set
  --with-lock-type-dlm: Test requires LVM_TEST_LOCK_TYPE_DLM to be set
  --with-lock-type-sanlock: Test requires LVM_TEST_LOCK_TYPE_SANLOCK to be set

Implementation:
  - Initialize REQUIRE_* variables to 1 (enabled by default)
  - When option is specified, read from environment variable with :-0 default
  - Skip test if REQUIRE_* = 0 (environment variable not set or set to 0)
  - Tests declare requirements upfront in inittest sourcing line

Benefits:
  - Declarative: Requirements visible at top of test file
  - Consistent: Follows --skip-with-* pattern
  - No code duplication: Eliminates manual [ -z "$VAR" ] && skip checks
  - Self-documenting: Clear what environment each test needs

Example usage:
    Before: . lib/inittest --skip-with-lvmpolld
            [ -z "$LVM_TEST_MULTI_HOST" ] && skip

    After:  . lib/inittest --skip-with-lvmpolld --with-multi-host

Updated 20 test scripts to use declarative options:
  - Multi-host tests: multi_hosts_*.sh (8 files)
  - IDM failure tests: idm_*_failure*.sh, lvmlockd_failure.sh (5 files)
  - Lock type setup: aa-lvmlockd-{idm,dlm}-prepare.sh (2 files)
  - Lock type removal: zz-lvmlockd-{idm,dlm}-remove.sh (2 files)
  - Stress tests: stress_*.sh (3 files)

test: optimize lvm-wrapper.sh bash

Change shebang from /bin/sh to /usr/bin/env bash for consistency.
Convert all test commands to [[ ]] for better performance.

test: optimize utils.sh bash

Convert test/[ ] to [[ ]] for better performance.
Consolidate multi-line test conditions into single [[ ]] blocks.

Key changes:
  - Replace all test/[ ] with [[ ]] for better performance
  - Combine multiple test conditions with && into single [[ ]] blocks
  - Convert test && { } patterns to if [[ ]] for multi-line blocks
  - Standardize LVM_TEST_NODEBUG to use numeric comparison :-0

Notable improvements:
  - STACKTRACE(): Changed from
      test -z "$LVM_TEST_NODEBUG" && test -f TESTNAME && {
    to
      if [[ "${LVM_TEST_NODEBUG:-0}" -eq 0 && -f TESTNAME ]]; then

  - Unified multi-condition checks into single [[ ]] expressions
    (avoids multiple external command forks)

Performance benefits:
  - [[ ]] is bash built-in, no fork overhead
  - Single [[ ]] for multiple conditions more efficient than chained test

test: fix aux heredoc indentation in generate_config()

Use tab indentation in heredocs with <<-EOF.
The - operator strips leading tabs for clean output.

test: optimize aux.sh bash

Convert test/[ ] to [[ ]] for better performance.
Replace $(seq) with brace expansion and C-style loops.
Use compact [[ ]] && command for single-line cases.
Keep if/then/fi blocks for multi-line readability.

Key optimizations:
  - Replace all test/[ ] with [[ ]] (eliminates external command forks)
  - Replace $(seq 1 N) with {1..N} brace expansion
  - Replace $(seq 0 $((N-1))) with C-style for (( i=0; i<N; i++ ))
  - Use [[ ]] && command pattern for simple one-line conditionals

Performance benefits:
  - Brace expansion {1..N} is built into bash, no fork needed
  - [[ ]] is bash keyword, faster than external [ command
  - C-style loops avoid subshell overhead of $(seq)

test: optimize check.sh bash

Replace test/[ commands with [[ ]] for better performance.
Consolidate multi-line conditions into single [[ ]] blocks.

Changes:
  - Convert all test/[ ] to [[ ]]
  - Simplify multi-line test conditions
  - Improve readability of conditional expressions
  - Maintain consistent style across check functions

This is part of a broader effort to modernize the test suite bash code,
improve performance, and ensure consistency.

test: optimize inittest.sh bash

Convert test/[ ] to [[ ]] for better performance and consistency.
Use compact [[ ]] && command form only for single-line cases.
Keep if/then/fi blocks for multi-line readability.

Change all boolean-style test variables from empty-string defaults
to numeric zero defaults for consistency and correctness.

Problem:
  Mixed initialization patterns caused bugs. For example,
  LVM_TEST_DEVICES_FILE=${LVM_TEST_DEVICES_FILE-} defaults to empty
  string, but when set to "0", tests like [[ -n "$VAR" ]] incorrectly
  treat "0" as true (non-empty string).

Solution:
  Standardize all boolean test variables to use :-0 (default to zero)
  and test with != 0 / = 0 (numeric comparison) instead of -n/-z
  (string empty/non-empty).

Changes:
  - Default to 0 for 17 boolean variables:
    LVM_TEST_NODEBUG, LVM_TEST_LVM1, LVM_TEST_FAILURE,
    LVM_TEST_MULTI_HOST, LVM_TEST_LVMLOCKD, LVM_TEST_LVMLOCKD_TEST,
    LVM_TEST_LVMPOLLD, LVM_TEST_DEVICES_FILE,
    LVM_TEST_LOCK_TYPE_DLM, LVM_TEST_LOCK_TYPE_SANLOCK,
    LVM_TEST_LOCK_TYPE_IDM, SKIP_WITHOUT_CLVMD, SKIP_WITH_CLVMD,
    SKIP_ROOT_DM_CHECK, SKIP_WITH_DEVICES_FILE, SKIP_WITH_LVMLOCKD,
    SKIP_WITH_LVMPOLLD

  - Update all conditional tests from -n/-z to != 0 / = 0

This makes the intent clear (numeric boolean), prevents string
comparison bugs, and works correctly with strict mode.

Key change: Updated from "13 boolean variables" to "17 boolean variables"
and added the 4 SKIP_* variables to the list (SKIP_ROOT_DM_CHECK,
SKIP_WITH_DEVICES_FILE, SKIP_WITH_LVMLOCKD, SKIP_WITH_LVMPOLLD).

test: don't use global wait

test: correct _reshape_layout arg order

Swapped args in lvconvert-raid-reshape-size.sh

test: comment unused invalid tests

These test never failed for the proper reason anyway,
and they are racy since we wait for sync anyway.

So fix the test, but keep them commented out.

test: correct checked parameter

Compare proper value is 512.00k.
Otherwise this is 100% failing check.

test: update for new vg attr field

spec: improve RPM packaging with backward-compatible fixes

Fix multiple RPM spec file issues while maintaining compatibility with
RHEL 6+ and Fedora 19+:

1. Add missing %{systemd_version} macro definition (set to 219)
   Fixes undefined macro warnings in dbusd package requirements

2. Fix %clean section syntax bug and make it conditional
   - Replaced buggy: test -n rpm-no-clean (missing quotes)
   - Made conditional: only for Fedora < 18 and RHEL < 7
   - %clean section deprecated in modern RPM

3. Make ldconfig scriptlets conditional using macro
   - Introduce ldconfig_scriptlets macro in macros.inc
   - Macro handles -n flag for subpackages (e.g., device-mapper-libs)
   - Expands to scriptlets on Fedora < 28 and RHEL < 8, %{nil} otherwise
   - Replaces 3 repetitive conditional blocks (18 lines -> 3 lines)
   - Affects: libs, device-mapper-libs, device-mapper-event-libs

4. Add conditional BuildRequires for systemd-rpm-macros
   - Only for Fedora 30+ and RHEL 8+
   - Provides proper dependencies for modern systemd macro usage

spec: add conditional SPDX license identifiers for Fedora

Add SPDX license identifiers conditionally for modern Fedora releases
while maintaining backward compatibility with older distributions.

Introduce license_gpl and license_lgpl macros in macros.inc to centralize
the license handling logic:
- Fedora 37+: GPL-2.0-only, LGPL-2.0-only (SPDX identifiers)
- Fedora <37, RHEL all: GPLv2, LGPLv2 (traditional names)

Replace all 10 conditional License blocks with macro references.