Skip to content

chore: [UpPort] Reduce the number of jobs running on PR triggers #3545

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

Merged
merged 8 commits into from
Jul 15, 2025

Conversation

EmandM
Copy link
Collaborator

@EmandM EmandM commented Jul 11, 2025

Reduce the number of jobs that are running on each PR from 43 jobs to 26.

Changelog

  • Changed: Updated the automated test configuration.

Testing and Documentation

Backport

This is an up-port of #3543

@michalChrobot
Copy link
Collaborator

Same comment as for original PR

Really nice improvement in terms of numbers! Thanks for doing this!

What we are doing in essence (is anyone else will stumble upon this PR) is

  • Migrating slowly to 2022.3 editor since 2021.3 will stop being supported in +- October. For now those tests will still run nightly so it's still covered but should start getting less attention
  • Running project/package tests on latest and oldest supported editor per each platform is replaced by specifically targeting combinations which should get us the same coverage (worst case all tests still run nightly but won't block PRs). The distribution times got quite high those days so this should help in reducing waiting time on PRs

@michalChrobot
Copy link
Collaborator

As for my commits

  1. I noticed that we progressed in editor versions so I used this occasion to add 6000.2 to the list of supported editors. Notice that this will only influence nightly/weekly tests and nothing PR related
  2. I noticed that in fix: PVP issues related to assemblies, file names, and other non-xml-API related #3444 we removed testproject-tools-integration project but since the Tools PR that aims to move the project there is still pending I re-added it (since your test definition also required it to be present)

EmandM added a commit that referenced this pull request Jul 14, 2025
Reduce the number of jobs that are running on each PR from 43 jobs to
26.

## Changelog

- Changed: Updated the automated test configuration.

## Testing and Documentation


## Backport

<!-- If this is a backport:
 - Add the following to the PR title: "\[Backport\] ..." .
 - Link to the original PR.
If this needs a backport - state this here
If a backport is not needed please provide the reason why.
If the "Backports" section is not present it will lead to a CI test
failure.
-->

Up ported by #3545

---------

Co-authored-by: michalChrobot <[email protected]>
… been moved to the tools repo and doesn't exist on the 2.x branch
@EmandM
Copy link
Collaborator Author

EmandM commented Jul 14, 2025

I noticed that in fix: PVP issues related to assemblies, file names, and other non-xml-API related #3444 we removed testproject-tools-integration project but since the Tools PR that aims to move the project there is still pending I re-added it (since your test definition also required it to be present)

This was my bad. I blindly copied the test definitions over from #3543 and completely missed that I'd also copied the tools integration tests. Seeing as we did remove that test project in #3444 yamato was right to be erroring out. Should be good now.

@EmandM EmandM enabled auto-merge (squash) July 14, 2025 18:16
@EmandM EmandM merged commit 1de6174 into develop-2.0.0 Jul 15, 2025
24 checks passed
@EmandM EmandM deleted the chore/2.x/update_yamato_pr_trigger branch July 15, 2025 00:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants