Skip to content

Conversation

@Narfinger
Copy link
Contributor

Switch ohos speedometer from hitrace-bench to servo-driver script.
This allows us to look at logs in the future and has easier parsing of the values.

Signed-off-by: Narfinger Narfinger@users.noreply.github.com

Testing: Custom run on https://github.com/Narfinger/servo/actions/runs/19575793934

Signed-off-by: Narfinger <Narfinger@users.noreply.github.com>
@Narfinger Narfinger marked this pull request as ready for review November 24, 2025 09:16
@Narfinger Narfinger requested a review from sagudev as a code owner November 24, 2025 09:16
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Nov 24, 2025
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Nov 24, 2025
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Nov 28, 2025
Signed-off-by: Narfinger <Narfinger@users.noreply.github.com>
Signed-off-by: Narfinger <Narfinger@users.noreply.github.com>
@yezhizhen yezhizhen added this pull request to the merge queue Dec 1, 2025
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Dec 1, 2025
Merged via the queue into servo:main with commit a1c37f8 Dec 1, 2025
29 checks passed
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Dec 1, 2025
@Narfinger Narfinger deleted the ohos-speedometer-test branch December 1, 2025 12:00
github-merge-queue bot pushed a commit that referenced this pull request Dec 1, 2025
The script seems to get sometimes stuck, so add a timeout that prevents
needing to wait until the default timeout is hit.
The timeouts might be related to the script change from
#40798.

Testing: Not needed.

Signed-off-by: Jonathan Schwender <schwenderjonathan@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-awaiting-review There is new code that needs to be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants