tree 7c9e5a3f0de7c4666c56d0b6c15fc6dcb9f6e78c
parent 281e178352f0f48a9a68c1de2ede60c7dacf3252
author Divij Vaidya <diviv@amazon.com> 1664493843 +0200
committer GitHub <noreply@github.com> 1664493843 -0700
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJjNikTCRBK7hj4Ov3rIwAA4qcIAIkk5visXfXYycv8wuBhLZib
 GWe73pIau+hg0432W+xG4KY0TbdLfzgBFqeK+38Fk3mgmVRuiiHI3cs1eQ1Z6GW6
 k8lCVOcSjqq4upMwBNnXM/iVvpCeORw2wVaTPozfxnKwYuqr1IXrF1Q1eUQhW1Rd
 3fnMf4uj8lRt9zm4WSAu5aLvu975/9qHXeKqEO1WzYGivdRSNL5OQPGkeRT8AZYX
 jTmS/gGy24ZceZnkqJoJC3GYMRv7TKIuHn7R0PwAyjAatqM847Km8FvPQbPKDE6V
 uF67wxnTf6ZGzOYG1W3q1QwpCPj4NUl1dYP+7Rng/hd/OSeIt8mxpNxWptzIbi0=
 =NC0g
 -----END PGP SIGNATURE-----
 

KAFKA-14248; Fix flaky test PlaintextAdminIntegrationTest.testCreateTopicsReturnsConfigs (#12669)

The test is failing intermittently because we do not wait for propagation of the altered config (LogRetentionTimeMillisProp) across all brokers before proceeding ahead with the test.

This PR makes the following changes:
1. Wait for propagation of altered configuration to propagate to all brokers.
2. Use the existing `killBroker` utility method which waits for shutdown using `awaitshutdown`.
3. Improve code readability by using `TestUtils.incrementalAlterConfigs` to send alter config requests.

Reviewers: Jason Gustafson <jason@confluent.io>