Merge pull request #2742 from lucab/ups/rust-regenerate-2022.6

rust: regenerate bindings after latest release (2022.6)

Merge pull request #2745 from lucab/ups/ci-bump-rust-linting

ci/rust: bump linting toolchain to latest stable (1.64)

Merge pull request #2746 from lucab/ups/lib-repo-refs-error

lib/repo-refs: properly return an error value

lib/repo-refs: properly return an error value

This tweaks a logic check in order to return a proper error value
when the related condition is not satisfied.

Merge pull request #2740 from lucab/ups/lib-sign-ed25519-assertions

lib/sign-ed25519: convert invariant checks to assertions

ci/rust: bump linting toolchain to latest stable (1.64)

This bumps the Rust toolchain for clippy/rustfmt to 1.64.

rust: update manual helpers

ostree: manually patch generated files

This manually adds a missing `ToGlibPtr` import, which seems to be result
of some bugs in `gir` code-generation.

rust: regenerate bindings after latest release (2022.6)

lib/sign-ed25519: convert invariant checks to assertions

This converts several invariant checks to asserts. Most of the
functions in this file were already using assertions, so this aligns
the remaining few outliers to the rest.

Merge pull request #2744 from lucab/ups/ci-memory-bump

ci: bump cosaPod memory requirement to 5Gi

ci: bump cosaPod memory requirement to 5Gi

We're hitting memory limits when running the reprovisioning tests. We
should investigate why we need 1Gi of overhead, but for now to get
unblocked let's just bump it.

Merge pull request #2663 from mstuehn/test-static-delta-file

lib/static-delta: document and check parameters format

rust/tests: fix static delta generation testcase

lib/static-delta: document and check parameters format

This enhances the logic handling GVariant parameters within
`ostree_repo_static_delta_generate()`.
Several of those entries are expected to be zero-terminated values,
and this implicit assumption has been observed to be an hidden trap
in languages where strings and arrays may not carry a terminator value
(e.g. Rust).
In order to improve the situation, this makes the documentation more
explicit and actively tries to catch invalid input parameters.

Add test for generating static delta to file

Merge pull request #2733 from ostreedev/dependabot/submodules/libglnx-e701578

build(deps): bump libglnx from `26375b5` to `e701578`

Merge pull request #2738 from lucab/ups/lib-zipl-check-open-errors

ci: Use new names for custom kola steps

The old `fcos*` versions are deprecated now:
https://github.com/coreos/coreos-ci-lib/pull/111

ci: Drop unnecessary outer stage

We define stages inside of this stage, so get rid of the outer `FCOS`
stage. This should help rendering the pipeline in BlueOcean more
clearly.

ci: Bump memory request, use fcosKolaTestIso

More fallout from https://github.com/coreos/coreos-ci-lib/pull/116. We
need to be more honest about how much memory we need.

While we're here, switch the `kola testiso` invocation to use
`fcosKolaTestIso` instead. This will add coverage for UEFI and UEFI
SecureBoot testing.

Merge pull request #2737 from lucab/ups/repo-checkout-fix-uid-typo

lib/repo-checkout: fix typo in error message

lib/bootloader-zipl: check for errors when opening initrd

This adds a missing error check when opening an initrd in s390x
Secure Execution (SE) logic.
It was detected by RH internal static checks.

lib/repo-checkout: fix typo in error message

This fixes a typo in an error message, resulting in a GID vs UID
mixup. It was detected by RH internal static checks.

build(deps): bump libglnx from `26375b5` to `e701578`

Bumps libglnx from `26375b5` to `e701578`.

---
updated-dependencies:
- dependency-name: libglnx
  dependency-type: direct:production
...

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

Merge pull request #2730 from ostreedev/release-2022.6

Release 2022.6

configure: post-release version bump

Release 2022.6

Merge pull request #2721 from cgwalters/link-otto

README.md: Link otto

Merge pull request #2717 from mangelajo/support-whiteouts

Support overlayfs whiteouts on checkout

README.md: Link otto

Another project in the ostree/container space.

Support overlayfs whiteouts on checkout

Introduces an intermediate format for overlayfs storage, where
.wh-ostree. prefixed files will be converted into char 0:0
whiteout devices used by overlayfs to mark deletions across layers.

The CI scripts now uses a volume for the scratch directories
previously in /var/tmp otherwise we cannot create whiteout
devices into an overlayfs mounted filesystem.

Related-Issue: #2712

Merge pull request #2720 from lucab/ups/lib-deploy-ignore-result

lib/sysroot-deploy: explicitly handle `g_variant_lookup` results

lib/sysroot-deploy: explicitly handle `g_variant_lookup` results

This explicitly ignores the results of two optional variant lookups,
in order to pacify a RHT internal static analyzer.

Merge pull request #2718 from lucab/ups/repo-initialize-valid-bootloader

lib/repo: properly initialize boolean variable

lib/repo: properly initialize boolean variable

This initializes a boolean variable that was previously left
uninitialized. It was detected by a RHT internal static analyzer.

Merge pull request #2716 from dbnicholson/max-metadata-size-doc

lib/pull: Fix max-metadata-size documentation

lib/pull: Fix max-metadata-size documentation

The documented option is incorrect and has been since it's introduction
in 2c55bc6997.

Merge pull request #2714 from andreamtp/patch-1

Fix recursive git archive reference

Fix recursive git archive reference

Broken link for "recursive git archive" example

Merge pull request #2711 from lucab/ups/sign-assertions

lib/sign: convert invariant checks to assertions

lib/sign: convert invariant checks to assertions

This converts several invariant checks to asserts. Most of the
functions in this file were already using assertions, so this aligns
the remaining few outliers to the rest.

Merge pull request #2709 from lucab/ups/otutils-variant-builder-error

otutil: add error handling to variant builders

otutil: add error handling to variant builders

This enhances a bunch of helpers related to GVariant building, in
order to properly handle errors and avoid some potential cases of
unexpected NULL results.

Merge pull request #2708 from nikita-dubrovskii/sh_to_c

s390x: drop shell code for `initrd` modification in SE case

s390x: use 'libarchive' to modify initrd in SE case

Merge pull request #2710 from lucab/ups/lib-mtree-redundant-validation

lib/mtree: drop redundant name checks

lib/mtree: drop redundant name checks

This drops several NULL checks against filename input argument.
Those checks are both redundant (as filename validation already
checks for that) and dangerous (as they return early without
setting an error value).

Merge pull request #2707 from lucab/ups/libostree-bootloader-asserts

lib/bootloader: assert invariants

s390x: simplify 's390x-se-luks-gencpio' script

Merge pull request #2706 from git-bruh/main

ostree-fetcher-curl: check for HTTP2 support before trying to use it

lib/bootloader: assert invariants

This tweaks some invariants checks into full assertions, in order
to avoid returning to the caller in case of known invalid states.

s390x: ensure both 'root' and 'boot' luks keys exist

ostree-fetcher-curl: check for HTTP2 support before trying to use it

Merge pull request #2704 from cgwalters/finalize-no-sigterm

finalize-staged: Don't listen to `SIGTERM`, just let kernel exit us

finalize-staged: Don't listen to `SIGTERM`, just let kernel exit us

Followup from discussion in
https://github.com/ostreedev/ostree/pull/2544#discussion_r958840936

This is more efficient; no need to have the kernel context switch
us in at shutdown time just so we can turn around and call
`exit()`.

Merge pull request #2544 from dbnicholson/finalize-block

finalize-staged: Ensure /boot and /sysroot automounts don't expire

finalize-staged: Ensure /boot automount doesn't expire

If `/boot` is an automount, then the unit will be stopped as soon as the
automount expires. That's would defeat the purpose of using systemd to
delay finalizing the deployment until shutdown. This is not uncommon as
`systemd-gpt-auto-generator` will create an automount unit for `/boot`
when it's the EFI System Partition and there's no fstab entry.

To ensure that systemd doesn't stop the service early when the `/boot`
automount expires, introduce a new unit that holds `/boot` open until
it's sent `SIGTERM`. This uses a new `--hold` option for
`finalize-staged` that loads but doesn't lock the sysroot. A separate
unit is used since we want the process to remain active throughout the
finalization run in `ExecStop`. That wouldn't work if it was specified
in `ExecStart` in the same unit since it would be killed before the
`ExecStop` action was run.

Fixes: #2543

main: Factor out sysroot loading

It can be useful to parse the options and initialize the sysroot without
actually loading it until later. Factor out the sysroot loading to a new
`ostree_admin_sysroot_load` and add a new
`OSTREE_ADMIN_BUILTIN_FLAG_NO_LOAD` flag to accommodate this.

Merge pull request #2695 from HuijingHei/kargs-stage

Fix `ostree admin kargs edit-in-place` assertion when deployments are pending

Fix `ostree admin kargs edit-in-place` assertion when deployments
are pending

This is to support pending deployments instead of rasing assertion.
For example:
```
$ sudo rpm-ostree kargs --append=foo=bar
$ sudo ostree admin kargs edit-in-place --append-if-missing=foobar
```
After reboot we get both `foo=bar foobar`.

Fix https://github.com/ostreedev/ostree/issues/2679

Merge pull request #2701 from cgwalters/bind-list-commits

 rust: Bind `ostree_repo_list_commits_starting_with`

rust: Bind `ostree_repo_list_commits_starting_with`

I wanted to use this as a fallback for
https://github.com/ostreedev/ostree-rs-ext/pull/351/commits/30dee81c22ad5cb90e77198d3ddbcc25d388afb5

Merge pull request #2702 from thesamesam/bashism

buildutil/glibtests.m4: fix bashism

Merge pull request #2700 from jlebon/pr/staged-docs

docs: Add section about staged deployments

buildutil/glibtests.m4: fix bashism

configure scripts need to be runnable with a POSIX-compliant /bin/sh.

On many (but not all!) systems, /bin/sh is provided by Bash, so errors
like this aren't spotted. Notably Debian defaults to /bin/sh provided
by dash which doesn't tolerate such bashisms as '=='.

This retains compatibility with bash.

Fixes configure warnings/errors like:
```

checking whether to build static libraries... no
./configure: 14795: test: unexpected operator
```

Signed-off-by: Sam James <sam@gentoo.org>

Merge pull request #2698 from cgwalters/update-gir

docs: Add section about staged deployments

I was explaining staged deployments to someone today and was looking for
a doc but we didn't have any. Fix that.

Merge pull request #2699 from cgwalters/drop-debian-testing

ci: Also drop seccomp on debian testing

ci: Also drop seccomp on debian testing

I didn't deep dive into debugging this but I'm pretty sure it's
 https://gitlab.gnome.org/GNOME/glib/-/issues/2580
which is us having an older Docker in the middle here.

rust: Update to latest git

In the future I may try to add CI that requires this to be
sync'd...

Merge pull request #2692 from GeorgesStavracas/gbsneto/transaction-leak

lib/commit: Unref repo on success

Merge pull request #2697 from tkfu/docs/add-related-projects

docs: Add aktualizr and TorizonCore to related projects

docs: Add aktualizr and TorizonCore to related projects

Signed-off-by: Jon Oster <jon.oster@toradex.com>

Merge pull request #2693 from lucab/ups/libostree-fix-annotation

libostree: fix a typo in annotation

libostree: fix a typo in annotation

This fixes a typo in the `allow-none` annotation on
`ostree_sysroot_deployment_set_kargs_in_place` argument.

Merge pull request #2691 from cgwalters/rev-parse-single

cli/rev-parse: Add `--single` option

In the current "ostree native container" flow, we're inserting
a commit object into the repo but with no refs.

We have hacks in a few places to find the commit digest via e.g.
`find repo/objects -name *.commit` but that's a horrible hack.
Add `ostree rev-parse --single` which will print the single commit,
and error out if there is not exactly one commit.

Co-authored-by: Jonathan Lebon <jonathan@jlebon.com>

Merge pull request #2690 from cgwalters/cli-less-goto-1

cli/rev-parse: Port to new code style

lib/commit: Unref repo on success

Commit 540e60c3 introduced _ostree_repo_auto_transaction_new(), a
private constructor to OstreeRepoAutoTransaction, by factoring out
some code from _ostree_repo_auto_transaction_start(). This factored
code increased the refcount of the 'repo' variable.

Subsequent commit 71304e854c made ostree_repo_prepare_transaction()
use ths newly introduced constructor. However, in this function, the
happy path assumed no ref was taken, and therefore did not unref it.
Commit 71304e854c didn't add the corresponding unref either.

This leaks a reference to OstreeRepo when calling
ostree_repo_prepare_transaction().

Plug this leak by using g_clear_object() to clear the repo field
of OstreeRepoAutoTransaction, instead of simply setting it to NULL.

Closes https://github.com/flatpak/flatpak/issues/4928

cli/rev-parse: Port to new code style

Prep for future changes.

Merge pull request #2684 from ostreedev/dependabot/submodules/libglnx-26375b5

build(deps): bump libglnx from `c59eb27` to `26375b5`

Bumps libglnx from `c59eb27` to `26375b5`.

---
updated-dependencies:
- dependency-name: libglnx
  dependency-type: direct:production
...

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

Merge pull request #2689 from jlebon/pr/ficlone-direct-payload

lib/commit: Directly use FICLONE for payload link

tests/kolainst/staged-deploy: parse `rpm-ostree status --json` instead

Don't parse `rpm-ostree status` output, it's not meant for that. Use
`--json` output instead.

While we're here, fix an obsolete reference to Ansible.

Related: https://github.com/coreos/rpm-ostree/pull/3938

lib/commit: Directly use FICLONE for payload link

The idea of payload linking is to reflink between objects where
possible. Instead of relying on `glnx_regfile_copy_bytes` to hit the
`FICLONE` path, just call `FICLONE` directly. At that point in the code,
we've already established that the source and dest repos are on the same
filesystem and that it supports `FICLONE`.

Related: https://gitlab.gnome.org/GNOME/libglnx/-/merge_requests/41
Related: https://github.com/ostreedev/ostree/pull/2684#issuecomment-1204068437

Merge pull request #2688 from LukasKalbertodt/main

Update to `libtest-mimic` 0.5.0

Update to `libtest-mimic` 0.5.0

Merge pull request #2686 from cgwalters/include-cleanups

Handle conflicts between newer glibc and kernel headers

Move FIFREEZE/FITHAW ioctl invocations into linuxfsutil.c

Should help avoid conflicts between glibc and linux headers.

Closes: https://github.com/ostreedev/ostree/issues/2685

Remove unused `linux/fs.h` includes

Prep for fixing conflicts introduced by newer glibc.
cc https://github.com/ostreedev/ostree/issues/2685

Merge pull request #2683 from cgwalters/sync-deny

deny.toml: Add `Unicode-DFS-2016`

deny.toml: Add `Unicode-DFS-2016`

This is used by the unicode crate now and is definitely a compatible
FOSS license.

Merge pull request #2678 from cgwalters/release-2022-5

Release 2022 5

rust-bindings: Fix `cargo fmt`

configure: post-release version bump

Release 2022.5

Merge pull request from GHSA-gqf4-p3gv-g8vw

Fix sign ed25519 verify

Merge pull request #2675 from HuijingHei/update-doc

Update doc about adding new function to libostree

Merge pull request #2676 from render-se/debos-docs

add debos to readme distribution build tools