Skip to content

Add cassanda jmx to discovery #5112

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

Merged
merged 11 commits into from
Jul 29, 2024
Merged

Add cassanda jmx to discovery #5112

merged 11 commits into from
Jul 29, 2024

Conversation

hughesjj
Copy link
Contributor

to test locally, first start a cassandra cluster with docker compose --file docker/docker-compose.yml --profile cassandra up

Then, make bundle.d && make otelcol && make integration-test-jmx-discovery

@hughesjj
Copy link
Contributor Author

timeout issue, gotta figure out how to best wait for

CassandraDaemon.java:738 - Startup complete

@hughesjj hughesjj force-pushed the add_cassanda_jmx branch 3 times, most recently from e7ce63e to ddd0068 Compare July 22, 2024 18:30
@hughesjj hughesjj marked this pull request as ready for review July 22, 2024 19:42
@hughesjj hughesjj requested review from a team as code owners July 22, 2024 19:42
@hughesjj hughesjj force-pushed the add_cassanda_jmx branch 3 times, most recently from 71294b2 to 5f383f2 Compare July 22, 2024 20:34
@samiura
Copy link

samiura commented Jul 24, 2024

@hughesjj LGTM but may be we should wait for jeff's comment.

@samiura
Copy link

samiura commented Jul 24, 2024

needs to rebase as well

@hughesjj hughesjj merged commit 1728aa7 into main Jul 29, 2024
129 of 131 checks passed
@hughesjj hughesjj deleted the add_cassanda_jmx branch July 29, 2024 15:49
@github-actions github-actions bot locked and limited conversation to collaborators Jul 29, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants