-
Notifications
You must be signed in to change notification settings - Fork 277
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
Consider forking weaveworks/ignite #1351
Comments
Hi @gaby |
@hellt Your go.mod still points to weaveworks though: https://github.com/srl-labs/containerlab/blob/main/go.mod#L39 |
Line 295 in 8c9b54f
|
😁 Didn't see that. Seems like even that one still has the CNI issue. Ignite was never updated to cni v1.0.0 while containerlab does use it. Not sure if that's an issue 🤔 Just wanted to make sure that containerlab was aware of ignite being abandoned. |
We are not using it much to be honest. Only cumulus nodes leverage ignite, and a few corner cases for Linux containers. We wont maintain it either, all the hope is with @networkop :) |
Yep, feel free to tag me in any ignite-related issues. I'll try to keep it alive for as long as it makes sense. |
Containerlab is the biggest project using
weaveworks/ignite
right now. The project seems to have been abandoned since weaveworks is now focused ongitops
. It may be worth for containerlab to fork and at least update deps for ignite.Rel weaveworks/ignite#971
The text was updated successfully, but these errors were encountered: