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

[Bug]: bazel-diff freezes if aspect CLI is added #776

Open
blockjon-dd opened this issue Nov 20, 2024 · 2 comments
Open

[Bug]: bazel-diff freezes if aspect CLI is added #776

blockjon-dd opened this issue Nov 20, 2024 · 2 comments
Labels
bug Something isn't working

Comments

@blockjon-dd
Copy link

blockjon-dd commented Nov 20, 2024

What happened?

I added .bazeliskrc with the following lines:

BAZELISK_BASE_URL=https://static.aspect.build/aspect
USE_BAZEL_VERSION=aspect/2024.42.3

Whereas bazel-diff works just fine and runs sub second normally, with aspect-cli, bazel-diff just freezes.

Version

Development (host) and target OS/architectures:

Output of bazel --version:
aspect 2024.42.3-e256e0fbf

Version of the Aspect rules, or other relevant rules from your
WORKSPACE or MODULE.bazel file:
bazel_dep(name = "aspect_bazel_lib", version = "2.9.4")
bazel-diff 8.1.4

Language(s) and/or frameworks involved:
n/a

How to reproduce

I'm running a command like this:

bazel run //:bazel-diff generate-hashes -- --workspacePath="$(pwd)" -b "$bazel_path" $current_hashes_json


### Any other information?

_No response_
@blockjon-dd
Copy link
Author

blockjon-dd commented Nov 20, 2024

Not sure if this is helpful, but here's what ps aux gives me:

dev      27912  0.0  0.0 1229864 8080 pts/1    Sl+  11:43   0:00 bazel run //:bazel-diff generate-hashes -- --workspaceP

then here's what i get with an strace:

$ strace -p 27912
strace: Process 27912 attached
futex(0x582500, FUTEX_WAIT_PRIVATE, 0, NULL

@alexeagle
Copy link
Member

alexeagle commented Dec 17, 2024

That's not too surprising to me. While we are investigating, I'd suggest just adding environment variables for the bazel-diff command to turn off Aspect, like BAZELISK_BASE_URL= USE_BAZEL_VERSION= bazel run ...

@alexeagle alexeagle removed the untriaged Requires traige label Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants