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

Adds lage-server binary - buildxl breakaway process part 1 #784

Merged
merged 6 commits into from
Sep 13, 2024
Merged

Conversation

kenotron
Copy link
Member

In this change, we add a lage-server binary to lage. It is a new command that work like this:

  1. lage-server launches cli/server.js
  2. the tool will find any existing servers - if it doesn't find it, it'll create one given listening host & port
  3. the tool will then execute what was given to lage-server (much like lage run)
  4. it will stay in the background for some period of time as specified by --timeout (in seconds)
  5. any lage-server run will ping for the server... if it finds a server, it'll actually act as a client passing along the execution args to the server.
  6. a running server in the background would handle these without worrying about target graph dependencies

Part 2 will include these:

  1. a way for the tasks to report which files were touched (read / write, etc.)
  2. a way for the client to then probe / touch the files, so BuildXL integration can register those as simulated reads and writes... this helps with BXL's sandbox -> hash calculations for the caching mechanism

Part 3 will include these:

  1. modify the info command to use "lage exec" or "lage-server" where appropriate

@kenotron kenotron merged commit 924b93e into master Sep 13, 2024
3 checks passed
@kenotron kenotron deleted the server branch September 13, 2024 18:01
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.

2 participants