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

Testing - Using Argo validation in Travis tests #3123

Conversation

Ark-kun
Copy link
Contributor

@Ark-kun Ark-kun commented Feb 19, 2020

Add Argo CLI so that the validation introduced in #3094 is triggered.

This change is Reviewable

@numerology
Copy link

/lgtm

@Ark-kun Ark-kun force-pushed the Testing---Using-Argo-validation-in-Travis-tests branch from 949d1ab to 12b097d Compare February 19, 2020 21:21
@k8s-ci-robot k8s-ci-robot removed the lgtm label Feb 19, 2020
@Ark-kun Ark-kun force-pushed the Testing---Using-Argo-validation-in-Travis-tests branch from 12b097d to 7cf2448 Compare February 19, 2020 21:33
:# Date:      Wed Feb 19 13:14:09 2020 -0800
@Ark-kun Ark-kun force-pushed the Testing---Using-Argo-validation-in-Travis-tests branch from 7cf2448 to 18c6372 Compare February 19, 2020 21:45
@Ark-kun
Copy link
Contributor Author

Ark-kun commented Feb 20, 2020

/retest

1 similar comment
@Ark-kun
Copy link
Contributor Author

Ark-kun commented Feb 20, 2020

/retest

@Ark-kun
Copy link
Contributor Author

Ark-kun commented Feb 20, 2020

/retest

@numerology
Copy link

/lgtm

@Ark-kun
Copy link
Contributor Author

Ark-kun commented Mar 11, 2020

/cc @IronPan

@Ark-kun
Copy link
Contributor Author

Ark-kun commented Mar 20, 2020

/retest

@Ark-kun
Copy link
Contributor Author

Ark-kun commented Apr 11, 2020

Friendly ping

@IronPan
Copy link
Member

IronPan commented May 27, 2020

/approve

@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

@k8s-ci-robot k8s-ci-robot merged commit 7d94de1 into kubeflow:master May 27, 2020
akartsky pushed a commit to akartsky/pipelines that referenced this pull request May 29, 2020
* Testing - Using Argo validation in Travis tests

:# Date:      Wed Feb 19 13:14:09 2020 -0800

* Fixed typo
akartsky pushed a commit to akartsky/pipelines that referenced this pull request May 29, 2020
* Testing - Using Argo validation in Travis tests

:# Date:      Wed Feb 19 13:14:09 2020 -0800

* Fixed typo
akartsky pushed a commit to akartsky/pipelines that referenced this pull request May 29, 2020
* Testing - Using Argo validation in Travis tests

:# Date:      Wed Feb 19 13:14:09 2020 -0800

* Fixed typo
akartsky pushed a commit to akartsky/pipelines that referenced this pull request May 29, 2020
* Testing - Using Argo validation in Travis tests

:# Date:      Wed Feb 19 13:14:09 2020 -0800

* Fixed typo
akartsky pushed a commit to akartsky/pipelines that referenced this pull request May 29, 2020
* Testing - Using Argo validation in Travis tests

:# Date:      Wed Feb 19 13:14:09 2020 -0800

* Fixed typo
akartsky pushed a commit to akartsky/pipelines that referenced this pull request May 29, 2020
* Testing - Using Argo validation in Travis tests

:# Date:      Wed Feb 19 13:14:09 2020 -0800

* Fixed typo
akartsky pushed a commit to akartsky/pipelines that referenced this pull request May 29, 2020
* Testing - Using Argo validation in Travis tests

:# Date:      Wed Feb 19 13:14:09 2020 -0800

* Fixed typo
Jeffwan pushed a commit to Jeffwan/pipelines that referenced this pull request Dec 9, 2020
* Testing - Using Argo validation in Travis tests

:# Date:      Wed Feb 19 13:14:09 2020 -0800

* Fixed typo
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.

4 participants