Re: [PR] MINOR: set test global timeout as 10 mins [kafka]

2024-05-07 Thread via GitHub
gaurav-narula commented on PR #15065: URL: https://github.com/apache/kafka/pull/15065#issuecomment-2098952056 Bumping this thread for more visibility. Would be nice if we give this a shot since we still run into builds which [time out](https://ci-builds.apache.org/blue/organizations/

Re: [PR] MINOR: set test global timeout as 10 mins [kafka]

2024-02-09 Thread via GitHub
gaurav-narula commented on code in PR #15065: URL: https://github.com/apache/kafka/pull/15065#discussion_r1484162204 ## clients/src/test/resources/junit-platform.properties: ## @@ -13,3 +13,4 @@ # See the License for the specific language governing permissions and # limitation

Re: [PR] MINOR: set test global timeout as 10 mins [kafka]

2024-02-08 Thread via GitHub
gaurav-narula commented on code in PR #15065: URL: https://github.com/apache/kafka/pull/15065#discussion_r1482924308 ## clients/src/test/resources/junit-platform.properties: ## @@ -13,3 +13,4 @@ # See the License for the specific language governing permissions and # limitation

Re: [PR] MINOR: set test global timeout as 10 mins [kafka]

2024-01-08 Thread via GitHub
ijuma commented on PR #15065: URL: https://github.com/apache/kafka/pull/15065#issuecomment-1881861076 This does actually work consistently? My experience with JUnit 5 timeout annotations has not been good, unfortunately (they often would simply not work at all). I haven't tried this approac

Re: [PR] MINOR: set test global timeout as 10 mins [kafka]

2024-01-08 Thread via GitHub
dajac commented on PR #15065: URL: https://github.com/apache/kafka/pull/15065#issuecomment-1881833093 I like the idea! Have we been able to get a stable build with this? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [PR] MINOR: set test global timeout as 10 mins [kafka]

2023-12-26 Thread via GitHub
jolshan commented on PR #15065: URL: https://github.com/apache/kafka/pull/15065#issuecomment-1869836473 Thanks @showuon for getting started with this. I noticed the first build still ran about 8h and hit an OOM issue. Looks like that will have to be addressed separately (which is ok). (edit

Re: [PR] MINOR: set test global timeout as 10 mins [kafka]

2023-12-22 Thread via GitHub
showuon commented on PR #15065: URL: https://github.com/apache/kafka/pull/15065#issuecomment-1867639044 @dajac @ijuma @jolshan @stanislavkozlovski , call for review. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[PR] MINOR: set test global timeout as 10 mins [kafka]

2023-12-22 Thread via GitHub
showuon opened a new pull request, #15065: URL: https://github.com/apache/kafka/pull/15065 As @stanislavkozlovski found [here](https://gist.github.com/stanislavkozlovski/8959f7ee59434f774841f4ae2f5228c2), there are some tests ran more than 1 ~ 2 hours, which make our CI build meet 8 hours