Skip to content
This repository has been archived by the owner on Nov 12, 2023. It is now read-only.

Initial metadata support for output parameters #12

Merged
merged 3 commits into from
Jun 5, 2020

Conversation

Tomcli
Copy link
Member

@Tomcli Tomcli commented Jun 3, 2020

Initial metadata support for output parameters

  • generalize metadata writer to support Tekton pods
  • add basic artifact support for parameter artifacts (Will add the rest of the artifact type once big data passing is finalized)

@kubeflow-bot
Copy link

This change is Reviewable

@Tomcli
Copy link
Member Author

Tomcli commented Jun 3, 2020

related #8, #9

Tomcli added 2 commits June 3, 2020 17:02
* init tekton backend

* add missing comments

* remove cache server changes in this pr

* initial parameter artifact support
Copy link
Contributor

@animeshsingh animeshsingh 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

@k8s-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: animeshsingh

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 32b502e into kubeflow:master Jun 5, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants