Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Start a kafka cluster for integration testing #14087

Closed
wants to merge 3 commits into from

Conversation

jsoriano
Copy link
Member

@jsoriano jsoriano commented Oct 16, 2019

No description provided.

@jsoriano jsoriano added in progress Pull request is currently in progress. module Metricbeat Metricbeat [zube]: In Progress Team:Integrations Label for the Integrations team labels Oct 16, 2019
@jsoriano jsoriano self-assigned this Oct 16, 2019
@ChrsMark
Copy link
Member

Any thoughts about exposing JMX metrics too (like https://github.com/wurstmeister/kafka-docker#jmx)? It would be sth we could need in the future.

@@ -0,0 +1,52 @@
version: '2.3'

services:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

really happy to see this new file :)

@jsoriano
Copy link
Member Author

Any thoughts about exposing JMX metrics too (like https://github.com/wurstmeister/kafka-docker#jmx)? It would be sth we could need in the future.

I guess you are thinking on #13366 or something similar? 🙂 For that we could install the Jolokia agent and add a light metricset based on jolokia to collect these metrics. I used Kafka as example to install the Jolokia agent in this article.

@mergify
Copy link
Contributor

mergify bot commented Sep 22, 2021

This pull request does not have a backport label. Could you fix it @jsoriano? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-v./d./d./d is the label to automatically backport to the 7./d branch. /d is the digit

NOTE: backport-skip has been added to this pull request.

@mergify mergify bot added the backport-skip Skip notification from the automated backport with mergify label Sep 22, 2021
@mergify
Copy link
Contributor

mergify bot commented Nov 10, 2021

This pull request is now in conflicts. Could you fix it? 🙏
To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b kafka-cluster-test upstream/kafka-cluster-test
git merge upstream/master
git push upstream kafka-cluster-test

@jsoriano jsoriano closed this Nov 10, 2021
@zube zube bot removed the [zube]: Done label Feb 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-skip Skip notification from the automated backport with mergify in progress Pull request is currently in progress. Metricbeat Metricbeat module Team:Integrations Label for the Integrations team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants