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

fix(Itest): fix GraphQLTest.java #316

Closed
wants to merge 20 commits into from

Conversation

aali309
Copy link
Contributor

@aali309 aali309 commented Mar 6, 2024

Welcome to Cryostat3! 👋

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: #294

Description of the change:

This change allows an environment variable to be configured so that...

Motivation for the change:

This change is helpful because users may want to...

How to manually test:

  1. Run CRYOSTAT_IMAGE=quay.io... bash smoketest.bash...
  2. ...

@aali309 aali309 added the fix label Mar 6, 2024
@github-actions github-actions bot added the needs-triage Needs thorough attention from code reviewers label Mar 6, 2024
@aali309 aali309 removed the needs-triage Needs thorough attention from code reviewers label Mar 6, 2024
@andrewazores andrewazores force-pushed the graphql branch 2 times, most recently from 8255856 to cead773 Compare March 11, 2024 14:39
@andrewazores andrewazores force-pushed the graphql branch 2 times, most recently from 1a90348 to 7eebd7d Compare March 20, 2024 17:55
andrewazores and others added 19 commits March 22, 2024 09:30
* feat(graphql): add top-level query for targetNodes

* return targets with distinct JVM IDs, implement archived recordings query

* implement active recordings query

* refactor enum registration

* implement subqueries (actually nested mutations) for doStartRecording and doSnapshot

* add mbeanMetrics query

* messaging server uses shared ObjectMapper

* add Jackson serialization customizer so Map<String, String> is encoded the same way GraphQL extension does
…ysis card (cryostatio#312)

* feat(graphql): implement features to support dashboard automated analysis card

* disable JVM ID node filtering, populate archived recordings query with data

* populate aggregate size

* refactor

* refactor to split out class
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants