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

node-installer: support bare-metal platforms #651

Merged
merged 1 commit into from
Jul 1, 2024

Conversation

msanft
Copy link
Contributor

@msanft msanft commented Jun 28, 2024

This adds support for the K3s- / RKE2-based bare-metal TDX deployment platforms in the node-installer. Depending on which platform is provided to the node-installer command line, different units are restarted, configs are written, etc. This does not yet add support of writing the appropriate commandline in the node-installer deployment, as this will be done in a follow-up PR. Thus, for now, to keep main healthy and deploy-able, we default to AKS-CLH-SNP in the node-installer if no platform is specified.

@msanft msanft requested a review from burgerdev June 28, 2024 11:37
@msanft msanft requested a review from katexochen as a code owner June 28, 2024 11:37
@msanft msanft added the no changelog PRs not listed in the release notes label Jun 28, 2024
Base automatically changed from msanft/kata-bare-metal-node-installer to main July 1, 2024 06:17
@msanft msanft force-pushed the msanft/bm-node-installer branch from b324c2f to 816a712 Compare July 1, 2024 07:10
node-installer/node-installer.go Outdated Show resolved Hide resolved
node-installer/node-installer.go Outdated Show resolved Hide resolved
node-installer/node-installer.go Outdated Show resolved Hide resolved
This adds support for the K3s- / RKE2-based bare-metal TDX deployment platforms in the node-installer. Depending on which platform is provided to the node-installer command line, different units are restarted, configs are written, etc.
This does not yet add support of writing the appropriate commandline in the node-installer deployment, as this will be done in a follow-up PR. Thus, for now, to keep main healthy and deploy-able, we default to AKS-CLH-SNP in the node-installer if no platform is specified.
@msanft msanft force-pushed the msanft/bm-node-installer branch from 816a712 to 0446714 Compare July 1, 2024 11:45
@msanft msanft requested a review from katexochen July 1, 2024 11:45
@msanft msanft merged commit 328a7f5 into main Jul 1, 2024
8 checks passed
@msanft msanft deleted the msanft/bm-node-installer branch July 1, 2024 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no changelog PRs not listed in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants