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

Make appliance build pipeline work on Ubuntu #6

Merged
merged 8 commits into from
Dec 17, 2024
Merged

Conversation

lunkwill42
Copy link
Member

The build stopped working a long time ago, and more cruft would accumulate.

The MacOS runner images were originally used because they were the only ones that would suppport VirtualBox and virtualization within virtualization. The macos-12 image we were dependent on has since been deprecated and macos-13, macos-14 and macos-15 have since been introduced: But these runners all operate on Apple Silicon and do not feature VirtualBox (which is an x86 hypervisor).

It seems the latest ubuntu runners do support virtualization within virtualization, so this updates the pipeline to build using Ubuntu instead.

Because macos-12 is deprecated on GitHub, and later macos images do not
feature Virtualbox at all.
We no longer build for these distros.
This plugin was Python 2 specific and is no longer available on bullseye
- and we don't need it either.
The `-e` wasn't working, becaus the she-bang was broken and the script
is being passed as an argument to bash anyway.  This changes to using
`set -e`, so we can fail the build properly if any command in the script
fails.
@lunkwill42 lunkwill42 self-assigned this Dec 17, 2024
@lunkwill42 lunkwill42 requested a review from a team December 17, 2024 13:15
Copy link
Contributor

@hmpf hmpf left a comment

Choose a reason for hiding this comment

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

Looks fine to me, if it works, it works

@lunkwill42 lunkwill42 merged commit 2f5068b into master Dec 17, 2024
2 checks passed
@lunkwill42 lunkwill42 deleted the test-ubuntu-build branch December 17, 2024 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants