Skip to content

[WIP] Container Providers and other fixes #75

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

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

Conversation

aramnhammer
Copy link

@aramnhammer aramnhammer commented Mar 23, 2025

Discovery of alternative local container providers

NOTE: in the current state this is just a proof of concept to unblock me on a project, If I can get the go ahead to complete the implementation I will update this PR with the full implementation and remove this note, see todo section.

Motivation

  1. I use podman on my development machine, and its api is mostly compatible with docker
  2. RedHat (biased source) has a nice writeup of why to use podman
  3. I like sidekick's design and want to expand it

Other changes

I discovered the image name does not include localhost/ before it when loaded into the vps, this may be a podman specific issue or its simply a bug. Since Sidekick does not support registry (that i know of), i think it makes sense to have images loaded with localhost/<imageName>

TODO

  • determine local container runtime earlier (deduplicate the POC)
  • support podman as a remote container provider (on the VPS)

Testing

Tested with a simple python app and my proxmox server running a VM

@aramnhammer aramnhammer marked this pull request as ready for review March 24, 2025 20:13
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.

1 participant