Merge pull request #33842 from DaanDeMeyer/test

Two fixes

resize-fs: Put minimal ext4 size in the same ballpark as the other filesystems

TEST-46-HOMED fails on ext4 because the filesystem is deemed to small
for activation by cryptsetup. Let's bump the minimal filesystem size for
ext4 a bit to be in the same ballpark as ext4 and btrfs to avoid weird
errors due to impossibly small filesystems.

Also use U64_MB while we're touching this.

Merge pull request #33825 from DaanDeMeyer/chattr

repart: Create disk image file with copy-on-write disabled on btrfs

shell completion: add support for sd-analyze capability --mask

Follow-up for 3e7a029c2856e7814b930443cc2d4fb089377592

Merge pull request #33840 from bluca/test_locale_dbus

test: fix D-Bus policy override for TEST-73-LOCALE

TEST-54-CREDS: Specify SMBIOS creds via corresponding mkosi option

This allows mkosi to combine fstab.extra with its own fstab.extra so
that it doesn't override the one we pass for the test.

sd-event: do not assert on invalid signal

The signalfd_siginfo struct is received from outside via a FD, hence
assert() is not appropriate way to check it. Just do a normal runtime
check.

logind-dbus: check auth. for all inhibitor operations

Fixes #33834

docs/CONTROL_GROUP_INTERFACE.md: document accounting information available via D-Bus

Merge pull request #33727 from intelfx/work/analyze-capability-masks

analyze: capability: add support for decoding capability masks

basic/log: do not treat all negative errnos as synthetic

Currently, IS_SYNTHETIC_ERRNO() evaluates to true for all negative errnos,
because of the two's-complement negative value representation.
Subsequently, ERRNO= is not logged for most of our own code.
Let's fix this, by formatting all synthetic errnos as positive.
Then, treat all negative values as non-synthetic.

While at it, mark the evaluation order explicitly, and remove
unneeded comment.

Fixes #33800

tree-wide: Don't explicity disable copy-on-write when copying images

Since the copy helpers now copy file attributes as well, let's not
explicitly disable copy-on-write anymore when we copy an image. If
the source already has copy-on-write disabled, the copy will have it
disabled as well. Otherwise, the copy will also have copy-on-write
enabled.

This makes sure that reflinks always work as reflink is only supported
if both source and target are copy-on-write or both source and target
are not copy-on-write.

copy: Copy file attributes as well

Let's make sure we copy all file attributes that can be copied as
well.

mkosi: update debian commit reference

* 7eebe8c0b1 autopkgtest: allow localectl in localed tests
* c08a88ffbb Update changelog for 256.4-1 release
* 03814c87fe Fix D-Bus policy for locale1 blocking
* 16f6130038 Drop last patch, all merged upstream
* fe6956e934 Install varlinkctl zsh completion file
* 9bc2a52832 Update upstream source from tag 'upstream/256.4'
* 8574241978 New upstream version 256.4

test: fix D-Bus policy override for TEST-73-LOCALE

We don't need to allow non-root, and the policy needs to specify destination
and interface too, to narrow it down

Follow-up for 7b5c38a91def6cf236605010a0a93a1cd4c137e9

Make read_attr_path() more generic

Let's make this an openat() like function so it can be used in more
scenarios.

repart: Create disk image file with copy-on-write disabled on btrfs

COW on btrfs generally does not play well lots of random writes so
let's make the disk images generated by repart NOCOW by default on
btrfs like we do elsewhere across the codebase.

repart: Make partition files NOCOW if the disk image is NOCOW

On btrfs, reflinks into a disk image that has copy-on-write disabled
only work if the source has copy-on-write disabled as well so let's
make sure that's the case if the disk image has copy-on-write disabled.

fs-util: Add XO_NOCOW flag

Let's add a flag for xopenat() that immediately makes a file NOCOW
after opening it if it's supported.

fs-util: Clean up properly in xopenat_full() on labelling error

If we fail to relabel the file, we should unlink the file or directory
again, so let's make sure we do that.

fs-util: Handle dangling symlinks in openat_report_new()

openat() will always resolve symlinks, except if O_NOFOLLOW is passed
or O_CREAT|O_EXCL is passed. This means that if a dangling symlink is
passed to openat_report_new(), the first call to openat() will always
fail with ENOENT and the second call to openat() will always fail with
EEXIST.

Let's catch this case explicitly and fallback to creating the file with
just O_CREAT and assume we're the ones that created the file. We can't
resolve the symlink with chase() because this function is itself called
by chase() so we could end up in weird recursive calls if we'd try to do
so.

test-fs-util: Modernize openat_report_new() test

test-id128: Use new assertion macros

test: Add ASSERT_EQ_ID128() and ASSERT_NE_ID128()

test: Add ASSERT_FAIL()

mkosi: Update to latest

Includes the required fix to make mkosi copy NOCOW disk images properly.

analyze: capability: add support for decoding capability masks

This adds support in `systemd-analyze capability` for decoding
capability masks (sets), e.g.:

```console
$ systemd-analyze capability --mask 0000000000003c00
NAME                 NUMBER
cap_net_bind_service     10
cap_net_broadcast        11
cap_net_admin            12
cap_net_raw              13
```

This is intended as a convenience tool for pretty-printing capability
values as found in e.g. `/proc/$PID/status`.

mkosi: update debian commit reference

* c7138e0b87 Configure default DNS servers for upstream CI builds
* bc5d1afe1e Drop out-of-tree localed patch and use D-Bus policy instead
* b5f8ababde autopkgtest: set Release= in mkosi.local.conf to distinguish testing vs unstable
* 323afafd80 autopkgtest: add allow-stderr to timedated test
* 0291f361e3 Install valrinkctl zsh completion file
* f40b9eba02 d/t/control: add Depends: lib{systemd,udev}-dev for upstream
* 3def595de3 d/t/upstream: ensure correct ubuntu codename is used
* 531bb6817e d/t/boot-and-services: fix a couple python sytax warnings
* 963ac13b7d d/t/boot-and-services: skip test_tmp_cleanup if tmp.mount is overridden

hwdb: add axis range corrections for the Lenovo Thinkpad E16

Merge pull request #33823 from YHNdnzj/varlink-deserialize-again

core: reliably check if varlink socket has been deserialized; switch varlink server to pidref

add udev rules for trezor hw wallet devices

resolved: report svc params as a json object

This representation is significantly more useful. The previous array
format omitted the svc param key, which is not very useful.

Merge pull request #33810 from YHNdnzj/find-executable-simplify

path-util: minor cleanup for find_executable()

Merge pull request #33809 from YHNdnzj/pidref-namespace

namespace-util: introduce pidref_namespace_open() and use it where appropriate; clean up mount-util a bit along the way

Merge pull request #33535 from neighbourhoodie/tests/dns-cache

Tests for DnsCache functions

Merge pull request #33534 from neighbourhoodie/tests/dns-query

Tests for DnsQuery functions

core-varlink: switch to PidRef + manager_get_unit_by_pidref()

core-varlink: do not log about ENOENT if oomd isn't available

This is simply too noisy, since every invocation of
manager_varlink_send_managed_oom_update() would try to
connect to oomd if not already.

core-varlink: add missing runtime_scope check for manager_varlink_init_user()

core: reliably check if varlink socket has been deserialized

Follow-up for 6906c028e83b77b35eaaf87b27d0fe5c6e1984b7

The mentioned commit uses access() to check if varlink socket
already exists in the filesystem, but that isn't sufficient.

> Varlink sockets are not serialized until v252, so upgrading from
> v251 or older means we will not listen anymore on the varlink sockets.
>
> See https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1074789
> for more details as this was found when updating from Debian Bullseye to a new version.

After this commit, the set up of varlink_server is effectively
split into two steps. manager_varlink_init_system(), which is
called after deserialization, would no longer skip listening
even if Manager.varlink_server is in place, but actually
check if we're listening on desired sockets.
Then, manager_deserialize() can be switched back to using
manager_setup_varlink_server().

Alternative to #33817

Co-authored-by: Luca Boccassi <bluca@debian.org>

mount-util: reorder params for mount_in_userspace, clean up logging

mount-util: do not unnecessarily acquire mountns fd twice

This also enables us to use pidref_namespace_open().

mount-util: clean up mount_exchange_graceful a bit, don't duplicate move_mount when fallback

machine-dbus: use pidref_namespace_open() where appropriate

namespace-util: introduce pidref_namespace_open()

namespace-util: explicitly specify namespace_info[]'s size

We do this already in all string lookup tables. This way
it's guaranteed that iterators which ends with _NAMESPACE_TYPE_MAX
wouldn't overrun the array.

path-util: simplify final path for find_executable()

Currently, systemd-run ./test.sh would set Description=
to $PWD/./test.sh. This is quite ugly.

path-util: trivial cleanup for find_executable_full()

core: clean up Set/LoadCredential= parsers

Make logging consistent, plus introduce helper function
for adding creds to ExecContext.set_credential too.

mkosi: Bump device timeout even more

I still manage to hit it in some cases so let's bump again.

Merge pull request #33533 from neighbourhoodie/tests/dns-synthesize

resolved: tests for dns_synthesize_family() and dns_synthesize_protocol()

resolved: tests for dns_synthesize_answer()

sd-dhcp-server: persist hardware addresses in leases

resolved: tests for dns_query_string()

resolved: tests for dns_query_go(); multiple search domains for dns_query_candidate_notify()

resolved: refactor environment management in dns_query_go() tests

resolved: tests for dns_query_go() when configured with search domains

resolved: tests for dns_query_go() when no scopes are available

resolved: tests for dns_query_go() using a bypass query

resolved: tests for dns_query_go() -- with and without network link

resolved: tests for dns_query_process_cname_one(); no match, DNAME, utf-8 handling

resolved: tests for dns_query_process_cname_many()

resolved: tests for common usage of dns_query_process_cname_one()

resolved: tests for dns_query_make_auxiliary()

resolved: tests for dns_query_new()

resolved: tests for dns_cache_lookup() clamping the TTL

resolved: tests for dns_cache_put() for  NXDOMAIN with no SOA

resolved: test that pseudo classes and types are not cached

resolves: tests for dns_cache_prune()

resolved: tests for dns_cache_check_conflicts()

resolved: tests for dns_cache_export_shared_to_packet()

resolved: tests for dns_cache_lookup(); mDNS and multiple matching entries

resolved: tests for dns_cache_lookup() returning the most recent input

resolved: tests for dns_cache_dump()

resolved: test cache misses

resolved: tests for dns_cache_lookup() for NXDOMAIN

resolved: first test for dns_cache_lookup()

resolved: tests for dns_cache_dump_to_json()

resolved: check that adding an expired response removes cache entry

resolved: tests for dns_cache_put(); CNAME success and name error

resolved: tests for dns_cache_put() with non-matching class, type, name

resolved: refactor DNS answer construction for cache tests

resolved: test for dns_cache_put() with empty answer

resolved: tests for dns_cache_put() with different RCODEs

resolved: tests for dns_cache_put(); successful A query

Co-Authored-By: jan@neighbourhood.ie

Merge pull request #33531 from neighbourhoodie/tests/dns-zone

resolved: tests for dns_zone_lookup()

resolved: tests for dns_zone_remove_rrs_by_key()

resolved: tests for dns_zone_remove_rr()

resolved: tests for dns_zone_put()

Merge pull request #33530 from neighbourhoodie/tests/dns-search-domain

resolved: tests for dns_search_domain_find()

resolved: tests for search domain marking and unlinking

resolved: extract function for checking search domains

resolved: tests for dns_search_domain_unlink()

resolved: tests for dns_search_domain_new()

Merge pull request #33529 from neighbourhoodie/tests/dns-link

Tests for DNS Link functions

Merge pull request #33528 from neighbourhoodie/tests/dns-answer

resolved: tests for dns_answer_order_by_scope()