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

Releasing component SDK #842

Merged
merged 4 commits into from
Feb 25, 2019
Merged

Conversation

hongye-sun
Copy link
Contributor

@hongye-sun hongye-sun commented Feb 21, 2019

The SDK is shipped with:

  1. A python package called kfp-component.tar.gz in the release folder
  2. A container called ml-pipeline-gcp with the sdk installed

This change is Reviewable

@hongye-sun
Copy link
Contributor Author

/test kubeflow-pipeline-e2e-test

@@ -43,6 +43,7 @@ images=(
"resnet-train"
"ml-pipeline-local-confusion-matrix"
"ml-pipeline-local-roc"
"ml-pipeline-gcp"
Copy link
Contributor

Choose a reason for hiding this comment

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

Did you get through the review process for new images?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the reminder. I just submitted the request.

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks. Let me know when it is accepted.

@vicaire
Copy link
Contributor

vicaire commented Feb 25, 2019

/lgtm
/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: vicaire

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

1 similar comment
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: vicaire

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 26622b7 into kubeflow:master Feb 25, 2019
cheyang pushed a commit to alibaba/pipelines that referenced this pull request Mar 28, 2019
* add container image uris support in train API

* Build GCP container

* Support releasing component SDK with docker and python package.

* Add licenses to image
Linchin pushed a commit to Linchin/pipelines that referenced this pull request Apr 11, 2023
* Add tf operator presubmit to share test infra config

* run optional-generate for tf-operator
HumairAK pushed a commit to red-hat-data-services/data-science-pipelines that referenced this pull request Mar 11, 2024
* m: typo in a comment

* remove separator param in loop spec

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

Successfully merging this pull request may close these issues.

3 participants