Skip to content

node-installer: fixes for containerd 2 #301

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

Merged

Conversation

shirok1
Copy link
Contributor

@shirok1 shirok1 commented Mar 27, 2025

  • node-installer: detect containerd config version instead of $IS_K3S
  • node-installer: K3S & RKE2 can detect spin shim, no need to configure
  • node-installer: fix K3S by linking shim to PATH

As discussed in #231 (comment)

@shirok1
Copy link
Contributor Author

shirok1 commented Mar 27, 2025

@shirok1

This comment was marked as resolved.

@shirok1 shirok1 force-pushed the fix-node-installer-containerd-2 branch from 639588e to 0d7dd1d Compare March 27, 2025 16:23
Copy link
Collaborator

@Mossaka Mossaka left a comment

Choose a reason for hiding this comment

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

LGTM! @kate-goldenring can you PTAL?

@Mossaka
Copy link
Collaborator

Mossaka commented Mar 27, 2025

Need a rebase

Copy link
Collaborator

@kate-goldenring kate-goldenring left a comment

Choose a reason for hiding this comment

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

LGTM thank you!

@shirok1 shirok1 force-pushed the fix-node-installer-containerd-2 branch 2 times, most recently from 08b1b91 to 8a5219d Compare March 28, 2025 06:58
shirok1 and others added 3 commits March 28, 2025 15:09
Signed-off-by: Shiroki Satsuki <[email protected]>

# Conflicts:
#	node-installer/script/installer.sh
node-installer: fix symbolic link to path containing $NODE_ROOT
node-installer: clearify containerd bug

Co-authored-by: Kate Goldenring <[email protected]>
Signed-off-by: Shiroki Satsuki <[email protected]>
@shirok1 shirok1 force-pushed the fix-node-installer-containerd-2 branch from 8a5219d to e513c36 Compare March 28, 2025 07:09
@kate-goldenring kate-goldenring merged commit 69dac2c into spinframework:main Mar 28, 2025
9 checks passed
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.

3 participants