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.

Sign up for GitHub

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

Jump to bottom

Add xz-utils as dependencies #1677

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add xz-utils as dependencies #1677

wants to merge 1 commit into from

Conversation

MarioZZJ
Copy link

Description

Fix licensed gem installation failure on ubuntu-latest (probably Ubuntu 24.04) runner. The issue was discovered during recent GitHub Actions ubuntu-latest image upgrade (announcement), where nokogiri compilation fails due to missing xzcat.

Problem

When running metrics action on Ubuntu 24.04 (new ubuntu-latest), licensed gem installation failed with:

ERROR: Failed to build gem native extension.
...
/var/lib/gems/3.1.0/gems/mini_portile2-2.8.8/lib/mini_portile2/mini_portile.rb:557:in `xzcat_exe': xzcat not found (RuntimeError)

Solution

Add xz-utils package to Dockerfile dependencies to provide xzcat command required by nokogiri compilation.

Testing

Successfully tested with action running on ubuntu-latest (workflow run)

Notes

Since GitHub's ubuntu-latest upgrade is rolling out until January 17th, 2025 (see announcement):

  • This PR could be merged after majority of users migrate to Ubuntu 24.04
  • Or affected users can temporarily use runs-on: ubuntu-22.04 as workaround

@MarioZZJ
fix: add xz-utils dependencies
ab6e708 
The licensed gem installation fails due to missing `xzcat` command during `nokogiri` compilation.
Adding `xz-utils` package resolves this dependency issue.
@MarioZZJ
Copy link
Author

@lowlighter Hi, I noticed this issue during recent GitHub Actions ubuntu-latest upgrade process.

Would you mind reviewing this PR?

SN-Koarashi added a commit to SN-Koarashi/metrics that referenced this pull request Dec 28, 2024
@SN-Koarashi
fix: add xz-utils dependencies
d969684 
refer to lowlighter#1677
Yomisana added a commit to Yomisana/metrics that referenced this pull request Dec 28, 2024
@Yomisana
Add xz-utils as dependencies lowlighter#1677
c33d1f5 
fix: add `xz-utils` dependencies, actions linux system not had this dependencies

refer to lowlighter#1677 PR
MrMasrozYTLIVE added a commit to MrMasrozYTLIVE/metrics that referenced this pull request Dec 29, 2024
@MrMasrozYTLIVE
Update Dockerfile
2fc7e00 
Add xz-utils as dependencies lowlighter#1677
fix: add `xz-utils` dependencies, actions linux system not had this dependencies

refer to lowlighter#1677 PR
BalaM314 added a commit to BalaM314/BalaM314 that referenced this pull request Jan 2, 2025
@BalaM314
Fix metrics, lowlighter/metrics#1677
efbfcb1
@tr7zw
Copy link

tr7zw commented Jan 9, 2025

Might this be related to repository stats no longer seeming to work for the past few days?
grafik

@MrMasrozYTLIVE
Copy link

Might this be related to repository stats no longer seeming to work for the past few days? grafik

Hey tr7zw, yeah that's the reason. It shouldve failed all builds for you for last few days

@tr7zw
Copy link

tr7zw commented Jan 9, 2025

I had the first failing one today (https://github.com/tr7zw/tr7zw/actions/runs/12682127087/attempts/1 but it worked on a second retry), outside of that I just noticed that all repo stat values vanish from time to time (tr7zw/tr7zw@2482f9a#diff-15c77d2bbedf6bd69a53f23e8705a32acfeeb9a45ca324f78e8f385924ab3f29L188), might depend on which ubuntu runner it happens to use.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MrMasrozYTLIVE MrMasrozYTLIVE MrMasrozYTLIVE approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@MarioZZJ @tr7zw @MrMasrozYTLIVE