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.

Sign up for GitHub

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

Jump to bottom

update pipeline system images to 0.1.13 release #2785

Merged
merged 27 commits into from
Mar 26, 2019
Merged

update pipeline system images to 0.1.13 release #2785

merged 27 commits into from
Mar 26, 2019

Conversation

IronPan
Copy link
Member

@IronPan IronPan commented Mar 23, 2019 •
edited
Loading

Also sync the pipelines branch to master(f32efbc). there are some breaking changes preventing test from passing.

This change is Reviewable

IronPan and others added 22 commits February 5, 2019 01:10
@IronPan
sync pipelines branch to master
2808edd
@IronPan
add pipeline team to the owner file (kubeflow#2299)
cdeccdd 
* add pipeline team to the owner file

* Update OWNERS
@IronPan
Merge remote-tracking branch 'upstream/pipelines' into pipelines
0154e92
@IronPan
fix
1842928
@neuromage @IronPan
Specify CRD version using v1beta1 schema (kubeflow#2424)
35cf64c
@IronPan
Bump pipeline version to 0.1.9 (kubeflow#2427)
28aaef5
@IronPan
deploy storage through kfctl.sh (kubeflow#2373)
7a3e15e 
* deploy storage through kfctl.sh

* revert the change to delete storage DM

* propagate feature flag

* skip set param conditionally

* avoid exposing permanent storage as high level flag

* delete storage in test

* update test template

* add quite

* delete storage after cluster deletion
@IronPan
Wait for at most 100s on namespace deletion (kubeflow#2409)
6b3c05f 
* delete namespace conditionally

* delete namespace conditionally

* address comments

* Update kfctl.sh
@IronPan
Merge remote-tracking branch 'upstream/pipelines' into pipelines
7155c3c
@IronPan
fix
fca726e
@IronPan
fix (kubeflow#2446)
1842c51
@neuromage @IronPan
Deploy viewer CRD controller for Kubeflow Pipelines. (kubeflow#2416)
4bbe4ff 
* Deploy viewer CRD controller for Kubeflow Pipelines.

* Add missing image variable

* Use new schema to specify versions.
@IronPan
Reduce NFS disk size from 200 -> 20 GB (kubeflow#2464)
b1b0a78 
Since the corresponding PVC only need 20Gb.

/assign @kunmingg
@IronPan
add selector for minio deploy (kubeflow#2466)
92155ef
@IronPan
Merge remote-tracking branch 'upstream/pipelines' into pipelines
eed5f37
@IronPan
merge
247cad4
@IronPan
default to minio (kubeflow#2503)
b824d51
@IronPan
Add script to upgrade kubeflow pipeline (kubeflow#2479)
5ede033 
* create default environment during  step

* add script to upgrade pipeline

* polish script

* add ks apply

* address comment

* update script
@IronPan
update pipeline image version to v0.1.10 (kubeflow#2534)
94bde86
@IronPan
merge
5929b40
@IronPan
update pipeline version to v0.1.13 (kubeflow#2783)
13578cc 
/assign @neuromage
@IronPan
update
5a3cd56
@IronPan
Copy link
Member Author

IronPan commented Mar 23, 2019

/assign @rileyjbauer

@IronPan
Copy link
Member Author

IronPan commented Mar 23, 2019

/test kubeflow-presubmit

1 similar comment
@neuromage
Copy link
Contributor

/test kubeflow-presubmit

@IronPan
Copy link
Member Author

IronPan commented Mar 25, 2019

/test kubeflow-presubmit

@rileyjbauer
Copy link

/lgtm

@IronPan
Copy link
Member Author

IronPan commented Mar 25, 2019

/approve

@k8s-ci-robot k8s-ci-robot removed the lgtm label Mar 25, 2019
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: IronPan

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

@rileyjbauer
Copy link

/lgtm

@Ark-kun
Copy link
Contributor

Ark-kun commented Mar 25, 2019

/test kubeflow-presubmit

1 similar comment
@IronPan
Copy link
Member Author

IronPan commented Mar 25, 2019

/test kubeflow-presubmit

@k8s-ci-robot k8s-ci-robot removed the lgtm label Mar 25, 2019
@Ark-kun
Copy link
Contributor

Ark-kun commented Mar 25, 2019

/lgtm

@IronPan
Copy link
Member Author

IronPan commented Mar 26, 2019

/test kubeflow-presubmit

@k8s-ci-robot
Copy link
Contributor

New changes are detected. LGTM label has been removed.

@k8s-ci-robot k8s-ci-robot removed the lgtm label Mar 26, 2019
@IronPan
Copy link
Member Author

IronPan commented Mar 26, 2019

/assign @jlewi

@jlewi
Copy link
Contributor

jlewi commented Mar 26, 2019

Force merge @IronPan's request.

@jlewi jlewi merged commit a5ca137 into kubeflow:pipelines Mar 26, 2019
@k8s-ci-robot
Copy link
Contributor

@IronPan: The following test failed, say /retest to rerun them all:

Test name Commit Details Rerun command
kubeflow-presubmit 88ed6f4 link /test kubeflow-presubmit

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ellistarn ellistarn Awaiting requested review from ellistarn

@neuromage neuromage Awaiting requested review from neuromage

Assignees

@jlewi jlewi

@Ark-kun Ark-kun

@rileyjbauer rileyjbauer

Labels
approved cla: yes size/XXL
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@IronPan @neuromage @rileyjbauer @k8s-ci-robot @Ark-kun @jlewi @googlebot