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

Increased size of go_bazelisk_test #637

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hauserx
Copy link

@hauserx hauserx commented Nov 21, 2024

Depending on the network one is using the test may go over default 5 minutes for medium size, causing spurious failures. Increased size of the test to "large" so timeout is now 15 minutes.

Sample execution on my machine:

$ bazel test //... --cache_test_results=no

//:bazelisk_version_test                                                 PASSED in 3.7s
//:go_bazelisk_test                                                      PASSED in 298.0s
//:py3_bazelisk_test                                                     PASSED in 52.0s
//:py_bazelisk_test                                                      PASSED in 54.5s
//core:core_test                                                         PASSED in 0.0s
//httputil:httputil_test                                                 PASSED in 0.0s
//httputil/progress:progress_test                                        PASSED in 0.0s
//platforms:platforms_test                                               PASSED in 0.0s

Depending on the network one is using the test may go over default 5
minutes for medium size, causing spurious failures. Increased size of
the test to "large" so timeout is now 15 minutes.

Sample execution on my machine:

$ bazel test //... --cache_test_results=no

//:bazelisk_version_test                                                 PASSED in 3.7s
//:go_bazelisk_test                                                      PASSED in 298.0s
//:py3_bazelisk_test                                                     PASSED in 52.0s
//:py_bazelisk_test                                                      PASSED in 54.5s
//core:core_test                                                         PASSED in 0.0s
//httputil:httputil_test                                                 PASSED in 0.0s
//httputil/progress:progress_test                                        PASSED in 0.0s
//platforms:platforms_test                                               PASSED in 0.0s
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant