You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@kafka.apache.org by GitBox <gi...@apache.org> on 2021/06/04 13:23:22 UTC
[GitHub] [kafka] showuon opened a new pull request #10820: KAFKA-12892: disable testChrootExistsAndRootIsLocked
showuon opened a new pull request #10820:
URL: https://github.com/apache/kafka/pull/10820
Detail is described in https://issues.apache.org/jira/browse/KAFKA-12892.
Thanks.
### Committer Checklist (excluded from commit message)
- [ ] Verify design and implementation
- [ ] Verify test coverage and CI build status
- [ ] Verify documentation (including upgrade notes)
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [kafka] showuon commented on pull request #10820: KAFKA-12892: disable testChrootExistsAndRootIsLocked
Posted by GitBox <gi...@apache.org>.
showuon commented on pull request #10820:
URL: https://github.com/apache/kafka/pull/10820#issuecomment-859979334
Thanks @omkreddy @ijuma @soarez , finally see the `trunk` build back to yellow light!
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [kafka] showuon commented on pull request #10820: KAFKA-12892: disable testChrootExistsAndRootIsLocked
Posted by GitBox <gi...@apache.org>.
showuon commented on pull request #10820:
URL: https://github.com/apache/kafka/pull/10820#issuecomment-854725411
@ijuma , KAFKA-12892 is the JIRA ticket. Thanks.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [kafka] ijuma commented on pull request #10820: KAFKA-12892: disable testChrootExistsAndRootIsLocked
Posted by GitBox <gi...@apache.org>.
ijuma commented on pull request #10820:
URL: https://github.com/apache/kafka/pull/10820#issuecomment-859633079
Can we file a JIRA to re-enable this test?
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [kafka] showuon commented on pull request #10820: KAFKA-12892: disable testChrootExistsAndRootIsLocked
Posted by GitBox <gi...@apache.org>.
showuon commented on pull request #10820:
URL: https://github.com/apache/kafka/pull/10820#issuecomment-854722099
@ijuma @omkreddy , could you take a look? This test keeps failing the jenkins build, I think we should disable it before we know how to avoid breaking jenkins build. Thanks.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [kafka] showuon commented on pull request #10820: KAFKA-12892: disable testChrootExistsAndRootIsLocked
Posted by GitBox <gi...@apache.org>.
showuon commented on pull request #10820:
URL: https://github.com/apache/kafka/pull/10820#issuecomment-858424436
@omkreddy @ijuma , the trunk build keeps failing with `InvalidAclException` at least 1 build group (sometimes 2 or 3 failed) after the fix merged (in build # 199). I think this is worse than flaky tests since the build terminated suddenly without build report output. I still think we need to disable the failing test soon, and have further investigation. What do you think?
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [kafka] ijuma commented on pull request #10820: KAFKA-12892: disable testChrootExistsAndRootIsLocked
Posted by GitBox <gi...@apache.org>.
ijuma commented on pull request #10820:
URL: https://github.com/apache/kafka/pull/10820#issuecomment-854724264
@omkreddy I will leave it to you to review this since you reviewed the original PR. If this is failing often, it makes sense to disable temporarily, but we need a JIRA to fix the test.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [kafka] showuon commented on pull request #10820: KAFKA-12892: disable testChrootExistsAndRootIsLocked
Posted by GitBox <gi...@apache.org>.
showuon commented on pull request #10820:
URL: https://github.com/apache/kafka/pull/10820#issuecomment-855153747
Triggering another test to make sure disabling the test won't break the build.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [kafka] omkreddy commented on pull request #10820: KAFKA-12892: disable testChrootExistsAndRootIsLocked
Posted by GitBox <gi...@apache.org>.
omkreddy commented on pull request #10820:
URL: https://github.com/apache/kafka/pull/10820#issuecomment-855231676
@showuon I have merged the PR https://github.com/apache/kafka/pull/10821. Let us monitor any build failures.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [kafka] showuon commented on pull request #10820: KAFKA-12892: disable testChrootExistsAndRootIsLocked
Posted by GitBox <gi...@apache.org>.
showuon commented on pull request #10820:
URL: https://github.com/apache/kafka/pull/10820#issuecomment-855233257
@omkreddy , sure. Hope it fixes the build. Thank you.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [kafka] omkreddy merged pull request #10820: KAFKA-12892: disable testChrootExistsAndRootIsLocked
Posted by GitBox <gi...@apache.org>.
omkreddy merged pull request #10820:
URL: https://github.com/apache/kafka/pull/10820
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [kafka] soarez commented on pull request #10820: KAFKA-12892: disable testChrootExistsAndRootIsLocked
Posted by GitBox <gi...@apache.org>.
soarez commented on pull request #10820:
URL: https://github.com/apache/kafka/pull/10820#issuecomment-859643949
Filed [KAFKA-12938 Fix and reenable testChrootExistsAndRootIsLocked](https://issues.apache.org/jira/browse/KAFKA-12938 ).
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [kafka] showuon commented on pull request #10820: KAFKA-12892: disable testChrootExistsAndRootIsLocked
Posted by GitBox <gi...@apache.org>.
showuon commented on pull request #10820:
URL: https://github.com/apache/kafka/pull/10820#issuecomment-855611057
@omkreddy , unfortunately, the `trunk` build is still failing with `InvalidAclException` after https://github.com/apache/kafka/pull/10821 merged. I think we should disable this test first (I run it 3 times and never failed the jenkins build), and maybe think another way to write the test (and have multiple runs to verify it won't break the tests). Thank you.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [kafka] showuon edited a comment on pull request #10820: KAFKA-12892: disable testChrootExistsAndRootIsLocked
Posted by GitBox <gi...@apache.org>.
showuon edited a comment on pull request #10820:
URL: https://github.com/apache/kafka/pull/10820#issuecomment-855611057
@omkreddy , unfortunately, the `trunk` build is still failing with `InvalidAclException` after https://github.com/apache/kafka/pull/10821 merged. I think we should disable this test first (This PR triggered PR build 3 times and never failed the jenkins build), and maybe we should think another way to do the test (and should have multiple runs to verify it won't break the tests). Thank you.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [kafka] showuon commented on pull request #10820: KAFKA-12892: disable testChrootExistsAndRootIsLocked
Posted by GitBox <gi...@apache.org>.
showuon commented on pull request #10820:
URL: https://github.com/apache/kafka/pull/10820#issuecomment-855197435
2 PR builds have no build failure. This PR can fix the trunk build failure issue. Thanks.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org