Skip to content

Comments

Remove --rbe from the build_wheel.yml#264

Open
Ruturaj4 wants to merge 2 commits intomasterfrom
ci_remove_rbe
Open

Remove --rbe from the build_wheel.yml#264
Ruturaj4 wants to merge 2 commits intomasterfrom
ci_remove_rbe

Conversation

@Ruturaj4
Copy link
Contributor

Motivation

Technical Details

Test Plan

Test Result

Submission Checklist

--jax-source-dir="./jax" \
dist_wheels \
--rbe
dist_wheels
Copy link
Collaborator

Choose a reason for hiding this comment

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

We should not be removing this. Rather, we should be fixing the build scripts so that RBE builds will work for PRs.

include:
- test-command: "python jax_rocm_plugin/build/rocm/run_single_gpu.py -c -s"
runner-label: '["linux-x86-64-1gpu-amd"]'
test-id: "single"
Copy link
Collaborator

Choose a reason for hiding this comment

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

These are unrelated to RBE, and these changes should be in a separate PR

@psanal35
Copy link
Contributor

This PR seems to be no longer needed. Should we close it?

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