[bazel] Update to Bazel 8 #16757
Draft
[bazel] Update to Bazel 8 #16757
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
selenium-ci
added
C-java
Member
|
Thank you, @shs96c for this code suggestion. The support packages contain example code that many users find helpful, but they do not necessarily represent After reviewing the change, unless it is a critical fix or a feature that is needed for Selenium We actively encourage people to add the wrapper and helper code that makes sense for them to their own frameworks. |
We see intermittent failures when builds fail on Windows when uploading
logs. This is because:
1. The **/*.log glob pattern in the "Upload test logs" step scans the
entire workspace, including bazel-out directories.
2. Bazel on Windows creates junction points for the `bazel-bin`,
`bazel-out`, `bazel-testlogs`, and so on directories.
3. The `upload-artifact` action has known issues with symlinks/junctions
on Windows - when it tries to `stat()` these paths, it can fail with
EPERM.
The intermittent nature is because:
1. The step only runs when tests fail.
2. The specific files/directories in bazel-out depend on what was built
in that run Windows file locking and junction handling can be
inconsistent.
Member
|
@shs96c should we remove |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This also updates all the bazel deps we have to their latest versions.