Skip to content

Find available port to run benchmarks - #9315

Merged
auto-submit[bot] merged 2 commits into
masterfrom
find_available_port
Jul 15, 2025
Merged

Find available port to run benchmarks#9315
auto-submit[bot] merged 2 commits into
masterfrom
find_available_port

Conversation

@mdebbar

@mdebbar mdebbar commented Jul 15, 2025

Copy link
Copy Markdown
Contributor

Instead of using a hardcoded port number to run benchmarks, let's dynamically find an available port.

Fixes #9314

This is not really a fix, but more of a workaround until the underlying package:test issue dart-lang/test#897 is resolved.

@mdebbar
mdebbar requested a review from a team as a code owner July 15, 2025 18:36
@mdebbar
mdebbar requested review from bkonyi and removed request for a team July 15, 2025 18:36
@mdebbar
mdebbar requested a review from kenzieschmoll July 15, 2025 18:36
@mdebbar mdebbar added the autosubmit Merge PR when tree becomes green via auto submit App label Jul 15, 2025
@auto-submit
auto-submit Bot merged commit e25288f into master Jul 15, 2025
45 checks passed
@auto-submit
auto-submit Bot deleted the find_available_port branch July 15, 2025 21:23
hrajwade96 pushed a commit to hrajwade96/devtools that referenced this pull request Sep 2, 2025
Instead of using a hardcoded port number to run benchmarks, let's dynamically find an available port.

Fixes flutter#9314

This is not really a fix, but more of a workaround until the underlying `package:test` issue dart-lang/test#897 is resolved.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App release-notes-not-required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[web_benchmarks] Chrome isn't cleaned up properly when test ends

2 participants