Skip to content

Add script that generates list of test targets - #4464

Merged
oxve merged 95 commits into
youtube:mainfrom
oxve:test-targets
Mar 14, 2025
Merged

oxve merged 95 commits into
youtube:mainfrom
oxve:test-targets

Conversation

@oxve

@oxve oxve commented Nov 21, 2024 •

Copy link
Copy Markdown
Member

Add script that calculates test targets based on gn deps is added to the build step. The output from this script is then passed through the build pipeline to each of the steps that use the list.

b/382508397

@oxve
oxve requested a review from a team as a code owner December 26, 2024 19:31
@oxve
oxve force-pushed the test-targets branch 17 times, most recently from a1f4e8d to dfe8727 Compare December 29, 2024 18:19
Comment thread .github/actions/build/action.yaml Resolved
@oxve
oxve marked this pull request as draft December 30, 2024 18:33
Comment thread cobalt/build/test_targets.py Resolved
@oxve
oxve merged commit 292be22 into youtube:main Mar 14, 2025
kaidokert added a commit to kaidokert/cobalt that referenced this pull request Mar 20, 2025
Partial revert of youtube#4464, as it currently doesn't reliably
find all changed tests.
Also - dynamic calculation should always be bypassed for post-submits

b/382508397
kaidokert added a commit to kaidokert/cobalt that referenced this pull request Mar 20, 2025
kaidokert added a commit to kaidokert/cobalt that referenced this pull request Mar 21, 2025
Partial revert of youtube#4464, as it currently doesn't reliably
find all changed tests.
Also - dynamic calculation should always be bypassed for post-submits

b/382508397
kaidokert added a commit to kaidokert/cobalt that referenced this pull request Mar 21, 2025
Partial revert of youtube#4464, as it currently doesn't reliably
find all changed tests.
Also - dynamic calculation should always be bypassed for post-submits

b/382508397
kaidokert added a commit that referenced this pull request Mar 21, 2025
Partial revert of #4464, as it currently doesn't reliably find all
changed tests.
Also - dynamic calculation should always be bypassed for post-submits

b/382508397
cobalt-github-releaser-bot pushed a commit that referenced this pull request Mar 21, 2025
Partial revert of #4464, as it currently doesn't reliably find all
changed tests.
Also - dynamic calculation should always be bypassed for post-submits

b/382508397

(cherry picked from commit a32c619)
kaidokert added a commit that referenced this pull request Mar 21, 2025
Partial revert of #4464, as it currently doesn't reliably find all
changed tests.
Also - dynamic calculation should always be bypassed for post-submits

b/382508397

(cherry picked from commit a32c619)
kaidokert added a commit that referenced this pull request Mar 21, 2025
Refer to the original PR: #5130

Partial revert of #4464, as it currently doesn't reliably find all
changed tests.
Also - dynamic calculation should always be bypassed for post-submits

b/382508397

Co-authored-by: Kaido Kert <[email protected]>
yell0wd0g added a commit that referenced this pull request Apr 12, 2025
This is a follow up to #4831 which landed with a TODO to move the newly
introduced `starboard_unittests` into a `if (current_toolchain ==
starboard_toolchain)` block, and let external users depend on a
`starboard_unittests_wrapper` instead.

#4831 landed unintendedly, so this CL follows up quickly; CI is likely
to fail while #4464 is being worked on.

Bug: b/384819454
cobalt-github-releaser-bot pushed a commit that referenced this pull request Apr 12, 2025
This is a follow up to #4831 which landed with a TODO to move the newly
introduced `starboard_unittests` into a `if (current_toolchain ==
starboard_toolchain)` block, and let external users depend on a
`starboard_unittests_wrapper` instead.

#4831 landed unintendedly, so this CL follows up quickly; CI is likely
to fail while #4464 is being worked on.

Bug: b/384819454
(cherry picked from commit bdaba2e)
cobalt-github-releaser-bot pushed a commit that referenced this pull request Apr 15, 2025
This is a follow up to #4831 which landed with a TODO to move the newly
introduced `starboard_unittests` into a `if (current_toolchain ==
starboard_toolchain)` block, and let external users depend on a
`starboard_unittests_wrapper` instead.

#4831 landed unintendedly, so this CL follows up quickly; CI is likely
to fail while #4464 is being worked on.

Bug: b/384819454
(cherry picked from commit bdaba2e)
yell0wd0g added a commit that referenced this pull request Apr 15, 2025
…5393)

Refer to the original PR: #5031

This is a follow up to #4831 which landed with a TODO to move the newly
introduced `starboard_unittests` into a `if (current_toolchain ==
starboard_toolchain)` block, and let external users depend on a
`starboard_unittests_wrapper` instead.

#4831 landed unintendedly, so this CL follows up quickly; CI is likely
to fail while #4464 is being worked on.

Bug: b/384819454

Co-authored-by: Miguel Casas <[email protected]>
kaidokert pushed a commit to kaidokert/cobalt that referenced this pull request Jun 15, 2025
Add script that calculates test targets based on gn deps is added to the
build step. The output from this script is then passed through the build
pipeline to each of the steps that use the list.

b/382508397
kaidokert added a commit to kaidokert/cobalt that referenced this pull request Jun 15, 2025
Partial revert of youtube#4464, as it currently doesn't reliably find all
changed tests.
Also - dynamic calculation should always be bypassed for post-submits

b/382508397
kaidokert pushed a commit to kaidokert/cobalt that referenced this pull request Jun 15, 2025
This is a follow up to youtube#4831 which landed with a TODO to move the newly
introduced `starboard_unittests` into a `if (current_toolchain ==
starboard_toolchain)` block, and let external users depend on a
`starboard_unittests_wrapper` instead.

youtube#4831 landed unintendedly, so this CL follows up quickly; CI is likely
to fail while youtube#4464 is being worked on.

Bug: b/384819454
briantting pushed a commit that referenced this pull request Jul 1, 2025
Add script that calculates test targets based on gn deps is added to the
build step. The output from this script is then passed through the build
pipeline to each of the steps that use the list.

b/382508397
original-hexsha: 292be22
briantting pushed a commit that referenced this pull request Jul 1, 2025
Partial revert of #4464, as it currently doesn't reliably find all
changed tests.
Also - dynamic calculation should always be bypassed for post-submits

b/382508397
original-hexsha: a32c619
briantting pushed a commit that referenced this pull request Jul 1, 2025
This is a follow up to #4831 which landed with a TODO to move the newly
introduced `starboard_unittests` into a `if (current_toolchain ==
starboard_toolchain)` block, and let external users depend on a
`starboard_unittests_wrapper` instead.

#4831 landed unintendedly, so this CL follows up quickly; CI is likely
to fail while #4464 is being worked on.

Bug: b/384819454
original-hexsha: bdaba2e
briantting pushed a commit that referenced this pull request Jul 11, 2025
Add script that calculates test targets based on gn deps is added to the
build step. The output from this script is then passed through the build
pipeline to each of the steps that use the list.

b/382508397
original-hexsha: 292be22
briantting pushed a commit that referenced this pull request Jul 11, 2025
Partial revert of #4464, as it currently doesn't reliably find all
changed tests.
Also - dynamic calculation should always be bypassed for post-submits

b/382508397
original-hexsha: a32c619
briantting pushed a commit that referenced this pull request Jul 11, 2025
This is a follow up to #4831 which landed with a TODO to move the newly
introduced `starboard_unittests` into a `if (current_toolchain ==
starboard_toolchain)` block, and let external users depend on a
`starboard_unittests_wrapper` instead.

#4831 landed unintendedly, so this CL follows up quickly; CI is likely
to fail while #4464 is being worked on.

Bug: b/384819454
original-hexsha: bdaba2e
sideb0ard pushed a commit to sideb0ard/cobalt that referenced this pull request Jul 25, 2025
Add script that calculates test targets based on gn deps is added to the
build step. The output from this script is then passed through the build
pipeline to each of the steps that use the list.

b/382508397
original-hexsha: 292be22
sideb0ard pushed a commit to sideb0ard/cobalt that referenced this pull request Jul 25, 2025
Partial revert of youtube#4464, as it currently doesn't reliably find all
changed tests.
Also - dynamic calculation should always be bypassed for post-submits

b/382508397
original-hexsha: a32c619
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

on_device Trigger on-device tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants