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

WatsonX-Serving: keep working on the scale test #13

Merged
merged 7 commits into from
Aug 24, 2023
Merged

WatsonX-Serving: keep working on the scale test #13

merged 7 commits into from
Aug 24, 2023

Conversation

kpouget
Copy link
Contributor

@kpouget kpouget commented Aug 23, 2023

No description provided.

@openshift-ci
Copy link

openshift-ci bot commented Aug 23, 2023

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please ask for approval from kpouget. For more information see the Kubernetes Code Review Process.

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

@kpouget
Copy link
Contributor Author

kpouget commented Aug 23, 2023

/test watsonx-serving-e2e
/var tests.scale.user_count=100

@kpouget
Copy link
Contributor Author

kpouget commented Aug 24, 2023 •
edited
Loading

/test watsonx-serving-e2e keep
/var tests.scale.user_count=100

@kpouget
Copy link
Contributor Author

kpouget commented Aug 24, 2023

/test watsonx-serving-e2e
/var tests.scale.user_count=100

@kpouget
Copy link
Contributor Author

kpouget commented Aug 24, 2023

/test watsonx-serving-light

@kpouget
cluster_preload_image: improve to be more reliable
68ed8ed 
before that, the role could crash when the daemonset controller wasnt fast enough to set the desired number of pods
@kpouget
testing: watsonx-serving: do not register the SMMR in the prepare phase
7e8e26d 
no need to register it to preload the image
@kpouget
testing: utils: ci_init_configure: do not crash if ARTIFACT_DIR is un…
24a8173 
…defined

no need to crash at this stage, 'secrets.sha256sum' is just for troubleshooting
@kpouget
testing: utils: openshift_clusters/clusters: allow running without CI…
73656bf 
…_ARTIFACTS_FROM_CONFIG_FILE

so that the 'keep' entrypoint can work
@kpouget
testing: utils: openshift_clusters/clusters: add a 'keep' entrypoint
08b9016 
so that we can easily manually 'keep' the CI cluster, even if the 'keep' flag was forgotten
@kpouget
testing: watsonx-serving: test_scale: mute set_project_annotation
93f629e 
so that the users' run.log is easier to read
@kpouget
testing: run: remove duplicated 'after' word in logging message
d95a992
@kpouget
Copy link
Contributor Author

kpouget commented Aug 24, 2023

/test watsonx-serving-light

@openshift-ci
Copy link

openshift-ci bot commented Aug 24, 2023

@kpouget: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/watsonx-serving-light 491abb0 link true /test watsonx-serving-light

Full PR test history. Your PR dashboard.

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.

@kpouget
Copy link
Contributor Author

kpouget commented Aug 24, 2023

/test watsonx-serving-light

@kpouget
Copy link
Contributor Author

kpouget commented Aug 24, 2023

test passed, merging

@kpouget kpouget merged commit 690108f into openshift-psap:main Aug 24, 2023
5 of 7 checks passed
@kpouget kpouget deleted the watsonx branch August 24, 2023 11:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@kpouget