Skip to content

refactor: migrate bootstrapper to components api#96

Merged
bcho merged 32 commits intomainfrom
hbc/components
Feb 27, 2026
Merged

refactor: migrate bootstrapper to components api#96
bcho merged 32 commits intomainfrom
hbc/components

Conversation

@bcho
Copy link
Member

@bcho bcho commented Feb 24, 2026

Thsi pull request refactored the bootstrapper steps to use components API. We also migrated the kubelet start and cni steps to components API. Unused bootstrapper steps are removed as part of the PR.

@bcho bcho marked this pull request as ready for review February 26, 2026 03:49
@@ -1,8 +1,11 @@
package components

import (
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm wondering if we add multiple versions of the same action in the future, do I need to register all with the hub? I assume not, since the action protobuf in unversioned, right?

Copy link
Member Author

Choose a reason for hiding this comment

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

I think we will have a step to convert into unversioned model. But I don't see the necessary to implement that layer at this moment, so i just assume the v20260301 is the base "unversioned" model for now.

@@ -0,0 +1,54 @@
[Unit]
Description=Kubelet
Copy link
Collaborator

Choose a reason for hiding this comment

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

So KubadmNodeJoin action cannot coexist with StartKubeletService action, do we have a way to enforce that?

Copy link
Member Author

Choose a reason for hiding this comment

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

this is on caller to make sure the sequence/actions selected will be working

@bcho bcho merged commit 59776c6 into main Feb 27, 2026
30 of 32 checks passed
@bcho bcho deleted the hbc/components branch February 27, 2026 00:30
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