Skip to content

[build] generate compact execution log - #17977

Merged
shs96c merged 1 commit into
SeleniumHQ:trunkfrom
iamricard:iamricard/produce-execlog
Sep 3, 2026
Merged

[build] generate compact execution log#17977
shs96c merged 1 commit into
SeleniumHQ:trunkfrom
iamricard:iamricard/produce-execlog

Conversation

@iamricard

@iamricard iamricard commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

🔗 Related Issues

N/A

💥 What does this PR do?

This patch modifies .bazelrc to produce a compact execution log for every build. It also adds the file to .gitignore because it shouldn't be checked in.

🔧 Implementation Notes

Bazel can produce a log file which details action execution for the given build. Its initial version (pre-7.x) was known to be far too large to parse on most cases and it also incurred non-trivial performance penalties (bazelbuild/bazel#23319). The new format (compact) was first released in bazel 7.x and stabilized in bazel 8.x. Since then a number of tools have emerged to analyzed this artifact and, usually, services that offer RE/RC/BES can make use of the execution log to provide more detailed insights on build execution.

🤖 AI assistance

None used.

  • No substantial AI assistance used
  • AI assisted (complete below)
    • Tool(s):
    • What was generated:
    • I reviewed all AI output and can explain the change

💡 Additional Considerations

N/A

🔄 Types of changes

  • None of the listed types apply

Context:
Bazel can produce a log file which details action execution for the given
build. Its initial version (pre-7.x) was known to be far too large to
parse on most cases and it also incurred non-trivial performance
penalties (bazelbuild/bazel#23319). The new
format (compact) was first released in bazel 7.x and stabilized in bazel
8.x. Since then a number of tools have emerged to analyzed this artifact,
and, usually, services that use RE/RC/BES can make use of the execution
log to provide more detailed insights on build execution.

This patch modifies `.bazelrc` to produce a compact execution log for
every build. It also adds the file to `.gitignore` because it shouldn't
be checked in.
@qodo-code-review

Copy link
Copy Markdown
Contributor

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@CLAassistant

CLAassistant commented Sep 3, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@selenium-ci selenium-ci added the B-build Includes scripting, bazel and CI integrations label Sep 3, 2026

@shs96c shs96c left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@shs96c
shs96c merged commit c71be6b into SeleniumHQ:trunk Sep 3, 2026
22 checks passed
This was referenced Sep 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

B-build Includes scripting, bazel and CI integrations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants