Skip to content

Conversation

kivistikliis
Copy link
Contributor

@kivistikliis kivistikliis commented Sep 23, 2021


Purpose of this PR

Changes android build images to fix SDK error.


Testing status

ShaderGraph green
URP_2D green
URP_Foundation green
URP_Foundation XR Some failed tests
URP_Lighting green


Comments to reviewers

Reverted changes for all performance jobs, as they are trickier to fix and will be fixed in a separate PR not to block URP fixes. Keeping the jobs listed here though to refer while prepping the follow-up PR.
BoatAttack performance build editmode
BoatAttack performance playmode runtime
LostCrypt performance_playmode_runtime
LostCrypt performance_build_editmode
VV builtin performance_build_editmode
VV builtin performance_playmode_runtime
VV URP performance_build_editmode
VV URP performance_playmode_runtime

@github-actions github-actions bot added the sdet label Sep 23, 2021
- cd TestProjects/BoatAttack && unity-config project add dependency com.unity.addressables@1.16.7 --project-path .
- cd TestProjects/BoatAttack && unity-config project add dependency com.unity.scriptablebuildpipeline@1.11.2 --project-path .
- cd TestProjects/BoatAttack && unity-config project add dependency com.unity.test-framework@1.1.18 --project-path .
- cd TestProjects/BoatAttack && unity-config project add dependency com.unity.test-framework@1.1.29 --project-path .
Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks for this! 🤦🏻‍♂️ I think I forgot to regenerated the yml files in my previous PR #5741.....

Copy link
Contributor

Choose a reason for hiding this comment

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

Causing the URP Perf tests to continue to fail

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, I didn't notice it myself 😅 but I'm glad!

@kivistikliis kivistikliis marked this pull request as ready for review September 23, 2021 14:06
@kivistikliis kivistikliis requested a review from a team as a code owner September 23, 2021 14:06
@kivistikliis kivistikliis merged commit 9e6285e into 2021.2/staging Sep 23, 2021
@kivistikliis kivistikliis deleted the 2021.2/yamato/android-sdk-fix branch September 23, 2021 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants