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

Support ignite backend #705

Open
3 tasks done
BenTheElder opened this issue Jul 15, 2019 · 9 comments
Open
3 tasks done

Support ignite backend #705

BenTheElder opened this issue Jul 15, 2019 · 9 comments
Labels
kind/feature Categorizes issue or PR as related to a new feature. lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. priority/awaiting-more-evidence Lowest priority. Possibly useful, but not yet enough support to actually get it done.

Comments

@BenTheElder
Copy link
Member

BenTheElder commented Jul 15, 2019

What would you like to be added: Support for using ignite as a docker-like backend with improved isolation.

Why is this needed: This provides a relatively fast but much more isolated option for nodes. On the surface the UX is very similar to the docker CLI, and it operates around standard docker images, with the requirement that you use an /sbin/init (which we do, though wrapped for docker in docker workarounds).

Some thoughts:

  • This likely wouldn't ever support extraMounts (or any mounts for that matter, firecracker doesn't support this). It might however support some form of persistent storage in the future, which we'd want to consider how to support as well... Add support for adding volumes to ignite run weaveworks/ignite#76
  • We'll probably want to resolve the following issues upstream first:
  • I already intend to decouple the internals of kind from docker better (and in fact, am working on that right now...) to support a mock node provider and podman, adding ignite seems like a natural next step, and will cause us to think a little more about things that are not exactly docker.

/priority backlog
/assign

@BenTheElder BenTheElder added the kind/feature Categorizes issue or PR as related to a new feature. label Jul 15, 2019
@k8s-ci-robot k8s-ci-robot added the priority/backlog Higher priority than priority/awaiting-more-evidence. label Jul 15, 2019
@aojea
Copy link
Contributor

aojea commented Jul 15, 2019

Just for clarification, the goal is to replace current provider or to allow kind to use multiple providers?
By the way, I was able to play with ignite and sounds and awesome idea, this also allow to test different kernels , ...

@BenTheElder
Copy link
Member Author

Definitely not replace (!) There's too many environments that won't have access to this, realistically, and we have our existing users etc. 🙃

As an additional backend this would be pretty useful and mostly can map pretty closely to what we do already 😁

@tao12345666333
Copy link
Member

SGTM.
I think this will be fun.
The main reason for choosing ignite is due to its UX?

@k8s-ci-robot k8s-ci-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Oct 13, 2019
@kubernetes-sigs kubernetes-sigs deleted a comment from fejta-bot Oct 13, 2019
@BenTheElder BenTheElder removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Oct 13, 2019
@BenTheElder BenTheElder changed the title Support ignite Support ignite backend Nov 1, 2019
@danielfbm
Copy link

I am working to add kubernetes as a new provider (using kubectl to manage pods, etcs), is this a viable/desirable goal for kind ? If yes, I would like to contribute (and probably would need some guidance)

@BenTheElder
Copy link
Member Author

BenTheElder commented Dec 10, 2019 via email

@danielfbm
Copy link

danielfbm commented Dec 10, 2019 via email

@BenTheElder
Copy link
Member Author

it's not in a good state, however there are many comments here about running kind in kubernetes clusters 😬

I highly recommend avoiding that if at all possible, it's tricky to avoid the various footguns, in general I don't recommend creating priviliged containers in kubernetes clusters unless you are working on a system component like kube-proxy.

@danielfbm
Copy link

danielfbm commented Dec 11, 2019

that is our main issue. Our team is mostly working with operators/CRDS/aggregations and the need to share resources (cluster of servers) is big deal for us, so we are willing to give it a shot to avoid creating another orchestration tool 😅, but really appreciate the heads up.

I looked through most of the issues that reference this kind of use case, I think #303 is the most important one here and I will post the issues I found there if it is ok for you.

Also, from what I gathered, kubernetes, istio and other projects use case is actually more like:
Pod using docker-in-docker then running kind inside, which is also a docker-in-docker solution. Would that approach be recommended instead of directly running a pod as a k8s node?

@k8s-ci-robot k8s-ci-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Mar 10, 2020
@BenTheElder BenTheElder added lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. and removed lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. labels Mar 15, 2020
@kubernetes-sigs kubernetes-sigs deleted a comment from fejta-bot Mar 15, 2020
@BenTheElder
Copy link
Member Author

I don't recommend it, unfortunately we must do it anyhow. This is all deeply on my backlog, currently focused on some kubernetes testing needs and #148

@BenTheElder BenTheElder added this to the 2020 goals milestone Apr 25, 2020
@BenTheElder BenTheElder removed this from the 2020 goals milestone Jan 25, 2021
@BenTheElder BenTheElder removed their assignment Jul 29, 2021
@BenTheElder BenTheElder added priority/awaiting-more-evidence Lowest priority. Possibly useful, but not yet enough support to actually get it done. and removed priority/backlog Higher priority than priority/awaiting-more-evidence. labels Jul 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature. lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. priority/awaiting-more-evidence Lowest priority. Possibly useful, but not yet enough support to actually get it done.
Projects
None yet
Development

No branches or pull requests

5 participants