chore: Up-port from develop-2.0.0 from week 32/33 - #4141
Conversation
* fix: GC allocations * Add message receive allocations test * Revert the fix to check ILPP build fails the test * Revert forced failure * Remove gc checks on NetworkListTests
* Re-enabled ps5 and webgl * ci: Add WebGL PlayMode test job (MTT-15569) Split the WebGL CI into a build phase and a run phase. webgl-build.yml now exposes the built player as a dedicated 'players' artifact, and a new webgl-test.yml run job consumes it and executes the PlayMode tests inside Firefox on a GPU-backed Ubuntu agent (Unity::VM::GPU, rtx2080). The Firefox browser flags are passed explicitly so UTR does not attempt to download the browser from Stevedore. The run job is wired into the Nightly and Weekly (QV) triggers only, not into PR triggers, since WebGL failures are infrequent and the build is slow. Runtime RuntimePlatform.WebGLPlayer exclusions are intentionally deferred to be added reactively after the first real CI run. * corrected images * disabled webgl test * adjustments * Removed WebGL changes * reverted change * added diagnostic for switch * corrected Switch sdk version * temporarily disabled switch * corrected ticket description
Changde users exclusion to pattern exclusion in pr description check
* Updated changelog and package version for Netcode in anticipation of v2.13.1 release * typo in build automation * trunk shadow changes: com.unity.netcode.gameobjects [skip ci] * Revert CHANGELOG changes * Added UNITY_TEST_FRAMEWORK_1_7_OR_NEWER guard and UnityCoreClrExplicitDisabledAttributeShim.cs to make attribute usage possible * corrected svc bot exclusion --------- Co-authored-by: netcode-automation <svc-netcode-sdk@unity3d.com> Co-authored-by: Michał Chrobot <michal.chrobot@unity3d.com> Co-authored-by: PETS automation <299490404+pets-svc[bot]@users.noreply.github.com> Co-authored-by: Emma <emma.mcmillan@unity3d.com>
Co-authored-by: unity-renovate[bot] <120015202+unity-renovate[bot]@users.noreply.github.com>
|
|
There was a problem hiding this comment.
💡 Harness Review
The allocation-focused runtime changes are generally coherent, but the new integration coverage is not excluded from a known failing CoreCLR configuration and two diagnostic/validation changes alter behavior incorrectly.
Reviewed commit c30cbc3
🤖 Helpful? 👍/👎
Codecov ReportAttention: Patch coverage is
@@ Coverage Diff @@
## develop-3.x.x #4141 +/- ##
=================================================
- Coverage 78.02% 78.00% -0.02%
=================================================
Files 153 153
Lines 26260 26264 +4
=================================================
- Hits 20489 20487 -2
- Misses 5771 5777 +6
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 1 file with indirect coverage changes
|
Purpose of this PR
PR that up-ports PRs that landed during last 2 weeks or otherwise since we did the last up-port.
Note that I won't "squash and merge" and I will rather merge them separately as then in the future it will be easier to reason about PRs history.
Also not all commits are ported because some of them were up-ported already (picture shows the range of commits I was evaluating)
Jira ticket
N/
Documentation
N/A
Testing & QA (How your changes can be verified during release Playtest)
Green CI
Up-port
This is an up-port
Backports
N/A