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

feat(agent): implement Agent HTTP recording retrieval #1607

Conversation

andrewazores
Copy link
Member

@andrewazores andrewazores commented Aug 4, 2023

Welcome to Cryostat! 👋

Before contributing, make sure you have:

  • Read the contributing guidelines
  • Linked a relevant issue which this PR resolves
  • Linked any other relevant issues, PR's, or documentation, if any
  • Resolved all conflicts, if any
  • Rebased your branch PR on top of the latest upstream main branch
  • Attached at least one of the following labels to the PR: [chore, ci, docs, feat, fix, test]
  • Signed all commits using a GPG signature

To recreate commits with GPG signature git fetch upstream && git rebase --force --gpg-sign upstream/main


Based on #1604
Depends on cryostatio/cryostat-agent#179
Fixes #1582

Description of the change:

This replaces stubs with real implementations for pulling JFR files from Cryostat Agent instances over HTTP.

Motivation for the change:

Users can deploy their workload applications with the Cryostat Agent and dynamically start/analyze/retrieve/stop/delete recordings all over the Agent HTTP API rather than over JMX.

How to manually test:

  1. See feat(agent): implement Agent HTTP dynamic JFR stop/delete #1604 and feat(api): implement GET /recordings/:id for streaming files cryostat-agent#179

@github-actions github-actions bot added needs-triage Needs thorough attention from code reviewers dependent labels Aug 4, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Aug 4, 2023

This PR/issue depends on:

@mergify mergify bot added the safe-to-test label Aug 4, 2023
@andrewazores andrewazores changed the base branch from main to 1578-epic-two-way-agent-communications August 4, 2023 19:15
@andrewazores andrewazores added feat New feature or request and removed needs-triage Needs thorough attention from code reviewers labels Aug 9, 2023
@andrewazores andrewazores marked this pull request as ready for review August 14, 2023 15:40
@andrewazores andrewazores merged commit 5b947e0 into cryostatio:1578-epic-two-way-agent-communications Aug 14, 2023
16 checks passed
@andrewazores andrewazores deleted the agent-recording-stream branch August 14, 2023 17:57
aali309 pushed a commit to aali309/cryostat-legacy that referenced this pull request Aug 27, 2023
aali309 pushed a commit to aali309/cryostat-legacy that referenced this pull request Sep 7, 2023
andrewazores added a commit that referenced this pull request Sep 19, 2023
* feat(agent): implement Agent HTTP dynamic JFR start (#1566)

* chore(svc): extract EventOptionsBuilder to -core and use new CryostatFlightRecorderService

* test(smoketest): enable API writes on one agent-equipped sample app

* chore(serial): extract recording descriptor to -core

* chore(activerecordings): clean up an error handler

* feat(agent): implement dynamic start of JFR over HTTP

* bump -core version

* feat(agent): implement Agent HTTP dynamic JFR stop/delete (#1604)

* feat(agent): implement Agent HTTP recording retrieval (#1607)

* feat(agent): implement HTTP JFR snapshot creation (#1627)

Co-authored-by: Atif Ali <[email protected]>
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: Done
Development

Successfully merging this pull request may close these issues.

[Story] Implement HTTP JFR file pull
2 participants