deploy: Remove global `sync` by default
authorColin Walters <walters@verbum.org>
Thu, 3 Aug 2023 23:08:02 +0000 (19:08 -0400)
committerColin Walters <walters@verbum.org>
Mon, 14 Aug 2023 18:16:59 +0000 (14:16 -0400)
commitfa69eaace6a7d15ada8a3c4c41b649a595dbb6f4
treeb6254bd2a56cc3d8deba049b163c5dd0b2483b4a
parent3620d3c709a4083a7bd63323b24bd17db4f3ed4b
deploy: Remove global `sync` by default

Our previous change here was not actually sufficient for
the ceph case, because what (I think) is happening is that
our other `syncfs()` invocation is getting blocked on some
kernel mutexes that are used in `sync`, and that's causing the
process to fully block.

We should not be dependent on a full filesystem `sync`, only
on the sync of the sysroot and boot filesystems.

Anyone who *does* want this behavior could inject an override
for `ostree-finalize-staged.service` that overrides `ExecStop`
to add a run of `sync`.
src/libostree/ostree-sysroot-deploy.c