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

CPU Architecture option #1

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

Conversation

Sadi-a
Copy link

@Sadi-a Sadi-a commented Jun 9, 2023

Typhoon on flatcar could previously only be run on x86_64 nodes.
This commit changes this by adding a new variable per node which is
cpu_architecture.

This value is set by default on "x86_64".
This variable is used for caching, selecting the architecture when
installing flatcar, and grabbing the right files with the remote
kernel and remote initrd.
This value can be set when defining a node at the same time
as other attributes such a machine's domain, mac address or
name.

This PR concerns both Flatcar Linux and Fedora CoreOS

Tested successfully on the test suite with x86_64 VMs.
I haven't tried with aarch64 VMs and mixed VMs (some aarch64, some x86_64) yet since I don't have KVM enabled for qemu-system-aarch64 (looks like I only have access to the tcg accelerator there) and thus can't use terraform's libvirt since it requires KVM.

@Sadi-a Sadi-a force-pushed the sadi/architecture-option branch from 59d521c to b103ef2 Compare June 13, 2023 10:49
@Sadi-a Sadi-a force-pushed the sadi/architecture-option branch 2 times, most recently from 4f15b62 to faf42c7 Compare July 11, 2023 15:44
@Sadi-a Sadi-a marked this pull request as draft July 12, 2023 13:23
@Sadi-a Sadi-a force-pushed the sadi/architecture-option branch from faf42c7 to a63514d Compare July 12, 2023 14:10
@Sadi-a Sadi-a marked this pull request as ready for review July 12, 2023 14:11
@Sadi-a
Copy link
Author

Sadi-a commented Jul 12, 2023

the PR was in draft mode for a short while since I wanted to make a small change that makes things a bit more elegant for controllers.

Sadi-a added 2 commits August 10, 2023 11:09
Typhoon on flatcar could previously only be run on x86_64 nodes.
This commit changes this by adding a new variable per node which is
cpu_architecture.
This value is set by default on "x86_64".
This variable is used for caching, selecting the architecture when
installing flatcar, and grabbing the right files with the remote
kernel and remote initrd
This value can be set when defining a node at the same time as other
attributes such a machine's domain, mac address or name.
Typhoon on fcos could previously only be run on x86_64 nodes.
This commit changes this by adding a new variable per node which is
cpu_architecture. This value is set by default on "x86_64" since
flatcar uses the "x86_64/aarch64" notation for cpu architectures. This
variable is used for caching, selecting the architecture when
installing flatcar, and grabbing the right files with the remote kernel
and remote initrd
@Sadi-a Sadi-a force-pushed the sadi/architecture-option branch from a63514d to d22c701 Compare August 10, 2023 10:09
Copy link
Member

@Snaipe Snaipe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, LGTM

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