Skip to content

[ci] Lengthen custom tests timeout #8715

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

Merged
merged 1 commit into from
Feb 26, 2025

Conversation

stuartmorgan-g
Copy link
Contributor

This test has been at the edge of its time limit for a while, and is now consistently timing out. Longer term we should see if we can shorten the Pigeon test time (it accounts for about 20 minutes), but for now, increase the timeout so that it doesn't fail.

This test has been at the edge of its time limit for a while, and is now
consisently timing out. Longer term we should see if we can shorten the
Pigeon test time (it accounts for about 20 minutes), but for now,
increase the timeout so that it doesn't fail.
Copy link
Contributor

@tarrinneal tarrinneal left a comment

Choose a reason for hiding this comment

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

Do you know why the pigeon tests are running so long?

@stuartmorgan-g
Copy link
Contributor Author

Do you know why the pigeon tests are running so long?

Nope, that's why I did the other PR. I'm guessing that adding the overflow version of the tests a while ago put us close to the limit though, and something else made things slightly longer (maybe the web_benchmark tests).

@stuartmorgan-g stuartmorgan-g added emergency Override tree-status signal (land even with closed tree), combine with the autosubmit label. autosubmit Merge PR when tree becomes green via auto submit App labels Feb 26, 2025
@auto-submit auto-submit bot merged commit 6a918c4 into flutter:main Feb 26, 2025
82 of 83 checks passed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Feb 26, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Feb 27, 2025
github-merge-queue bot pushed a commit to flutter/flutter that referenced this pull request Feb 27, 2025
flutter/packages@c44c228...01d3d5c

2025-02-27 engine-flutter-autoroll@skia.org Manual roll Flutter from
043b719 to 1659206 (19 revisions) (flutter/packages#8728)
2025-02-27 stuartmorgan@google.com Manual roll Flutter from 911aa75
to 043b719 (flutter/packages#8693)
2025-02-26 magder@google.com Dependabot to update major and minor
versions of test dependencies, ignore patch (flutter/packages#8712)
2025-02-26 reidbaker@google.com [local_auth] Update to use
flutter.targetSdkVersion (flutter/packages#8695)
2025-02-26 57854043+CaoGiaHieu-dev@users.noreply.github.com
[go_router_builder]: Handle invaild params (flutter/packages#8405)
2025-02-26 stuartmorgan@google.com [pigeon] Timestamp test steps in CI
(flutter/packages#8716)
2025-02-26 43054281+camsim99@users.noreply.github.com
[camera_android_camerax] Fix 90°-off preview rotation
(flutter/packages#8629)
2025-02-26 108667553+aprzedecki@users.noreply.github.com [go_router]
Secured empty matches in canPop (flutter/packages#8557)
2025-02-26 reidbaker@google.com [tool] Update targetsdk version to 35
from 32 (flutter/packages#8694)
2025-02-26 stuartmorgan@google.com [various] Bump androidx.test:core to
1.4.0 (flutter/packages#8710)
2025-02-26 stuartmorgan@google.com [camera] Disable flaky tests
(flutter/packages#8708)
2025-02-26 mdebbar@google.com [url_launcher][web] Prevent browser from
navigating when followLink isn't called (flutter/packages#8675)
2025-02-26 stuartmorgan@google.com [various] Remove plugin-level
`integration_test` dependencies (flutter/packages#8711)
2025-02-26 stuartmorgan@google.com [ci] Lengthen custom tests timeout
(flutter/packages#8715)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages-flutter-autoroll
Please CC flutter-ecosystem@google.com on the revert to ensure that a
human
is aware of the problem.

To file a bug in Flutter:
https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
androidseb pushed a commit to androidseb/packages that referenced this pull request Jun 8, 2025
This test has been at the edge of its time limit for a while, and is now consistently timing out. Longer term we should see if we can shorten the Pigeon test time (it accounts for about 20 minutes), but for now, increase the timeout so that it doesn't fail.
FMorschel pushed a commit to FMorschel/packages that referenced this pull request Jun 9, 2025
This test has been at the edge of its time limit for a while, and is now consistently timing out. Longer term we should see if we can shorten the Pigeon test time (it accounts for about 20 minutes), but for now, increase the timeout so that it doesn't fail.
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 emergency Override tree-status signal (land even with closed tree), combine with the autosubmit label.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants