pprovenzano commented on PR #13114:
URL: https://github.com/apache/kafka/pull/13114#issuecomment-1453878000
I've added the MetadataVersion fixes with tests.
--
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
pprovenzano commented on PR #13114:
URL: https://github.com/apache/kafka/pull/13114#issuecomment-1450839352
> We need a test like
`./metadata/src/test/java/org/apache/kafka/image/ScramImageTesst.java` ,
similar to the tests for the other Image classes.
I'm confused by this comment. T
pprovenzano commented on PR #13114:
URL: https://github.com/apache/kafka/pull/13114#issuecomment-1422710574
I plan to update existing tests after bootstrap support is committed.
I'll check on the `handleDescribeUserScramCredentialsRequest`
--
This is an automated message from th
pprovenzano commented on PR #13114:
URL: https://github.com/apache/kafka/pull/13114#issuecomment-1411037660
I just updated AlterUserScramCredentialsRequestTest to work in KRaft after
finding it while looking into the tests you wanted. It covers a lot of what you
want testing ScramControlMan
pprovenzano commented on PR #13114:
URL: https://github.com/apache/kafka/pull/13114#issuecomment-1408812047
>
The changes to `BrokerMetadataSnapshotterTest.scala` are needed and small.
The size of the SCRAM records is larger than 1024 bytes and so I increased it
to 4096 bytes in the
