Skip to content

Conversation

priteau
Copy link
Member

@priteau priteau commented Jan 20, 2023

No description provided.

mnasiadka and others added 15 commits January 11, 2023 11:53
(cherry picked from commit ee30588)
The file 50-edk2-ovmf-cc.json is used by Nova in get_loader() when
lauching UEFI instances without Secure Boot.

It only supports machine type pc-q35-rhel8.5.0. However, with recent
nova-compute container images built on CentOS Stream 8, the machine type
is pc-q35-rhel8.6.0, which results in failures to launch instances with:

    nova.exception.UEFINotSupported: UEFI is not supported

There is a fix for RHEL9/c9s [1], but not for RHEL8/c8s. Also see [2]
for more details.

[1] https://bugzilla.redhat.com/show_bug.cgi?id=2090752
[2] https://bugs.launchpad.net/nova/+bug/1955035
Patch edk2-ovmf file to fix UEFI support
Use Rocky Linux 8.7 repositories
While not pretty, this workaround avoids manual intervention after the
runner is power cycled.

Co-Authored-By: Pierre Riteau <pierre@stackhpc.com>
CI: Workaround container image builder power cycle issue
Deploy Nova images with edk2-ovmf UEFI fix
@priteau priteau requested a review from a team as a code owner January 20, 2023 10:06
@priteau priteau self-assigned this Jan 20, 2023
markgoddard
markgoddard previously approved these changes Jan 20, 2023
@priteau
Copy link
Member Author

priteau commented Jan 20, 2023

Is this the proper way of merging changes from one branch to another? I believe CI will automatically create a Git tag after the merge, and it will look like this tag has the UEFI fix (because there is the release note), while I still need to build Nova images for Xena and bump the container image tag.

@markgoddard
Copy link
Contributor

Is this the proper way of merging changes from one branch to another? I believe CI will automatically create a Git tag after the merge, and it will look like this tag has the UEFI fix (because there is the release note), while I still need to build Nova images for Xena and bump the container image tag.

I usually just merge one branch into the next, then check the diff and make any changes necessary for the current branch. In this case that should probably include building a container and bumping the tag.

@priteau
Copy link
Member Author

priteau commented Jan 20, 2023

I have triggered a nova container build using this branch: https://github.com/stackhpc/stackhpc-kayobe-config/actions/runs/3967356367

@priteau priteau merged commit b81a95f into stackhpc/xena Jan 25, 2023
@priteau priteau deleted the xena-wallaby-merge branch January 25, 2023 10:15
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.

4 participants