particle-os/debian-forge
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

stages/dnf: log more clearly in case the repo metadata changes

Browse source 
This is an expected error case, so we should not assert, but log
and return failure. In the future we should probably also return
the error as structured data.

Signed-off-by: Tom Gundersen <teg@jklm.no>
This commit is contained in:
Tom Gundersen 2019-12-17 20:26:30 +01:00 • committed by Lars Karlitski
parent f67a649805
commit 1f7766596f
1 changed files with 5 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

7
stages/org.osbuild.dnf
View file

@ -234,12 +234,15 @@ def main(tree, options):
# verify metadata checksum
for repoid, repo in enumerate(repos):
algorithm, checksum = repo["checksum"].split(":")
algorithm, expected_checksum = repo["checksum"].split(":")
assert algorithm == "sha256"
cachedir = dnf_cachedir(f"repo{repoid}", repo, releasever, basearch)
with open(f"{tree}/var/cache/dnf/{cachedir}/repodata/repomd.xml", "rb") as f:
repomd = f.read()
assert hashlib.sha256(repomd).hexdigest() == checksum
checksum = hashlib.sha256(repomd).hexdigest()
if checksum != expected_checksum:
print(f"repo {repoid} has checksum {checksum}, expected {expected_checksum}")
return 1
# delete cache manually, because `dnf clean all` leaves some contents behind
fd = os.open(f"{tree}/var/cache/dnf", os.O_DIRECTORY)