tests: add udev_wait before tearing scsi_debug

Make less probably there will be some udev work going while we try
to remove scsi_debug target.

tests: aux.sh fix double quote

Patch 72a58ce4b0f68f6e45ae30606fe5af21caa42b43 was wronly placing
double quotes around this variable which we want to pass expanded,
as it's just set of 'space' device args ATM.

TODO consider using array[@] to make this cleaner.
Add shellcheck directive to skip warning here

tests: utils.sh six stacktrace

Changes:

- BASH_SOURCE index was one off.
- The first line of stacktrace was pure confusion displaying executed
  script together with innermost line number (which was either 125 when
  STACKTRACE or 229 when skip was called.)
- We can safely ignore innermost call, as stack trace is always produced
  by stacktrace function.
- It is safer to test for array length, instead of testing FUNCNAME is
  main - if main function were introduced.
- Bashishm is safe to use as this function as a whole is relying on bash.

raid: add data_offset incompatibility segment type flag

In order to reject out of place reshaping with segment data_offset
field on old runtime, add a respective segment type incompatibility
flag causing "+RESHAPE_DATA_OFFSET" to be suffixed to the segment
type name.

raid: use return_0 for better backtracking

report: fix data_offset/new_data_offset reporting

lvconvert: allow reshaping in the cluster and on open devices

The previous commit fixed allocation/activation of reshape space.

Remove conditionals prohibiting reshaping in these cases.

Related: rhbz1447812
Related: rhbz1448116
Related: rhbz1461562

raid: fix allocation/activation of reshape space

When reshape space is allocated anew, an update and reload is needed to
promote the new size to the cluster node with the exclusively active RaidLV
or reloading the RaidLV will fail with a size related error.  Additionally,
store "data_offset <sectors>" with the RaidLV in the lvm2 metadata so that
it can be retrieved on cluster nodes.

Process allocation of reshape space on a 2-legged raid4/5 (interim layout
to convert from/to linear via raid1) properly in the cluster.

Resolves: rhbz1461562
Resolves: rhbz1448116

tests: aux.sh integrate both teardown loops

Use 1 logic for 2 loops tearing down left device.
First loops tries to remove all closed devices with 'normal' remove.
Second loop tries to replace those left devices with 'error' target.

tests: skip test for non dirty cache

Seems some cache LV can't be made dirty easily with older
cache policy implementations....

tests: aux.sh better teardown loop

We can't really sleep that much in teardown as it slows test too much.
So do a nested loop (similar to 'dmsetup remove_all') and keep
removing devices with open count == 0 as long as it works.

tests: wait for udev to minize race windows

mkfs can complain about volume in use when meets udev rule processing.

tests: bound variables

tests: delay read of cache pool data

It's better to delay read of cached data.
Limit loop checking for cleaner policy.

tests: add some delay after dmsetup

When we want to squash as much device as possible,
it's better to give it some delay, so devices have
some time to release it's resouces for next removal.

Also drop surrounding cookie processing and let each
dmsetup call run on its own.

format_text: Extend FIXME to reduce label scans

It's unnecessarily scanning all invalid labels even when nothing changed
instead of first just scanning the ones under the lock.

docs: Refresh kernel target docs

tests: fix for new hardened bash

Add missing get_devs.
When $7 is not given use empty string.
See if we can live with less RAM disk for PVs.
Drop limitation on single core as presence 1.12 should address this.

pvs: use dummy structs in duplicate processing

Use the same dummy struct approach from report.c
when a pv needs to be processed without a real vg.

tests: properly check variables were defined

When test suite is installed some vars are not defined.

tests: use properly quoted device list

tests: fix detection of running dmeventd after test

The checking order here has happend after TESTDIR was removed
resulting in weird further error on trap path.

Properly check for unexpected dmeventd before removing TESTDIR
since  'trap' codepath is still using it.

Also try to kill this unexpected dmeventd so testing is
not skipping all next  dmeventd tests.

(Downside would be - if user would be accidentally starting
dmeventd by some regular system admin work - such dmeventd
might be killd if it's unused. It can't kill dmeventd in-use.

tests: utils.sh  drop gdb_commands.txt

Remove file after use (fixes recursive STACKTRACE calls)

tests: bounds still more vars

tests: drop type

Assuming test wanted to check $lv3 does not exists.

tests: more exact test

Check there is really just linear.

Also short-circuit mimages_are_on_ when file is empty
(avoids unbound  mimages[@] access on older bash)

tests: more vars bounded

tests: aux.sh better loop logic

Instead of using pipe for read and haveing no chance
to modify shell vars inside such loop  use  <( )
and keep using bash arrays.

tests: improve logic for a race to catch abortion

tests: hardening - no unbound vars

Dissallow usage of any unbound shell variable in tests.
All vars needs to have some defined value.

tests: check.sh simplify in_sync

Avoid calling sed.

tests: double quote

tests: correct typo

Use defined MARKED_AS_USED_MSG shell var.

tests: avoid unbound shell var usage

tests: improve test logic

Use existing vars.
Reduce extra lvchange -an and use directly activation with lvcreate.

cleanup: whitespace

commands: fix lvm shell segfaults

Don't check for -h/--help if argc is 1.

lvm: report error if options are set before command

Previously, strange parsing errors were reported.

lvm: run help from -h|--help

tests: avoid using readarray

Real bashman can live without readarray.

Support older tested machines with bash <4.0....

man: specify slow device when creating origin LV

Also make the slow device name consistent everywhere.

Signed-off-by: Eric Ren <zren@suse.com>

cleanup: fix some typos

form -> from
accomodate -> accommodate

Signed-off-by: Eric Ren <zren@suse.com>

comment: update

Use 'is' for both forms.

tests: using prepare_vg

tests: assign path

tests: fix array usage

Use bash arrays to pass devs.

tests: simplify and doublequote

tests: use well defined test

Prefer [ p ] && [ q ] as [ p -a q ] is not well defined.

Apparently && and || "short-circuit" while -a and -o do not.

tests: cleaner written eval

Although code work this coding style is better.
This \c will be a regular 'c' in this context.

tests: mixes string and array

Argument mixes string and array. Use * or separate argument

tests: fix caller of mirror_images_on

Missed vg name.

tests: fix working with devs in array

Also fixes one instance of invalid  'mirror_image_on' test
and places there correct indexed set of PVs.

tests: check mirror_images_on fixed

Fix mirror_images_on() to actually report something useful (thought
it might be tuned later).

So for now the function got through all '_mimages_' and compares
where the order of them is matching given list of devices.

tests: correcting usage of $@

Use "$@" (with quotes) to prevent whitespace problems.

tests: assign as string

Assigning an array to a string! Assign as array, or use * instead of @ to concatenate.

tests: secure rm call

Use "${var:?}" to ensure this never expands to / .
(Should never happen as 'dir' was supposed to exist from preceding
test.)

tests: improve loop

tests: check result directly

tests: replace cat usage

tests: add space to split words

Fixing shellcheck warning:
This word is outside of quotes. Did you intend to 'nest '"'single quotes'"' instead'?

tests: fix tests variable

Actually check $r instead or 'r' for test failure.

tests: replace legacy code

Use $(..) instead of legacy `..`.

tests: fix quoting

tests: drop extra sourcing of utils

This sourceing is automatic by . lib/inittest

tests: drop useless literal

Here was usage of '\' unneeded.

tests: drop unneeded ;

tests: drop unneeded check

Test should actually fail in this condition as it's testing lvmetad.

tests: drop unneeded echo

tests: drop unneeded cat

tests: fix possible misspelling

Possible misspelling: FAILED_MIXED_STR may not be assigned, but FAIL_MIXED_STR is.
Possible misspelling: FAILED_MULTI_STR may not be assigned, but FAIL_MULTI_STR is.
Possible misspelling: FAILED_BLACK_STR may not be assigned, but FAIL_BLACK_STR is.

tests: avoid masking return values

Declare and assign separately to avoid masking return values.

tests: math drop uncessary $/${}

$/${} is unnecessary on arithmetic variables.

Use $((..)) instead of deprecated $[..]

tests: aux.sh  tweaks

Some improvements.

tests: aus.sh avoid duplicating arg list

tests: aux.sh common pattern for starting daemons

tests: aux.sh move corosyn

tests: cleanup msg

tests: scripts double quote

tests: use read -r

tests: replace egrep

tests: use bash array

tests: drop wc -l when grep -c works

tests: use bash

tests: double quote

tests: correcting usage of '==' in bash

tests: utils.sh assign directly

Assign properly quoted vars in eval directly.

tests: get.sh use simplier tr

tests: aux.sh check for already present config

'aux lvmconf' detects if new config needs to be regenerated.

tests: aux.sh double quote

tests: aux.sh replace sed call

Use internal shell vars manips

tests: aux.sh use grep -E

egrep is nonstandard.

tests: aux.sh drop useless echo

Simply trim on space.

tests: aux.sh check cd result

Use 'cd ... || exit' or 'cd ... || return' in case cd fails.

tests: export many shell vars with 1 command

udev: also create /dev/disk/by-part{label,uuid} and gpt-auto-root symlinks

The blkid we call in 13-dm-disk.rules also returns identifiers for
partitions based on which the /dev/disk/by-part{uuid,label} and
gpt-auto-root symlinks should be created in the same manner as we
already create symlinks for filesystem labels and uuids.

This is because we handle blkid calls and symlink creation under
/dev/disk ourselves in our 13-dm-disk.rules for device-mapper devices
for us to have more control over this process.

See also https://lists.freedesktop.org/archives/systemd-devel/2017-July/039220.html
and original report http://tracker.ceph.com/issues/19489 for
the exact case where these symlinks were missing.

commands: fix handing of ENO_SUCH_CMD in script

Fix the error messages when an unrecognized command is
run from a script.  We shouldn't attempt to parse options
for an unrecognized command name, which causes misleading
errors about bad options, but rather exit right when we
know the command name is not valid.  Also don't complain
about exiting without an error message when running a
script if no command didn't exist.

commands: centralize shell vs script vs command decision

This clarifies the point where lvm decides if it is
running an lvm shell, command or a script.

lvmlockd: use DM_UUID_LEN for buffer size

lvmlockd: miss adopt orphaned resources

1. dm_uuid is 68 byte length, but buf is 64 which
   will cause miss match uuid from lv lock manager
2. no lv lock_type path in dm config, use lock_args instead

Signed-off-by: Zhang Huan <zhanghuan@chinac.com>

lvmlockd: fix revert in lvcreate

If the activation step in lvcreate fails (e.g. the specified
minor number is already used), then the lvcreate is reverted,
but the LV lock in lvmlockd was not being unlocked or properly
freed.