Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Github Actions for Ubuntu 24.04 base image #160

Closed
wants to merge 2 commits into from

Conversation

Stormheg
Copy link
Member

@Stormheg Stormheg commented Dec 24, 2024

Github actions has now defaulted to using its Ubuntu 24.04 base image.
Compared to the previous base image, this is missing many pre-installed
binaries. Such as imagemagick.

Comparison:

@Stormheg Stormheg changed the title CI: update apt package lists before attempting to install Fix Github Actions for Ubuntu 24.04 base image Dec 24, 2024
Github actions has now defaulted to using its Ubuntu 24.04 base image.
Compared to the previous base image, this is missing many pre-installed
binaries. Such as imagemagick.

Comparison:

- Ubuntu 24.04: https://github.com/actions/runner-images/blob/ubuntu24/20241215.1/images/ubuntu/Ubuntu2404-Readme.md
- Ubuntu 22.04: https://github.com/actions/runner-images/blob/ubuntu24/20241215.1/images/ubuntu/Ubuntu2204-Readme.md
@Stormheg
Copy link
Member Author

Not sure what causes the CI to fail... Can't seem to reproduce this locally in a Ubuntu 24.04 docker container.

run: |
sudo apt-get update -y
# For Wand tests
sudo apt-get install -y imagemagick
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

given 22.04 has imagemagick, do we want this?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

or.. perhaps try installing libmagick++-dev libmagickcore-dev libmagickwand-dev in addition to imagemagick on 24.04?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@zerolab I tried to ascertain that the test suite does pass on on 22.04, which it does. That manual install is leftover from trying to fix the 24.04 build.

Not sure why it is failing on 24.04. I know that 24.04 does not ship with ImageMagick installed, so we do that manually. That does not help however, there is something wrong where it fails somewhat silently by returning None.

I'd like to get to the bottom of this, but don't have a lot of capacity to invest at the moment. Perhaps we should pin the workflow to 22.04 for now so the testsuite continues working and create an issue to move to 24.04

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's do that. solve the immediate problem, then spend more time looking into 24.04

@Stormheg
Copy link
Member Author

Closing this in favor of stop-gap #162

@Stormheg Stormheg closed this Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants