KAFKA-10050: kafka_log4j_appender.py fixed for JDK11

[nizhikov]

kafka_log4j_appender.py broken on JDK11 by #befd80b38d3ccb1aa0c6d99a899129fd5cf27774
This fix just setup node version for log4j appender.

Committer Checklist (excluded from commit message)

• Verify design and implementation
• Verify test coverage and CI build status
• Verify documentation (including upgrade notes)

[nizhikov]

Hello, @ijuma

I found that kafka_log4j_appender.py broken for JDK11.
This was broken by my patch - befd80b

I've prepared oneliner fix

Can you, please, take a look?

[ijuma]

Thanks for the PR. Can you elaborate how the original change broke it? I don't see anything obviously related.

[nizhikov]

I added call of fix_opts_for_new_jvm here.

But node variable for kafka_log4j_appender.py don't have an attribute 'version'.
The fix just adds this attribute from the corresponding Kafka version.

  File "/opt/kafka-dev/tests/kafkatest/services/kafka_log4j_appender.py", line 48, in start_cmd
    cmd = fix_opts_for_new_jvm(node)
  File "/opt/kafka-dev/tests/kafkatest/services/kafka/util.py", line 36, in fix_opts_for_new_jvm
    if node.version == LATEST_0_8_2 or node.version == LATEST_0_9 or node.version == LATEST_0_10_0 or node.version == LATEST_0_10_1 or node.version == LATEST_0_10_2 or node.version == LATEST_0_11_0 or node.version == LATEST_1_0:
AttributeError: 'ClusterNode' object has no attribute 'version'

[ijuma]

Thanks for the explanation, LGTM.