pvcreate: add prompt when setting dev size

If the device size does not match the size requested
by --setphysicalvolumesize, then prompt the user.

Make the pvcreate checking/prompting code handle
multiple prompts for the same device, since the
new prompt can be in addition to the existing
prompt when the PV is in a VG.

dbus: log_debug needs qualifier

Adding qualifier makes the only unqualified log_debug occurence
consistent with other uses in the same file.

Other possible ways to fix this:

- using `from .utils import log_debug`
- moving the line below `from . import utils` line

pvresize: Missing a message on error path.

test: add -y to raid1 up conversions

lvconvert: preserve region size on raid1 image count changes (v2)

Unless a change of the regionsize is requested via "lvconvert -R N ...",
keep the region size when the number of images changes in a raid1 LV.

Related: rhbz1443705

test: Update pattern to match code

Fix commit a3fdc966b5

Revert "lvconvert: preserve region size on raid1 image count changes"

This reverts commit 8333d5a969042a321f00656271bd55a37c1ea179.

test: Add -y to pvresize --setphysicalvolumesize

pvresize: Prompt when non-default size supplied.

Seek confirmation before changing the PV size to one that differs
from the underlying block device.

typo: s/extends/extents/ in lvmthin man page

bug 1439905

lvmdbusd: Improve error msg to include PV

Include PV device path when we believe it already exists.

lvmdbustest.py: Add nested testing

Make sure when a LV is used as a PV the dbus service works correctly.

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

lvmdbustest.py: Handle nested setUp & tearDown

Handle cleaning up correctly if a LV is used as a PV.

lvmdbusd: Make sure we don't hang on lvcreate

If we happen to create a lv that has a previous signature we hang on y/n
prompt, add '--yes'.

commands: improve error messages for rules

Make the error messages more consistent,
and use less code-centric wording.

lvconvert: FIXME

Add FIXME to move error path processing out of tool into library.

Related: rhbz1437653

lvconvert: fix inactive mirror up converting regression

Up converting an inactive mirror with insufficient
devs results in an over concerned warning.

Resolves: rhbz1437653

lvconvert: preserve region size on raid1 image count changes

Unless a change of the regionsize is requested via "lvconvert -R N ...",
keep the region size when the number of images changes in a raid1 LV.

Resolves: rhbz1443705

raid: handle insufficent PVs on takeover to/from raid4

Commit 7bc85177b08c7a6f4f0c3384c8d05b9089267c14
felt short relative to striped/raid0* -> raid4
and raid4 -> raid6.

Related: rhbz1438013

raid: avoid superfluous variable

test: add raid4 to upconvert allocation failure tests

test: consider changed default

commit b81b4aad24807d2aa4968aa9fc7d970a65623620
raised the region size so demand the sizes the
test checks for.

test: add upconvert allocation failure tests

test: also prevent lvconvert-raid-reshape.sh on single core

test: Adjust previous commit

Change have_single_core to  have_multi_core and go back to || logic in related test scripts.

test: Fix skip some reshape tests that hang on single core machines

Fix commit c7fb0cb861104e99c11e8b4e031d981dab7891a1.

raid: remove double minimum area check on takeover

raid: define return value on error paths

raid: avoid superfluous reload on takeover

Allow any reset rebuild flags to trigger the second update on takeover.
Use descriptive callback names.
Fix typo and add comments.

test: Skip some reshape tests that hang on single core machines

Skip hanging raid reshape tests until https://bugzilla.redhat.com/1443999
is fixed

mirror/raid: display adjusted region size with units

Display adjusted region size in units (e.g. "4.00 MiB") rather than sectors.

man: quote the word no

configure: Fix notify-dbus and dmfilemapd options.

WHATS_NEW: configure option enable-lvmlockd

spec: rename lockd to lvmlockd

configure: rename lockd to lvmlockd

test: correcting binary usage

Ensure 'test suite' run uses fsadm and dmeventd from compiled dir,
while for 'rpm' installed test use binaries installed in system.

lvconvert: typo in message

tests: use raid_region_size 512

given the new default from 5ae7a016b8e5796d36cf491345b1cf8e43ec9ea5

WHATS_NEW: raise region_size default

conf: Remove exec_prefix from confdir.

Some distros now want to put /etc outside prefix/exec_prefix.

lvcreate: raise default raid regionsize to 2MiB

Related: rhbz1392947.

conf: Add missing prefix to installation directory.

Our Makefiles also support systems that set the installation prefix
at configuration time - the use of DESTDIR is not required.

post-release

pre-release

autoconf

make generate

tests: use egrep to check for message

Seems we may get different '' around option on different nixes.

tests: aux defines fsadm path

raid: use log_error

Turn log_print into log_error for error path.

lvconvert: return error without conversion

lvconvert parameters not causing a conversion (i.e. no type,
number of stripes, stripesize or regionsize changes) will
remove any allocated reshape space in which case the command
returns success.  If reshape space does not exist though,
return error.

tests: test for new supported commands

New tests for:

lvconvert --regionsize | --swapmetadata | --startpoll |
          --mergesnapshot | mergethin

tests: restore some test changes

Restore existing tests and separate new tests
for new options later.

Keep coverage for existing syntax

Reverts bits from commits:

a801b92b2c81213dddccc9f9a09794208f92bfc9.
d71aaca07b2c14049130ab368221510c65236af7
fa2a728a395639a32707bb75e6bb4197b02769de
1e2420bca85da9a37570871cd70192e9ae831786
46abc28a480069b3e8b2c5c7c3448ffa25ec17ba
0e3e611a137a264a343335d18d500fce7af8e0f7

fsadm: support configurable full path

Just like with other tools lvm2 is using allow to define
fully configurable path.

Default is selected by $PREFIX/sbin/fsadm

thin: properly check for status for max sizes metadata

When metadata LV size was over DM_THIN_MAX_METADATA_SIZE sectors,
the info() routine was incorrectly trying to match bigger size,
while we do never pass any bigger device.

Fixing a case, where lvs should be displaying status for metadata
LV with 16GB size.

cmdline: ensure command struct is defined

Make it clear to Coverity cmd will be defined.

libdm: set min,maj

Reset derefered pointers instead of pointers.

dmfilemapd: free path

Cleanup resources before exit.

dmfilemapd: keep 1 byte for \0

Coverity noticed the buffer should have 1 byte for \0.

commands: improve error message for option rules

When a cmd def RULE fails because of a disallowed
combination of options, improve the error message
to show the option combination, not just the options
that broke the rule.

WHATS_NEW: add items since last release

commands: add rule for lvconvert commands that create pools

For an lvconvert command that creates a thin pool or cache pool,
--poolmetadata cannot be used with --readahead, --stripes or --stripesize.

WHATS_NEW: add recent RAID entries

lvconvert: better message on --regionsize

Enhance message on "lvconvert --regionsize size RaidLV".
in case the regionsize does not change and return error.

WHATS_NEW_DM: add recent dmsetup and dmfilemapd entries

Typo: change loose to lose

loose (v): set free; release
lose (v) : be deprived of or cease to have or retain

We 'lose' redundancy or 'lose' meaning.

lvconvert: fix failing valid regionsize change

Reshape check failed when regionsize changed and current raid type
was provided with no other change requested (stripes or stripesize).

E.g. "lvconvert --type raid6 --regionsize 256K" on a raid6 LV
with != 256K regionsize.

Enable --type in test script.

lvconvert: add segment type raid10_near

Introducing this alias for "raid10", avoid allocating
reshape space when converting between them.

Resolves: rhbz1441347

raid: handle insufficent PVs on takeover from striped/raid0

Remove any newly allocated sub LV (pair) remnants in case
allocation fails due to lag of (parallel) free PV space
and keep initial raid type.

Resolves: rhbz1438013

man lvmraid: wrap long lines

Break lines in examples to keep line length under 80.

raid_manip: fix typo warning message

man lvmraid: fix line wrapping in examples

man lvmraid: use same style as generated pages

lvconvert: require different LV for pool

When convering an LV to a cache or thin LV, require
that an LV being converted to cache pool or thin pool
at the same time be a different LV.

lvconvert: prevent duplicate use of metadata lv

The LV being processed cannot be used as the
pool metadata LV.

man lvmraid: improve text layout

lvconvert: validate vg name prefix for pool metadata name

man lvcreate/lvconvert: minor clean up rewording

commands: check required option value when matching command

A command def can include a specific constant option value,
but the value was not being checked for optional opts.

e.g. this is an incorrect command and does not match any
command defs:

lvconvert --type cache --cachepool vg/lv

However, it was mistakely being matched to this cmd def,
where the required options match, but the optional options
do not:

lonvert --cachepool LV_linear_striped_raid_cachepool
OO: --type cache-pool, ...

The optional options were mistakely considered matching
because 'cache' and 'cache-pool' were not being compared.

signals: fix comparison flaw

commit 9a689fb8f0a6b1675a4e819f4837bf2e0269832f
fixed SIGINT blocking but introduced a comparision
flaw to sigint_restore().

Related: rhbz1440766

signals: fix SIGINT blocking flaw causing inconsistent metadata

SIGINT isn't blocked properly after a sigint_allow(),
sigint_restore() cycle leading to illicit interruptable
metadata updates.  These can leave corrupted metadata behind.

Issues addressed in this commit:

sigint_allow() fails to set _oldmasked[] members properly due
to an offset by one bug on indexing the members of the array.

It bails out prematurely comparing to MAX_SIGINTS causing nesting
depths to be one less than MAX_SIGINTS.  Fix the comparision.

Correct the related comparison flaw in sigint_restore().

Initialize all sig_atomic_t variables consequently.

Resolves: rhbz1440766

lvconvert: cleanup prompting

lvconvert: fixe conversion message

When selecting a convenience RAID type only display
the selected type when it changed.  Display proper
current raid type when prompting.

lvconvert: raid1 -> linear prompt

Avoid 2 prompts when downconverting raid1 to linear
(related commit 0f65d7ec3a7325165af1a9f722cb760fd363e932).

lvconvert: avoid error message on raid1 -> raid4 conversion

Avoid error message
"Logical Volume *_rimage_0 already exists in volume group,,,"
on takeover conversion from a 2-legged raid1 to raid4
(aiming to reshape it adding images).

Resolves: rhbz1439398

lvconvert: prompt on raid1 image changes

Don't change resilience of raid1 LVs without --yes.

Adjust respective tests.

raid: reload on removing images

Requesting _raid_remove_images() to commit the
metadata missed to reload the origin causing a
kernel takeover error converting a 2-legged raid1
(with previously removed images) to raid5.

man: lvmraid.7 raid6_n_6 fix

raid: tidying

lvconvert: allow --type with --regionsize

Allow the combination of both arguments keeping
the raid level but changing the regionssize
(e.g. "lvconvert --type raid1 --regionsize 1M RaidLV"
 on an existing raid1 LV).

Resolves: rhbz1438396

lvchange/lvconvert: fix missing lvmlockd LV locks

lvchange/lvconvert operations that do not require the LV
to already be active need to acquire a transient LV lock
before changing/converting the LV to ensure the LV is not
active on another host.  If the LV is already active,
a persistent LV lock already exists on the host and the
transient LV lock request is a no-op.

Some lvmlockd locks in lvchange were lost in the cmd def
changes.  The lvmlockd locks in lvconvert seem to have
been missed from the start.

lvchange: tidy switch code in _lvchange_properties_single()

lvchange: fix missing return value

A return value from lvchange_persistent_cmd() was missed
in commit 1c41898c07ad750820fb39770355fded8e9b030a

lvmconf: fix systemd unit existence check for lvmconf --services --startstopservices

We have to unset the LoadState variable from previous use when we check
for systemd unit state. We use this variable to check if systemd services
are loaded or not and if it is loaded, we issue systemctl commands to
enable/disable and start/stop the service. We don't issue these commands
if the unit is not loaded to avoid error messages which may confuse users.

commands: fix misspelling of option value

(The cmd def parser really needs to catch mistakes
like this as it did when it was external.)

lvchange: fix --poll value when set from option

The actual value specified by the --poll y|n option was not
being used.  The way the --poll value is used is hidden
through an indirection where the value is stored in a
global variable at the start of the command, and then the
value is read from there later.  Setting the global
variable early in the command had been lost with the
cmd def changes.

vgchange/lvchange: fix poll and monitor use

Fill in some gaps where old versions of lvm allowed
--poll and --monitor in combination with other operations,
but those combinations had been lost since the cmd def work.
(The new cmd def code also added some combinations that
had been missed by the old code.)

Changes:

lvchange --activate: add poll and monitor options, and
                     add calls to them in implementation.

lvchange --refresh: add monitor option (poll already there),
                    and call to monitor in implementation.

lvchange <metadata ops>: add poll and monitor options, and
                         add calls to them in implementation.

vgchange <metadata ops>: add poll option (call to poll
                         already in implementation).

vgchange --refresh: remove monitor option (not used by code)

lvchange --persistent y: add poll and monitor options, and
                         add calls to them, and to activate
                         in the implementation. (Making it
                         match the main lvchange metadata
                         command.)

Summary of current usage:

lvchange --activate: monitor, poll
vgchange --activate: monitor, poll

lvchange --refresh: monitor, poll
vgchange --refresh: poll

lvchange --monitor: ok
lvchange --poll: ok
lvchange --monitor --poll: ok
vgchange --monitor: ok
vgchange --poll: ok
vgchange --monitor --poll: ok

lvchange <metadata ops>: monitor, poll
vgchange <metadata ops>: poll

lvchange: enhance avoiding multiple metadata updates/reloads/backups

Enhance commit 25b5915c9b5260c59d627bd1f6db8220bd4ad61e
to process options requiring immediate metadata commits
and reloads after those we can group together doing just
one commit and an optional reload for the whole group.

Backup metadata after processing options successfully.

Related: rhbz1437611

man: '-pool' regexp

lvconvert: fix converting hidden lv to pool

A hidden LV can not be converted to a thin pool or cache pool.