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

Adding JMX support to e2e tests #1248

Merged

Conversation

carlosroman
Copy link
Contributor

@carlosroman carlosroman commented Nov 15, 2024

What does this PR do?

It adds support to be able to use the JMX container version of the Agent in e2e tests.

Which scenarios this will impact?

Just AML tests.

Motivation

I'd like to not have to manually test JMXFetch changes.

Additional Notes

Have also updated the JMXFetch sample app with some helper manifests to allow JMXFetch to grab metrics.

@carlosroman carlosroman requested review from a team as code owners November 15, 2024 16:06
@carlosroman carlosroman force-pushed the carlosroman/AMLII-1076-adding-JMX-support-to-e2e-tests branch from 3466bcf to 5929c82 Compare November 20, 2024 10:44
@carlosroman carlosroman force-pushed the carlosroman/AMLII-1076-adding-JMX-support-to-e2e-tests branch from 5929c82 to f6b9625 Compare November 21, 2024 09:54
@carlosroman
Copy link
Contributor Author

/merge

@dd-devflow
Copy link

dd-devflow bot commented Nov 21, 2024

Devflow running: /merge

View all feedbacks in Devflow UI.


2024-11-21 12:14:59 UTC ℹ️ MergeQueue: pull request added to the queue

The median merge time in main is 20m.

@dd-mergequeue dd-mergequeue bot merged commit aa7598a into main Nov 21, 2024
24 checks passed
@dd-mergequeue dd-mergequeue bot deleted the carlosroman/AMLII-1076-adding-JMX-support-to-e2e-tests branch November 21, 2024 12:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants