diff --git a/test/TEST-14-IMSM/.shchkdir b/test/TEST-14-IMSM/.shchkdir new file mode 100644 index 00000000..e69de29b diff --git a/test/TEST-14-IMSM/create-root.sh b/test/TEST-14-IMSM/create-root.sh index a565a054..555a3fdb 100755 --- a/test/TEST-14-IMSM/create-root.sh +++ b/test/TEST-14-IMSM/create-root.sh @@ -77,7 +77,7 @@ mdadm --detail --export /dev/md0 | grep -F MD_UUID > /tmp/mduuid echo "MD_UUID=$MD_UUID" { echo "dracut-root-block-created" - echo MD_UUID=$MD_UUID + echo MD_UUID="$MD_UUID" } | dd oflag=direct,dsync of=/dev/sda mdadm --wait-clean /dev/md0 sync diff --git a/test/TEST-14-IMSM/cryptroot-ask.sh b/test/TEST-14-IMSM/cryptroot-ask.sh index 7318fa70..a6b7ac70 100755 --- a/test/TEST-14-IMSM/cryptroot-ask.sh +++ b/test/TEST-14-IMSM/cryptroot-ask.sh @@ -1,5 +1,5 @@ #!/bin/sh -[ -b /dev/mapper/$2 ] && exit 0 +[ -b /dev/mapper/"$2" ] && exit 0 printf test > /keyfile -/sbin/cryptsetup luksOpen $1 $2 < /keyfile +/sbin/cryptsetup luksOpen "$1" "$2" < /keyfile diff --git a/test/TEST-14-IMSM/test-init.sh b/test/TEST-14-IMSM/test-init.sh index 1f91210e..9853e18c 100755 --- a/test/TEST-14-IMSM/test-init.sh +++ b/test/TEST-14-IMSM/test-init.sh @@ -1,114 +1,10 @@ #!/bin/sh -getcmdline() { - while read -r _line || [ -n "$_line" ]; do - printf "%s" "$_line" - done < /proc/cmdline -} +. /lib/dracut-lib.sh -_dogetarg() { - local _o _val _doecho - unset _val - unset _o - unset _doecho - CMDLINE=$(getcmdline) - - for _o in $CMDLINE; do - if [ "${_o%%=*}" = "${1%%=*}" ]; then - if [ -n "${1#*=}" -a "${1#*=*}" != "${1}" ]; then - # if $1 has a "=", we want the exact match - if [ "$_o" = "$1" ]; then - _val="1" - unset _doecho - fi - continue - fi - - if [ "${_o#*=}" = "$_o" ]; then - # if cmdline argument has no "=", we assume "=1" - _val="1" - unset _doecho - continue - fi - - _val="${_o#*=}" - _doecho=1 - fi - done - if [ -n "$_val" ]; then - [ "x$_doecho" != "x" ] && echo "$_val" - return 0 - fi - return 1 -} - -getarg() { - local _deprecated _newoption - while [ $# -gt 0 ]; do - case $1 in - -d) - _deprecated=1 - shift - ;; - -y) - if _dogetarg $2 > /dev/null; then - if [ "$_deprecated" = "1" ]; then - [ -n "$_newoption" ] && warn "Kernel command line option '$2' is deprecated, use '$_newoption' instead." || warn "Option '$2' is deprecated." - fi - echo 1 - return 0 - fi - _deprecated=0 - shift 2 - ;; - -n) - if _dogetarg $2 > /dev/null; then - echo 0 - if [ "$_deprecated" = "1" ]; then - [ -n "$_newoption" ] && warn "Kernel command line option '$2' is deprecated, use '$_newoption=0' instead." || warn "Option '$2' is deprecated." - fi - return 1 - fi - _deprecated=0 - shift 2 - ;; - *) - if [ -z "$_newoption" ]; then - _newoption="$1" - fi - if _dogetarg $1; then - if [ "$_deprecated" = "1" ]; then - [ -n "$_newoption" ] && warn "Kernel command line option '$1' is deprecated, use '$_newoption' instead." || warn "Option '$1' is deprecated." - fi - return 0 - fi - _deprecated=0 - shift - ;; - esac - done - return 1 -} - -getargbool() { - local _b - unset _b - local _default - _default="$1" - shift - _b=$(getarg "$@") - [ $? -ne 0 -a -z "$_b" ] && _b="$_default" - if [ -n "$_b" ]; then - [ $_b = "0" ] && return 1 - [ $_b = "no" ] && return 1 - [ $_b = "off" ] && return 1 - fi - return 0 -} export PATH=/sbin:/bin:/usr/sbin:/usr/bin -strstr() { [ "${1##*"$2"*}" != "$1" ]; } -CMDLINE=$(while read line || [ -n "$line" ]; do echo $line; done < /proc/cmdline) -plymouth --quit +command -v plymouth > /dev/null 2>&1 && plymouth --quit exec > /dev/console 2>&1 + echo "dracut-root-block-success" | dd oflag=direct,dsync of=/dev/sda export TERM=linux export PS1='initramfs-test:\w\$ ' @@ -119,4 +15,5 @@ echo "made it to the rootfs!" getargbool 0 rd.shell && sh -i echo "Powering down." mount -n -o remount,ro / +sync poweroff -f diff --git a/test/TEST-14-IMSM/test.sh b/test/TEST-14-IMSM/test.sh index f228bdca..39efd577 100755 --- a/test/TEST-14-IMSM/test.sh +++ b/test/TEST-14-IMSM/test.sh @@ -1,4 +1,5 @@ #!/bin/bash +# shellcheck disable=SC2034 TEST_DESCRIPTION="root filesystem on LVM PV on a isw dmraid" KVERSION=${KVERSION-$(uname -r)} @@ -8,62 +9,64 @@ KVERSION=${KVERSION-$(uname -r)} #DEBUGFAIL="$DEBUGFAIL udev.log-priority=debug" client_run() { - echo "CLIENT TEST START: $@" + echo "CLIENT TEST START: $*" - rm -f -- $TESTDIR/marker.img - dd if=/dev/zero of=$TESTDIR/marker.img bs=1M count=1 + rm -f -- "$TESTDIR"/marker.img + dd if=/dev/zero of="$TESTDIR"/marker.img bs=1M count=1 - $testdir/run-qemu \ - -drive format=raw,index=0,media=disk,file=$TESTDIR/marker.img \ - -drive format=raw,index=1,media=disk,file=$TESTDIR/disk1 \ - -drive format=raw,index=2,media=disk,file=$TESTDIR/disk2 \ + "$testdir"/run-qemu \ + -drive format=raw,index=0,media=disk,file="$TESTDIR"/marker.img \ + -drive format=raw,index=1,media=disk,file="$TESTDIR"/disk1 \ + -drive format=raw,index=2,media=disk,file="$TESTDIR"/disk2 \ -append "panic=1 systemd.crash_reboot $* root=LABEL=root rw debug rd.retry=5 rd.debug console=ttyS0,115200n81 selinux=0 rd.info rd.shell=0 $DEBUGFAIL" \ - -initrd $TESTDIR/initramfs.testing + -initrd "$TESTDIR"/initramfs.testing - if ! grep -U --binary-files=binary -F -m 1 -q dracut-root-block-success $TESTDIR/marker.img; then - echo "CLIENT TEST END: $@ [FAIL]" + if ! grep -U --binary-files=binary -F -m 1 -q dracut-root-block-success "$TESTDIR"/marker.img; then + echo "CLIENT TEST END: $* [FAIL]" return 1 fi - echo "CLIENT TEST END: $@ [OK]" + echo "CLIENT TEST END: $* [OK]" return 0 } test_run() { - read MD_UUID < $TESTDIR/mduuid + read -r MD_UUID < "$TESTDIR"/mduuid if [[ -z $MD_UUID ]]; then echo "Setup failed" return 1 fi client_run rd.auto rd.md.imsm=0 || return 1 - client_run rd.md.uuid=$MD_UUID rd.dm=0 || return 1 + client_run rd.md.uuid="$MD_UUID" rd.dm=0 || return 1 # This test succeeds, because the mirror parts are found without # assembling the mirror itsself, which is what we want - client_run rd.md.uuid=$MD_UUID rd.md=0 rd.md.imsm failme && return 1 - client_run rd.md.uuid=$MD_UUID rd.md=0 failme && return 1 + client_run rd.md.uuid="$MD_UUID" rd.md=0 rd.md.imsm failme && return 1 + client_run rd.md.uuid="$MD_UUID" rd.md=0 failme && return 1 # the following test hangs on newer md - client_run rd.md.uuid=$MD_UUID rd.dm=0 rd.md.imsm rd.md.conf=0 || return 1 + client_run rd.md.uuid="$MD_UUID" rd.dm=0 rd.md.imsm rd.md.conf=0 || return 1 return 0 } test_setup() { # Create the blank file to use as a root filesystem - rm -f -- $TESTDIR/marker.img - rm -f -- $TESTDIR/disk1 - rm -f -- $TESTDIR/disk2 - dd if=/dev/zero of=$TESTDIR/marker.img bs=1M count=1 - dd if=/dev/zero of=$TESTDIR/disk1 bs=1M count=104 - dd if=/dev/zero of=$TESTDIR/disk2 bs=1M count=104 + rm -f -- "$TESTDIR"/marker.img + rm -f -- "$TESTDIR"/disk1 + rm -f -- "$TESTDIR"/disk2 + dd if=/dev/zero of="$TESTDIR"/marker.img bs=1M count=1 + dd if=/dev/zero of="$TESTDIR"/disk1 bs=1M count=104 + dd if=/dev/zero of="$TESTDIR"/disk2 bs=1M count=104 kernel=$KVERSION # Create what will eventually be our root filesystem onto an overlay ( + # shellcheck disable=SC2030 export initdir=$TESTDIR/overlay/source - . $basedir/dracut-init.sh + # shellcheck disable=SC1090 + . "$basedir"/dracut-init.sh ( - cd "$initdir" + cd "$initdir" || exit mkdir -p -- dev sys proc etc var/run tmp mkdir -p root usr/bin usr/lib usr/lib64 usr/sbin for i in bin sbin lib lib64; do @@ -71,7 +74,7 @@ test_setup() { done ) inst_multiple sh df free ls shutdown poweroff stty cat ps ln ip \ - mount dmesg dhclient mkdir cp ping dhclient dd + mount dmesg dhclient mkdir cp ping dhclient dd sync for _terminfodir in /lib/terminfo /etc/terminfo /usr/share/terminfo; do [ -f ${_terminfodir}/l/linux ] && break done @@ -79,18 +82,27 @@ test_setup() { inst_simple /etc/os-release inst "$basedir/modules.d/35network-legacy/dhclient-script.sh" "/sbin/dhclient-script" inst "$basedir/modules.d/35network-legacy/ifup.sh" "/sbin/ifup" + + inst_simple "${basedir}/modules.d/99base/dracut-lib.sh" "/lib/dracut-lib.sh" + inst_binary "${basedir}/dracut-util" "/usr/bin/dracut-util" + ln -s dracut-util "${initdir}/usr/bin/dracut-getarg" + ln -s dracut-util "${initdir}/usr/bin/dracut-getargs" + inst_multiple grep inst ./test-init.sh /sbin/init find_binary plymouth > /dev/null && inst_multiple plymouth - cp -a /etc/ld.so.conf* $initdir/etc - mkdir -p $initdir/run + cp -a /etc/ld.so.conf* "$initdir"/etc + mkdir -p "$initdir"/run ldconfig -r "$initdir" ) # second, install the files needed to make the root filesystem ( + # shellcheck disable=SC2030 + # shellcheck disable=SC2031 export initdir=$TESTDIR/overlay - . $basedir/dracut-init.sh + # shellcheck disable=SC1090 + . "$basedir"/dracut-init.sh inst_multiple sfdisk mke2fs poweroff cp umount grep dd sync inst_hook initqueue 01 ./create-root.sh inst_simple ./99-idesymlinks.rules /etc/udev/rules.d/99-idesymlinks.rules @@ -99,46 +111,49 @@ test_setup() { # create an initramfs that will create the target root filesystem. # We do it this way so that we do not risk trashing the host mdraid # devices, volume groups, encrypted partitions, etc. - $basedir/dracut.sh -l -i $TESTDIR/overlay / \ + "$basedir"/dracut.sh -l -i "$TESTDIR"/overlay / \ -m "bash lvm mdraid dmraid udev-rules base rootfs-block fs-lib kernel-modules qemu" \ -d "piix ide-gd_mod ata_piix ext2 sd_mod dm-multipath dm-crypt dm-round-robin faulty linear multipath raid0 raid10 raid1 raid456" \ --no-hostonly-cmdline -N \ - -f $TESTDIR/initramfs.makeroot $KVERSION || return 1 - rm -rf -- $TESTDIR/overlay + -f "$TESTDIR"/initramfs.makeroot "$KVERSION" || return 1 + rm -rf -- "$TESTDIR"/overlay # Invoke KVM and/or QEMU to actually create the target filesystem. - $testdir/run-qemu \ - -drive format=raw,index=0,media=disk,file=$TESTDIR/marker.img \ - -drive format=raw,index=1,media=disk,file=$TESTDIR/disk1 \ - -drive format=raw,index=2,media=disk,file=$TESTDIR/disk2 \ + "$testdir"/run-qemu \ + -drive format=raw,index=0,media=disk,file="$TESTDIR"/marker.img \ + -drive format=raw,index=1,media=disk,file="$TESTDIR"/disk1 \ + -drive format=raw,index=2,media=disk,file="$TESTDIR"/disk2 \ -append "root=/dev/dracut/root rw rootfstype=ext2 quiet console=ttyS0,115200n81 selinux=0" \ - -initrd $TESTDIR/initramfs.makeroot || return 1 - grep -U --binary-files=binary -F -m 1 -q dracut-root-block-created $TESTDIR/marker.img || return 1 - eval $(grep -F --binary-files=text -m 1 MD_UUID $TESTDIR/marker.img) + -initrd "$TESTDIR"/initramfs.makeroot || return 1 + grep -U --binary-files=binary -F -m 1 -q dracut-root-block-created "$TESTDIR"/marker.img || return 1 + eval "$(grep -F --binary-files=text -m 1 MD_UUID "$TESTDIR"/marker.img)" if [[ -z $MD_UUID ]]; then echo "Setup failed" return 1 fi - echo $MD_UUID > $TESTDIR/mduuid + echo "$MD_UUID" > "$TESTDIR"/mduuid ( + # shellcheck disable=SC2031 export initdir=$TESTDIR/overlay - . $basedir/dracut-init.sh + # shellcheck disable=SC1090 + . "$basedir"/dracut-init.sh inst_multiple poweroff shutdown inst_hook shutdown-emergency 000 ./hard-off.sh inst_hook emergency 000 ./hard-off.sh inst_simple ./99-idesymlinks.rules /etc/udev/rules.d/99-idesymlinks.rules ) - $basedir/dracut.sh -l -i $TESTDIR/overlay / \ + "$basedir"/dracut.sh -l -i "$TESTDIR"/overlay / \ -o "plymouth network kernel-network-modules" \ -a "debug" \ -d "piix ide-gd_mod ata_piix ext2 sd_mod" \ --no-hostonly-cmdline -N \ - -f $TESTDIR/initramfs.testing $KVERSION || return 1 + -f "$TESTDIR"/initramfs.testing "$KVERSION" || return 1 } test_cleanup() { return 0 } -. $testdir/test-functions +# shellcheck disable=SC1090 +. "$testdir"/test-functions