Add ability to create Snapshot recordings #67
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
andrewazores
force-pushed
the
react-snapshot-recordings
branch
2 times, most recently
from
3c13520 to
c456e4a
Compare
jiekang
approved these changes
May 7, 2020
There was a problem hiding this comment.
Looks fine.
Follow up nit:
Automated analysis for empty recordings (such as a snapshot taken when no recordings have been started) displays just the text:
java.io.IOException: No recording data available
When UX is revisited for the automated analysis, we should address it there too.
andrewazores
force-pushed
the
react-snapshot-recordings
branch
from
c456e4a to
6e68aad
Compare
andrewazores
added a commit
to andrewazores/cryostat-web
that referenced
this pull request
May 19, 2020
* Extract custom recordings form component * Add tabset around card body * Add Snapshot recording form
andrewazores
added a commit
that referenced
this pull request
May 20, 2020
* Initial rewrite commit using patternfly-react-seed * Initial port of command channel service * Initial port of API service * Very rough auth flow support * Use function components to avoid 'this' binding complications * Add component for (dis)connecting targets * Update Select state on selection * Include target connectUrl in select option label * Add ContainerNode icon to Select * Add title label to Select * Use Grid layout * Allow message ID to be supplied by caller * 'Dynamic' routes only added when target connected * Update label using is-connected listener * Remove default Dashboard and Support pages * Add rudimentary recording table view * Update README * Split Login component into React Form components (#52) Basic login form asks for separate username and password, Bearer form asks for just auth token Additionally, use Card styling * Add TargetSelect widget on RecordingList (#51) * Add TargetSelect on recordings list view * Only allow target disconnection from dashboard select * Do not send messages before command channel is ready * Add precondition to sendMessage ensuring channel is ready before messages processed * Refactor to extract reusable TargetView container component * allowDisconnect prop is optional, default false * Each TargetView child becomes its own GridItem * Dashboard reuses TargetView * Update Route page titles to match components * TargetView uses Stack layout rather than Grid * Wrap TargetSelect contents within Card * Wrap RecordingList in Card * Include target port in render and command args * Send user back to login when command channel connection is lost (#53) * Clean up Patternfly Seed remnants (#62) * Replace Seed app title * Remove unused header "Toolbar" example * "Create Recording" view (#59) * Set up "Create Recording" view routing * Add public function for generating message IDs * Implement custom recording creation form * Render continuous recording duration as Continuous rather than 0s * Nest routes, style nav active if route/child match * Add navigable breadcrumbs replacing page titles * Extract BreadcrumbPage component * Validate recording name Add form property to display invalid style when recording name provided is invalid * Constrain duration input to minimum of 0 * Validate recording event specifier string Add form properties to style event specifier string text-area as invalid if the specifier is invalid * Update Template form input if user manually inputs matching name * Add event specifier string tooltip icon * Add global toast notifications * Inform user of form issues and recording creation status via toast * CommandChannel uses Notifications for user error messages * Reformat tooltip for readability * Recording List overhaul as Data List (#63) * Replace React Table with Data List for flexibility Render recording download and report URLs as downloadable links * Add row checkboxes * Implement button for deleting recordings * Implement button for stopping recordings * Extract toolbar to local function component * Append .jfr to recording download request filename * Enable Stop button only when some selected recording is RUNNING/STARTING * Initial Events view (#54) * Add first-pass impls of Events components * Add event template filtering * Add event-types table pagination * Move event type options into collapsible rows And set event tables to compact styling * Add event type filtering * Apply changes from 'npm run format' * Fix uppercase/lowercase filtering bug * Allow comma-delimited category string matches * Re-apply 'npm run format' * Recalculate page number from previous perPage Maintain state for previous count of items per page and recalculate the page number and displayed slice to keep the current items in view when the user changes the count per page * Correct item count when filter text is non-empty * Use eval-source-map in prod mode (#68) Fixes an issue where invalid prod mode source maps cause browsers to be unable to open the client when built in prod mode. See webpack/webpack#8302 * Prefix request paths with /api/v1 (#66) Related to cryostatio/cryostat-legacy#148 * Add ability to create Snapshot recordings (#67) * Extract custom recordings form component * Add tabset around card body * Add Snapshot recording form * Add "Archived Recordings" view tab (#69) * Extract active recording list component * Add Archived Recordings view Add Archived Recordings view under Recordings List tab, visible/enabled only when the ContainerJFR instance supports archiving * Add "Archive" button to active recordings view Add button allowing users to select active recordings and save them to the archives Fixes #57 * Immediately refresh archived list when save message observed * Refactor to extract shared data list table element * Use PF DataToolbar instead of plain Toolbar * Add action button for creating recordings from template (#70) * Add action button for creating recordings from Event Templates Related to #58 * Restyle toolbars using PF DataToolbar * Do not (un)mount child components on tab selection
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #58
Based on top of #66
This PR puts the existing recording creation form into a tab within its card and adds a second tab which contains a simple "form" explaining what JFR Snapshots are and allowing the user to create a Snapshot by clicking the "Create" button.
Also, the two tabbed form components are not unmounted when not visible, so the "custom recording" form state will be preserved if the user switches between custom and snapshot tabs.