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

test(smoketest): use -agent-equipped -reports sidecar #1048

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

andrewazores
Copy link
Member

@andrewazores andrewazores commented Aug 18, 2022

Based on top of #1028
Related to #520

This PR builds on top of #1028 with the pluggable discovery API and combines cryostatio/cryostat-reports#45 / cryostatio/cryostat-agent#4 for testing/demonstration purposes. The end result when everything is rebuilt, configured (as in smoketest.sh here), and run, is that the cryostat-reports sidecar container is deployed with a cryostat-agent attached. -reports no longer enables JMX and so is not connectable for typical Cryostat actions like listing event types or manually starting recordings, but it is still available for its report generation sidecar purpose. The attached -agent is configured to talk to the Cryostat server instance. It registers itself as a Discovery Plugin via the new #1028 API, then publishes a self-description discovery node with some metadata about itself and a dummy http connectUrl. It then starts a managed Flight Recording within the cryostat-reports application and periodically pushes JFR binary files to the Cryostat server archives.

@andrewazores andrewazores added the feat New feature or request label Aug 18, 2022
@andrewazores andrewazores force-pushed the reports-with-agent branch 3 times, most recently from 62b830d to 980f1a8 Compare August 23, 2022 19:02
@andrewazores andrewazores force-pushed the reports-with-agent branch 5 times, most recently from 5ba4483 to a194ed7 Compare September 20, 2022 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat New feature or request safe-to-test
Projects
No open projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

None yet

1 participant