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

Add wait for deployment ready #1996

Conversation

asanzgom
Copy link
Contributor

Added check for deployment ready since execution together with other Test Cases that remove model mesh deployment can cause the recreation of deployment to not be ready at the moment of execution

@asanzgom asanzgom self-assigned this Oct 30, 2024
Suite Setup Suite Setup
Suite Teardown Suite Teardown


*** Variables ***
${APPLICATIONS_NS} ${APPLICATIONS_NAMESPACE}

Check notice

Code scanning / Robocop

Variable '{{ name }}' is assigned but not used Note test

Variable '${APPLICATIONS_NS}' is assigned but not used
${TRAINING_LABEL_SELECTOR} app.kubernetes.io/name=training-operator
${DATASCIENCEPIPELINES_LABEL_SELECTOR} app.kubernetes.io/name=data-science-pipelines-operator
${MODELMESH_CONTROLLER_LABEL_SELECTOR} app.kubernetes.io/instance=modelmesh-controller
${ODH_MODEL_CONTROLLER_LABEL_SELECTOR} app=odh-model-controller

Check notice

Code scanning / Robocop

Variable '{{ name }}' is assigned but not used Note test

Variable '${ODH_MODEL_CONTROLLER_LABEL_SELECTOR}' is assigned but not used
${MODELMESH_CONTROLLER_LABEL_SELECTOR} app.kubernetes.io/instance=modelmesh-controller
${ODH_MODEL_CONTROLLER_LABEL_SELECTOR} app=odh-model-controller
${MODELREGISTRY_CONTROLLER_LABEL_SELECTOR} control-plane=model-registry-operator
${KSERVE_CONTROLLER_MANAGER_LABEL_SELECTOR} control-plane=kserve-controller-manager

Check notice

Code scanning / Robocop

Variable '{{ name }}' is assigned but not used Note test

Variable '${KSERVE_CONTROLLER_MANAGER_LABEL_SELECTOR}' is assigned but not used
modelmeshserving modelmesh-controller odh_modelmesh_controller_image
datasciencepipelines data-science-pipelines-operator-controller-manager odh_data_science_pipelines_operator_controller_image
trainingoperator kubeflow-training-operator odh_training_operator_image
codeflare codeflare-operator-manager ${CODEFLARE_LABEL_SELECTOR} odh_codeflare_operator_image

Check warning

Code scanning / Robocop

Line is too long ({{ line_length }}/{{ allowed_length }}) Warning test

Line is too long (160/120)
modelmeshserving modelmesh-controller odh_modelmesh_controller_image
datasciencepipelines data-science-pipelines-operator-controller-manager odh_data_science_pipelines_operator_controller_image
trainingoperator kubeflow-training-operator odh_training_operator_image
codeflare codeflare-operator-manager ${CODEFLARE_LABEL_SELECTOR} odh_codeflare_operator_image

Check warning

Code scanning / Robocop

Trailing whitespace at the end of line Warning test

Trailing whitespace at the end of line
kueue kueue-controller-manager ${KUEUE_LABEL_SELECTOR} odh_kueue_controller_image
workbenches odh-notebook-controller-manager ${WORKBENCHES_LABEL_SELECTOR} odh_notebook_controller_image
workbenches notebook-controller-deployment ${WORKBENCHES_LABEL_SELECTOR} odh_kf_notebook_controller_image
dashboard ${DASHBOARD_DEPLOYMENT_NAME} ${DASHBOARD_LABEL_SELECTOR} odh_dashboard_image

Check warning

Code scanning / Robocop

Line is too long ({{ line_length }}/{{ allowed_length }}) Warning test

Line is too long (151/120)
kueue kueue-controller-manager ${KUEUE_LABEL_SELECTOR} odh_kueue_controller_image
workbenches odh-notebook-controller-manager ${WORKBENCHES_LABEL_SELECTOR} odh_notebook_controller_image
workbenches notebook-controller-deployment ${WORKBENCHES_LABEL_SELECTOR} odh_kf_notebook_controller_image
dashboard ${DASHBOARD_DEPLOYMENT_NAME} ${DASHBOARD_LABEL_SELECTOR} odh_dashboard_image

Check warning

Code scanning / Robocop

Trailing whitespace at the end of line Warning test

Trailing whitespace at the end of line
workbenches odh-notebook-controller-manager ${WORKBENCHES_LABEL_SELECTOR} odh_notebook_controller_image
workbenches notebook-controller-deployment ${WORKBENCHES_LABEL_SELECTOR} odh_kf_notebook_controller_image
dashboard ${DASHBOARD_DEPLOYMENT_NAME} ${DASHBOARD_LABEL_SELECTOR} odh_dashboard_image
modelmeshserving modelmesh-controller ${MODELMESH_CONTROLLER_LABEL_SELECTOR} odh_modelmesh_controller_image

Check warning

Code scanning / Robocop

Line is too long ({{ line_length }}/{{ allowed_length }}) Warning test

Line is too long (162/120)
workbenches notebook-controller-deployment ${WORKBENCHES_LABEL_SELECTOR} odh_kf_notebook_controller_image
dashboard ${DASHBOARD_DEPLOYMENT_NAME} ${DASHBOARD_LABEL_SELECTOR} odh_dashboard_image
modelmeshserving modelmesh-controller ${MODELMESH_CONTROLLER_LABEL_SELECTOR} odh_modelmesh_controller_image
datasciencepipelines data-science-pipelines-operator-controller-manager ${DATASCIENCEPIPELINES_LABEL_SELECTOR} odh_data_science_pipelines_operator_controller_image

Check warning

Code scanning / Robocop

Line is too long ({{ line_length }}/{{ allowed_length }}) Warning test

Line is too long (184/120)
dashboard ${DASHBOARD_DEPLOYMENT_NAME} ${DASHBOARD_LABEL_SELECTOR} odh_dashboard_image
modelmeshserving modelmesh-controller ${MODELMESH_CONTROLLER_LABEL_SELECTOR} odh_modelmesh_controller_image
datasciencepipelines data-science-pipelines-operator-controller-manager ${DATASCIENCEPIPELINES_LABEL_SELECTOR} odh_data_science_pipelines_operator_controller_image
trainingoperator kubeflow-training-operator ${TRAINING_LABEL_SELECTOR} odh_training_operator_image

Check warning

Code scanning / Robocop

Line is too long ({{ line_length }}/{{ allowed_length }}) Warning test

Line is too long (159/120)
Copy link
Contributor

Robot Results

✅ Passed ❌ Failed ⏭️ Skipped Total Pass %
560 0 0 560 100

@asanzgom asanzgom enabled auto-merge October 30, 2024 14:19
@asanzgom asanzgom requested a review from bdattoma October 30, 2024 14:20
@asanzgom
Copy link
Contributor Author

Running rhods-ci-pr-test/3423/ together with all Operator Test Cases

@asanzgom asanzgom requested review from jstourac and jgarciao October 30, 2024 14:37
@asanzgom
Copy link
Contributor Author

rhods-ci-pr-test/3423/ passed successfully

@asanzgom asanzgom requested a review from jgarciao October 30, 2024 15:13
Copy link
Contributor

@MarianMacik MarianMacik left a comment

Choose a reason for hiding this comment

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

Looks good to me!

Copy link

@jgarciao jgarciao dismissed their stale review October 30, 2024 15:56

My proposal could be done in a follow-up PR

@asanzgom asanzgom merged commit 55aa1c0 into red-hat-data-services:master Oct 30, 2024
8 checks passed
@asanzgom asanzgom deleted the hotfix/add_deployment_ready_check branch October 30, 2024 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants