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 a ca-derivations required machine feature #4905

Merged
merged 1 commit into from
Jun 23, 2021

Conversation

thufschmitt
Copy link
Member

@thufschmitt thufschmitt commented Jun 11, 2021

Make ca-derivations require a ca-derivations machine feature, and
ca-aware builders expose it.

That way, a network of builders can mix ca-aware and non-ca-aware
machines, and the scheduler will send them in the right place.

Fix #4755

Make ca-derivations require a `ca-derivations` machine feature, and
ca-aware builders expose it.

That way, a network of builders can mix ca-aware and non-ca-aware
machines, and the scheduler will send them in the right place.
@thufschmitt thufschmitt added the ca-derivations Derivations with content addressed outputs label Jun 11, 2021
@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/tweag-nix-dev-update-14/13754/1

@edolstra edolstra merged commit d9a43d3 into master Jun 23, 2021
@edolstra edolstra deleted the ca-derivations-machine-feature branch June 23, 2021 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ca-derivations Derivations with content addressed outputs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow mixing ca-enabled and non-ca-enabled builders
4 participants