{"push_id":1081245768,"size":5,"distinct_size":1,"ref":"refs/heads/zundel/junit-concurrency-param","head":"9f29744f71525c9cd0c224d3e0899c4003a012d3","before":"4d2f6c7b189822afd8d01d745efa2e3d4168512b","commits":[{"sha":"3db4ccead7f275e3fa9f283a523257951a072c16","author":{"email":"86f39098c3d6e8fa24cceea9f4753c2782977b2d@gmail.com","name":"John Sirois"},"message":"Skip flaky pytest timeout failure ITs.\n\nThe `test_pytest_run_timeout_cant_terminate` test is causing the\npantsbuild-osx/pants Travis-CI job to go red roughly 50% of the time.\nLocal testing of this skip application found\n`test_pytest_run_timeout_fails` also failing intermittently on my\nmachine so a skip is applied to that similarly fragile test as well.\n\nTesting Done:\nRan the following locally and confirmed 2 skips:\n```\n./pants test \\\n  tests/python/pants_test/backend/python/tasks/:pytest_run_integration\n```\n\nBefore the skips, I was able to get failure in one or both of the\nskipped tests >50% of the time (linux box).\n\nCI went green here:\n  https://travis-ci.org/pantsbuild/pants/builds/125117190\n\nBugs closed: 3255, 3258\n\nReviewed at https://rbcommons.com/s/twitter/r/3748/","distinct":false,"url":"https://api.github.com/repos/ericzundel/pants/commits/3db4ccead7f275e3fa9f283a523257951a072c16"},{"sha":"438d299bac8c58e5cbe236e9edeac3497baf5bcc","author":{"email":"695d76d5a2f51dc9340d65ccd0d72ad7560773f9@squareup.com","name":"Eric Ayers"},"message":"Enhance parallel testing junit_tests\n\nSurfaces EXPERIMENTAL support for parallel testing of methods within a test class.\n - Adds a 'concurrency' parameter to set test concurrency to 'serial' 'parallel', or 'parallel_method'\n - Adds a 'threads' parameter to junit_tests to control concurrency.\n - Adds a --test-junit-parallel-methods to allow methods to run in parallel for the entire test run\n - Added a number of unit test cases and integration tests to exercise these features\n\nNote that there is a bug in that the @TestSerial annotation is not respected when the -parallel-methods flag is in use.  See https://github.com/pantsbuild/pants/issues/3209\n\nFollowup work for this change is to add an `@TestParallelMethods` annotation and come up with a more rational way to\npass concurrency options to the junit-runner backend.\n\nTesting Done:\nIntegration test and additional unit tests added.\nCI is green at https://travis-ci.org/pantsbuild/pants/builds/123712609\n\nBugs closed: 3191, 3210\n\nReviewed at https://rbcommons.com/s/twitter/r/3707/","distinct":false,"url":"https://api.github.com/repos/ericzundel/pants/commits/438d299bac8c58e5cbe236e9edeac3497baf5bcc"},{"sha":"ef1e7f6771437ca36b48952f97736661a3090cbd","author":{"email":"695d76d5a2f51dc9340d65ccd0d72ad7560773f9@squareup.com","name":"Eric Ayers"},"message":"Revert \"Enhance parallel testing junit_tests\"\n\nThis reverts commit 438d299bac8c58e5cbe236e9edeac3497baf5bcc.","distinct":false,"url":"https://api.github.com/repos/ericzundel/pants/commits/ef1e7f6771437ca36b48952f97736661a3090cbd"},{"sha":"b912668bd53d7394259d5eff8d47b2b2e1737750","author":{"email":"695d76d5a2f51dc9340d65ccd0d72ad7560773f9@squareup.com","name":"Eric Ayers"},"message":" Enhance parallel testing junit_tests\n\n    Surfaces EXPERIMENTAL support for parallel testing of methods within a test class.\n     - Adds a 'concurrency' parameter to set test concurrency to 'SERIAL' 'PARALLEL_CLASSES', 'PARALLEL_METHODS', or 'PARALLEL_BOTH'\n     - Adds a 'threads' parameter to junit_tests to control concurrency.\n     - Adds a --test-junit-default-concurrency option\n     - PARALLEL_BOTH will allow both classes and methods to run in parallel for the entire test run\n     - PARALLEL_METHODS (just run methods within a class in parallel) is flagged as not implemented\n     - Added a number of unit test cases and integration tests to exercise these features\n\n    Note that there is a bug in that the @TestSerial annotation is not respected when the -parallel-methods flag is in use.  See https://github.com/pantsbuild/pants/issues/3209\n\n    Followup work for this change is to add an `@TestParallelMethods` annotation and come up with a more rational way to\n    pass concurrency options to the junit-runner backend.","distinct":false,"url":"https://api.github.com/repos/ericzundel/pants/commits/b912668bd53d7394259d5eff8d47b2b2e1737750"},{"sha":"9f29744f71525c9cd0c224d3e0899c4003a012d3","author":{"email":"695d76d5a2f51dc9340d65ccd0d72ad7560773f9@squareup.com","name":"Eric Ayers"},"message":"New -default-concurrency parameter","distinct":true,"url":"https://api.github.com/repos/ericzundel/pants/commits/9f29744f71525c9cd0c224d3e0899c4003a012d3"}]}