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 EngFlow to list of RE servers #1847

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

plaindocs
Copy link

Hey folks,

Disclaimer: I'm an EngFlow employee

I found this in my travels and thought I'd add to the list.

@nanonyme
Copy link
Contributor

Has it been tested with this?

@plaindocs
Copy link
Author

I believe so, but don't have a link to proof.

It's mentioned on the main website https://www.engflow.com/product/remoteExecution?r=menu****

image

I can ask around if you need something more specifc?

@nanonyme
Copy link
Contributor

nanonyme commented Jul 3, 2023

It's just that simply "Supports Remote Execution API's" has historically not been enough to mean it actually works due to various implementation details. Afaik there is no automated end-to-end testing in context of this project with EngFlow (assumably due to lack of testing cluster) so I was asking if that had been done by someone manually.

@abderrahim
Copy link
Contributor

I can ask around if you need something more specifc?

I think the main thing buildstream needs is the ability to use the input root as the filesystem root. Like the chrootIntoInputRoot option of buildbarn.

Then some testing should indeed be in order just to validate that it indeed works.

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

Successfully merging this pull request may close these issues.

3 participants