Change from QEMU to OpenStack Builder#86
Merged
Conversation
DavidFair
requested changes
Dec 3, 2025
khalford
force-pushed
the
qemu_to_openstack_builder
branch
4 times, most recently
from
0d52102 to
5a6f937
Compare
Collaborator
|
@khalford is this running the baseline roles currently? Not sure if this is ready for review or still pending problems |
khalford
force-pushed
the
qemu_to_openstack_builder
branch
from
f694c7e to
6f9f3d5
Compare
Member
Author
|
Fixed the issue with roles not runnning and goss failing |
DavidFair
reviewed
Jan 6, 2026
Collaborator
|
Could we confirm that the image still applies the roles from our baseline now we've changed it to |
Member
Author
I have tested this. It applies the roles |
DavidFair
previously approved these changes
Jan 6, 2026
khalford
force-pushed
the
qemu_to_openstack_builder
branch
4 times, most recently
from
0a121d7 to
17fd2bf
Compare
Use the openstack builder as we do for the os images. As we are now building images remotely we need separate variables for dev and prod
OpenStack provisioner uses node instead of first boot. This fixes the vm_baseline role not running
Goss fails but we don't care. Adding this flag allows packer to build the image anyways as it would fail and not build previously
Update the README.md for the build-all script. The usage has changed as packer uploads the image for us
Move common vars between environments into a single file to avoid drift
Override image name to include the date as we normally do. This is changed in the build-all.sh script and needs to be manually changed when doing manual builds
khalford
force-pushed
the
qemu_to_openstack_builder
branch
from
17fd2bf to
1af9dbc
Compare
Update the readme for better formatting and more guidance on different image building scenarios.
DavidFair
requested changes
Jan 12, 2026
Using this template makes the sed simpler and it more obvious.
khalford
force-pushed
the
qemu_to_openstack_builder
branch
from
0928523 to
82a8bee
Compare
DavidFair
approved these changes
Jan 12, 2026
jose-caballero
approved these changes
Jan 12, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Change to using the OpenStack builder which we are using for the os image builds.
Updates the README.md to provide up to date instructions.
Split the variables file into a dev and prod variables. This is because we will be remote provisioning to 2 different clouds