dajac commented on PR #15349: URL: https://github.com/apache/kafka/pull/15349#issuecomment-1936915364
@gharris1727 I see. So if I understand correctly, two tests are flaky in the suite so we move the entire suite. This does not seem right to me. This particular suite is the one that you want to run when doing changes in the admin client. I have been working on https://github.com/apache/kafka/pull/14862 which changes how we report flaky tests in Jenkins. I believe that this is a better was to handle them. What do you think? However, I do agree with marking the slow tests as integration tests. I suppose that they should have been integration tests anyway. -- 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 unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
