TEST-64-UDEV-STORAGE: Fix drive ID

test: Only add tpm2.target dependency to TEST-70-TPM2.service

Let's not add the dependency unless required.

Merge pull request #32766 from DaanDeMeyer/test

Various test improvements

test: Run TEST-64-UDEV-STORAGE with mkosi

We add a configure script for each subtest to add the required
qemu arguments.

Co-authored-by: Richard Maw <richard.maw@codethink.co.uk>

mkosi: Mask mdmonitor and isc-dhcp-server services

They always fail and do not ship with proper units so we can't
disable them with presets so we mask them instead.

TEST-64-UDEV-STORAGE: Skip LVM subtests on Ubuntu

These fail for unknown reasons on Ubuntu. Until someone from Ubuntu
who cares can figure out why, let's skip these tests on Ubuntu.

TEST-64-UDEV-STORAGE: Bump timeout for testcase_simultaneous_events_1

The timeout is reliably hit when running tests in mkosi.

TEST-64-UDEV-STORAGE: Skip btrfs_basic if btrfs module is not available

TEST-64-UDEV-STORAGE: Check for tgt and tgtd services

On Debian/Ubuntu, the unit is named tgt.service instead of tgtd.service,
so let's make sure we take that into account.

On CentOS, tgtd.service is not available, so let's skip the test if we
can't find the service.

TEST-64-UDEV-STORAGE: Add missing udevadm settle

Otherwise we try to do the check before the udev queue is empty which
will make it fail.

TEST-64-UDEV-STORAGE: Use virtio-scsi-pci instead of ahci

The debian cloud kernel does not support ahci, so let's use
virtio-scsi-pci everywhere instead.

TEST-64-UDEV-STORAGE: Use ID based paths

test: Ignore configure scripts in minimal images

These are always aimed at the system image so ignore any specified
configure scripts in the minimal images.

test: Add cmdline field to configure extra kernel command line args

Configuring extra cmdline arguments as a list is much nicer than
having to pass the --kernel-command-line-extra argument manually.

test: Generate basic testsuite services with meson

Most of these are identical, let's just generate from a meson
template.

TEST-46-HOMED: Simplify service unit

Let's remove the unneeded NotifyAccess=all and start the socket
and service in the test itself instead of via the service unit. This
makes the test unit identical to the other test units which will allow
us to autogenerate it in a later commit.

TEST-06-SELINUX: Simplify auto-relabeling

Let's ship a .autorelabel file so we can get rid of
firstboot-autorelabel.service.

test: Rename mkosi_args to mkosi-args

test: Disable tests via 'enabled' field

This allows doing other stuff with the test without actually defining
the test, such as generating a service unit automatically.

test: Simplify argument/variable names

test: Rename testsuite-XX units to match test name

Having these named differently than the test itself mostly creates
unecessary confusion and makes writing logic against the tests harder
so let's rename the testsuite-xx units and scripts to just use the
test name itself.

test: Rework integration test definitions

Let's make this behave more like all the rest of the meson stuff.
This also is the first step to making it a bit more flexible so we
can define integration tests in different ways as will be seen in
the next commits.

TEST-01-BASIC: Drop logic to run test without install-tests=true

We don't support this for any other tests either so let's drop the
support for running TEST-01-BASIC without installing as well to make
the upcoming commit easier to implement.

mkosi: Update to latest

core: Imply DefaultDependencies=no for credential mounts

Currently, on soft-reboot, /run/credentials/@system is unmounted
because it has DefaultDependencies=yes and as such will have
Conflicts=umount.target and Before=umount.target. Let's make sure
credential mounts survive soft-reboot by implying DefaultDependencies=no
for credential mounts.

sd-event: increase test-event timeout to 120s

The test-event test seems to be taking quite a bit more time than
the other 'simple tests', which usually complete in < 1s. In case
of a slower or loaded machine the default 30s timeout is not enough.

man: fix typo

Follow-up for 7df0297ac5aa784209d4c719c87fdd707a9f34cf.

dlfcn-util: fix typo

Follow-up for 2f6bd11c1a0147f6bd24bce3ed00c3d91c0884e6.

TODO: fix typo

Follow-up for 84bdf71592be032fa4100fb7b0ed0a9345942ece.

bootctl: fix crash when parsing addon without .cmdline section

Follow-up for 706ca67d3074b2a405ee8fe5de307416e4915b9f

update TODO

network: IPv6 Compliance RFC4862: Address Lifetime Expiry (Hosts Only) [v6LC.3.2.2]

RFC 4862 Section 5.5.3, bullet e, sub-bullet 3 applies to existing
addresses, i.e. when address_get() returns success. If the address is
new (i.e. address_get() fails), then we should not be adding 2 hours to
the lifetime_valid_usec. Instead, use the valid_lifetime from the RA's
Prefix Information Option.

This change allows v6LC.3.2.2 to pass. Also verified all v6LC3.2.* tests
pass. This covers all the v6LC tests from Group2: Router Advertisement
Processing and Address Lifetime.

Fixes #32652.

network: IPv6 Compliance: Router Advertisement Processing, Reachable Time [v6LC.2.2.15] (#32792)

Previously, RA option fields were being ignored when the Router Lifetime
value was zero. Remove this logic to be compliant with RFC4861.

Extract from: https://www.ietf.org/rfc/rfc4861.html#section-4.2, p.21,
first paragraph:

    The Router Lifetime applies only to
    the router's usefulness as a default router; it
    does not apply to information contained in other
    message fields or options.

This affected IPv6 Conformance test:
    v6LC.2.2.15: Router Advertisement Processing, Reachable Time.

Fixes #31842.

Co-authored-by: Matt Muggeridge <Matt.Muggeridge@hpe.com>

curl-glue: catch libcurl attempting to change timeout handler when we destroy a curl context on exit

If we destroy both an event loop and a curl contect object at the same
time, then we get into this weird situation where curl wants us to
reconfigure a timout event source right before destruction, which
sd-event will refuse however, since it is already being shutdown.

Hence, catch that and simply don't bother adjusting the timeout, since
we cannot get back from there anyway.

machined: return recognizable error when we try to register the same machine name twice

core/service: remove unnecessary reset of notify_access_override

This already gets reset in service_enter_dead()
and service_start().

varlink: rename vsockCid → vSockCid field

hwdb: ASRock LED Controller classified incorrectly as joystick due to buttons and axis (#32775)

The function `builtin-input_id` incorrectly identifies the ASRock LED Controller
as an input device due to the presence of buttons and axis. To fix this we add
a new rule in `hwdb.d/60-input-id.hwdb`.
The properties have been set to empty instead of `0` because some programs
might check if the value is set at all instead of checking its value, as discussed
in #32773.

The device has no real keys. The devices is controlled by i2c interface and some
settings in UEFI, and it provides an header to connect LED strips and similar devices.
I suppose it's possible that ASRock intended to connect devices with buttons for
controlling LEDs to it, but: (i) the controller itself does not have key, (ii) to my
knowledge no such device exists. So I think we can unset that property as well.
On a sidenote, unsetting those properties does not affect the i2c interface,
OpenRGB still interacts normally with the device.

Fixes #32773.

TEST-38-FREEZER: Relax regex a little

The state might be "freezing-by-parent" as well so let's take that
into account.

Fixes #32746

hwdb: ID_INPUT_XYZ allows an empty string

This is for making CIs for #32775 green.

Merge pull request #32758 from yuwata/journal-ratelimit

journal: modernize journal ratelimiting

build(deps): bump pkg/arch from `b13e94a` to `b578e90`

Bumps pkg/arch from `b13e94a` to `b578e90`.

---
updated-dependencies:
- dependency-name: pkg/arch
  dependency-type: direct:production
...

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

build(deps): bump pkg/debian from `7cc898c` to `824c52f`

Bumps pkg/debian from `7cc898c` to `824c52f`.

---
updated-dependencies:
- dependency-name: pkg/debian
  dependency-type: direct:production
...

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

man: add v257 tag

Even though v256-final is not released yet, let's v257 tag now to make
not PRs for v257 conflict with each other.

journal-rate-limit: replace in-house management of JournalRateLimitGroup with OrderedHashmap

No functional change, just refactoring.

hwdb: updated Librem 11 accelerometer (#32772)

hwdb: update Dere N12 / Juno Tablet 3 accelerometer (#32765)

po: Translated using Weblate (Hindi)

Currently translated at 100.0% (233 of 233 strings)

Co-authored-by: Scrambled 777 <weblate.scrambled777@simplelogin.com>
Translate-URL: https://translate.fedoraproject.org/projects/systemd/main/hi/
Translation: systemd/main

journal-rate-limit: split out journal_ratelimit_group_acquire()

No functional change, just refactoring.

journal-rate-limit: introduce cleanup function for JournalRateLimitGroup

This also makes journal_ratelimit_group_new() use structured initializer.

journal-rate-limit: rename variables

'r' is usually used for storing return value.

journal-rate-limit: use FOREACH_ELEMENT() and usec_add()

journal-rate-limit: make journal_ratelimit_group_free() return NULL

To make it follow our recent coding style.

journal-rate-limit: add trailing comma in table

test: introduce test cases for journal_ratelimit_test()

tree-wide: use LOG_PRI() and LOG_FAC()

debug-generator: Allow specifying name of unit-dropin credential

A fixed name is too rigid, let's give users the ability to define
custom drop-in names which at the same time also allows defining
multiple dropins per unit.

We use ~ as the separator because:
- ':' is not allowed in credential names
- '=' is used to separate credential from value in mkosi's --credential
  argument.
- '-' is commonly used in filenames
- '@' already has meaning as the unit template specifier which might be
  confusing when adding dropins for template units

Merge pull request #32755 from yuwata/test-network-cleanups

test-network: several cleanups

test-network: simplify and unify waiting loop

Also,
- drop unused fail_assert arguments,
- use wait_links() and check_networkd_log() in wait_activated().

test-network: make link_exists() take multiple arguments

This also improves performance of test_delete_links().

Fix Positivo N14EPE and N15EPE key toggle touchpad and search key

libcrypt-util: fix wrong errno value assignment

Follow-up for 9de324c3c919f20fd49e1d25579f5a66cac0eaa0.

Merge pull request #32747 from YHNdnzj/tmpfiles-ret-gather

tmpfiles: some cleanups/follow-up

Merge pull request #32748 from yuwata/test-network-improvements

test-network: several more improvements

test-network: check existence of kernel bug

This adds checks for the kernel bug caused by
https://github.com/torvalds/linux/commit/3ddc2231c8108302a8229d3c5849ee792a63230d,
it will be fixed by
https://patchwork.kernel.org/project/netdevbpf/patch/20240510072932.2678952-1-edumazet@google.com/

test-network: introduce --no-journal option

This should be useful when the test run as a service, e.g.
running on a mkosi image.

Merge pull request #32738 from yuwata/trivial-cleanups

Trivial follow-ups for recent changes

NEWS: update for latest changes

Merge pull request #32491 from yuwata/journalctl-fix-boot

journalctl: several fixes and cleanups for --boot= option handling

tmpfiles: use RET_GATHER more, add missing assertions

Note that item_do() now aborts on OOM, since it's
pretty pointless to iterate further if memory allocation
doesn't work.

tmpfiles: don't compare errno with negative value

Follow-up for 677430b3c7fcd1b352eb66f19b8746741459b91a

tmpfiles: clean up hardlinks_vulnerable a bit

dangerous_hardlinks() -> hardlinks_protected(),
and the meaning of the function is now in line
with fs.protected_hardlinks value.

Plus, We ship 50-default.conf where the sysctl
is enabled. Mention it in the comment.

Merge pull request #32741 from yuwata/network-dhcp4-route-to-dns

network/dhcp4: fix assignment of routes to DNS or NTP servers

Merge pull request #32739 from yuwata/vmspawn-journal-forward

vmspawn: fix journal forwarding

mkosi: Update centos submodule to latest

Merge pull request #32743 from yuwata/test-network-for-mkosi

test-network: several improvements for running with mkosi

core/unit: use FOREACH_ELEMENT() to add dependencies for journal namespace instances

Follow-up for 06c0f569e94cc7406886079d241f1c1f82711937.

test: wait for network interface reconfigured after updating config

Otherwise, at this stage, the interface may be in e.g. initialized or
pending state, and the drop-in file introduced by the previous command
may not be registered to the state file for the interface.

Fixes #32685.

test-network: do not fail when /etc/protocols does not exist

Also this makes several checks more strict.

test-network: do not fail test_macvlan() with old kernel or ip command

test-network: do not fail if macvlan module is not available

network/dhcp4: do not set gateway if DNS server or friends is in the acquired prefix

Previously, even if a DNS server is in the acquired prefix, the route to the
server might have gateway address.
This makes the prefix route, which is always configured, is also handled
as same as static routes, and do not use any gateway if the prefix route
is the most suitable route to access the destination.
The same change is also applied to route to NTP servers and semi-static
routes.

Fixes a regression introduced by 0ce86f5eeb0921b44a9782260a8c88aafb15ffde.

Fixes #32715.

test: add basic tests for in_addr_prefix_covers_full()

journalctl: make --list-boots support -n/--lines= option

Also mention that -r/--reverse is supported by the command.

journalctl: fail and show error message when no boot ID found

No boot ID in journal should be definitly spurious.
Let's warn about that and exit with failure.

logs-show: use GREEDY_REALLOC_APPEND()

logs-show: fix stored timestamp when advance_older is true

Currently, the parsed timestamp is only used when advance_older is
false. Hence, this does not change any behavior. But, let's fix it anyway.

journalctl: fix support of --boot=ID±offset format

Fixes a regression introduced by e44f06065bf20e8d0e4adacff61350ebd36f299e.

After the offending commit, if a boot ID suffixed with an offset is
specified to --boot=, the boot ID was ignored.
This fixes the issue.

To fix the issue, this merges journal_find_boot_by_id() and
journal_find_boot_by_offset().

logs-show: flush matches before and after finding boots

Otherwise, if several matches already set, then the first seek to head
or tail may move the cursor to an invalid place, hence they provide
wrong ID(s). Also, reading journal after calling these function may
provide unexpected data.

Currently, the caller does not install any matches before calling the
functions, and does not read any journal entry after journal_get_boots()
succeeds or journal_find_boot_by_offset() succeeds with 0. Hence, this
should not change any behavior. Just for safety.

journalctl: fix --boot=0 with --file=- (from stdin)

Follow-up for 592855c3189549fed93b1060b72299910c6ab1d0.

journalctl: split out journal_acquire_boot() from add_boot()

No functional change, just refactoring and prepration for later changes.

vmspawn: fix finding systemd-journal-remote binary

Follow-up for 258d26940fb6bfa1b5b8acf03cc4a37f2657e4ac.

Fixes https://github.com/systemd/systemd/pull/31218#discussion_r1596168449.

vmspawn: trivial cleanups for start_systemd_journal_remote()

- prefix argument for storing result with 'ret_',
- align arguments for strv_new().

core/service: shorten code a bit

Follow-up for 026335e9aab5ccf1fc9477215beadd0acef94073.

time-util: define TIMESPEC_OMIT

Follow-up for 34c3d574742e867ef97e79509e4051a82f1b7d9b.

journalctl: several cleanups for parse_boot_descriptor()

- rename to parse_id_descriptor(), to make it usable for other kind of
  ID later.
- add missing assertions,
- prefix arguments for storing results with 'ret_',
- drop unnecessary 'else'.

logs-show: add missing strempty()

Follow-up for 8e976dc9209853d5d4d2db3016289f2a5ab99fd9.

logs-show: drop uid argument from add_matches_for_user_units()

It is always equivalent to getuid(). Let's call getuid() in the
function instead.

basic/linux: Copy netfilter.h to the source tree

This fixes build with old toolchains prior to Linux < 4.2 which do not
have a definition for NFPROTO_NETDEV.

shell-completion: on zsh, systemd-cat is a precommand

_precommand lets zsh complete other commands and their arguments
e.g. it can complete grep with "systemd-cat gr" and complete grep options
with "systemd-cat grep -"