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

First take at getting spintainer working #278

Merged
merged 1 commit into from
Sep 18, 2024

Conversation

calebschoepp
Copy link
Contributor

@calebschoepp calebschoepp commented Jul 18, 2024

Spintainer is an alternative executor that runs a spin app in a container rather than via the shim. The user can leverage the image published by the Spin project or they can create their own image. A user might use spintainer if:

  • They can't install the shim.
  • They want to use a different version of Spin or trigger not supported by the shim.

@adamreese
Copy link
Member

I really love the name spintainer but it's not totally intuitive what it is. Giving it a cute name makes it sound like it's something more than just a container.

@calebschoepp
Copy link
Contributor Author

I really love the name spintainer but it's not totally intuitive what it is. Giving it a cute name makes it sound like it's something more than just a container.

I hear that. What would you prefer we call it?

Copy link
Contributor

@endocrimes endocrimes left a comment

Choose a reason for hiding this comment

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

first pass lgtm 🎉

This executor runs SpinApps directly in a docker container rather than via the shim.
You can use the default docker images published by the Spin project or you can craft
your own images to run custom triggers, Spin versions, plugins, etc.

Signed-off-by: Caleb Schoepp <[email protected]>
@calebschoepp
Copy link
Contributor Author

calebschoepp commented Sep 16, 2024

Comments addressed — looking for some approvals.

Copy link
Contributor

@endocrimes endocrimes left a comment

Choose a reason for hiding this comment

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

Code LGTM - one feature compatibility question/comment/concern, but happy to punt into follow up.

internal/controller/spinapp_controller.go Show resolved Hide resolved
@calebschoepp calebschoepp merged commit ea924ac into spinkube:main Sep 18, 2024
11 checks passed
@calebschoepp calebschoepp deleted the spintainer branch September 18, 2024 01:45
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.

4 participants