ci(basic): correct syntax to exclude several tests

Make sure tests 20 and 26 are excluded as intended by fixing
GitHub Action syntax.

Follow-up to 1798677 .

ci: skip running failing tesst due to 2021

Temporary skip running tests until
https://github.com/dracut-ng/dracut-ng/issues/2021
is resolved.

Follow-up to 1798677 .

ci: simplify GitHub Action job naming

network-manager dracut module is the default networking provider.

Remove forcing network-manager networking and simplify naming.

ci: skip running failing tesst due to 2021

Temporary skip running tests until
https://github.com/dracut-ng/dracut-ng/issues/2021
is resolved.

ci: skip running tests that are failing due to 1988

Temporary skip running tests until
https://github.com/dracut-ng/dracut-ng/issues/1988
is resolved.

ci: run network-systemd tests on arm as well

Add arm to the test matrix for network-systemd.

ci: move mkosi testing into daily testsuite

mkosi is not default/native in any of the test containers and it
does not seem to be stable.

This PR moves running mkosi reference test into
daily testing so that the dedicated GitHub Action can be just
disabled without a PR.

ci(openSUSE): container build fix

ruby3.4-rubygem-asciidoctor is now called ruby4.0-rubygem-asciidoctor.

ci: run network-legacy tests on arm as well

Add arm to the test matrix for network-legacy.

ci: remove leftover and duplicated GitHub Action

daily-systemd-arm64 GitHub Action is no longer needed as it is
covered by daily-systemd.

Follow-up to a43bd7a .

ci: fix GitHub Action syntax

excluding architectures requires {runner: 'ubuntu-24.04', tag: 'amd'} .

ci: combine omitsystemd x64 and arm64 into one GitHub action

Running tests on arm64 is just as stable as x64, there is
no need to separate them. Combine them into one GitHub
action should make it easier to reason about regressions.

ci: combine systemd x64 and arm64 into one GitHub action

Running tests on arm64 is just as stable as x64, there is
no need to separate them. Combine them into one GitHub
action should make it easier to reason about regressions.

ci: combine hostonlystrict x64 and arm64 into one GitHub action

Running tests on arm64 is just as stable as x64, there is
no need to separate them. Combine them into one GitHub
action should make it easier to reason about regressions.

ci: combine network-manager x64 and arm64 into one GitHub action

Running tests on arm64 is just as stable as x64, there is
no need to separate them. Combine them into one GitHub
action should make it easier to reason about regressions.

ci: combine basic x64 and arm64 into one GitHub action

Running tests on arm64 is just as stable as x64, there is
no need to separate them. Combine them into one GitHub
action should make it easier to reason about regressions.

ci(busybox): make test 80 compatible with busybox

BusyBox ln does not support the -r argument. This argument
can be removed from the test suite without losing test coverage.

Now that the test runs with busybox, add it to the busybox daily
testsuite.

ci(busybox): make test 30 compatible with busybox

busybox dd understands bs=1M but it does not understands bs=1MiB.

Now that the test runs with busybox, add it to the busybox daily
testsuite.

ci: run TEST_CONTAINER_COMMAND before configure runs

Move running TEST_CONTAINER_COMMAND during CI tests
earlier so that packages can be installed/uninstalled
before configure runs.

ci: daily busybox tests

Add a GitHub action to run and pass as many of the
existing tests as possible without installing coreutils.

ci: modularize GitHub Actions for daily tests

Move each job into its own GA yaml file so that each job
can be individually enabled/disabled via the GitHub web interface.

This change prevents triggering all test runs every time
the monolithic yaml file is modified.

test(KERNEL-INSTALL): layout and uki_generator set must be set on Gentoo

Set layout, initrd_generator, and uki_generator for kernel-install.

Fixes https://github.com/dracut-ng/dracut-ng/issues/2018 .

ci(alpine-busybox): grep is no longer required

busybox grep is supports all the required functionality and
command line arguments to pass dracut's basic test.

Installing GNU grep on top of busybox is no longer required.

Follow-up to 2de5129 .

ci: verify marker images consistently

Use grep -F -a to check marker images after test runs.

This affects test evaluation only and does not modify dracut
core logic.

ci: add a test run for basic tests using busybox

This test prevents regressions when running basic tests on
Alpine with busybox.

fix(dracut): use proper dracut logging when possible

The recent restructuring of the code allows using proper dracut logging
in more areas of the code.

Follow-up to 8552a0f .

ci(alpine-busybox): alpine container with busybox

The coreutils binaries are not installed and only busybox
binaries are available.

This container is meant to prevent regressions when running
basic test with busybox.

refactor(dracut-install): simplify boolean evaluation

Checking a boolean with `== false` can be simplified to use the `not`
operator.

test: use syntax compatible with busybox implementation

busybox du only support `-s` for summary and `-b` for block-size.

Follow-up to d4af288f0 .

ci: drop alpine:latest and run tests only on alpine:edge

alpine:edge has been very stable CI environment for this project,
so let's promote it to the main CI and at the same time
let's retire alpine:latest.

The goal of this PR is to reduce time and computation on the CI
without loosing significant test coverage.

feat: always preserve file ownership

Currently file ownership is almost always preserved, except when
DRACUT_CP variable is used.

Make all file copying consistent and always preserve file ownership
by passing `-p` to `cp`.

Follow-up to 9ef73b6ad .

fix(dracut): ensure that inst is called after DRACUT_NO_XATTR is computed

`inst` calls dracut-install which requires DRACUT_NO_XATTR.

Move code block around to make sure DRACUT_NO_XATTR is set before
`inst` is called.

Follow-up to 941b18d .

feat(no-xattr): no longer support no-xattr config

Starting with dracut release v110, no-xattr is the default
config, so it is no longer desired to maintain a separate config
file for no-xattr.

Follow-up to 941b18d .

test(SYSTEMD): run with --no-hostonly-cmdline

This test aims to run without the `initqueue` module, but the `base` module
depends on it if `$hostonly_cmdline == "yes"` [1]. Although the test ends
successfully, the log shows some errors because `dracut-lib.sh` is not present
if the `base` module cannot be installed:

```
dracut[E]: Module 'base' depends on module 'initqueue', which can't be installed
...
/usr/bin/dracut-cmdline-ask: line 3: /lib/dracut-lib.sh: No such file or directory
[    2.108191] dracut-cmdline[254]: /usr/bin/dracut-cmdline: line 6: /lib/dracut-lib.sh: No such file or directory
[    2.129548] dracut-pre-udev[259]: /usr/bin/dracut-pre-udev: line 6: /lib/dracut-lib.sh: No such file or directory
...
[    3.176822] dracut-pre-pivot[368]: /usr/bin/dracut-pre-pivot: line 7: /lib/dracut-lib.sh: No such file or directory
...
All OK
```

[1] https://github.com/dracut-ng/dracut-ng/blob/bf29f4b79aee9771800dffe78857014970be117e/modules.d/80base/module-setup.sh#L8-L12

chore(Makefile): use cp -a consistently

`cp -a` is used in all call sites except one.
Update the outlier to use the -a flag for consistency.

Follow-up to a0d12aa7d .

test(NETWORK): do not set net.ifnames=0

Increase test coverage by letting CI test containers to decide
the value of net.ifnames.

Change the test expectation to handle both net.ifnames=0
and net.ifnames=1.

feat(dracut): drop unreachable code for dracutbasedir/initdir

`dracut.sh` sets the variables `dracutbasedir` and `initdir`. Then it
creates the `initdir` directory. So checking those variables again can
be dropped.

Follow-up: 4cb4917e1222 ("refactor: fold dracut-init.sh into dracut.sh")

chore(deps): bump luizm/action-sh-checker from 0.9.0 to 0.10.0

Bumps [luizm/action-sh-checker](https://github.com/luizm/action-sh-checker) from 0.9.0 to 0.10.0.
- [Release notes](https://github.com/luizm/action-sh-checker/releases)
- [Commits](luizm/action-sh-checker@v0.9.0...v0.10.0)

Removed the `-s` argument (stands for `--simplify`) as this
argument now overrules .editorconfig and moved setting simplify
to .editorconfig.

test: drop --auto=yes from mdadm call

mdadm 4.5 emits following warning:

```
mdadm: --auto is deprecated and will be removed in future releases.
```

So drop `--auto=yes` from the `mdadm --create` calls.

fix(dracut): move comment back to related dracut_args variable

Commit 58dad7025b01 added the comment "store for logging" next to the
`dracut_args` variable. Move that comment back to this place.

refactor(dracut-functions): put functions first

For better readability put all the functions first in
`dracut-functions.sh` and put the executed code last.

feat(dracut): disable xattr when creating cpios

Disable xattr when creating cpio, but do not change
the default for squashfs/erofs.

Move setting the default for DRACUT_CP later in the code to
account for the newly set defaults.

Fixes: https://github.com/dracut-ng/dracut-ng/issues/1561

fix(dracut): source dracut-functions.sh before calling dwarning

`dracut-functions.sh` sources `dracut-logger.sh` which defines the
`dwarning` function. `dracut.sh` might call `dwarning` before
`dracut-functions.sh` is sourced.

`dracut-functions.sh` calls `find_binary` and `dlog_init` when sourced.
`find_binary` needs `dracutsysrootdir` set and `dlog_init` needs
`stdloglvl`, `sysloglvl`, `kmsgloglvl`, `maxloglvl`, `fileloglvl`,
`logfile`. Move sourcing `dracut-functions.sh` as early as possible, but
after setting those variables.

Follow-up for 8d9887b
Fixes: https://bugs.debian.org/1124479

chore: move net udev rules to net-lib dracut modules

The net-lib dracut module is designed to be always installed when
networking is enabled in the generated initramfs.

Move networking related udev rules to net-lib dracut modules from the
udev-rules dracut module.

The motivation behind this change is to increase modularity,
stay consistent with other dracut modules and potentially
increase performance when networking is not required in the
initramfs.

ci(alpine): install openRC and build-base

alpine-base package is required to make the
container bootable and have the openRC services
available.

The alpine-base package is not installed by default in the alpine
container (only the alpine-baselayout package is installed).

Instead of explicitly installing make, install build-base meta-package
for future proofing.

feat(dracut): print $uefi with --printconfig

Allow the enviroment (e.g. package managers) to discover
if uefi is enabled or not.

Similar to 71a713bb2 .

ci: remove stalebot config file

There is no plan for this project to maintain the a stalebot
(or introduce any other stalebot alternative).

fix(dm): add explicit rootfs-block dependency

Device mapper operates on local storage typically
configured by the rootfs-block module.

This change tightens dependencies to ensure that including only
the dm module via `dracut -m dm` produces a functional initramfs.

chore: remove support for 64-device-mapper.rules

64-device-mapper.rules use to be a Gentoo specific file,
which is now obsolete even on Gentoo.

Reverts 5d7298.

chore: reduce the scope of shellcheck exceptions

The motivation behind this commit is to be able easily
identify unused functions in dracut.sh going forward.

feat(dracut): add support for --ukify, --no-ukify

Introduce a dracut option to enable/disable using ukify to
create UKI even if ukify is installed.

Fixes: https://github.com/dracut-ng/dracut-ng/issues/1592

fix: grammatical errors

chore: move inst_opt_decompress to i18n dracut module

`inst_opt_decompress`, `inst_decompress` and get_decompress_cmd
functions are only used in the i18n dracut module.

These functions were never documented as public functions
for dracut modules.

Fixes: https://github.com/dracut-ng/dracut-ng/issues/1966

refactor: move defining require_X functions to dracut-functions.sh

require_X are public interfaces to dracut modules, move them
to `dracut-functions.sh`.

Add documentation for these public functions to make it clear that
it is expected to be used in out-of-tree dracut modules as well.

ci(alpine): fix regression regarding make is not installed

It turns out make it not a dependency for dracut-tests downstream.
Add make dependency to continue upstream development.

Follow-up to 52606ca .

ci(alpine): simplify CI container after Alpine v3.23 release

The upstream dracut-tests package in Alpine v3.23 already installs
dracut build dependencies, no need to repeat them here.

Simplify the package list of the dracut CI.

Alpine v3.23 stable has been released on 2025-12-03.

Follow-up to 353373e99b37ef30e9fc218a3766d379e6c1af5b .

chore(dasd): combine two inst_rules calls into one

Instead of sequentially calling inst_rules twice, combine
the two function calls into one.

ci: remove iscsi test for network-legacy runs

Further reduce CI coverage for iscsi tests to remove non-determinism
from the CI.

Follow-up to 04db824.

ci: extend run-qemu function to pass test 82 on Void arm64 as well

Extend the rules to find the kernel inside the Void container.
Void on arm64 uses the vmlinux filename pattern.

ci: add omitsystemd and hostonlystrict tests to the ARM CI

Run omitsystemd and hostonlystrict tests on Daily arm64
CI the same way as they run on amd64.

feat: remove rev_lib_symlinks

rev_lib_symlinks is not used any more by dracut and it is not
a dependency for any of the out-of-tree dracut modules.

`rev_lib_symlinks` function was introduced by a10a141 to be used by
`inst_library`, however `inst_library` no longer uses it
(neither any other function).

It is hard to maintain code that is not documented,
not used and has no test coverage.

fix(nfs): simplify DRACUT_CP arguments to increase portability

DRACUT_CP defaults to preserving mode and timestamps already,
which allows to just use `-p` argument here.

`busybox` `cp` does not support `--preserve=`, but it does support
`-p` argument.

Follow-up to 2f5a759.

refactor: move defining inst_libdir_X and instmods to dracut-functions.sh

inst_libdir_dir, inst_libdir_file and instmods are public interfaces to
dracut modules, move them to `dracut-functions.sh`.

These public functions are all already documented in the man page
and they do not have a dependency on other non-yet-public functions.

test: drop root=LABEL=dracut from TEST_KERNEL_CMDLINE

Several tests (for example the NFS or iSCSI tests) set the kernel
cmdline `root=` and therefore overwrite `root=LABEL=dracut` set by
`TEST_KERNEL_CMDLINE`.

Shorten the kernel cmdline and avoid confusion, drop `root=LABEL=dracut`
and specify it only in tests that do not set `root=`.

ci: expose /dev/kvm on arm64 as well

Similar to amd64 expose `/dev/kvm` to the arm64 container for faster
QEMU execution.

As of 2025-12-19 the arm64 runners do not provide nested virtualization.
Nested virtualization is possible on AArch64 since Linux 6.16, so it
should be possible to offer even without bare metal instances.

Similar to amd64 expose `/dev/kvm` to the arm64 container for faster
QEMU execution once the runners provide nested virtualization.

perf(nfs): do not source /lib/nfs-lib.sh twice

Follow-up for bf29f4b79aee9771800dffe78857014970be117e

refactor: move defining inst_rules to dracut-functions.sh

inst_rules is a public interface to dracut modules, move it
to `dracut-functions.sh`.

To make `dracut-functions.sh` useable on its own, move all the dependent
functions to `dracut-functions.sh` but make internal by prepending
their names with _.

ci: re-enable running iscsi tests parallel

iscsi tests have been stabilized (based on the CI runs
in the last few days), let's try to open them back
up and re-enable running iscsi tests parallel.

Reverts 9b63e4e .

fix(nfs): do not execute logic in nfs hooks if netroot is not nfs

If the nfs module is present in the initrd, it always perform some operations
(mount/umount rpc_pipefs, exec/kill rpcbind, rpc.statd and rpc.idmapd), even in
initrds not specifically created for nfsroot (e.g., in non-hostonly mode).

This can create a conflict with some VMware tools that use the rpc_pipefs
filesystem and can run in the initrd.

chore(deps): bump actions/upload-artifact from 5 to 6

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 5 to 6.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](https://github.com/actions/upload-artifact/compare/v5...v6)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

test(sysroot): run with --no-hostonly-cmdline

TEST-13-SYSROOT fails in a Ubuntu VM. This is the `lsblk -f` output in
the VM:

```
 NAME    FSTYPE FSVER LABEL           UUID                                 FSAVAIL FSUSE% MOUNTPOINTS
 fd0
 sr0
 vda
 ├─vda1  ext4   1.0   cloudimg-rootfs 606e50c2-c444-4c6c-86a1-a6e16ff87015     19G    12% /
 ├─vda13 ext4   1.0   BOOT            30a0e670-dd1c-4d4f-8489-02af8f2f9a18  844.4M     8% /boot
 ├─vda14
 └─vda15 vfat   FAT32 UEFI            5150-4E06                              98.1M     6% /boot/efi
```

TEST-13-SYSROOT calls dracut with `--hostonly` which leads to
`hostonly_cmdline` default to `yes`. This leads to the code path for
"save host_devs which we need bring up" in
`modules.d/80base/module-setup.sh` being executed. In this VM case the
variables are:

```
host_devs: /dev/vda1 /dev/vda13 /dev/vda15
user_devs:
root_devs: /dev/vda1
```

The code in `modules.d/80base/module-setup.sh` creates systemd units for
`/dev/vda13` and `/dev/vda15` which can be seen in the initrd file:

```
lrwxrwxrwx   1 root     root           78 Nov 24 13:35 etc/systemd/system/initrd.target.wants/dev-disk-by\x2duuid-30a0e670\x2ddd1c\x2d4d4f\x2d8489\x2d02af8f2f9a18.device -> ../dev-disk-by\x2duuid-30a0e670\x2ddd1c\x2d4d4f\x2d8489\x2d02af8f2f9a18.device
lrwxrwxrwx   1 root     root           42 Nov 24 13:35 etc/systemd/system/initrd.target.wants/dev-disk-by\x2duuid-5150\x2d4E06.device -> ../dev-disk-by\x2duuid-5150\x2d4E06.device
```

These two units hang forever in TEST-13-SYSROOT.

So run TEST-13-SYSROOT with `--no-hostonly-cmdline`.

Fixes: https://github.com/dracut-ng/dracut-ng/issues/1871

ci: split install dependencies step from test run step

To ease looking at the test results, split the step that installs the
dependencies step from the step that runs the actual test (in this case
`make syncheck`).

feat(dracut-systemd): preserve the content of /var/lib/systemd

Persist the content of /var/lib/systemd in memory during initramfs
execution by linking it to /run/initramfs/systemd .

One of the motivation of this PR is to preserved coredumps
when systemd-coredump dracut module is included in the initrfamfs.

In general, this approach would requires less customization of
systemd configuration for dracut.

docs: functions starting with an underscore are private

Document that functions starting with an underscore are private
functions and must not be called directly.

chore: ignore dracut-cpio build artifacts

ci: fix YAML indentation

yamllint complains about wrong indentation.

chore: order functions as they are documented in dracut.modules

Improve maintanability by ordering the code and corresponding
documentation the same way.

Follow-up to 883eba5.

feat: remove inst_any

inst_any does not seem to be used. It is hard to maintain code
that is not documented, not used and has no test coverage.

chore: remove default argument to cp

Adding argument processing for arguments that are the default
anyways just adds to cognitive load of reading the code and
a little bit of performance overhead.

--sparse=auto is the default. There is no need to re-specify it.

fix(systemd): only install systemd-coredump with systemd-coredump dracut module

systemd-coredump is not meant to be installed "by default" and expected
to be an opt-in binary, which is achieved by adding systemd-coredump
dracut module.

refactor: move defining DRACUT_INSTALL to dracut-functions.sh

Functions in `dracut-functions.sh` call `DRACUT_INSTALL`. To make
`dracut-functions.sh` useable on its own, move the code to define
`DRACUT_INSTALL` from `dracut.sh` there.

feat(dmsquash-live): allow booting from compressed ostree filesystems

When combined with the ostree module it is possible to boot from a
compressed ostree filesystem, but the root of this has /ostree not /usr
so it fails.

Related: HMS-8933

ci: limit iscsi testing to Debian/Ubuntu on Daily testsuite

Based on CI test tun observations iscsi tests seems to be more flaky
on Arch, Fedora and openSUSE. Remove these flaky test runs from the CI
and keep Debian/Ubuntu, which seems to be more stable.

Follow-up to 79867b5.

ci(alpine): simplify CI container after Alpine v3.23 release

The upstream dracut-tests package in Alpine v3.23 already installs
networkmanager-cli and networkmanager-initrd-generator.

Simplify the package list of the dracut CI.

Alpine v3.23 stable has been released on 2025-12-03.

ci: limit iscsi testing to Debian/Ubuntu

Based on CI test tun observations iscsi tests seems to be more flaky
on Arch and openSUSE. Remove these flaky test runs from the CI and
keep Debian/Ubuntu, which seems to be more stable.

ci: simplify syntax ot exclude test 72 on ubuntu:devel

Instead of including and than later excluding test 72 on ubuntu:devel
let's just not include it to increase readbility.

ci: run iscsi tests sequentially

Limit the concurrency to a single job at a time within the matrix strategy
to run iscsi tests sequentially.

ci: run iscsi tests with network-manager

iscsi tests seems to be more reliable with network-manager.

ci: separate out iscsi and nbd tests into its own GA

iscsi tests are known to be flaky, let's make sure we run
them as a separate GA to be able to manage flakiness better.

chore: upgrade shellcheck

Switch CI to a newer version of Ubuntu to get a newer version of
shellcheck.

fix(Makefile): improve support for shfmt v3.8.0+

Add the --apply-ignore flag to respect ignore rules even when processing
specific files by name or via stdin.

Fixes: https://github.com/dracut-ng/dracut-ng/issues/1866

fix(dracut): ignore shellcheck SC2329 in addition to SC2317

Shellcheck <= 0.10 complains about SC2317 (info): Command appears to be
unreachable. Check usage (or ignore if invoked indirectly). Shellcheck
0.11 changes this complaint to SC2329 (info): This function is never
invoked. Check usage (or ignored if invoked indirectly).

So ignore shellcheck SC2329 in addition to SC2317 to make both
shellcheck versions happy.

fix(dracut): ignore failing find_binary dracut-install

The call `find_binary dracut-install` might fail. Ignore this failure,
because a proper error message will be printed later in the code.

test(run-qemu): support RISC-V

Use the `virt` machine on RISC-V. This machine does not have a PCI bus.
So use `virtio-rng-device` as RNG device there.

feat(dracut): always check DRACUT_INSTALL being executable

If `DRACUT_INSTALL` is set externally (to `valgrind dracut-install` for
example), dracut will skip the check for the dracut-install command.

Better check the first parameter in `DRACUT_INSTALL` being executable
(e.g. check `valgrind` being present when `DRACUT_INSTALL` is set to
`valgrind dracut-install`).

feat: source dracut-logger.sh from dracut-functions.sh

Some functions in `dracut-functions.sh` use logger functions defined in
`dracut-logger.sh`, for example `inst_hook` might call `dfatal`.

So move the inclusion of `dracut-logger.sh` into `dracut-functions.sh`
to make `dracut-functions.sh` useable on its own.

Move including `dracut-functions.sh` further up in `dracut.sh` because
the logger function `dfatal` is needed afterwards.

fix(dracut): remove newlines from libdirs variable

The `libdirs` variable might contain newlines:

```
$ dracut --printconfig
[...]
dracutconfig: libdirs= /lib64 /usr/lib64 /lib /usr/lib /usr/lib/x86_64-linux-gnu
/usr/lib/x86_64-linux-gnu/libfakeroot
```

This is confusing. Replace those newlines by spaces:

```
$ dracut --printconfig
[...]
dracutconfig: libdirs=/lib64 /usr/lib64 /lib /usr/lib /usr/lib/x86_64-linux-gnu /usr/lib/x86_64-linux-gnu/libfakeroot
```

docs(test): document numbering of tests

Document the numbering of the tests. See commit 5bb613dc902f
("test: reorganize tests").

fix(dracut): --printconfig does not work without --force

```
$ dracut --printconfig
dracut[F]: Will not override existing initramfs (/boot/initrd-6.17.0-2-default) without --force
```

Follow-up for 8e24c4bf06850da1bc89022081b7e793548d9b0b

feat(dracut): print $initrdname with --printconfig

kiwi will need this to properly get the name of the initrd output file.

See https://github.com/OSInside/kiwi/issues/2918