Skip to content

ci: Initial BuildAutomation module implementation - #4168

Merged
michalChrobot merged 3 commits into
develop-3.x.xfrom
build-automation/user-recipe-engine-module
Sep 25, 2026
Merged

michalChrobot merged 3 commits into
develop-3.x.xfrom
build-automation/user-recipe-engine-module

Conversation

@michalChrobot

@michalChrobot michalChrobot commented Sep 23, 2026 •

Copy link
Copy Markdown
Member

Purpose of this PR

This PR removes local implementation of Build Automation and replaces it with a RecipeEngine module that was created in order to consolidate and easily share this functionality

Jira ticket

Not really listed there but related to https://jira.unity3d.com/browse/MTT-15054

Documentation

N/A

Testing & QA (How your changes can be verified during release Playtest)

I will make the builds as usual to double check that everything works the same as before

Up-port

N/A

Backports

Done

@michalChrobot michalChrobot self-assigned this Sep 23, 2026
@michalChrobot michalChrobot changed the title Initial BuildAutomation module implementation ci: Initial BuildAutomation module implementation Sep 24, 2026
@michalChrobot
michalChrobot marked this pull request as ready for review September 24, 2026 10:49
@u-pr

u-pr Bot commented Sep 24, 2026

Copy link
Copy Markdown

⚠️ Some settings were ignored in '.pr_agent.toml (repository root)' repo settings

Check the configuration's TOML syntax, permitted settings, and size limit. See the error below.


Error message:
Ignored prohibited settings: github_app.handle_pr_actions. Permitted settings were applied.

Configuration content:
# This is the configuration for the u-pr bot
# https://github.cds.internal.unity3d.com/unity/u-pr
# For configuration of this file:
# https://developer.portal.internal.unity.com/catalog/default/component/u-pr/docs/configuration/automatic_runs/

[github_app]
handle_pr_actions = ['opened', 'ready_for_review']  # PR events that auto-run pr_commands
pr_commands       = ["/harness-review"]             # what runs on those events
reviewer_commands = ["/harness-review"]             # what runs when @u-pr is added as reviewer

@u-pr u-pr Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Harness Review

Needs changes

The generated build workflow is internally coherent, but existing automation does not yet fully account for its new outputs and contract.

P1

The reachable release-preparation flow still points every jobDefinition at the deleted .yamato/project-builders/project-builders.yml, so Step 3 gets non-success responses for all sample launches; the trigger helper only logs these failures and continues, allowing a release to proceed without playtest builds. Update ReleaseConfig to use the new file and lowercase job IDs, and migrate its platform/backend keys, editor matrix, and obsolete Burst option to the new contract.

Reviewed commit 9767c54

🤖 Helpful? 👍/👎

Comment thread Tools/CI/Program.cs
@michalChrobot
michalChrobot merged commit 341d9dc into develop-3.x.x Sep 25, 2026
12 of 13 checks passed
@michalChrobot
michalChrobot deleted the build-automation/user-recipe-engine-module branch September 25, 2026 08:33
michalChrobot added a commit that referenced this pull request Sep 25, 2026
* Initial BuildAutomation module implementation

* Renamed Wrench settings for beter readibility

* Updated MinSupportedEditorBySample values
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.

1 participant