fix(github): yml syntax and commit message for dependabot

Also fix commit message prefix.
chore(github):(deps): bump ... --> chore(deps): bump ...

test: use -device instead of -watchdog to remove qemu warnings

test: make test 01 and 02 use the test dracut modules

test: explicitly check if systemctl is available for systemd tests

Dracut tests already do similar checks for test 40, 62, 63 and 98.

This is useful when the full testsuite is run and systemd is not
available.

chore(ci): bump docker/setup-buildx-action from 1 to 2

v2 has node 16 as default runtime.

ci: add mkfs.ntfs and ntfs-3g to test containers

ntfs-3g is needed to test dmsquash-live-ntfs

docs: correct "rd.blacklist"

fix(dracut-systemd): run systemctl daemon-reload after remove_hostonly_files

Dependencies to systemd .device unit files are marked as "hostonly" in the
special file hostonly-files. These .device unit files can be removed using the
`rd.hostonly=0` command line parameter. But, since systemd v252
(https://github.com/systemd/systemd/pull/24054), the initrd-parse-etc.service
service does not run `systemctl daemon-reload`, so systemd keeps waiting
for these .device units to show up.
This means that if a hostonly device is not available for any reason at boot
time, using `rd.hostonly=0` has no effect and the system will not boot.

ci: upgrade to actions/checkout@v3

It resolves the following warning:
Please update the following actions to use Node.js 16: actions/checkout

fix(kernel-modules): add sysctl to initramfs to handle modprobe files

Users were seeing errors like this:

[     2.917246] dracut-pre-udev[717]: sh: line 1: /sbin/sysctl: No such file or directory

This was the result of modprobe.d files that needed to call sysctl
and failing because sysctl wasn't included in the initramfs.

This change makes it so that we have the binary included so those
modprobe configuration files work properly.

test(ISCSI-NM): fix failing test

server fails to startup with /sbin/sysctl: not found

ci: remove non-existing modules from labeler

ci: teach labeler about test dracut modules

test: remove layering violation from the make system

Tests should not try to build dracut itself. Test should assume dracut
is built and availabnle to run.

test: only set basedir if it is not set by the invoker

Set basedir in test-functions instead of Makefile.
This change will allow to use a different basedir without
modifying Makefiles.

test: remove suppressing shellcheck messages

They are no longer needed.

test: move more execution to dracut command line

use the -I argument to install dd and mkfs.btrfs
use the -i argument to install the hook
btrfs is already installed by the btrfs dracut module
sfdisk is not used

test: move more common dependencies from test case to test-root module

umount setsid sync binaries are used in all test cases to build rootfs

test: remove unused dependencies

test: move more execution to dracut command line

test: move plymouth installation from test case to test-root module

test: remove unused variable

test: install debug module into what test case is booting into

This will also make some of the non-essential tools optional
instead of a hard dependency.

test: building ld cache explicitly is no longer needed

dracut.sh will build the ld cache now

test: move common dependencies from test case to test-root module

feat(test-root): add new module to share code between tests

Use dracut to make a root filesystem instead of each test case.

test: move finished-false.sh from test case to test-makeroot module

test: move common dependencies from test case to test-makeroot module

poweroff cp umount sync dd are commonly used by test cases to make
root filesystem.

feat(test-makeroot): add new module to share code between tests

This module is used for tests to create root filesystem.

test: move poweroff handling from test case to test module

feat(test): add new module to share code between tests

test: shutdown is not used for running tests

ci: fix Arch container

packages should be installed with -Syu instead of -Sy.

feat(overlayfs): add a new module called overlayfs

Also split ovlfs variable with two different meaning.

fix(dmsquash-live): add support for NFS

Move overlayfs mount out into its own dedicated file
so that we can call into it directly for NFS support.

Add a new test case for overlayfs on top of NFS.

ci: use prebuilt Linux kernel binaries for Gentoo container

Also remove dmsetup. It does not build and it is not required for ci.

refactor(fs-lib.sh): udevadm info --query=property instead of --query=env

env is a long time deprecated alias to property, not listed in the man page for more than a decade.

fix(dracut-init.sh): instmods: wrong variable name

It worked most of the time because `dracut-install -o` returns 0 in normal circumstances.

fix(dmsquash-live-ntfs): remove unnecessary command

feat(qemu): add efi_secret driver

Despite the generic name, the only current use of the `efi_secret`
is exposing data in /sys that's been injected into a SEV/SEV-ES
virtual machine. More info:

https://docs.kernel.org/security/secrets/coco.html

This secret is probably a luks password, so we need this driver at
boot time.

Signed-off-by: Cole Robinson <crobinso@redhat.com>

fix(90kernel-modules): add (nonstandard) NVMe drivers

The M1/etc hardware has an quirky NVMe disk controller that needs
a dedicated NVMe driver. Rather than picking up just that
bit of hardware pick up drivers that register NVMe controllers.

This also picks up NVMe-OF style drivers, but seems like a better
long term solution on Arm platforms which tend to have a lot
of odd hardware.

Signed-off-by: Jeremy Linton <jeremy.linton@arm.com>

refactor(multipath): do not install the pidof binary

dracut-lib.sh provides a custom version of pidof, so there is no need to install
the pidof binary. Also, source dracut-lib.sh in hook scripts when needed.

Fixes issue #2027

fix(dracut-functions.sh): suppress findmnt error msg if /etc/fstab not exist

When the rootfs of a virtual machine is virtiofs shared folder,
the /etc/fstab file may not exist. As a result, the "findmnt --fstab"
will complain "findmnt: can't read (null): No such file or directory"
when trying to rebuild kdump initramfs within the vm.

This patch checks if /etc/fstab exist before calling "findmnt --fstab"
to suppress the error message.

Before:
$ kdumpctl rebuild
kdump: Rebuilding /boot/initramfs-5.14.0-182.el9.x86_64kdump.img
findmnt: can't read (null): No such file or directory
findmnt: can't read (null): No such file or directory
findmnt: can't read (null): No such file or directory

After:
$ kdumpctl rebuild
kdump: Rebuilding /boot/initramfs-5.14.0-182.el9.x86_64kdump.img

Signed-off-by: Tao Liu <ltao@redhat.com>

fix(dracut-logger.sh): this fixes the dlog_init check for /dev/log

The /dev/log check was not correct, it would continue working if
/dev/log was a broken symlink. This changes the logic to be more clear
-- /dev/log needs to be a socket, and writable, and logger needs to be
available. If any of those conditions fail, turn off syslog and only use
console logging.

feat(multipath): install tmpfiles.d config file

Recent multipath-tools (0.9.2) added a tmpfiles.d template. Install it
if present.

feat(dmsquash-live): add support for dash

Using dash instead of bash saves about 0.5M initramfs size.

Co-authored-by: nabijaczleweli <nabijaczleweli@nabijaczleweli.xyz>

feat(systemd-pcrphase): introducing the systemd-pcrphase module

Introducing the systemd-pcrphase module

fix(dracut.sh): make omit-drivers option do exact match for names

Modify the basic test case to use --omit-drivers and make it fail
without the PR and make it pass with the PR.

The test would fail with the following error without the PR:
FATAL: iscsiroot requested but kernel/initrd does not support iscsi

ci: integration test for openrc and musl

chore(base): remove dead code

RD_TIMESTAMP is no longer used.
systemd-timestamp is not used.

ci(suse.conf.example): update SUSE-specific config

feat(github): automating dependency updates

Let github's dependabot handle updating github's actions.

fix(multipath): install multipathd.socket

perf(90kernel-modules): use awk instead of shell monster

~120ms -> ~2ms

Downstream: https://bugs.debian.org/1017411

feat(systemd-portabled): introducing the systemd-portabled module

Introducing the systemd-portabled module.

It's intended to work only with raw binary disk images contained in regular
files, but not with directory trees.

fix(network-manager): always install the library plugins directory

The library plugins directory is automatically added to the initrd if either
libnm-device-plugin-team.so or libnm-settings-plugin-ifcfg-rh.so are present on
the system, but both are optional libraries, and if it does not exist, the
NetworkManager issues a warning.

feat(dracut-init.sh): add inst_libdir_dir() helper

This helper function is similar to inst_libdir_file(), but it installs plain
directories located on a library directory instead of files.

chore: adjust .kateconfig for C code

chore: remove editor configuration files from subdirectories

Keep only the toplevel ones.

Closes #1958

fix(dracut): replace invalid lzo command with lzop for LZO compression

Fixes issue #1999

refactor(dracut-install): resolve_deps: improve error message

Prefix + error description at the end.

refactor(shutdown): move module specific code out of base module

dracut-lib.sh and the base module now does not have a knowledge
of oldroot and plymouth.

It improves the separation of concerns between the base module
and the shutdown module.

fix(network-legacy): check if dhclient has --timeout option

dhclient's --timeout option is not upstream, so a pre-check is needed before
using it.

fix(man): correct typo

feat(dmsquash-live): add new dmsquash-live-autooverlay module

ci: revert to actions/checkout@v1 in manualtest

This enables checkout to work in the Gentoo container.

See: https://github.com/actions/runner/issues/2115

docs: fix a typo in HACKING.md

fix(dracut.spec): tpm2-tools is required for crypt module to work

tpm2-tools dependency can be can be pulled in case tmp2-tss package
is installed, using a soft conditional dependency.

https://bugzilla.redhat.com/show_bug.cgi?id=2077697

fix(man): dracut.cmdline.7: clarify "rd.nvmf.discover=fc,auto"

fix(network): avoid double brackets around IPv6 address

This code would add brackets twice. Fix it.

feat(nvmf): set rd.neednet=1 if tcp records encountered

This is currently always the case for NBFT records.
We can do this now, as we run before parse-ip-options.sh

fix(man): dracut.cmdline(7): correct syntax for rd.nonvmf

rd.nonvmf=0 is a double negation, and wrong. Fix it.

fix(network): don't use same ifname multiple times

If multiple ibft interfaces exist, set_ifname() may reuse the same
name (usually "ibft0"), because it's usually called cmdline time,
when no interfaces exist in sysfs yet. Remember the number used
set and don't use it again.

fix(nvmf): run cmdline hook before parse-ip-opts.sh

This way we can set "rd.neednet" and have it seen by parse-ip-options.sh

fix(nvmf): avoid calling "exit" in a cmdline hook

"exit" should never be executed in dracut hooks, because the
hooks are sourced by the main script.

fix(nvmf): make sure "rd.nvmf.discover=fc,auto" takes precedence

The command line may contain several rd.nvmf.discover options.
The "fc,auto" option should take precedence.

fix(nvmf): don't use "finished" queue for autoconnect

The "finished" initqueue is for testing if everything is alright,
not for triggering any actions.

fix(nvmf): don't create did-setup file

did-setup files are meant to indicate that an interface setup
was successful. Don't do it here.

fix(nvmf): no need to load the nvme module

The module "nvme" is not required for NVMeoF.

fix(nvmf): don't try to validate network connections in cmdline hook

The cmdline hook runs before any network interfaces have been brought
up. There's no point in trying to validate the connections at this
stage.

fix(nvmf): nvme list-subsys prints the address using commas as separator

nvme-cli 1.x printed the address using spaces as separator, but nvme-cli 2.x
prints the address using commas as separator (exact output from sysfs). E.g.,
output from `cat /sys/class/nvme/nvme0/address`:

traddr=nn-0x201700a09890f5bf:pn-0x201900a09890f5bf,host_traddr=nn-0x200000109b579ef5:pn-0x100000109b579ef5

Also, I suppress rd.nvmf.discover= cmdline option if all fields are empty.

ci: login and push containers as repository_owner

test(SYSTEMD): make the man command succeed

Document the reason of the existing workaround.

Move the existing logic to the initramfs.testing generation phase
and make the steps more generic.

This change makes the test pass on debian.

test: do not bring up the network interface if it is not needed

Notice that TEST-04 does not explicitelly includes dhclient
and dhclient-script for the rootfs.

Apply the same principle for all local tests that do not
need networking.

This should make the tests run faster and slightly more reliable.

This change also brings us to a better place where network-legacy
files are only gets included via proper dracut module inclusion.

fix(dracut-init.sh): add missing hostonly code in the inst_multiple function

It seems that commit https://github.com/dracutdevs/dracut/commit/4ef45f13 forgot
to add the code to handle the hostonly parameter in the inst_multiple function.

refactor(network-manager): use more precise global variables

refactor(network-legacy): use more precise global variables

refactor(systemd-timesyncd): use more precise global variables

refactor(systemd-networkd): use more precise global variables

fix(network-manager): correct wrong local network configuration path

fix(network-legacy): correct wrong local network configuration path

fix(dracut.sh): correct wrong systemd variable paths

systemdsystemconfdir usually expands to /etc/systemd/system, but the local
configuration of systemd-networkd, systemd-timesyncd, portablectl and systemd
users is saved into directories whose parent is /etc/systemd
(systemdutilconfdir).

Also add missing exports.

test(DMSQUASH): add test for overlay on top of a block device

Add a test for mounting root filesystem with overlay for /dev/sdb.

The existing test was testing mounting root filesystem with overlay
from a squashfs image file.

test(NFS): remove a workaround for systemd v230

Remove some more dead code.
Improve style and formatting.
Remove duplicated modules.

refactor(base): remove `str_replace` from `dracut-lib.sh`

Remove duplicate implementation of `str_replace` to follow-up
commit 148e420be5b5809aa8d5033f47477573bbbf3e60

ci: add Gentoo Linux container

The Gentoo container is configured to use openrc and musl to
test in those enviroments as well.

chore(udev-rules): remove obsolete code

Arch linux no longer needs a special load-modules.sh file.

chore(base): do not output root entry while installing base

Silence the printing out root entry of passwd file
while including base module.

dracut: *** Including module: base ***
root:x:0:0:root:/root:/bin/bash
dracut: *** Including module: fs-lib ***

ci: improve CI for containers

Login to GitHub Container Registry as the actor instead
of the repository owner.

Upgrade to login-action@v2 and build-push-action@v3 Github Actions.

fix(squash): build ld cache for squash loader

Commit dc21638c3f0a fixes kdump kernel crash, due to non-conventional
library path in powerpc, by copying /etc/ld.so.cache under initdir.
While that works in general, it is vulnerable to failure because of
missing links, when the CPU is reconfigured to run in compatibility
mode of older CPU version. Avoid this by running ldconfig for squash
loader to create the necessary links & ld cache for it. Doing this
also saves a few kilobyes of space as the copied ld cache is bigger
in size than the one needed for squash loader environment.

Fixes: dc21638c3f0a ("fix(squash): keep ld cache under initdir")
Cc: Pingfan Liu <piliu@redhat.com>
Signed-off-by: Hari Bathini <hbathini@linux.ibm.com>