Skip to content
This repository has been archived by the owner on Dec 2, 2021. It is now read-only.

[MLMD Lineage] Remove LineageView and use version from kubeflow/frontend #208

Merged
merged 24 commits into from
Jan 24, 2020

Conversation

kwasi
Copy link
Contributor

@kwasi kwasi commented Jan 23, 2020

/kind feature
/area metadata
/area front-end
/assign @avdaredevil

This PR removes the version of LineageView that was developed in the metadata
repo in favor the shared version in kubeflow/frontend.

Changes

  • There should be no changes in this PR, visually or logic wise

Validation:

Deployment: http://kubeflow-ui-07.endpoints.kwasinti-kubeflow-dev.cloud.goog


This change is Reviewable

Will update to kubeflow/frontend once changes are merged
The relevant classes are available in frontend/mlmd.
This is the last essential change, so this branch is ready for PR once
frontend changes are merged and synced with outstanding PRs.
@kwasi kwasi moved this from To do to In progress in Pipelines: MLMD Lineage FE Jan 23, 2020
@kwasi kwasi moved this from In progress to Review in progress in Pipelines: MLMD Lineage FE Jan 23, 2020
@kwasi
Copy link
Contributor Author

kwasi commented Jan 24, 2020

@avdaredevil I've updated to the mainline version of kubeflow/frontend so this PR is tested and ready for review.

Copy link
Contributor

@avdaredevil avdaredevil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
/approve

Looks great!

@avdaredevil
Copy link
Contributor

/approve

@k8s-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

Approval requirements bypassed by manually added approval.

This pull-request has been approved by: avdaredevil

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot merged commit c831a2a into kubeflow:master Jan 24, 2020
Pipelines: MLMD Lineage FE automation moved this from Review in progress to Done Jan 24, 2020
@avdaredevil avdaredevil deleted the integrate-frontend-pr branch January 24, 2020 22:51
@avdaredevil avdaredevil assigned kwasi and unassigned avdaredevil Feb 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

Move LineageView components to a standalone repo
3 participants