kernel-install: add helper for logging

import-ceds: use the right error variables at four places

update TODO

update TODO

growfs: don't actually resize on dry-run

This causes systemd-growfs to exit before resizing the partition when
`--dry-run` is passed. Resizing during a dry run of a change breaks the
users expectations.

update TODO

Merge pull request #24002 from yuwata/network-hw-addr

network: fix hardware address handling

update TODO

sd-bus: do not pass NULL when received message with invalid type

Fixes #24003.

sd-dhcp6-client: fix off-by-one error in parsing dhcp6 options

This fixes error in parsing message when the rapid commit option is
located at the end of the message.

Fixes an issure reported in #24002.

sd-dhcp6-client: allow NULL option value when length is zero

sd-dhcp6-client: add more debugging logs on parsing message

Merge pull request #23882 from dtardon/logind-set-display-test

Add tests for org.freedesktop.login1.Session SetType and SetDisplay

network: do not set invalid MAC address for non-ethernet interface

network: NDisc does not require MAC address

This effectively revert ba4c7184b320bb8698d470530d46a6c94641cc6e.

Fixes #23546.

network: dhcp4: disable DHCPv4 client on interfaces with non-supported types

Replaces f42d41cc5f9cd8cac538a1c30fda04d346b5bae2.

Revert "network: configure DHCP clients after MAC address is assigned"

This reverts commit f42d41cc5f9cd8cac538a1c30fda04d346b5bae2.

DHCPv6 client does not require MAC address.
DHCPv4 client will be handled in a different way in a later commit.

Partially fixes #23546.

Do not fail EFI build with newer binutils

Newer binutils versions currently trigger the following warnings due to
a bug in gnu-efi

on arm64:
/usr/bin/ld.bfd: warning: src/boot/efi/systemd-bootaa64.elf has a LOAD segment with RWX permissions

on amd64:
/usr/bin/ld.bfd: warning: /usr/lib/crt0-efi-x86_64.o: missing .note.GNU-stack section implies executable stack

This results in a build failure due to --fatal-warnings.
Work around this issue by suppressing those warnings until gnu-efi has
been fixed.

See https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1013341

man: lift pam_systemd_homed description to Summary

Also change the title to describe the module more comprehensively.
Follow-up for 90bc309aa2c1430941f4c50f73e681ab3e488bd3. Suggested
in https://bugzilla.redhat.com/show_bug.cgi?id=2085485#c5.

test: add test for org.freedesktop.login1.Session SetDisplay

test: add test for org.freedesktop.login1.Session SetType

test: ensure cleanup functions return success

Otherwise the return value of the last command is propagated, which may
cause spurious test failures. E.g., pkill returns 1 if no process
matched, which may be a problem in cleanup session:

cleanup_session() {
...

    pkill -u "$(id -u logind-test-user)"
    sleep 1
    pkill -KILL -u "$(id -u logind-test-user)"
}

If there are no remaining processes when the final pkill runs, it will
return 1 and therefore cleanup_session will return 1 as well.

Add coredump daemons to bug/RFE template component options

Merge pull request #23817 from yuwata/sd-device-send-udev-database-version

sd-device: send udev database version

ci: limit which env variables we pass through `sudo`

to work around #23987.

test-network: save and restore timezone

Several DHCP client tests change the system timezone.
Let's save the current timezone at the beginning, and restore it with
the saved value at the end.

Merge pull request #23881 from keszybz/kernel-install-strikes-yet-again

kernel-install: fix invocation as installkernel, add tests, tweak documentation

test-network: wait for L2TP tunnels being removed

Fixes #23985.

logind: don't delay login for root even if systemd-user-sessions.service is not activated yet

If for any reason something goes wrong during the boot process (most likely due
to a network issue), system admins should be allowed to log in to the system to
debug the problem. However due to the login session barrier enforced by
systemd-user-sessions.service for all users, logins for root will be delayed
until a (dbus) timeout expires. Beside being confusing, it's not a nice user
experience to wait for an indefinite period of time (no message is shown) this
and also suggests that something went wrong in the background.

The reason of this delay is due to the fact that all units involved in the
creation of a user session are ordered after systemd-user-sessions.service,
which is subject to network issues. If root needs to log in at that time,
logind is requested to create a new session (via pam_systemd), which ultimately
ends up waiting for systemd-user-session.service to be activated. This has the
bad side effect to block login for root until the dbus call done by pam_systemd
times out and the PAM stack proceeds anyways.

To solve this problem, this patch orders the session scope units and the user
instances only after systemd-user-sessions.service for unprivileged users only.

smack: Add DefaultSmackProcessLabel to user.conf and system.conf

DefaultSmackProcessLabel tells systemd what label to assign to its child
process in case SmackProcessLabel is not set in the service file. By
default, when DefaultSmackProcessLabel is not set child processes inherit
label from systemd.

If DefaultSmackProcessLabel is set to "/" (which is an invalid character
for a SMACK label) the DEFAULT_SMACK_PROCESS_LABEL set during compilation
is ignored and systemd act as if the option was unset.

Merge pull request #23982 from medhefgo/boot-misc

boot: Misc cleanups

Merge pull request #23979 from DaanDeMeyer/nspawn-relative-paths

nspawn: Support relative paths for --bind and --overlay

nspawn: Support relative source paths for --bind and --overlay

bcd: Clean up includes

Also, now that bcd.h does not depend on efi.h anymore we can now
properly include it in the test instead.

bcd: Use std alignof

boot: Remove _cleanup_freepool_

boot: Use int instead of INTN

boot: Use uintptr_t when converting EFI_PHYSICAL_ADDRESS

uintptr_t is the more appropriate type when casting to/from pointers.

kernel-install: suppress bogus shellcheck hint

I don't quite understand this, but '{ ! true; }' is not the same as '( ! true )'.
In interactive mode, it seems to work as expected. But in a script, it doesn't.

kernel-install: use set -e

This means that we'll fail hard if something goes wrong, e.g. reading
of a config file. I think this is appropriate. If errors should be ignored,
the caller should do that on their end.

kernel-install: return 0 for unknown verbs in plugins

In practice this makes little difference, because kernel-install will
only call the plugins for 'add' or 'remove', and if we were to add a
new verb to kernel-install, we'd just change the plugins at the same
time. But our plugins serve as documentation for external plugins too,
and there it's better to silently ignore unknown verbs so that we can
add new verbs in the future.

(50-depomod.install was already like that.)

kernel-install: raise fuss if plugins are called without the expected parameters

test-kernel-install: add a simple test that kernel-install copies the files

I opted to tweaking kernel-install to allow overriding config
(with $KERNEL_INSTALL_CONF_ROOT, $KERNEL_INSTALL_PLUGINS). An alternative
would be to build a test environment in test/. We can still do that,
but I think it's nice to have a simple test that is very quick and easy
to debug.

Invocation as installkernel is for #23681.

kernel-install: mark the plugins as executable in git

We mark them as executable during installation, but it's also nice to mark
them as such in sources, so they can be executed directly.

kernel-install: allow overriding the plugin list too

The use of IFS=<newline> is dropped. Let's just iterate over the list, using
any whitespace as separator.

Merge pull request #23980 from yuwata/test-network

test-network: tiny output improvements

Merge pull request #23972 from yuwata/sd-device

sd-device: do not return -ENOENT by sd_device_get_is_initialized()

Fix automatic screen rotation for Asus Transformer T100TAM

Merge pull request #23974 from yuwata/align-tables

basic: align tables

test-network: support to remove routing policy rules with l3mdev flag

test-network: merge stdout and stderr of invoked command by call()

Otherwise, the logs may be messed up.

man: explain why pam_systemd_home wants to be in all four stacks

Suggested here:

https://bugzilla.redhat.com/show_bug.cgi?id=2085485#c5

Merge pull request #23969 from yuwata/test-network

test-network: fixlets and performance improvements

nspawn: Rewrap --overlay docs

nspawn: Drop unnecessary intermediate variable

sd-netlink: add sizes for some IP and Ethernet addresses

Add sizes for some uses of NETLINK_TYPE_IN_ADDR and NETLINK_TYPE_ETHER_ADDR
types.

test-network: various cleanups

- introduce several helper functions
- do not list unit files, but remove the runtime unit directory in
  tearDown().
- do not list used interfaces, but remove all interfaces previously not
  exists in tearDown().
- save routes and routing policy rules before running tests, and flush
  unnecessary routes and rules in each tearDown() calls.
- drop many time.sleep() calls.
- call tearDown() after each sub tests.
- shorten code.
- several coding style fixes.
- etc, etc...

Hopefully, this improves performance of the test.

update TODO

test-network: extend timeout for DHCP lease to be expired

Fixes #23964.

sd-device: shorten code a bit

tree-wide: trivial tweaks

update TODO

mkosi: update to latest main

Fix build failure on SUSE Tumbleweed due to config changes

unit-def: align string tables

virt: align tables

sd-device: make sd_device_get_is_initialized() not return -ENOENT

Merge pull request #23962 from keszybz/taint-flag-support-ended

Taint flag "support-ended"

manager: add taint flag "support-ended"

In the welcome line, use NAME= as the fallback for PRETTY_NAME=.
PRETTY_NAME= doesn't have to be set, but NAME= should.

Example output:
---
Welcome to Fedora Linux 37 (Rawhide Prerelease)!

[  !!  ] This OS version (Fedora Linux 37 (Rawhide Prerelease)) is past its end-of-support date (1999-01-01)
Queued start job for default target graphical.target.
[  OK  ] Created slice system-getty.slice.
---

man: update the description of taint flags

We had a description in README, and an outdated list in the man page.
I think we should keep a reference-style list in the man page. The description
in README is more free-form.

man: redefine SUPPORT_END= to mean one day earlier

I thought it would be nice to specify the last day of support, because I
thought it'd seem more natural. But in practice this doesn't work well, because
such a truncated timestamp is usually taken to mean midnight that starts the
given date. I.e. 2011-12-13 is a shorthand for 2011-12-13 00:00:00 and not
2011-12-13 23:59:59.999999999999. Let's instead specify that the given date is
the first unsupported day, which is meaningful for humans, and let the computer
treat it as midnight, which gives consistent interpratation.

resolve: introduce FORMAT_DNS_RCODE() macro

Fixes #23958.

Merge pull request #23959 from yuwata/test-network

test-network: several performance improvements

test-network: use wait_operstate() at one more place

test-network: suppress periodic output in wait_operstate()

test-network: disable debugging logs from networkctl, resolvectl, and so on

test-network: merge two DHCPv6 client tests

test-network: replace sleep with wait_address() and wait_address_dropped()

test-network: merge two more tests with test_dhcp_client_ipv4_only

test-network: merge tests for UseDNS=

test-network: merge two IPv4LL tests

test-network: drop test_dhcp_client_ipv4_ipv6

As the test is covered by test_dhcp_client_reuse_address_as_static.

sd-dhcp-client: fix log message

test-network: check DHCPv4 address in more detail

Also, this drops redundant address check through networkctl.

test-network: drop 25-dhcp-v4-server-veth-peer.network

test-network: drop meaningless test

On DHCPv6 lease renew, the kernel do not drop static routes, and hence
we do not try to reassign them. Hence, the test is mostly meaningless.

test-network: introduce read_dnsmasq_log_file() helper function

This reduces the number of times that the log file is read.
Also, dropped unnecessary dump of the log file.

test-network: shorten sleep time a bit

Network interfaces will be ready after lease is acquired.
Hence, it is not necessary to wait more than lease time.

test-network: introduce read_link_state_file() helper function

network: drop redundant warning

If file is world readable, then `read_full_file_full()` will warn
about that.

Merge pull request #23940 from yuwata/core-extract-first-word

core: fix error value in log_syntax()

Merge pull request #23731 from bluca/bootctl_image

bootctl: add --root and --image

Merge pull request #23954 from yuwata/resolve-overflow

resolve: fix heap-buffer-overflow

sd-device: send udev database version

Otherwise, sd-device object received through sd-device-monitor does not
show current tags.

Fixes #23799.

sd-device: make device_get_properties_{nulstr,strv}() take NULL for result value

In most cases, it is not necessary to call them without retrieving
result. But, most of other getter functions for sd-device can take NULL.
Let's follow the way for consistency.

sd-device: change type of properties nulstr from uint8_t* to char*

core/cgroup: drop unnecessary else

core: shorten code a bit

core/load-fragment: fix error value in log_syntax()

`extract_first_word()` may return positive value on success.

test: start test user session before idle action setting is changed

Otherwise, idle action may be triggered before starting the test user
session.

This also introduce create_session() and cleanup_session() helper
functions.

Fixes #23952.

Update TODO