[rb] streamline tests on github actions runners#17550
Draft
titusfortner wants to merge 9 commits into
Draft
Conversation
selenium-ci
added
C-rb
Contributor
Review Summary by QodoStreamline Ruby CI tests on GitHub Actions runners
WalkthroughsDescription• Streamline Ruby CI by passing affected targets from main workflow • Reduce unit test matrix by relying on OS-specific integration tests • Replace commit message/PR title parsing with target-based filtering • Consolidate browser-specific tests into unified OS test matrix • Update test tags from no-sandbox to semantic labels (unit, lint) Diagramflowchart LR
A["ci.yml<br/>read-targets"] -->|"rb_targets"| B["ci-ruby.yml<br/>workflow_call"]
B --> C["unit-tests<br/>reduced matrix"]
B --> D["os-tests<br/>ubuntu/windows/macos"]
D --> E["browser-specific<br/>tag filters"]
F["rb/spec/tests.bzl<br/>rb_unit_test"] -->|"tags: unit"| C
G["rb/BUILD.bazel<br/>lint"] -->|"tags: lint"| B
File Changes1. .github/workflows/ci-ruby.yml
|
Contributor
Code Review by Qodo
1. Invalid matrix.tag-filters access
|
![qodo-code-review[bot]](https://avatars.githubusercontent.com/in/484649?s=60&v=4){kind=small}
Contributor
|
Persistent review updated to latest commit fa50a25 |
Contributor
There was a problem hiding this comment.
Pull request overview
Streamlines Ruby CI on GitHub Actions runners by switching Ruby workflow execution to be driven by the Bazel “affected targets” list, and consolidating more signal into OS-specific test jobs (instead of a broader unit-test OS matrix).
Changes:
- Pass Ruby affected targets from
ci.ymlintoci-ruby.ymland gate Ruby CI on those targets being present. - Replace the prior Ruby CI job layout with OS-specific test runs using Bazel tag filters, plus a smaller Ruby interpreter unit-test matrix.
- Update Bazel tags for Ruby unit and lint targets to support the new filtering approach.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
rb/spec/tests.bzl |
Retags Ruby unit test targets to support CI tag filtering. |
rb/BUILD.bazel |
Retags the Ruby lint target to be filterable via Bazel tag filters. |
.skipped-tests |
Removes a previously skipped Ruby BiDi test label from the skip list. |
.github/workflows/ci.yml |
Emits Ruby affected targets (rb_targets) and passes them into the Ruby reusable workflow. |
.github/workflows/ci-ruby.yml |
Restructures Ruby CI to run OS-specific Bazel tests over affected targets with tag filters and a reduced unit-test matrix. |
Contributor
|
Persistent review updated to latest commit 2f0d00f |
Contributor
|
Persistent review updated to latest commit 035aaf6 |
Contributor
|
Persistent review updated to latest commit 7e9d57a |
Contributor
|
Persistent review updated to latest commit 6c8325e |
Contributor
|
Persistent review updated to latest commit 2f4b024 |
Contributor
|
Persistent review updated to latest commit e6d425f |
Contributor
|
Persistent review updated to latest commit 6d6cb6f |
Contributor
|
Persistent review updated to latest commit f0a30f7 |
Contributor
|
Persistent review updated to latest commit 02d2d5c |
titusfortner
force-pushed
the
ci-ruby
branch
3 times, most recently
from
c9df9f6 to
f05ec3d
Compare
5 tasks
titusfortner
force-pushed
the
ci-ruby
branch
3 times, most recently
from
0ecfe6a to
31fff07
Compare
Contributor
CI Feedback 🧐A test triggered by this PR failed. Here is an AI-generated analysis of the failure:
|
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
🔗 Related Issues
Ruby implementation of #17539
Supersedes #16813
💥 What does this PR do?
[rb]to decide what to run for a PR, everything is gated off affected targets evaluated🔧 Implementation Notes
These OS tests will include multiple browsers, but need to specify browser name as safari to get safaridriver setup, and browser name can't be blank so I used true
🤖 AI assistance
💡 Additional Considerations
Currently running local and remote, considering limiting it to just local, especially for safari. I'll time the runs to compare