-
Notifications
You must be signed in to change notification settings - Fork 297
Add MAUI Android inner loop deploy measurement scenario #5165
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
Open
davidnguyen-tech
wants to merge
26
commits into
dotnet:main
Choose a base branch
from
davidnguyen-tech:feature/measure-maui-android-deploy
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
8123eed
Add AndroidInnerLoopParser for build/deploy binlog analysis
davidnguyen-tech f6c5869
Add ANDROIDINNERLOOP test type to Python test framework
davidnguyen-tech 7740568
Add MAUI Android inner loop scenario scripts
davidnguyen-tech 3f5e643
Wire MAUI Android inner loop into CI pipeline
davidnguyen-tech 746709a
Make screen timeout configurable and validate cold startup LaunchState
davidnguyen-tech 986c7bf
Address review notes
davidnguyen-tech 26ea3ea
Address review notes 2
davidnguyen-tech 59d3d40
Add SDK version capture + diagnostics to mauiandroidinnerloop
davidnguyen-tech efd2217
Address copilot review notes
davidnguyen-tech d73f5fb
Log rollback_maui.json contents on Helix machine
davidnguyen-tech d5da6c1
Switch MAUI Android Inner Loop to 'dotnet run -p:WaitForExit=false'
davidnguyen-tech c47d731
Migrate MAUI Android Inner Loop startup measurement to logcat polling
davidnguyen-tech 7c1e3bf
Remove stale Android cold startup helper
davidnguyen-tech e2abd8d
Address PR review findings
davidnguyen-tech 27ef49a
Reject non-Debug BuildConfig for MAUI Android Inner Loop
davidnguyen-tech 57d68ff
Address PR review: align with Android verifier-disable + clarify semi…
davidnguyen-tech 0f1a76c
Revert verifier-disable change; keep semicolon-list help text
davidnguyen-tech d019e81
Revert ValidateInnerLoopBuildConfig guard and -c $(BuildConfig) switch
davidnguyen-tech 802b12f
Document why -c Debug is hardcoded in inner loop scenario
davidnguyen-tech b03125d
MAUI Android Inner Loop: pass AndroidManifestType=GoogleV2 for API 37
davidnguyen-tech dffb8fc
[mauiandroidinnerloop] Repair silently-skipped Android SDK deps
davidnguyen-tech 362bf00
Consolidate inner-loop proj into maui_scenarios_android.proj
kotlarmilos 530503f
Pass RunKind to Send job to Helix step
kotlarmilos 053f80c
Guard Helix traces cleanup on non-null upload dir
kotlarmilos 7e3e8e3
Address review feedback: source restore, adb timeout, PEP 604 compat
kotlarmilos 7ec125c
androidhelper: use grep -F so dots in package name aren't regex wildc…
kotlarmilos File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,28 @@ | ||
| ''' | ||
| post cleanup script | ||
| ''' | ||
|
|
||
| import subprocess | ||
| import sys | ||
| import traceback | ||
| from performance.logger import setup_loggers, getLogger | ||
| from shared.postcommands import clean_directories | ||
| from shared.util import xharness_adb | ||
| from test import EXENAME | ||
|
|
||
| setup_loggers(True) | ||
| logger = getLogger(__name__) | ||
|
|
||
| try: | ||
| # Uninstall the app from the connected device so re-runs start from a clean state | ||
| package_name = f'com.companyname.{EXENAME.lower()}' | ||
| logger.info(f"Uninstalling {package_name} from device") | ||
| subprocess.run(xharness_adb() + ['uninstall', package_name], check=False) | ||
|
|
||
| logger.info("Shutting down dotnet build servers") | ||
| subprocess.run(['dotnet', 'build-server', 'shutdown'], check=False) | ||
|
|
||
| clean_directories() | ||
| except Exception as e: | ||
| logger.error(f"Post cleanup failed: {e}\n{traceback.format_exc()}") | ||
| sys.exit(1) |
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since we're settings these here instead of using https://github.com/dotnet/performance/pull/5165/changes#diff-652d0d84aa2897441204ca7f3c3e5f8b3dd75fb44edaa69c2095a610270297d3L18-R28, maybe we should create a separate .proj file for the inner loop? I don't see many things which are re-used among the two scenarios and it would remove the
<PropertyGroup Condition="'$(_IsInnerLoop)' == 'true'">checks