[Tracker]: Support versions for pipelines

[andrewballantyne]

Description

We don't currently show Pipeline Versions in the Dashboard UI, only the first default version associated to a Pipeline. We would like to show all versions and have the proper selection throughout to support them.

Target Branch

f/pipelines-enhancement

Requirements

P0: Ability to add new pipeline versions and access previous versions in the UI
Can do runs against previous versions

Itemized UX Issues

TBD

Itemized Dev Issues

Infrastructure:

• [Story]: Create the Pipeline Version API #2191

Global pipelines page:

• [Story]: Refactor pipelines table list #2196
• [Story]: Create pipeline versions table in pipeline table expandable section on global pipelines page #2199

Pipeline details page:

• [Story]: Improve the toolbar on the pipeline details page #2197

Pipeline runs:

• [Story]: Add pipeline version selection when creating pipeline runs #2194
• [Story]: Improve pipeline runs table #2200

Components:

• [Story]: Create the menu for pipeline or pipeline version selection #2192
• [Story]: Upload pipeline version modal #2190
• [Story]: Refactor deletion modal for pipelines and pipeline versions #2204
• [Story]: Pipeline versions list in the dropdown menu on the pipeline details page #2198

Other items:

• [Bug]: Manually triggered pipeline runs do not include the latest pipeline version #2014
• [Feature Request]: Allow simultaneous deletion of several pipelines. #1961

Related artifacts

• Mocks: https://www.sketch.com/s/6c72b6b2-2c32-45ff-b07c-381d9d8c8267/a/lRzqa1x
• API Docs: https://www.kubeflow.org/docs/components/pipelines/v1/reference/api/kubeflow-pipeline-api-spec/

[andrewballantyne]

cc @yannnz @kywalker-rh

[yannnz]

This is the latest design mockup: https://www.sketch.com/s/6c72b6b2-2c32-45ff-b07c-381d9d8c8267/a/lRzqa1x

There are still two things that we need to decide:

1. Can the backend auto generates the pipeline version name with version number rather than following the KBF's long timestamp format? Waiting for Harshad to confirm.
2. Do we want to provide filtering Runs/Jobs by Pipeline names?

cc: @andrewballantyne @uidoyen @kywalker-rh
cc: @kaedward Could you help to review the content? Thanks!

[andrewballantyne]

@yannnz

(1) might be possible, but that shouldn't hold us up right? What design hinders on the answer to this question?

(2) Is not possible based on Humair's findings. My opinion is we just scratch that one out fully -- no filtering on Pipelines at all; only Pipeline Versions (which is very unfortunate for our UI).

[yannnz]

@yannnz

(1) might be possible, but that shouldn't hold us up right? What design hinders on the answer to this question?

(2) Is not possible based on Humair's findings. My opinion is we just scratch that one out fully -- no filtering on Pipelines at all; only Pipeline Versions (which is very unfortunate for our UI).

The first point won't hold us up. It will only need to display very long names in UI.
Agree with the second point.

[yannnz]

@andrewballantyne @uidoyen I got two questions when I was preparing the demo for Thursday's UX review meeting. It is specific for the Pipeline dropdown and Pipeline version dropdown. Will we have the same issue as what we have on table list to load pipeline options and version options all in once when the list is long? I think we will, so I added a solution for it. The demo only shows the dropdown for Create a run modal. But if this case effects all Pipeline/Pipeline version dropdowns, we can apply the same approach: https://www.sketch.com/s/6c72b6b2-2c32-45ff-b07c-381d9d8c8267/a/L495k0L

Demo recording (24min27s)
Question 1: 15:45
Question 2: 17:00

[yannnz]

The Pipeline & Pipeline version dropdown are updated. Please help to review the design :
Mockup of Pipeline & Pipeline version dropdown @andrewballantyne @kywalker-rh

[andrewballantyne]

@yannnz are you completely done your first pass of Versions or are we still looking for more? I'm wondering if I should log you a UX issue to finalize the effort here or if the latest mocks (https://www.sketch.com/s/6c72b6b2-2c32-45ff-b07c-381d9d8c8267/p/52D4BC1C-F002-4775-9C69-7E82C519E931/canvas) are good to go? I'd like to break down the UI stories soon too.

cc @kywalker-rh @dgutride

[yannnz]

@yannnz are you completely done your first pass of Versions or are we still looking for more? I'm wondering if I should log you a UX issue to finalize the effort here or if the latest mocks (https://www.sketch.com/s/6c72b6b2-2c32-45ff-b07c-381d9d8c8267/p/52D4BC1C-F002-4775-9C69-7E82C519E931/canvas) are good to go? I'd like to break down the UI stories soon too.

cc @kywalker-rh @dgutride

@andrewballantyne The latest mockups are good to go from my side.

[yannnz]

@andrewballantyne I am taking back my words. One last thing I forgot to check is the delete action. I am proposing to have the bulk action to the pipeline and pipeline version data lists, and change the Delete pipeline button to Delete depends on what are selected.

(This model is similar to Delete run modal )

[andrewballantyne]

@yannnz @kywalker-rh and I talked about it on slack... we will keep the delete the same effort all around and use the toolbar kebab "delete selected" functionality. Bulk delete is bulk delete. We may need additional wording around deleting a Pipeline as all Versions will go with it (99% certain I asked the backend team of this and got that answer)

[yannnz]

@yannnz @kywalker-rh and I talked about it on slack... we will keep the delete the same effort all around and use the toolbar kebab "delete selected" functionality. Bulk delete is bulk delete. We may need additional wording around deleting a Pipeline as all Versions will go with it (99% certain I asked the backend team of this and got that answer)

@kaedward Could you help on the content of the delete modal? I tagged you in the sketch file.
https://sketch.com/s/6c72b6b2-2c32-45ff-b07c-381d9d8c8267/a/mPw0gLL?annotation=bcb561f4-c120-4fd8-8e71-031a6ef6b216

[yannnz]

This is the latest design link for implementation: https://www.sketch.com/s/6c72b6b2-2c32-45ff-b07c-381d9d8c8267/a/lRzqa1x

@kaedward There were several design updates after your latest review and needs your help. I @ you in those updates.

[dgutride]

Replaced by: https://issues.redhat.com/browse/RHOAIENG-262

[dgutride]

Moving to closed. . .