-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
no such package '@com_github_axboe_liburing//' building Envoy docker image #33862
Comments
cc @soulxu |
see below |
actually - i was thinking it was this PR that had landed - #33833 not sure, but i think this may be related to the workaround i added for one of the tests breaking |
either way - im reasonably confident that #33833 will fix this |
@isaaccarrington i have landed the likely fix if you can confirm we can close this ticket |
Thanks @phlax but unfortunately the same error
I actually cloned the repo fresh
and deleted |
hmm, i see, thanks for checking thinking more - i think that its not arm-specific as we run that command in our arm ci - so im thinking its more to do with mac i dont have a mac to test with but i think there is some parts of the code that require would be happy to review (insofar as i can) any PRs to fix that if you got chance to test any of the release branches that would be helpful as we might be able to ~bisect the change ... and backport any fixes |
not sure whether this PR is going to help #33107 |
emm...no, you failed on another test '//test/common/io:io_uring_worker_factory_impl_test', but I think you can doing something similar to this one https://github.com/envoyproxy/envoy/pull/33107/files#diff-8e3c597bf96603a6c558022b88c5339e40dcb2da67375e7617db1d614f3d6226R438 |
Commit id being built - 35d00a5
Command run
./ci/run_envoy_docker.sh './ci/do_ci.sh release'
Error output
Output of
bazel version
Machine details:
MacBook Pro
14 inch, 2021
Chip: Apple M1 Pro
MacOS: Sonoma 14.4.1
The text was updated successfully, but these errors were encountered: