lorax: Write package lists in run_transaction

A change in glibc now requires /proc be mounted in order to run mknod
which is needed in order to run rpm from runtime-postinstall.

This drops that code from the template and moves writing the package
list into run_transaction, which already has all of the needed
information to generate the list.

Resolves: rhbz#1812895
This commit is contained in:
Brian C. Lane 2020-03-12 12:21:26 -07:00
parent a30531d41b
commit e2e8deb906
2 changed files with 17 additions and 27 deletions

View File

@ -119,12 +119,6 @@ remove etc/lvm/cache
remove etc/lvm/lvm.conf remove etc/lvm/lvm.conf
append etc/lvm/lvm.conf "global {\n\tuse_lvmetad = 1\n}\n" append etc/lvm/lvm.conf "global {\n\tuse_lvmetad = 1\n}\n"
## Record the package versions used to create the image
## rpm initializes nss, which requires /dev/urandom to be present, hence the mknod
runcmd chroot ${root} /usr/bin/mknod -m 666 /dev/random c 1 8
runcmd chroot ${root} /usr/bin/mknod -m 666 /dev/urandom c 1 9
runcmd chroot ${root} /usr/bin/rpm -qa --pipe "sort | tee /root/lorax-packages.log"
## TODO: we could run prelink here if we wanted? ## TODO: we could run prelink here if we wanted?
## fix fonconfig cache containing timestamps ## fix fonconfig cache containing timestamps

View File

@ -247,35 +247,31 @@ class LoraxTemplateRunner(TemplateRunner):
def _getsize(self, *files): def _getsize(self, *files):
return sum(os.path.getsize(self._out(f)) for f in files if os.path.isfile(self._out(f))) return sum(os.path.getsize(self._out(f)) for f in files if os.path.isfile(self._out(f)))
def _write_debuginfo_log(self): def _write_package_log(self):
""" """
Write a list of debuginfo packages to /root/debug-pkgs.log Write the list of installed packages to /root/ on the boot.iso
If lorax is called with a debug repo find the corresponding debuginfo package If lorax is called with a debug repo find the corresponding debuginfo package
names and write them to /root/debubg-pkgs.log on the boot.iso names and write them to /root/debubg-pkgs.log on the boot.iso
The non-debuginfo packages are written to /root/lorax-packages.log
""" """
for repo in self.dbo.repos: os.makedirs(self._out("root/"), exist_ok=True)
repo = self.dbo.repos[repo]
if any(True for url in repo.baseurl if "debug" in url):
break
if repo.metalink and "debug" in repo.metalink:
break
if repo.mirrorlist and "debug" in repo.mirrorlist:
break
else:
# No debug repos
return
available = self.dbo.sack.query().available() available = self.dbo.sack.query().available()
pkgs = []
debug_pkgs = [] debug_pkgs = []
for p in list(self.dbo.transaction.install_set): for p in list(self.dbo.transaction.install_set):
pkgs.append(f"{p.name}-{p.epoch}:{p.version}-{p.release}")
if available.filter(name=p.name+"-debuginfo"): if available.filter(name=p.name+"-debuginfo"):
debug_pkgs += ["{0.name}-debuginfo-{0.epoch}:{0.version}-{0.release}".format(p)] debug_pkgs.append(f"{p.name}-debuginfo-{p.epoch}:{p.version}-{p.release}")
os.makedirs(self._out("root/"), exist_ok=True) with open(self._out("root/lorax-packages.log"), "w") as f:
with open(self._out("root/debug-pkgs.log"), "w") as f: f.write("\n".join(sorted(pkgs)))
for pkg in debug_pkgs: f.write("\n")
f.write("%s\n" % pkg)
if debug_pkgs:
with open(self._out("root/debug-pkgs.log"), "w") as f:
f.write("\n".join(sorted(debug_pkgs)))
f.write("\n")
def install(self, srcglob, dest): def install(self, srcglob, dest):
''' '''
@ -640,8 +636,8 @@ class LoraxTemplateRunner(TemplateRunner):
if len(self.dbo.transaction) == 0: if len(self.dbo.transaction) == 0:
raise Exception("No packages in transaction") raise Exception("No packages in transaction")
# If a debug repo has been included, write out a list of debuginfo packages # Write out the packages installed, including debuginfo packages
self._write_debuginfo_log() self._write_package_log()
pkgs_to_download = self.dbo.transaction.install_set pkgs_to_download = self.dbo.transaction.install_set
logger.info("Downloading packages") logger.info("Downloading packages")