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

[Bazel CI] aspect_rules_js no such file error or directory with Bazel@HEAD in CI #421

Closed
sgowroji opened this issue Mar 11, 2024 · 13 comments · Fixed by #445
Closed

[Bazel CI] aspect_rules_js no such file error or directory with Bazel@HEAD in CI #421

sgowroji opened this issue Mar 11, 2024 · 13 comments · Fixed by #445

Comments

@sgowroji
Copy link
Member

CI: https://buildkite.com/bazel/bazel-at-head-plus-downstream/builds/3711#018e2bad-eced-489f-aa96-d3974826f324

Platform: MacOS

Logs:

ERROR: �[0m/Users/buildkite/builds/bk-imacpro-17/bazel-org-repo-root/examples/frontend/BUILD.bazel:10:22: Copying directory aspect_rules_js~~npm~npm__typescript__4.9.3/package [for tool] failed: Exec failed due to IOException: /private/var/tmp/_bazel_buildkite/234bc2f97ee57c6cd035cc0acf2083ba/execroot/_main/external/aspect_rules_js~~npm~npm__typescript__4.9.3/package (No such file or directory)
ERROR: �[0m/Users/buildkite/builds/bk-imacpro-17/bazel-org-repo-root/examples/frontend/BUILD.bazel:10:22: Copying directory aspect_rules_js~~npm~npm__he__1.2.0/package [for tool] failed: Exec failed due to IOException: /private/var/tmp/_bazel_buildkite/234bc2f97ee57c6cd035cc0acf2083ba/execroot/_main/external/aspect_rules_js~~npm~npm__he__1.2.0/package (No such file or directory)
ERROR: �[0m/Users/buildkite/builds/bk-imacpro-17/bazel-org-repo-root/examples/frontend/vue/libraries/simple/BUILD.bazel:55:12 Copying files to directory vue/libraries/simple/simple failed: Exec failed due to IOException: /private/var/tmp/_bazel_buildkite/234bc2f97ee57c6cd035cc0acf2083ba/execroot/_main/external/aspect_rules_js~~npm~npm__typescript__4.9.3/package (No such file or directory)
ERROR: �Build did NOT complete successfully

Culprit:

Steps:

git clone https://github.com/bazelbuild/examples
git reset 7285d2e400b04f81df4d9edcad5a8fc8d43c607c  --hard
export USE_BAZEL_VERSION=3697397569e25a29a828fb46b26302876d372e26
bazel build ... 

CC Greenteam @Wyverald

@sgowroji sgowroji changed the title [Bazel CI] [Logs] Exec failed due to IOException on Linux/macOS [Bazel CI] aspect_rules_js no such file error or directory with Bazel@HEAD in CI Mar 11, 2024
@Wyverald
Copy link
Member

The same failure happens on release Bazel (https://buildkite.com/bazel/bazel-bazel-examples/builds/2766#018e2acf-d2b3-4ad7-88ae-e73c3f46c796), so I think we should just move this issue to the Bazel examples repo? also cc @alexeagle

@Wyverald
Copy link
Member

sorry, I didn't realize we were already in the examples repo...

@meteorcloudy
Copy link
Member

sorry, I didn't realize we were already in the examples repo...

Np! ;D

@alexeagle Any idea on how to address this flaky failure?

@alexeagle
Copy link
Collaborator

@gregmagolan and @tjgq have had an email thread about it. I think it's a matter of finding some time for it. Our clients observe it sometimes too, we are just having trouble understanding the cause.

@sgowroji
Copy link
Member Author

Hi Team, Any Update/progress on the above issue ?

@gregmagolan
Copy link

gregmagolan commented Apr 19, 2024

1.40.0 rules_js release has a fix/work-around for the issue for most cases. See the PR summary of aspect-build/rules_js#1538 for more info.

@fmeum
Copy link

fmeum commented Apr 22, 2024

I'm pretty sure this is caused by a race in skymeld that isn't related to caching and can't be worked around reliably: bazelbuild/bazel#22073

alexeagle added a commit that referenced this issue Apr 22, 2024
@fmeum
Copy link

fmeum commented Apr 22, 2024

@alexeagle Did you mean to close this with #445?

@alexeagle
Copy link
Collaborator

Yes, that change includes the version bump for rules_js that @gregmagolan mentioned 4d ago

@fmeum
Copy link

fmeum commented Apr 22, 2024

Yes, that change includes the version bump for rules_js that @gregmagolan mentioned 4d ago

Based on bazelbuild/bazel#22073, I don't think that this can be worked around reliably on the ruleset side, it appears to be an honest race in Skymeld that can affect virtually any external repo. I would suggest keeping this open until it doesn't reproduce in CI anymore.

@alexeagle alexeagle reopened this Apr 22, 2024
@alexeagle
Copy link
Collaborator

Okay sure, but does that "honest race in Skymeld" have an owner? I don't think Greg or I are planning to do more here, and I don't love having an issue "rules_js is broken" on the examples repo.

@fmeum
Copy link

fmeum commented Apr 22, 2024

As far as I know @joeleba is actively working on bazelbuild/bazel#22073.

@sgowroji
Copy link
Member Author

sgowroji commented May 7, 2024

Examples CI is Green now. We can close this issue. Thank you all for the great support.

@sgowroji sgowroji closed this as completed May 7, 2024
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 a pull request may close this issue.

6 participants