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] Fix travis error: "virtualenv 20.0.1 has requirement six<2,>=1.12.0, but you'll have six 1.11.0 which is incompatible." #3039

Closed
wants to merge 2 commits into from

Conversation

Bobgy
Copy link
Contributor

@Bobgy Bobgy commented Feb 11, 2020

/cc @numerology
This is blocking all presubmit tests.
/priority p0
/kind bug


This change is Reviewable

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign hongye-sun
You can assign the PR to them by writing /assign @hongye-sun in a comment when ready.

The full list of commands accepted by this bot can be found 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
Copy link
Contributor

@Bobgy: The following tests failed, say /retest to rerun all failed tests:

Test name Commit Details Rerun command
kubeflow-pipeline-sample-test b739c90 link /test kubeflow-pipeline-sample-test
kubeflow-pipeline-e2e-test b739c90 link /test kubeflow-pipeline-e2e-test

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.

@Bobgy
Copy link
Contributor Author

Bobgy commented Feb 11, 2020

/close
Duplicate of #3035

@k8s-ci-robot
Copy link
Contributor

@Bobgy: Closed this PR.

In response to this:

/close
Duplicate of #3035

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.

magdalenakuhn17 pushed a commit to magdalenakuhn17/pipelines that referenced this pull request Oct 22, 2023
* First commit

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

* Removing unused code

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

* Added handling for ensemble use case

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

* Bumping up resources for ISVCs to come up quickly

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

* Using latest version of controller-gen

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

* Adding in testing resources

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

* Reverting res req and limits

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

* Enabling live logging for pytest to enable logs to prin to cli for easier debugging

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

* Adding images build step in e2e tests.

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

* Fixing test to use service_name

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

* Adding build of router image to use latest router image everytime for tests

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

* Adding router enrty for kserve deps setup to update router for IGs

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

* Fixed linting errors

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

* Asserting in pythonic way

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

* Updating kserve version to 0.11.0

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

* Adding more e2e tests

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

* Some refactoring to run tests locally easily

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

* Adding some test deps and adding more tests

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

* Fixing a bug and Adding tests for switch use cases

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

* Fixing a bug in e2e-test script

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

* Added e2e test cases for ensemble.

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

* Reducing parallelism to avoid pressure on minikube and timeouts

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

* Bump up timeout interval for the action.

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

* Bump up timeout interval for the test_triton_runtime_with_transformer as this ISVC has both predictor and transformer that needs to come up and it fails sometime with ISVC Timeout error.

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

* Running path-based routing tests as their own job

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

* Moved path-based-routing test as it's own job

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

* Reducing parallelism to 2

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

* Reducing parallelism to 2 for test-fast job and increasing timeout for job as well as new tests have been added

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

* Update test/e2e/graph/test-resources/ig_test_ensemble_scenario_7.yaml

Co-authored-by: Dan Sun <dsun20@bloomberg.net>
Signed-off-by: Rachit Chauhan <rachitchauhan43@gmail.com

* Removing parallelism and bumping up timeout to 120 mins to avoid resource contention issues.

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>

---------

Signed-off-by: rachitchauhan43 <rachitchauhan43@gmail.com>
Signed-off-by: Rachit Chauhan <rachitchauhan43@gmail.com>
Co-authored-by: Dan Sun <dsun20@bloomberg.net>
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.

2 participants