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 support for building AlmaLinux 9 aarch64 parallels vagrant box #115

Closed
wants to merge 13 commits into from
Closed

Add support for building AlmaLinux 9 aarch64 parallels vagrant box #115

wants to merge 13 commits into from

Conversation

paulmaunders
Copy link
Contributor

These changes add support for building 9/aarch64 vagrant boxes using the parallels provider so they can be run on M1 Macs. It mostly works, but the parallels tools do not seem to be installed correctly.

@paulmaunders paulmaunders mentioned this pull request Nov 15, 2022
Open
@LKHN LKHN requested a review from srbala November 15, 2022 16:20
@LKHN LKHN added virtualbox VirtualBox related issues parallels Parallels Desktop related issues 9 AlmaLinux OS 9 labels Nov 15, 2022
@LKHN
Copy link
Member

LKHN commented Nov 15, 2022

Hi Paul,

Thanks for the contribution!

How the VirtualBox vagrant boxes works on the macOS / Arm64 (M1/M2) hosts ?

@paulmaunders
Copy link
Contributor Author

Hi Elkhan,
No problem!
The packer build process runs and completes, but I am not able to load the vagrant boxes because the parallels tools weren't installed correctly. If Parallels can fix their tools, then I think this will work well!
See: https://forum.parallels.com/threads/parallels-tools-do-not-install-on-rhel9.357546/

@LKHN
Copy link
Member

LKHN commented Nov 15, 2022

As I see from the changes, you also added the support for the virtualbox provider too. I wonder how the box is working on the Developer preview for macOS / Arm64 (M1/M2) hosts

@paulmaunders
Copy link
Contributor Author

paulmaunders commented Nov 15, 2022 •
edited
Loading

The virtualbox developer preview didn't seem to work when I tried it, but I will keep trying!

==> Some builds didn't complete successfully and had errors:
--> virtualbox-iso.almalinux-9-aarch64: Error starting VM: VBoxManage error: VBoxManage: error: Unsupported CPU. (VERR_UNSUPPORTED_CPU)
VBoxManage: error: Details: code NS_ERROR_FAILURE (0x80004005), component ConsoleWrap, interface IConsole

@srbala
Copy link
Contributor

srbala commented Nov 16, 2022

@paulmaunders what version of macOS you are using? I am getting CPU error as above in 12.5

@paulmaunders
Copy link
Contributor Author

@srbala I am using Ventura 13.0.1 (22A400)

VirtualBox does not work at all at the moment, but Parallels should build a box - it just fails to install the Parallels tools.

srbala
srbala reviewed Jan 13, 2023
View reviewed changes
variables.pkr.hcl
@@ -10,7 +10,7 @@ variables {
iso_checksum_8_ppc64le = "file:https://repo.almalinux.org/almalinux/8.7/isos/ppc64le/CHECKSUM"
iso_url_9_x86_64 = "https://repo.almalinux.org/almalinux/9.1/isos/x86_64/AlmaLinux-9.1-x86_64-boot.iso"
iso_checksum_9_x86_64 = "file:https://repo.almalinux.org/almalinux/9.1/isos/x86_64/CHECKSUM"
iso_url_9_aarch64 = "https://repo.almalinux.org/almalinux/9.1/isos/aarch64/AlmaLinux-9.1-aarch64-boot.iso"
Copy link
Contributor

Choose a reason for hiding this comment

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

Change back to boot iso. 750MB download vs 8GB, prefer to stay with boot iso.

srbala
srbala suggested changes Jan 13, 2023
View reviewed changes
Copy link
Contributor

@srbala srbala left a comment

Choose a reason for hiding this comment

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

@paulmaunders Please make proposed change, I'll review again

almalinux-9-vagrant.pkr.hcl
@@ -142,7 +183,7 @@ build {
ansible_env_vars = [
"ANSIBLE_PIPELINING=True",
"ANSIBLE_REMOTE_TEMP=/tmp",
"ANSIBLE_SSH_ARGS='-o ControlMaster=no -o ControlPersist=180s -o ServerAliveInterval=120s -o TCPKeepAlive=yes'"
Copy link
Contributor

Choose a reason for hiding this comment

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

Please roll back this change, since it affects all of the cloud-images build processes.

If you would like to propose the Ansible SSH options, create a separate PR/issue and justify it.

http/almalinux-9.vagrant-aarch64.ks
@@ -0,0 +1,64 @@
# AlmaLinux 9 aarch64 kickstart file for Vagrant boxes
cdrom
Copy link
Contributor

Choose a reason for hiding this comment

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

KS file might need an update since it has to use the boot iso and repo URLs.

@srbala
Copy link
Contributor

srbala commented Jan 13, 2023

@paulmaunders one more, if you can create PR from a fix-branch, instead of the master branch where you can squash the multiple commits Prior to PR.

This was referenced Jan 19, 2023
Merged
Merged
@srbala srbala closed this Jan 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@srbala srbala srbala requested changes

Assignees
No one assigned
Labels
9 AlmaLinux OS 9 parallels Parallels Desktop related issues virtualbox VirtualBox related issues
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@paulmaunders @LKHN @srbala