Skip to content

Bump bazel version to 6.1.1 #17664

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

Closed
wants to merge 2 commits into from

Conversation

drewmacrae
Copy link
Contributor

This is a WIP, it will probably need to be announced at some point to get people to update appropriately.

@timothytrippel
Copy link
Contributor

Unless there is a critical work stream this is blocking, I would highly suggest we postpone doing this for a couple months. This could impact the infrastructure that runs our nightlies, and may require medium -- significant effort to get things working again.

CC: @moidx

@moidx moidx self-requested a review March 23, 2023 15:42
@moidx
Copy link
Contributor

moidx commented Mar 23, 2023

Agreed. At this moment we don't have enough bandwidth to support updates to nightlies and physical design workflows. We can pick this up again later once we have a separate branch for earlgrey.

dmcardle and others added 2 commits March 23, 2023 20:50
This prepares for better FPGA test management and remote execution.
Signed-off-by: Drew Macrae <[email protected]>
@drewmacrae
Copy link
Contributor Author

For reference this blocks:

  • remote execution which needs 6.0.0+ and could be useful to developing CI and other test frameworks that deliver resources to teams instead of individuals and on a per-action basis instead of per-test job (which should help to cache CI runs more flexibly)
  • better FPGA allocation which needs 6.1.0+ and runs the first FPGA test as soon as it's ready which should improve CI performance and performance on everyone's workstations.
  • There are also improvements to dependency management that I haven't yet looked into.

In all, the better test performance will offer a nice ROI, but it's only critical in the sense that it gives us better tools to address some of the performance issues and dependency management it looks like we're struggling with at present.

@drewmacrae drewmacrae marked this pull request as ready for review June 2, 2023 14:21
@moidx moidx closed this Jul 25, 2023
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.

4 participants