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

Add an option to use contracts on illumos #1371

Open
sunshowers opened this issue Mar 11, 2024 · 0 comments
Open

Add an option to use contracts on illumos #1371

sunshowers opened this issue Mar 11, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@sunshowers
Copy link
Member

Seems like the better way to do it, and depending on how things go we may also want to make it the default.

From @jclulow:

I think contracts is probably what you want, generally; it's much easier to make sure something doesn't escape, and you can listen to events around processes dumping core, etc

buildomat currently does a very early draft implementation of using contracts to run what we're told to run by the user; e.g., https://github.com/oxidecomputer/buildomat/blob/main/agent/src/exec.rs#L398-L416

There are library interfaces and so on but we're just using ctrun(1) there today because it was expedient

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant