core: Use source root for repos if set

The goal here is to have the new (more container-native) flow for `experimental compose rootfs` to always operate on a "repos container" that we've mounted. I noticed that `rpmostree_find_and_download_packages()` has similar behavior, but that's apparently only used in override flows? Signed-off-by: Colin Walters <[email protected]>
coreos · Feb 7, 2025 · 8f82c3b · 8f82c3b
1 parent 3ac642e
commit 8f82c3b
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/src/libpriv/rpmostree-core.cxx b/src/libpriv/rpmostree-core.cxx
@@ -668,6 +668,14 @@ rpmostree_context_setup (RpmOstreeContext *self, const char *install_root, const
 
   dnf_context_set_install_root (self->dnfctx, install_root);
   dnf_context_set_source_root (self->dnfctx, source_root);
+  if (source_root)
+    {
+      // Override what rpmostree_context_set_cache_root() did.
+      // TODO disentangle these things so that we only call set_repo_dir if
+      // we didn't have a source root.
+      g_autofree char *source_repos = g_build_filename (source_root, "etc/yum.repos.d", NULL);
+      dnf_context_set_repo_dir (self->dnfctx, source_repos);
+    }
 
   /* Hackaround libdnf logic, ensuring that `/etc/dnf/vars` gets sourced
    * from the host environment instead of the install_root:

diff --git a/tests/compose-rootfs/Containerfile b/tests/compose-rootfs/Containerfile
@@ -17,12 +17,6 @@ COPY . /src
 WORKDIR /src
 RUN --mount=type=bind,from=repos,src=/,dst=/repos <<EORUN
 set -xeuo pipefail
-# Synchronize the dnf/rpm configs from the repos container.
-for x in etc/dnf etc/yum.repos.d etc/pki/rpm-gpg; do
-  rm -rf /"$x" && cp -a /repos/${x} /$x
-done
-# And copy to the workdir; TODO fix this in rpm-ostree
-cp /etc/yum.repos.d/*.repo .
 exec rpm-ostree experimental compose rootfs --source-root=/repos manifest.yaml /target-rootfs
 EORUN