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

Ubuntu 24 runner support #8326

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

Ubuntu 24 runner support #8326

wants to merge 3 commits into from

Conversation

hexchain
Copy link
Contributor

@hexchain hexchain commented Jan 28, 2025

  • Update Linux runner OS to Ubuntu 24.04 again
  • Ensure unprivileged user namespace is available to normal user
  • Applied some workarounds for ARM64 Linux runner issues

@hexchain hexchain mentioned this pull request Jan 28, 2025
@hexchain
Copy link
Contributor Author

hexchain commented Jan 28, 2025

Hmmm, strange... the exact same job works without problem in my fork: https://github.com/hexchain/insomnia/actions/runs/13013477749/job/36296739340

@hexchain
Copy link
Contributor Author

hexchain commented Jan 28, 2025

I see, the failing one is a different workflow. Fixed.

@hexchain hexchain force-pushed the ci-fixes branch 2 times, most recently from bf8dc2d to 1c32476 Compare January 29, 2025 04:59
@hexchain
Copy link
Contributor Author

I've rebased this against develop and also applied a fix for snap/rpm/deb generation.

Comment on lines +75 to +82
if: runner.os == 'Linux'
run: |
# install snapcraft
sudo snap install snapcraft --classic

# install fpm for electron-builder
sudo gem install fpm
fpm --version
Copy link
Contributor

Choose a reason for hiding this comment

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

Can this be arm only? I think x64 already works, or was snap removed in Ubuntu 24?

Copy link
Contributor

@jackkav jackkav left a comment

Choose a reason for hiding this comment

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

Looks like we are using two workarounds here

  1. lift ubuntu 24 restrictions
  2. electron-builder doesn't bundle a arm64 fpm

Both of these appear to be open issues, which will hopefully be solved in time.

What motivation do we have to add these workarounds now?

What exactly do you need for the downstream to make your life easier? ubuntu 24 or a specific linux distributable runnable on arm64 (deb,rpm,tar.gz,appimage)?

@hexchain
Copy link
Contributor Author

hexchain commented Jan 29, 2025

Looks like we are using two workarounds here

  1. lift ubuntu 24 restrictions
  2. electron-builder doesn't bundle a arm64 fpm

Both of these appear to be open issues, which will hopefully be solved in time.

Yes, indeed. The userns stuff is for Ubuntu 24 (otherwise electron refuses to start in smoke tests) while the rest are for electron-builder/snapcraft/fpm.

What motivation do we have to add these workarounds now?

What exactly do you need for the downstream to make your life easier? ubuntu 24 or a specific linux distributable runnable on arm64 (deb,rpm,tar.gz,appimage)?

I added those fixes mainly for the sake of completeness (i.e. artifact parity between x64 and arm64 linux). For Flatpak though, I guess having an AppImage is enough (the tar.gz doesn't contain a desktop file IIRC).

I understand that keeping these workarounds can be a real burden. If building snap and deb/rpm isn't required right now for arm64, do you prefer removing those workarounds, and only keep AppImage + tar.gz for arm64 Linux?

@jackkav
Copy link
Contributor

jackkav commented Jan 29, 2025

Let's keep this PR open for now to track when those workarounds are fixed upstream, and open a separate PR solely attempting to add AppImage to arm64 to get you unblocked. I think we would benefit from waiting on ubuntu24 for now, and we have no time pressure on snap/deb/rpm arm64 builds so those workarounds can be avoided too.

@jackkav jackkav changed the title Fix CI test and build issues Ubuntu 24 runner support Jan 29, 2025
@hexchain
Copy link
Contributor Author

Sounds good to me. I've opened #8331 to only enable AppImage for ARM64 Linux.

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