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: adds project-level executions view #135

Merged
merged 7 commits into from
Jan 9, 2021
Merged

Conversation

schottra
Copy link
Contributor

@schottra schottra commented Jan 7, 2021

flyteorg/flyte#581

This adds a new tab/view to the ProjectDetails page which will show executions for a given project/domain combo. It uses the existing infrastructure for rendering a scoped project-level view and uses the recent react-query patterns and useInfiniteQuery to power the list (instead of the legacy useWorkflowExecutions hook).

  • Added routes and view component for project executions
  • Refactored list/table components to not be dependent on a State object just to determine if a fetch is in progress. They now accept a simple boolean which can be derived from a State using isLoadingState or from a QueryObserver using query.isFetching.
  • Added a simple query wrapper for our common pagination pattern using token that is compatible with useInfiniteQuery
  • Added workflowExecutionQueries module with a query generator for a list of Workflow Executions based on the new query wrapper.
  • Added logic for displaying a WorkflowExecutionsTable based on the results of our new executions query.
  • Added prop to WorkflowExecutionsTable that controls showing the workflow name of an execution in place of the Last run... string. This is useful in the generic case where the table is showing a mixture of executions from different workflows and the execution name/link isn't enough information to distinguish between them.
  • Updated styling on the table to wrap the name column in the case that the workflow name is too long. Made the column slightly wider (table still fits in a 1440x900 screen).
  • Split up WorkflowExecutionsTable and added a simplified mock version which does not use react-virtualized to make it possible to test views which use the table (Autosizer and CellMeasurer do not work in jsdom).
  • Added support to mock server for specifying a listWorkflowExecutions response, which is used in the tests for ProjectExecutions

image

@codecov-io
Copy link

codecov-io commented Jan 7, 2021

Codecov Report

Merging #135 (d84390b) into master (928f094) will increase coverage by 1.07%.
The diff coverage is 83.61%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #135      +/-   ##
==========================================
+ Coverage   73.35%   74.43%   +1.07%     
==========================================
  Files         411      415       +4     
  Lines        7222     7310      +88     
  Branches     1129     1154      +25     
==========================================
+ Hits         5298     5441     +143     
+ Misses       1924     1869      -55     
Impacted Files Coverage Δ
src/components/Executions/Tables/constants.ts 100.00% <ø> (ø)
src/components/Project/ProjectLaunchPlans.tsx 0.00% <0.00%> (ø)
src/components/Project/ProjectRecentExecutions.tsx 0.00% <0.00%> (ø)
src/components/Project/ProjectSchedules.tsx 0.00% <0.00%> (ø)
src/components/Tables/DataList.tsx 27.58% <ø> (ø)
src/components/Tables/DataTable.tsx 28.26% <ø> (ø)
src/components/Tables/LoadMoreRowContent.tsx 47.05% <0.00%> (-2.95%) ⬇️
src/routes/routes.ts 50.00% <0.00%> (-2.28%) ⬇️
src/components/Navigation/ProjectNavigation.tsx 39.53% <16.66%> (-3.71%) ⬇️
...ents/Executions/Tables/WorkflowExecutionsTable.tsx 37.14% <60.00%> (+7.65%) ⬆️
... and 33 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b6c8ca2...d84390b. Read the comment docs.

@schottra schottra requested review from BobNisco and podehnal January 9, 2021 01:11
BobNisco
BobNisco previously approved these changes Jan 9, 2021
@service-github-lyft-semantic-release
Copy link
Contributor

🎉 This PR is included in version 0.19.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants