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

chore: switch to Aspect Workflows for RBE #612

Merged
merged 1 commit into from
May 29, 2024
Merged

Conversation

gregmagolan
Copy link
Member

@gregmagolan gregmagolan commented May 24, 2024

@gregmagolan gregmagolan mentioned this pull request May 24, 2024
Copy link

aspect-workflows bot commented May 24, 2024

Test

All tests were cache hits

77 tests (100.0%) were fully cached saving 11s.


Buildifier      Format

@gregmagolan gregmagolan force-pushed the workflows_rbe branch 2 times, most recently from 8fd1f6e to 49663b4 Compare May 29, 2024 21:53
@gregmagolan gregmagolan marked this pull request as ready for review May 29, 2024 21:58
@gregmagolan gregmagolan force-pushed the workflows_rbe branch 2 times, most recently from 9fc4603 to 9c374f6 Compare May 29, 2024 21:59
build --host_platform=@aspect_bazel_lib//platforms:x86_64_linux_remote
build --extra_execution_platforms=@aspect_bazel_lib//platforms:x86_64_linux_remote
build --remote_executor=unix:///mnt/ephemeral/buildbarn/.cache/bb_clientd/grpc
build --jobs=8 # current max scaling for the OSS RBE cluster
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's the parallelism on each worker? Is this too low?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gregmagolan gregmagolan merged commit db924be into main May 29, 2024
23 checks passed
@gregmagolan gregmagolan deleted the workflows_rbe branch May 29, 2024 22:21
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