Skip to content
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

[DRAFT][PERF][MAUI] Add startup time and size mesauresments for dotnet-podcast app #65398

Closed

Conversation

naricc
Copy link
Contributor

@naricc naricc commented Feb 15, 2022

Add startup time measuresments and size measurements to perf runs for the dotnet-podcast app.

This is just a test run to see if there are issues building the app in CI. Needs better integration with existing .ymls before it can be merged for real.

Added main Maui build pipeline yml file with target and prop files to properly build Maui for Android with the latest runtime bits, updated the perf.yml file to run the Maui build steps, and piped the Android app artifact through the same line as the AndroidScenario so the perf size on disk tests are run.

(cherry picked from commit ed91cea)
* Update the file location and android runtimes. The Maui android architecture targets are being updated to include all android targets.

* Updated the targets and props copy location.

* Add in the correct depends on for the maui build.

* The initial file move did not actually delete the folder being moved, delete it.

(cherry picked from commit 3159d62)
* Test allowing 'unstable' packages since maui is not finding stable packages for some of the extension packages.

(cherry picked from commit bc9682c)
* Fix artifact match string.

* Completed first attempt for a complete Maui SOD run. Added ios_scenarios tests, setup file copies, uploads, and downloads; and setup perf branch to use as partner testing branch.

* Reset performance-setup.ps1 file to use main perf branch.

* Reenable yml jobs disabled for testing.

(cherry picked from commit 06c2dba)
Add app with debug instrumentation build and download. Add android scenario calls and include xharness.

Co-authored-by: Bill Wert <billwert@microsoft.com>
(cherry picked from commit c8ede5c)
* Setup test pipeline for closest possible setup to what is currently working for android startup with the new adb method being used.

* Fix android scenario problems.

* Use latest xharness version for testing popen fix.

* Wrong branch changed.

* Final run setup for runtime part of the repo.

* Use test branch for full run test and keep it until the test finishes.

* Change perf pull branch back to main, hold until test run is complete.

* Add in iOS runtime builds for sure, and change branch for testing.

* Add in the file extraction of the built ios packs and changed workload typo to runtime to better match actual meaning.

* Revert back to main branch for setup.

* Git missed ios-arm runtime extraction, add that in.
*  Add dotnet install and run to test if new dotnet version can be installed similar to how it is done in the Maui repo.

* Download and use local dotnet version.

* Use updated dotnet for more of the pipeline.

* Test with made runtime packs and and add ios arm pack substitution.

* Try building instead of publishing.

* Reenable everything for full test run.

* Package name now seems to default to lower case, at least for maui.
Added main Maui build pipeline yml file with target and prop files to properly build Maui for Android with the latest runtime bits, updated the perf.yml file to run the Maui build steps, and piped the Android app artifact through the same line as the AndroidScenario so the perf size on disk tests are run.

(cherry picked from commit ed91cea)
* Update the file location and android runtimes. The Maui android architecture targets are being updated to include all android targets.

* Updated the targets and props copy location.

* Add in the correct depends on for the maui build.

* The initial file move did not actually delete the folder being moved, delete it.

(cherry picked from commit 3159d62)
* Test allowing 'unstable' packages since maui is not finding stable packages for some of the extension packages.

(cherry picked from commit bc9682c)
* Fix artifact match string.

* Completed first attempt for a complete Maui SOD run. Added ios_scenarios tests, setup file copies, uploads, and downloads; and setup perf branch to use as partner testing branch.

* Reset performance-setup.ps1 file to use main perf branch.

* Reenable yml jobs disabled for testing.

(cherry picked from commit 06c2dba)
Add app with debug instrumentation build and download. Add android scenario calls and include xharness.

Co-authored-by: Bill Wert <billwert@microsoft.com>
(cherry picked from commit c8ede5c)
*  Add dotnet install and run to test if new dotnet version can be installed similar to how it is done in the Maui repo.

* Download and use local dotnet version.

* Use updated dotnet for more of the pipeline.

* Test with made runtime packs and and add ios arm pack substitution.

* Try building instead of publishing.

* Reenable everything for full test run.

* Package name now seems to default to lower case, at least for maui.
…cessful. Disabled non-android/ios scenario stuff for first test.
Added main Maui build pipeline yml file with target and prop files to properly build Maui for Android with the latest runtime bits, updated the perf.yml file to run the Maui build steps, and piped the Android app artifact through the same line as the AndroidScenario so the perf size on disk tests are run.

(cherry picked from commit ed91cea)
* Update the file location and android runtimes. The Maui android architecture targets are being updated to include all android targets.

* Updated the targets and props copy location.

* Add in the correct depends on for the maui build.

* The initial file move did not actually delete the folder being moved, delete it.

(cherry picked from commit 3159d62)
@ghost ghost assigned naricc Feb 15, 2022
@dotnet-issue-labeler
Copy link

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@naricc
Copy link
Contributor Author

naricc commented Feb 23, 2022

Closing in favor of: #65813

@naricc naricc closed this Feb 23, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Mar 26, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants