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

Amphora image tweaks #38

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

Amphora image tweaks #38

wants to merge 7 commits into from

Conversation

DavidFair
Copy link
Collaborator

Adds various tweaks to the Amphora build steps

See each commit for further details

Instead of configuring en-ZW for example, replace the default locale
with en_US which is shipped anyway since these VMs are not user facing
in any-way
Adds the output directory and any log.txt files from the build to the
gitignore to ensure we don't pollute the git history with large or
obsolete files
Switches the builder to use Ubuntu 22.04
Unfortunately, upstream disk image builder requires a fix as they use
pkg_resources for 24.04 support, so bump a single version instead
Improves the various git clones for the amphora:
- Use OpenDev directly for Octavia instead of a mirror
- Set depth to 1 for all clones, since these are ephemeral
@DavidFair DavidFair added the enhancement New feature or request label Nov 12, 2024
Update the upload step to use the dedicated output directory, instead of
making the user manually cd to the correct place
@DavidFair DavidFair force-pushed the Amphora_image_tweaks branch 4 times, most recently from 95ea0b5 to 991800f Compare November 12, 2024 18:19
Validates the qemu image then creates artifacts from the Github Actions
including the SHA256 (which OpenStack can use as part of the image
upload) as artifacts that stay around for ~1 week
Adds steps for enabling the Amphora image after uploading it
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants