[Kafka integration] Provide a clear guidance for Kafka hosts

[lucabelluccini]

Proposed commit message

In order for Kafka integration to work properly, a user must provide all broker hosts.
The description should help users

Checklist

• None applies to this PR

Related issues

• Relates Kafka: Can we specify a single broker to represent all the brokers in a cluster? beats#34053

[elasticmachine]

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

[elastic-sonarqube]

Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

[kush-elastic]

LGTM!

[lucabelluccini]

Hello @kush-elastic - I've also updated the docs and the changelog. Is there a way to preview the docs & the Fleet UI if it renders fine?
Once done, I think we can merge.

[kush-elastic]

Hey @lucabelluccini,
You can test this by following the steps mentioned in the Developer guide.

You can setup local stack using elastic-package and checkout how changes looks like in local kibana.

[kush-elastic]

/test

[lucabelluccini]

Tested locally.
Learnt about the _dev for the README.md TIL.

As I am not usually contributing to integration, would it be possible for the maintainers to follow up with the version bump, merge & release?

[ishleenk17]

Please address the review comments @lucabelluccini .
We can then proceed

[ishleenk17]

/test

[ishleenk17]

Learnt about the _dev for the README.md TIL.

@lucabelluccini : You have to make the change just to README under _dev. And run elastic-packagae build.
The actual README will automatically get generated.

[lucabelluccini]

Learnt about the _dev for the README.md TIL.

@lucabelluccini : You have to make the change just to README under _dev. And run elastic-packagae build. The actual README will automatically get generated.

Yep, that's the Today I Learnt :) moment

[botelastic]

Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as Stale to make it hit our filters and make sure we get back to it as soon as possible. In the meantime, it'd be extremely helpful if you could take a look at it as well and confirm its relevance. A simple comment with a nice emoji will be enough :+1. Thank you for your contribution!

[devamanv]

@lucabelluccini Can we please address the suggested changes by @ishleenk17, so that we can close this one?

[botelastic]

Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as Stale to make it hit our filters and make sure we get back to it as soon as possible. In the meantime, it'd be extremely helpful if you could take a look at it as well and confirm its relevance. A simple comment with a nice emoji will be enough :+1. Thank you for your contribution!

[elasticmachine]

💔 Build Failed

• Buildkite Build
• Commit: 5e21142

Failed CI Steps

• Check integrations kafka

History

• 💔 Build #10211 failed c368c13
• 💔 Build #10192 failed c368c13
• 💔 Build #10074 failed 18086ed
• 💔 Build #10073 failed 18086ed
• 💔 Build #10072 failed 6d332af

cc @lucabelluccini

[lucabelluccini]

Hello all,

I would like to get suggestions on which would be the messaging we should promote (to replace the link to the Beats repo issue).

To me it seems an implementation limitation. Removing the link we lose the background/context of the issue.

Let me know how you would like to proceed.

[botelastic]

Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as Stale to make it hit our filters and make sure we get back to it as soon as possible. In the meantime, it'd be extremely helpful if you could take a look at it as well and confirm its relevance. A simple comment with a nice emoji will be enough :+1. Thank you for your contribution!

[lucabelluccini]

bump

[botelastic]

Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as Stale to make it hit our filters and make sure we get back to it as soon as possible. In the meantime, it'd be extremely helpful if you could take a look at it as well and confirm its relevance. A simple comment with a nice emoji will be enough :+1. Thank you for your contribution!

[lucabelluccini]

Bump

[botelastic]

Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as Stale to make it hit our filters and make sure we get back to it as soon as possible. In the meantime, it'd be extremely helpful if you could take a look at it as well and confirm its relevance. A simple comment with a nice emoji will be enough :+1. Thank you for your contribution!