-
Notifications
You must be signed in to change notification settings - Fork 169
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
mantle/gcp: support c3 metal instance types #3878
Conversation
This requires the IDPF guest OS feature flag to be set when creating GCP images and also a TERMINATE maintenance policy and hyperdisk storage. With this change, after uploading a disk you can then test with kola with something like: ``` cosa kola run -p=gcp \ --gcp-json-key=key.json \ --gcp-project=project \ --gcp-machinetype=c3-highcpu-192-metal \ --gcp-zone=us-central1-c basic ``` Related to coreos/fedora-coreos-tracker#1794
ignore golangci-lint - those failures are handled in #3876 |
Whenever we change the disk/cloud code we should also double check what the image builder code does since ideally in the future we unify with those, was that done? |
No I didn't. It turns out I made a bad assumption. The RHEL9 image somehow does have that guestOS feature enabled:
and I assumed that since it did OSBuild already had that setting baked in. I guess there is some other source of truth for those images? I'll open a PR over there too. |
@dustymabe GCP builds these images with their own tooling. We could escalate it to them via internal channels if needed. |
Thanks @major - no need. Just wanted to understand the delta and now we do. |
@cgwalters - opened osbuild/images#929 |
/lgtm |
merging.. |
This requires the IDPF guest OS feature flag to be set when creating GCP images and also a TERMINATE maintenance policy and hyperdisk storage. With this change, after uploading a disk you can then test with kola with something like:
Related to coreos/fedora-coreos-tracker#1794