{"action":"created","issue":{"url":"https://api.github.com/repos/prestodb/presto-admin/issues/203","repository_url":"https://api.github.com/repos/prestodb/presto-admin","labels_url":"https://api.github.com/repos/prestodb/presto-admin/issues/203/labels{/name}","comments_url":"https://api.github.com/repos/prestodb/presto-admin/issues/203/comments","events_url":"https://api.github.com/repos/prestodb/presto-admin/issues/203/events","html_url":"https://github.com/prestodb/presto-admin/pull/203","id":164402451,"number":203,"title":"Fix travis - job timeout and test timeout","user":{"login":"zachyee","id":7135060,"avatar_url":"https://avatars.githubusercontent.com/u/7135060?v=3","gravatar_id":"","url":"https://api.github.com/users/zachyee","html_url":"https://github.com/zachyee","followers_url":"https://api.github.com/users/zachyee/followers","following_url":"https://api.github.com/users/zachyee/following{/other_user}","gists_url":"https://api.github.com/users/zachyee/gists{/gist_id}","starred_url":"https://api.github.com/users/zachyee/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zachyee/subscriptions","organizations_url":"https://api.github.com/users/zachyee/orgs","repos_url":"https://api.github.com/users/zachyee/repos","events_url":"https://api.github.com/users/zachyee/events{/privacy}","received_events_url":"https://api.github.com/users/zachyee/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2016-07-07T20:41:23Z","updated_at":"2016-07-11T20:03:07Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/prestodb/presto-admin/pulls/203","html_url":"https://github.com/prestodb/presto-admin/pull/203","diff_url":"https://github.com/prestodb/presto-admin/pull/203.diff","patch_url":"https://github.com/prestodb/presto-admin/pull/203.patch"},"body":"Created more jobs to fix the case where a single job would take more than 50 minutes to complete.\r\n\r\nMoved creating the docker images for the product tests to before the test executes rather than during the test itself. This helps tests avoid their 10 minute limit for not outputting anything.\r\n\r\nAdded some logging that can be used in the future to figure out why tests are taking a long time (if this ever happens).\r\n\r\nWorks successfully with presto version 148, but not with the newest version. I can hardcode the URL where the tests download the presto rpm to fetch the 148 version rather than the newest version. This might be useful to test the release? (This PR fetches 149, not 148, so it will fail on travis)"},"comment":{"url":"https://api.github.com/repos/prestodb/presto-admin/issues/comments/231847759","html_url":"https://github.com/prestodb/presto-admin/pull/203#issuecomment-231847759","issue_url":"https://api.github.com/repos/prestodb/presto-admin/issues/203","id":231847759,"user":{"login":"ebd2","id":12751788,"avatar_url":"https://avatars.githubusercontent.com/u/12751788?v=3","gravatar_id":"","url":"https://api.github.com/users/ebd2","html_url":"https://github.com/ebd2","followers_url":"https://api.github.com/users/ebd2/followers","following_url":"https://api.github.com/users/ebd2/following{/other_user}","gists_url":"https://api.github.com/users/ebd2/gists{/gist_id}","starred_url":"https://api.github.com/users/ebd2/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ebd2/subscriptions","organizations_url":"https://api.github.com/users/ebd2/orgs","repos_url":"https://api.github.com/users/ebd2/repos","events_url":"https://api.github.com/users/ebd2/events{/privacy}","received_events_url":"https://api.github.com/users/ebd2/received_events","type":"User","site_admin":false},"created_at":"2016-07-11T20:03:07Z","updated_at":"2016-07-11T20:03:07Z","body":"Just had a thought: Parallelizing the tests almost requires that we create the docker images prior to starting the tests, otherwise, it's highly likely that multiple tests will think they're the first one that needs a particular image. This will result in committing different containers to the same image names.\r\n\r\nAt best we burn a little extra disk space (multiple layers with presto installed, of which only one is :latest). At worst, we get intermittent errors owing to the race condition."}}
{"actor":{"display_login":"ebd2"}}