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>

feat(dracut.sh): populate uefi_cmdline if no other cmdline is given

When creating uefi image in hostonly mode (with using --hostonly-cmdline),
it makes sense to copy current kernel commandline from /proc/cmdline,
in case there are no other options specified.

Usually, the cmdline.d/*.conf file is generated by module rootfs-block
(or other modules), but it might, not handle all cases correctly,
and specifying --kernel-cmdline every time is not much user-friendly.

feat(systemd): install systemd-sysroot-fstab-check

systemd-sysroot-fstab-check is a symlink to systemd-fstab-generator added in
systemd commit https://github.com/systemd/systemd/commit/cd7ad0cb

fix(i18n): do not fail if FONT in /etc/vconsole.conf has the file extension

If the FONT option defined in /etc/vconsole.conf refers to a file with its
extension, not just the file name, dracut should not fail and install it. The
systemd-vconsole-setup service ends up calling setfont, which supports both file
names and files with extensions.

fix(load_fstype): use $1 if $2 is missing

Use parameter expansion default value if the second argument is missing.
The current code mistakenly reuses the null second argument.

fix(dracut-initramfs-restore.sh): initramfs detection not working

The path detection is not working on latest Fedora and some other
distros, and it fails to extract the initramfs. It seems the if
statement is broken by a previous commit, so let's fix it.

Fixes: 3d8e1ad ('fix(dracut-initramfs-restore.sh): add missing default paths')
Signed-off-by: Kairui Song <kasong@tencent.com>

fix(systemd): add missing modprobe@.service

sys-kernel-config.mount needs modprobe@configfs.service since systemd v246.7
(https://github.com/systemd/systemd/commit/42cc2855), so the kernel configfs
fails to mount in the initrd.

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

Introducing the systemd-pstore module.

ci: add workflow for manual test run

This workflow allows it to run any of the 29 tests
on any of the 4 containers.

On commit, currently the CI only runs 24 tests out of the 116
possible combinations of test runs.

As the project grows, the usefulness of this tool will also grow.

docs: correct typo

fix(dracut.sh): remove duplicate "dracut:" string in logger functions

dracut-logger functions already prepend the "dracut:" prefix to the message.

refactor(dracut.sh): use logger functions instead of printf

dracut-logger is sourced in dracut-init, so any printf calls after sourcing
dracut-init should be replaced with logger functions.

fix(man): point man pages to github.com instead of kernel.org

docs: fix path to CONTRIBUTING.md

docs: clarify that rd.break stops before the breakpoint starts

chore: add execute permission to all scripts

This avoids most of the rpmlint non-executable-script warnings.

test: list only the required modules explicitly

rootfs-block module would always include the following
modules: udev-rules base fs-lib

lvm module always includes rootfs-block

This is a follow-up to a1d4041cee5f7b2d8da9192338d5e81315e758ef

fix(shell-completion): add missing -p and --parallel options

fix(nfs,virtiofs): check kernel for builtin fs drivers

Check first for fs support in `/proc/filesystems` before attempting
to load the kernel module.

This is needed in the --no-kernel use case to avoid the error:
modprobe: FATAL: Module overlay not found in directory /lib/modules/<kver>

fix(dracut-initramfs-restore.sh): hide unpack errors

With the current "try and fail" approach, the system log contains errors until
the correct initrd compression is reached.

test: remove dracut.conf.test

Empty file and it is not used. Has not been used for over a decade :-)

fix(skipcpio): ignore broken pipe

If lsinitrd is called from a context in which SIGPIPE is ignored (e.g.,
from a systemd unit with default setting of IgnoreSIGPIPE=), the
following line will result in an error being issued:

bin="$($SKIP "$image" | { read -r -N 6 bin && echo "$bin"; })"

An example error from `kdumpctl start` (which internally just calls
`systemctl start kdump.service`):

kdumpctl[1287]: ERROR: src/skipcpio/skipcpio.c:191:main(): fwrite

A minimal reproducer:

systemd-run -t sh -c '/path/to/skipcpio /path/to/any/file | false'

test: remove unused ci-matrix.txt file

The ci matrix is described in .github/workflows instead.
ci-matrix.txt file is unreferenced.

docs: create CONTRIBUTING.md

Note down some commonly agreed, but not yet documented guidelines
to have a baseline that can be improved later.

fix(systemd-networkd): typo in systemd-networkd.socket local conf path

fix(virtiofs): make shebangs work on split-usr systems

Fixes: c29325a982013a315418cdb2318e90cf77e387b5
Signed-off-by: Mike Gilbert <floppym@gentoo.org>

fix(dracut-install): use stripped kernel module path as hash key

in install_dependent_modules we use &path[kerneldirlen] as the key for inserting,
let's do the same for checking.

otherwise installing kernel module has circular dependency from a custom kernel
module directory will cause infinite recursion and segfault.

$ grep ipmi kbuilt/lib/modules/5.10.121/modules.dep
kernel/drivers/char/ipmi/ipmi_msghandler.ko:
kernel/drivers/char/ipmi/ipmi_devintf.ko: kernel/drivers/char/ipmi/ipmi_msghandler.ko

$ grep ipmi kbuilt/lib/modules/5.10.121/modules.softdep
softdep ipmi_msghandler post: ipmi_devintf

$ ./dracut-install -D /tmp --kerneldir ~/working/kernel/linux-5.10.121/kbuilt/lib/modules/5.10.121 -m ipmi-devintf
Segmentation fault (core dumped)

(gdb) b install_dependent_modules
Breakpoint 1 at 0x7db0: file src/install/dracut-install.c, line 1513.
(gdb) bt
#0  install_dependent_modules (modlist=0x0) at src/install/dracut-install.c:1513
#1  0x000055555555c027 in install_dependent_modules (modlist=modlist@entry=0x555555579e90) at src/install/dracut-install.c:1553
#2  0x000055555555bf1c in install_dependent_modules (modlist=0x5555555799d0) at src/install/dracut-install.c:1548
#3  0x000055555555c034 in install_dependent_modules (modlist=modlist@entry=0x55555557a3f0) at src/install/dracut-install.c:1554
#4  0x000055555555bf1c in install_dependent_modules (modlist=0x555555579d60) at src/install/dracut-install.c:1548
#5  0x000055555555c034 in install_dependent_modules (modlist=modlist@entry=0x55555557b170) at src/install/dracut-install.c:1554
#6  0x000055555555bf1c in install_dependent_modules (modlist=0x55555557a0f0) at src/install/dracut-install.c:1548
#7  0x000055555555c034 in install_dependent_modules (modlist=modlist@entry=0x555555575320) at src/install/dracut-install.c:1554
#8  0x000055555555bf1c in install_dependent_modules (modlist=0x55555557ab30) at src/install/dracut-install.c:1548
#9  0x000055555555c034 in install_dependent_modules (modlist=modlist@entry=0x55555557dd60) at src/install/dracut-install.c:1554
#10 0x000055555555bf1c in install_dependent_modules (modlist=0x55555557b640) at src/install/dracut-install.c:1548
#11 0x000055555555c034 in install_dependent_modules (modlist=modlist@entry=0x55555557e0f0) at src/install/dracut-install.c:1554
#12 0x000055555555bf1c in install_dependent_modules (modlist=0x55555557b9d0) at src/install/dracut-install.c:1548
#13 0x000055555555c034 in install_dependent_modules (modlist=modlist@entry=0x555555574340) at src/install/dracut-install.c:1554
#14 0x000055555555bf1c in install_dependent_modules (modlist=0x55555557cf70) at src/install/dracut-install.c:1548
#15 0x000055555555c034 in install_dependent_modules (modlist=modlist@entry=0x5555555768d0) at src/install/dracut-install.c:1554
#16 0x000055555555bf1c in install_dependent_modules (modlist=0x55555557d750) at src/install/dracut-install.c:1548
#17 0x000055555555c034 in install_dependent_modules (modlist=modlist@entry=0x55555557e700) at src/install/dracut-install.c:1554
#18 0x000055555555bf1c in install_dependent_modules (modlist=0x55555557de90) at src/install/dracut-install.c:1548
#19 0x000055555555c034 in install_dependent_modules (modlist=modlist@entry=0x555555581c90) at src/install/dracut-install.c:1554
#20 0x000055555555bf1c in install_dependent_modules (modlist=0x555555571e60) at src/install/dracut-install.c:1548
#21 0x000055555555c034 in install_dependent_modules (modlist=modlist@entry=0x55555556b620) at src/install/dracut-install.c:1554
#22 0x000055555555bf1c in install_dependent_modules (modlist=0x555555583000) at src/install/dracut-install.c:1548
#23 0x000055555555c034 in install_dependent_modules (modlist=modlist@entry=0x55555556b640) at src/install/dracut-install.c:1554
#24 0x000055555555bf1c in install_dependent_modules (modlist=0x555555571b40) at src/install/dracut-install.c:1548
#25 0x000055555555c034 in install_dependent_modules (modlist=modlist@entry=0x555555574100) at src/install/dracut-install.c:1554
#26 0x000055555555c4b0 in install_module (mod=mod@entry=0x555555573bc0) at src/install/dracut-install.c:1617
#27 0x000055555555c93d in install_modules (argc=argc@entry=1, argv=argv@entry=0x7fffffffd6e0) at src/install/dracut-install.c:1952
#28 0x000055555555862a in main (argc=<optimized out>, argv=0x7fffffffd6a8) at src/install/dracut-install.c:2090

Signed-off-by: runsisi <runsisi@hust.edu.cn>

fix(virtiofs): ismounted has a dependency on the base module

When dracut.sh is called with "--modules virtiofs", make sure
dracut-lib.sh is installed by making the base module a dependency.

fix(drm): add video drivers needed on hyper-v and similar

Due to non-availability of Hyper-V video driver hyperv_drm in kdump
initramfs, the console seems to be in hang state with no text over it.

We should also go through the /sys/bus/vmbus/devices and include drivers
referenced there.

fix(kernel-modules): always include nvmem driver on nvmem_on_arm

These drivers are needed to boot on some SoCs like NXP i.MX
We should include them so installation images will work.

refactor(convertfs): drop duplicate def. of ismounted()

fix(dmsquash-live): check kernel for built-in squashfs drivers

Check first for squashfs in `/proc/filesystems`.
This is needed in the --no-kernel use case to avoid the error:
modprobe: FATAL: Module overlay not found in directory /lib/modules/<kver>

Introduce load_fstype function to make it easier to check `/proc/filesystems`
before calling modprobe.

ci: dependencies for testing nfs module for Debian

This change allows executing TEST-20-NFS on Debian.

fix(test): remove unnecessary setup steps

var/lib/nfs/rpc_pipefs is only needed for NFS mounts.

test(16-DMSQUASH): make it pass on arch

Include additional kernel drivers and install squashfs-tools
into the arch container.

docs(README.md): update wiki link

Point to the github (and remove kernel.org). Add link to News.
Remove broken badge links.

fix(90multipath): use RemainAfterExit=yes for multipathd-configure.service

Services of type `oneshot` should almost always use
`RemainAfterExit=yes`. This is a one-time action that we don't want to
rerun if the unit gets pulled in multiple times in the systemd
transaction.

fix(90multipath): create `/etc/multipath` only

The dropins path is `/etc/multipath/conf.d`, not
`/etc/multipath/multipath.conf.d`:

https://github.com/opensvc/multipath-tools/blob/686797b7d0da4fc24fdfcb0fc6364df78a0b232d/Makefile.inc#L100

But anyway, we don't need to create that directory. We only need to
create `/etc/multipath`.

While we're here, add a comment to explain this.

fix(systemd): set right permissions for the machine-id file

Fixes issue #1864

fix(dracut-install): do not try to copy files from the root directory

HWCAP library path needs at least two parent directories, for example:

/lib64/power6x/libc.so.6
/lib64/glibc-hwcaps/power9/libpthread-2.28.so (glibc 2.33+)

Before:

dracut-install: Lib install: '/lib64/libz.so.1'
dracut-install: dracut_install('/lib64/libz.so', '/lib64/libz.so', 0, 0, 1)
dracut-install: dracut_install('//libz.so.1', '//libz.so.1', 0, 0, 1)
dracut-install: dracut_install('//libz.so', '//libz.so', 0, 0, 1)

After:

dracut-install: Lib install: '/lib64/libz.so.1'
dracut-install: dracut_install('/lib64/libz.so', '/lib64/libz.so', 0, 0, 1)

refactor(dracut-install): do not call dirname() directly

Avoid its horrible API. Borrow dirname_malloc() from systemd:

https://github.com/systemd/systemd/blob/v251/src/basic/path-util.c#L775
https://github.com/systemd/systemd/blob/v251/src/basic/path-util.h#L150

(licence is LGPLv2.1 or later)

fix(dracut.sh): do not fail on irregular files

If file is not a regular file (test -f), dracut.sh fails,
which is unexpected change of behaviour.
The workaround would be to create an empty file.

Fixes: #1835

feat(dracut.sh): pass engine flag to sbsign allowing use with hardware devices

fix(dmsquash-live): run checkisomd5 on correct device

When the new grub2 iso is written to a usb drive the disk label points
to a partition that does not include the full iso image. This causes
checkisomd5 to run with the wrong data and it fails.

This patch adds a check that will test to see if there is a parent
device that is a disk, and to run checkisomd5 on it instead of on the
partition pointed to by the label.

When running from an iso this will return the original
/dev/disk/by-label/ path, and when running from a usb drive it will
return the parent device (eg. /dev/sda).

Resolves: rhbz#2107858

fix(dracut-init.sh): correct dracut-install source path

fix(lsinitrd.sh): correct skipcpio source path

fix(dracut-init.sh): propagate the result code returned by dracut-install

Currently only the inst_multiple function propagates the result code returned by
dracut-install.
This was missing since 043 (e.g.: https://github.com/dracutdevs/dracut/blob/043/dracut-functions.sh#L751)

fix(dmsquash-live-root): check kernel for built-in `overlay` drivers

Do not assume that `overlay` is always a module.  Check first for
`overlay` in `/proc/filesystems`. This is needed in the --no-kernel
use case to avoid the error: modprobe:
  FATAL: Module overlay not found in directory /lib/modules/<kver>

Signed-off-by: Federico Vaga <federico.vaga@cern.ch>

ci: add glibc to the Arch Linux container

tgt has a dependency on glibc

refactor(url-lib): write curl output directly to file

The redirection to stdout was originally added to workaround
https://bugzilla.redhat.com/show_bug.cgi?id=989133 (curl would create no
file if the source was empty), but I don't see that problem with current
curl.

A nice side-effect of this change is that curl shows download progress.

fix(i18n): add required includes for keymaps

Commit https://github.com/dracutdevs/dracut/commit/f3441cc7 removed shell
options but didn't adapt all the code that requires nullglob and globstar.
This caused the initrd to be generated without the required includes for
keymaps.

fix(man): correct typo

Fixes issue #1852