-
Notifications
You must be signed in to change notification settings - Fork 221
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
Remove support for Windows #710
Comments
I humbly wanted to provide some reasons against this breaking change: Some may say that these are niche reasons in the era of the cloud. But I think these 2 reasons are both very common AND very important for users that work on on-prem enterprise networks:
These reasons are actually the main reason why I am looking for krustlet as a project. Lastly, wasm is about being able to run your code everywhere. So shouldn't k8s follow suit? I really like this project either way, and I really hope you consider this 🙏 |
I understand where you're coming from, @RaananHadar. But the question was more about who is either developing or running Krustlet on Windows. We understand the benefits. But right now the issues stemming from Windows support is actively preventing us from shipping 1.0 due to the issues mentioned earlier. So we're trying to gauge how much interest there is in attempting to fix this (which will cause further delays to 1.0's release) or we remove Windows support for now and re-introduce in a future update when things stabilize. I'm +1 on removing support for now and re-introducing it in a future update. |
I'm all for shipping 1.0 and fully understand that you can't support everything from the get go. I just wanted to make sure that you understand that that this is a viable use case and re-introduce this as part of the roadmap and not permanently remove this from the scope of the project. |
ACK. That was always part of the plan :)
|
In #702 we discussed the possibility of removing support for Windows. This is primarily for two reasons:
Number 1 can eventually be addressed when there is better support for
UnixStream
andUnixListener
for windows. UDS support does exist in Windows itself, but because Rust supports versions of Windows older than Windows 10, they don't have those structs in Windows.So if we choose to go this way, there will be two important details. First, this is a major breaking change. This is acceptable under the guidelines we sent out for the
alpha
series releases of 1.0, but we will need to communicate this well. Second, we will need to create an issue for adding Windows support back in.If you are a Krustlet (or
kubelet
crate) user doing something on Windows PLEASE LET US KNOW. The last thing we want to do is unintentionally screw things up for someone doing something real with WindowsThe text was updated successfully, but these errors were encountered: