fix: remove apt-get upgrade from pdf-prep-force to fix CI snap failure - #619
Open
izmalk wants to merge 1 commit into
Open
fix: remove apt-get upgrade from pdf-prep-force to fix CI snap failure#619izmalk wants to merge 1 commit into
izmalk wants to merge 1 commit into
Conversation
The 'apt-get upgrade -y' line in the pdf-prep-force target was causing CI to fail on the PDF dependencies installation step. The upgrade pulls in ALL system packages, including the firefox snap on Ubuntu 24.04 runners, which cannot be installed in the containerized CI environment. The upgrade is unnecessary for PDF generation - only the PDF packages need to be installed, not the entire system upgraded. Removing it makes CI deterministic and avoids snap-related failures.
izmalk
requested review from
SecondSkoll,
a-velasco,
akcano,
dwilding,
medubelko,
rkratky and
tang-mm
as code owners
July 22, 2026 21:19
Contributor
Author
|
@medubelko I have encountered it in my own Ulwazi repo, where we have the same problem - https://github.com/canonical/ulwazi/actions/runs/29955798613/job/89046899743#step:3:739 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Removes the
apt-get upgrade -yline from thepdf-prep-forcetarget indocs/Makefile. This line was causing CI failures on the PDF dependencies installation step.Root cause
apt-get upgrade -yupgrades all system packages, not just the PDF-related ones. On the GitHub Actionsubuntu-latestrunner, this pulls in a new version of thefirefoxpackage. In Ubuntu 24.04,firefoxis a transitional package that depends on the firefox snap, and snapd doesn't work properly in the containerized CI environment — hence:This is an environmental failure: whenever a new firefox snap version is published in the Ubuntu archive,
apt-get upgradetries (and fails) to install it.The fix
The
apt-get upgrade -yline is unnecessary for PDF generation — you only need to install the PDF packages, not upgrade the entire system. Removing it makes CI deterministic and avoids snap-related failures.pdf-prep-force: apt-get update - apt-get upgrade -y apt-get install --no-install-recommends -y $(DOCS_PDFPACKAGES) \