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

2025-01-08 Thread via GitHub
showuon closed pull request #15065: MINOR: set test global timeout as 10 mins URL: https://github.com/apache/kafka/pull/15065 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

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

2025-01-04 Thread via GitHub
github-actions[bot] commented on PR #15065: URL: https://github.com/apache/kafka/pull/15065#issuecomment-2571488470 This PR is being marked as stale since it has not had any activity in 90 days. If you would like to keep this PR alive, please leave a comment asking for a review. If the P

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