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: enable packing riscv64 cloud images #214

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

Conversation

xypron
Copy link

@xypron xypron commented Mar 7, 2024

add riscv64 architecture to ubuntu-cloudimg

The name of the directory for the riscv64 EDK 2 firmware does not match the prefix of the firmware files. An additional variable was needed.

ubuntu: set bootindex=1 for packer-cloudimg

EDK 2 at least on riscv64 will not use the first disk as first boot option automatically. We need to assign a bootindex.

Testing

For testing the riscv64 build Ubuntu 24.04, Packer v1.10.2, qemu-system-misc 8.2.1+ds-1ubuntu1, qemu-efi-riscv64 2024.02-1 were used.

cd packer-maas/ubuntu/
packer init .
packer build -var ubuntu_series=noble -var architecture=riscv64 -only=cloudimg.* .

or

SERIES=noble ARCH=riscv64 make custom-cloudimg.tar.gz

@xypron
Copy link
Author

xypron commented Mar 8, 2024

Building on arm64 failed without if=none for the drives. I have force pushed the change.

@xypron
Copy link
Author

xypron commented Mar 8, 2024

The parameter acpi for the the RISC-V virt machine was introduced in QEMU v8.0.0. RISC-V EDK II requires QEMU >= v8.1.2.

@SK1Y101
Copy link
Member

SK1Y101 commented Mar 19, 2024

Could you rebase this on top of main? I landed a small change to make the ARCH field in ubuntu images a makefile parameter so we can test arches with our tooling

@xypron
Copy link
Author

xypron commented Mar 20, 2024

@SK1Y101 thank you for reviewing.

I have rebased the patches. With your patch I don't have use packer directly anymore.

SERIES=noble ARCH=riscv64 make custom-cloudimg.tar.gz

works fine.

@xypron
Copy link
Author

xypron commented Jul 23, 2024

Rebased upon origin/main considering 1c81266 ("Noble build fixes + More (#228)") which introduced the OVMF_SFX variable.

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