-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
windows: support sandboxing [blocking #5640] #5136
Comments
I'm sure you're aware, but Windows has something called the Host Compute Service which is used to implement containers for things like Docker. This blog post has the high level picture and they link to wrappers+sample code for C# and Go (which looks more complete and is what Docker actually uses.) Maybe something from there would be useful? |
I wasn't aware, thanks for the info! Could be useful indeed. Though supporting the Docker-based sandbox strategy on Windows may be more convenient than implementing native sandboxing. |
Cool! Makes sense. |
FYI: https://github.com/Microsoft/BuildXL has an implementation of Windows Sandboxing |
@ilya-klyuchnikov -- Thanks for the heads-up! Very interesting, I'll take a look and try to build it. |
@laszlocsomor could you please add a priority to this issue? |
Setting priority to p3 -- I don't expect much focus on this before Bazel 1.0 (though this is not ruled out either). |
Since Windows builds aren’t sandboxed properly (see bazelbuild/bazel#5136), concurrent compilations interact with the same tmp_db directory which leads to flaky build failures.
Since Windows builds aren’t sandboxed properly (see bazelbuild/bazel#5136), concurrent compilations interact with the same tmp_db directory which leads to flaky build failures.
Since Windows builds aren’t sandboxed properly (see bazelbuild/bazel#5136), concurrent compilations interact with the same tmp_db directory which leads to flaky build failures.
Since Windows builds aren’t sandboxed properly (see bazelbuild/bazel#5136), concurrent compilations interact with the same tmp_db directory which leads to flaky build failures.
Since Windows builds aren’t sandboxed properly (see bazelbuild/bazel#5136), concurrent compilations interact with the same tmp_db directory which leads to flaky build failures.
Since Windows builds aren’t sandboxed properly (see bazelbuild/bazel#5136), concurrent compilations interact with the same tmp_db directory which leads to flaky build failures.
@rongjiecomputer is working on this feature \o/ |
I wonder if you looked at FUSE implementations for Windows, such as https://dokan-dev.github.io/ or http://www.secfs.net/winfsp/ ? As Bazel already makes use of sandboxfs for Linux and Mac it may be beneficial to utilize as similar tech as possible on Windows. |
@laszlocsomor @rongjiecomputer What's the status of this? Is the current progress documented somewhere? |
@aherrmann : @rongjiecomputer implemented experimental sandbox support as part of Google Summer of Code 2019 (GSoC). All about the project is in a public Google doc: https://docs.google.com/document/d/1dbF5cGYJouGF5zd0GDE-rbUvkKvH36DnNdtEvE4yV6w/edit The Bazel team does not maintain this feature. We currently have no plans to work on it either. |
Thank you for contributing to the Bazel repository! This issue has been marked as stale since it has not had any activity in the last 2.5 years. It will be closed in the next 14 days unless any other activity occurs or one of the following labels is added: "not stale", "awaiting-bazeler". Please reach out to the triage team ( |
This issue has been automatically closed due to inactivity. If you're still interested in pursuing this, please reach out to the triage team ( |
Description of the problem / feature request:
This is an umbrella bug for the feature request of supporting sandboxing on Windows.
What operating system are you running Bazel on?
Windows 10
What's the output of
bazel info release
?release 0.13.0
Have you found anything relevant by searching the web?
Microsoft recently opensourced Detours, which could be useful: https://github.com/Microsoft/Detours
The text was updated successfully, but these errors were encountered: