pull_requests: 1285598862
This data as json
id | node_id | number | state | locked | title | user | body | created_at | updated_at | closed_at | merged_at | merge_commit_sha | assignee | milestone | draft | head | base | author_association | auto_merge | repo | url | merged_by |
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
1285598862 | PR_kwDOAMm_X85MoK6O | 7657 | closed | 0 | add timeouts for tests | 14808389 | The `macos` 3.11 CI seems stall very often at the moment, which makes it hit the 6 hours mark and get cancelled. Since our tests should never run that long (the ubuntu runners usually take between 15-20 minutes), I'm introducing a pretty generous timeout of 5 minutes. By comparison, we already have a timeout of 60 seconds in the upstream-dev CI, but that's on a ubuntu runner which usually is much faster than any of the macos / windows runners. Tests that time out raise an error, which might help us with figuring out which test it is that stalls, and also if we can do anything about that. | 2023-03-22T10:20:04Z | 2023-03-24T16:42:48Z | 2023-03-24T15:49:22Z | 2023-03-24T15:49:22Z | 635b9d0f2db4cc3ff005983c4af46ef6801258e2 | 0 | bf019197ae95493918a4b3b6e8fdd616fbe11ae4 | 6364cc068c56c1bb854f6ff8287b4e7f9638319c | MEMBER | { "enabled_by": { "login": "keewis", "id": 14808389, "node_id": "MDQ6VXNlcjE0ODA4Mzg5", "avatar_url": "https://avatars.githubusercontent.com/u/14808389?v=4", "gravatar_id": "", "url": "https://api.github.com/users/keewis", "html_url": "https://github.com/keewis", "followers_url": "https://api.github.com/users/keewis/followers", "following_url": "https://api.github.com/users/keewis/following{/other_user}", "gists_url": "https://api.github.com/users/keewis/gists{/gist_id}", "starred_url": "https://api.github.com/users/keewis/starred{/owner}{/repo}", "subscriptions_url": "https://api.github.com/users/keewis/subscriptions", "organizations_url": "https://api.github.com/users/keewis/orgs", "repos_url": "https://api.github.com/users/keewis/repos", "events_url": "https://api.github.com/users/keewis/events{/privacy}", "received_events_url": "https://api.github.com/users/keewis/received_events", "type": "User", "site_admin": false }, "merge_method": "squash", "commit_title": "add timeouts for tests (#7657)", "commit_message": "* install `pytest-timeout` into all environments\r\n\r\n* don't also install it into the upstream-dev environment\r\n\r\n* set a timeout of 5 minutes\r\n\r\nThis is plenty for a test to run: we shouldn't have tests that take\r\nmore than 30 seconds, but the windows / macos runners tend to be\r\nslower than the linux ones.\r\n\r\n* add `pytest-timeout` to the remaining test environments\r\n\r\n* add `pytest-timeout` to the ignored dependencies [skip-rtd]\r\n\r\n* reduce the timeout from 5 minutes to 90 seconds\r\n\r\n* double the test timeout" } |
13221727 | https://github.com/pydata/xarray/pull/7657 |
Links from other tables
- 4 rows from pull_requests_id in labels_pull_requests