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.

lvmlockd: use lock on thin pool when command names tdata

Some lvconvert commands can be used directly on the data sublv:
lvconvert ... vg/pool_tdata

The correct LV lock to use in lvmlockd is the one on the pool LV.

lvmetad: Suppress accept error on clean shutdown

Also fix error message after pthread_create() error in handle_connect().

test: return 0 if ignoring 'a' device health chars is requested

This allows lvconvert-raid-reshape-striped_to_linear.sh to succeed when
it reduces raid5 to two stripes before converting to raid1 and linear.

clvmd: Fix client list corruption

Centralise editing of the client list into _add_client() and
_del_client().  Introduce _local_client_count to track the size of the
list for debugging purposes.  Simplify and standardise the various ways
the list gets walked.

While processing one element of the list in main_loop(),
cleanup_zombie() may be called and remove a different element, so make
sure main_loop() refreshes its list state on return.  Prior to this
patch, the list edits for clients disappearing could race against the
list edits for new clients connecting and corrupt the list and cause a
variety of segfaults.

An easy way to trigger such failures was by repeatedly running shell
commands such as:
  lvs &; lvs &; lvs &;...;killall -9 lvs; lvs &; lvs &;...

Situations that occasionally lead to the failures can be spotted by
looking for 'EOF' with 'inprogress=1' in the clvmd debug logs.

clvmd: add client id to debug log messages

Use standard format to make it easier to find the client to which each
debug log message refers.

clvmd: add debuglog mutex

Log messages issued by different threads occasionally got intertwined.

tests: aux.sh fix pattern

Commit 72a58ce4b0f68f6e45ae30606fe5af21caa42b43 fixed 'translation' of
pattern by bash by using "" - so we need to drop '\' here to get
wanted behavior.

coverity: just make impossible division by zero

Visible for analyzer  code will not try to use 0 for division.

man_generator: release buffer after use

Although ATM this code has no memory management at all,
it will not make any harm to release some memory,
when it's not needed anymore.

raid: ensure enum is defined

Just making sure enum is always defined.

TODO: code path using this enum needs closer inspection.

tests: aux.sh use subshell for simple redirection

tests: correcting  globbing usage

Ensure globbing uses current dir.
Break loop when no match is found.
Let STACKTRACE finish its output when gdb fails.

tests: aux prepares lvmdbusd

Correctly skip the test when lvmdbusd is found already running.
For pgrep usage we need to add '-f -l' options to get python3 name
printed.
Remove no longer used 'pids' local var.

tests: aux do not specify bitmap for raid0

tests: check LV in proper VG

tests: check.sh uses array for list

For properly quoted args, switch to use arrays for arg list.

tests: api updates

tests: get.sh declare and assign separately

Declare and assign separately to avoid masking return values.

Also comment-out currently unused  orig variable.

tests: get.sh double quote

tests: utils.sh separately assign

Declare and assign separately to avoid masking return values.

tests: utils.sh avoid iterating over ls output

tests: utils.sh arithmetic

expr is antiquated. Consider rewriting this using $((..)), ${} or [[ ]].
$ is unnecessary on arithmetic variables.

tests: utils.sh no mixed string array

Argument mixes string and array. Use * or separate argument.

tests: utils.sh double quote

tests: lvm-wrapper.sh legacy syntax

tests: lvm-wrapper.sh double quote

tests: inittest.sh  double quote and mixing string

tests: check.sh eliminate useless cat

Useless cat. Consider 'cmd < file | ..' or 'cmd file | ..' instead.

tests: check.sh better if

Prefer [ p ] || [ q ] as [ p -o q ] is not well defined.

tests: check.sh drop unnecessary $

$/${} is unnecessary on arithmetic variables.

tests: check.sh using grep -F

fgrep is non-standard and deprecated. Use grep -F instead.

tests: check.sh no mixing string

Argument mixes string and array. Use * or separate argument.

tests: check.sh assign separately

Declare and assign separately to avoid masking return values.

tests: check.sh double quote

tests: aux.sh fix pid handling

Modification of pids is local (to subshell caused by pipeline).
pids was modified in a subshell. That change might be lost.

tests: aux.sh add -r to read

tests: aux.sh array assing

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

tests: aux.sh separately assign

Declare and assign separately to avoid masking return values.

tests: aux.sh using grep -c

Consider using grep -c instead of grep|wc -l.

tests: aux.sh no mixing string

Argument mixes string and array. Use * or separate argument.

tests: aux.sh drop unnecessary $

$/${} is unnecessary on arithmetic variables.