Skip to content

chore: Updated WRENCH_LOCALAPV_URL for APV jobs - #4201

Merged
michalChrobot merged 1 commit into
develop-3.x.xfrom
fix/apv-failures
Oct 20, 2026
Merged

michalChrobot merged 1 commit into
develop-3.x.xfrom
fix/apv-failures

Conversation

@michalChrobot

@michalChrobot michalChrobot commented Oct 20, 2026 •

Copy link
Copy Markdown
Member

Purpose of this PR

PR to fix failing APV tests (see the failing jobs slack channel). Issue is fixed directly in Wrench (LINK) and I included temp version of new Wrench in APV references. You can see that the job is passing now properly

Plan would be to land this as it is for now (manual version override) and this will be overridden when new official Wrench version is released

N/A

Documentation

N/A

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

APV jobs run green

Up-port

N/A

Backports

#4204

@michalChrobot michalChrobot changed the title Updated WRENCH_LOCALAPV_URL for APV jobs ci: Updated WRENCH_LOCALAPV_URL for APV jobs Oct 20, 2026
@michalChrobot
michalChrobot marked this pull request as ready for review October 20, 2026 13:07

@u-pr u-pr Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

💡 Harness Review

LGTM

The change consistently updates the local APV archive across both editor versions and all three platforms.

Reviewed commit 3cc9d05

🤖 Helpful? 👍/👎

UNITY_LICENSING_SERVER_TOOLSET: pro
UPMPVP_CONTEXT_WRENCH: 3.11.5.0
WRENCH_LOCALAPV_URL: https://artifactory.prd.it.unity3d.com/artifactory/stevedore-unity-internal/wrench-localapv/1-3-19_94a756eb601d84c127a989866172fdd530ec62aa5d9611a9c9898a625926ec62.zip
WRENCH_LOCALAPV_URL: https://artifactory.prd.it.unity3d.com/artifactory/stevedore-unity-internal/wrench-localapv/1-3-24_1493dea1e0eb932938365a4a53ec2091ec007ceec57eea844d15a058eddedc9f.zip

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P2

The next CI regeneration before the official Wrench update will undo this fix and cause the generated-job check on release/* branches to fail. RecipeEngine.Modules.Wrench remains pinned to 3.11.5 in NGO.Cookbook.csproj. Program.Main regenerates the jobs without applying this temporary URL override. test_-_wrench_jobs_up_to_date runs on release-branch pushes and rejects any regenerated differences under .yamato/wrench. Apply the temporary URL override during recipe generation. Commit the resulting YAML so regeneration preserves the fix.

🤖 Helpful? 👍/👎

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yes, it's a known thing, I will be able to update Wrench to that latest version before Friday anyway

@EmandM EmandM changed the title ci: Updated WRENCH_LOCALAPV_URL for APV jobs chore: Updated WRENCH_LOCALAPV_URL for APV jobs Oct 20, 2026
@michalChrobot
michalChrobot merged commit 39f1f1f into develop-3.x.x Oct 20, 2026
19 of 25 checks passed
@michalChrobot
michalChrobot deleted the fix/apv-failures branch October 20, 2026 21:28
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.

2 participants