Fixed advertised.listeners block in kafka

Open ankur-gupta-guavus opened this issue 6 years ago • 2 comments

What changes were proposed in this pull request?

advertised.listeners were not set properly and they remain as localhost:<portNumber>. Code responsible for the same was under else block of params.kerberos_security_enabled. Advertised.listeners were just being set in case if kerberos security is disabled. Advertised.listeners block should be called irrespective of kerberos enabled or disabled and depending on if someone has set this property in config or not.

How was this patch tested?

Made changes to file and restarted kafka-broker. Advertised.listeners have hostname instead of localhost

Please review Ambari Contributing Guide before opening a pull request.

Mar 18 '19 08:03 ankur-gupta-guavus

Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5009/ Test PASSed.

Mar 18 '19 09:03 asfgit

@ishanbha : can you please review this PR ?

Mar 26 '19 16:03 ankur-gupta-guavus