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

Fix model mesh tests and update runtime images #2060

Merged
merged 18 commits into from
Nov 27, 2024

Conversation

rnetser
Copy link
Contributor

@rnetser rnetser commented Nov 21, 2024

verified via jenkins job/devops/job/rhoai-test-flow/1779/robot/ - failues are not related to this PR and will be fixed in seperate PRs
for the images - tested against disconnected env

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
Page Should Contain Element xpath://span[contains(., "${runtime}")]
${selected}= Run Keyword And Return Status
... Click Element xpath://span[contains(., "${runtime}")]
IF ${selected}==${TRUE} BREAK

Check notice

Code scanning / Robocop

'{{ block_name }}' condition can be simplified Note test

'INLINE IF' condition can be simplified
@@ -54,7 +54,7 @@
Recreate S3 Data Connection project_title=${PRJ_TITLE} dc_name=model-serving-connection
... aws_access_key=${S3.AWS_ACCESS_KEY_ID} aws_secret_access=${S3.AWS_SECRET_ACCESS_KEY}
... aws_bucket_name=ods-ci-s3
Create Model Server token=${TRUE} runtime=${ONNX_RUNTIME_NAME} server_name=${ONNX_RUNTIME_NAME} existing_server=${TRUE}
Create Model Server token=${TRUE} runtime=${ONNX_RUNTIME_NAME} server_name=${ONNX_RUNTIME_NAME} existing_server=${FALSE}

Check warning

Code scanning / Robocop

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

Line is too long (136/120)
@@ -130,7 +130,7 @@
Recreate S3 Data Connection project_title=${PRJ_TITLE} dc_name=model-serving-connection
... aws_access_key=${S3.AWS_ACCESS_KEY_ID} aws_secret_access=${S3.AWS_SECRET_ACCESS_KEY}
... aws_bucket_name=ods-ci-s3
Create Model Server token=${TRUE} runtime=${ONNX_RUNTIME_NAME} server_name=${ONNX_RUNTIME_NAME} existing_server=${TRUE}
Create Model Server token=${TRUE} runtime=${ONNX_RUNTIME_NAME} server_name=${ONNX_RUNTIME_NAME} existing_server=${FALSE}

Check warning

Code scanning / Robocop

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

Line is too long (136/120)
@rnetser rnetser changed the title [ui] Fix model mesh tests and add kserve test [wip][ui] Fix model mesh tests and add kserve test Nov 21, 2024
 into fix-model-mesh

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
@@ -226,7 +226,7 @@
[Arguments] ${model_name}
# TODO: Open model serving home page if needed?
# Click on Inference Endpoints link
${endpoint_link}= Set Variable //a[@data-testid="metrics-link-test-model" and text()="${model_name}"]/ancestor::tr//td//button[@data-testid="internal-external-service-button"]
${endpoint_link}= Set Variable //a[contains(@data-testid, 'metrics-link')][text() = '${model_name}']/ancestor::tr//td//button[@data-testid="internal-external-service-button"]

Check warning

Code scanning / Robocop

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

Line is too long (184/120)
... existing_data_connection=${TRUE} data_connection_name=model-serving-connection
... model_path=inception_resnet_v2.pb
${runtime_pod_name}= Replace String Using Regexp string=${RUNTIME_NAME} pattern=\\s replace_with=-
${runtime_pod_name}= Convert To Lower Case ${runtime_pod_name}
Wait Until Keyword Succeeds 5 min 10 sec Verify Openvino Deployment runtime_name=${RUNTIME_POD_NAME}
Wait Until Keyword Succeeds 5 min 10 sec Verify Serving Service
Wait Until Keyword Succeeds 5 min 10 sec Verify Openvino Deployment runtime_name=${RUNTIME_POD_NAME} project_name=${namespace}

Check warning

Code scanning / Robocop

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

Line is too long (141/120)
... existing_data_connection=${TRUE} data_connection_name=model-serving-connection
... model_path=inception_resnet_v2.pb
${runtime_pod_name}= Replace String Using Regexp string=${RUNTIME_NAME} pattern=\\s replace_with=-
${runtime_pod_name}= Convert To Lower Case ${runtime_pod_name}
Wait Until Keyword Succeeds 5 min 10 sec Verify Openvino Deployment runtime_name=${RUNTIME_POD_NAME}
Wait Until Keyword Succeeds 5 min 10 sec Verify Serving Service
Wait Until Keyword Succeeds 5 min 10 sec Verify Openvino Deployment runtime_name=${RUNTIME_POD_NAME} project_name=${namespace}

Check warning

Code scanning / Robocop

Variable '{{ name }}' has inconsistent naming. First used as '{{ first_use }}' Warning test

Variable '${RUNTIME_POD_NAME}' has inconsistent naming. First used as '${runtime_pod_name}'
Verify Model Status ${MODEL_NAME} success
Set Suite Variable ${MODEL_CREATED} ${TRUE}
${url}= Get Model Route Via UI ${MODEL_NAME}
${status_code} ${response_text}= Send Random Inference Request endpoint=${url} name=input
${status_code} ${response_text}= Send Random Inference Request endpoint=${url} name=input:0

Check notice

Code scanning / Robocop

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

Variable '${response_text}' is assigned but not used
... existing_data_connection=${TRUE} data_connection_name=model-serving-connection
... model_path=inception_resnet_v2.pb
${runtime_pod_name}= Replace String Using Regexp string=${RUNTIME_NAME} pattern=\\s replace_with=-
${runtime_pod_name}= Convert To Lower Case ${runtime_pod_name}
Wait Until Keyword Succeeds 5 min 10 sec Verify Openvino Deployment runtime_name=${RUNTIME_POD_NAME}
Wait Until Keyword Succeeds 5 min 10 sec Verify Serving Service
Wait Until Keyword Succeeds 5 min 10 sec Verify Openvino Deployment runtime_name=${RUNTIME_POD_NAME} project_name=${namespace}

Check warning

Code scanning / Robocop

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

Line is too long (141/120)
... existing_data_connection=${TRUE} data_connection_name=model-serving-connection
... model_path=inception_resnet_v2.pb
${runtime_pod_name}= Replace String Using Regexp string=${RUNTIME_NAME} pattern=\\s replace_with=-
${runtime_pod_name}= Convert To Lower Case ${runtime_pod_name}
Wait Until Keyword Succeeds 5 min 10 sec Verify Openvino Deployment runtime_name=${RUNTIME_POD_NAME}
Wait Until Keyword Succeeds 5 min 10 sec Verify Serving Service
Wait Until Keyword Succeeds 5 min 10 sec Verify Openvino Deployment runtime_name=${RUNTIME_POD_NAME} project_name=${namespace}

Check warning

Code scanning / Robocop

Variable '{{ name }}' has inconsistent naming. First used as '{{ first_use }}' Warning test

Variable '${RUNTIME_POD_NAME}' has inconsistent naming. First used as '${runtime_pod_name}'
rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
 into fix-model-mesh

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
 into fix-model-mesh

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
@@ -64,7 +64,7 @@
Verify Model Status ${MODEL_NAME} success
Set Suite Variable ${MODEL_CREATED} ${TRUE}
${url}= Get Model Route Via UI ${MODEL_NAME}
${status_code} ${response_text}= Send Random Inference Request endpoint=${url} name=input
${status_code} ${response_text}= Send Random Inference Request endpoint=${url} name=input:0

Check notice

Code scanning / Robocop

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

Variable '${response_text}' is assigned but not used
@@ -1347,9 +1347,9 @@
[Arguments] ${gpu_type}
IF "${RUNTIME_IMAGE}" == "${EMPTY}"
IF "${gpu_type}" == "nvidia"
Set Test Variable ${runtime_image} quay.io/modh/vllm@sha256:167aa05982214c3941afbdde7c5aff6f81040193a0db1dc06a5ad16af0a8780e
Set Test Variable ${runtime_image} quay.io/modh/vllm@sha256:c86ff1e89c86bc9821b75d7f2bbc170b3c13e3ccf538bf543b1110f23e056316

Check warning

Code scanning / Robocop

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

Line is too long (142/120)
@@ -1347,9 +1347,9 @@
[Arguments] ${gpu_type}
IF "${RUNTIME_IMAGE}" == "${EMPTY}"
IF "${gpu_type}" == "nvidia"
Set Test Variable ${runtime_image} quay.io/modh/vllm@sha256:167aa05982214c3941afbdde7c5aff6f81040193a0db1dc06a5ad16af0a8780e
Set Test Variable ${runtime_image} quay.io/modh/vllm@sha256:c86ff1e89c86bc9821b75d7f2bbc170b3c13e3ccf538bf543b1110f23e056316

Check notice

Code scanning / Robocop

{{ set_variable_keyword }} can be replaced with VAR Note test

Set Test Variable can be replaced with VAR
ELSE IF "${gpu_type}" == "amd"
Set Test Variable ${runtime_image} quay.io/modh/vllm@sha256:1db9fc11f3cac723dee8c4de71fd36aafc94c6ff4d9f51cb785a6f4461f027af
Set Test Variable ${runtime_image} quay.io/modh/vllm@sha256:10f09eeca822ebe77e127aad7eca2571f859a5536a6023a1baffc6764bcadc6e

Check warning

Code scanning / Robocop

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

Line is too long (142/120)
ELSE IF "${gpu_type}" == "amd"
Set Test Variable ${runtime_image} quay.io/modh/vllm@sha256:1db9fc11f3cac723dee8c4de71fd36aafc94c6ff4d9f51cb785a6f4461f027af
Set Test Variable ${runtime_image} quay.io/modh/vllm@sha256:10f09eeca822ebe77e127aad7eca2571f859a5536a6023a1baffc6764bcadc6e

Check notice

Code scanning / Robocop

{{ set_variable_keyword }} can be replaced with VAR Note test

Set Test Variable can be replaced with VAR
@rnetser rnetser changed the title [wip][ui] Fix model mesh tests and add kserve test [wip][ui] Fix model mesh tests and update runtime images Nov 25, 2024
 into fix-model-mesh

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
@@ -426,6 +427,7 @@
Sleep 1s
END
Switch Model Serving Project project_name=All projects
@{projects}= Get WebElements xpath://table/tbody/tr/td[@data-label="Project"]

Check notice

Code scanning / Robocop

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

Variable '@{projects}' is assigned but not used
Copy link
Contributor

github-actions bot commented Nov 26, 2024

Robot Results

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

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
 into fix-model-mesh

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
@@ -68,7 +68,7 @@
... token_auth=${TRUE} token=${first_token}
Run Keyword And Warn On Failure Should Contain ${inf_out} Log in with OpenShift
[Teardown] Run Keywords Run Keyword If Test Failed Get Kserve Events And Logs
... model_name=${MODEL_NAME} project_title=${PRJ_TITLE} AND Clean All Models Of Current User
... model_name=${MODEL_NAME} project_title=${PRJ_TITLE} AND model_name=${SECOND_MODEL_NAME} project_title=${PRJ_TITLE}

Check warning

Code scanning / Robocop

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

Line is too long (137/120)
@@ -68,7 +68,7 @@
... token_auth=${TRUE} token=${first_token}
Run Keyword And Warn On Failure Should Contain ${inf_out} Log in with OpenShift
[Teardown] Run Keywords Run Keyword If Test Failed Get Kserve Events And Logs
... model_name=${MODEL_NAME} project_title=${PRJ_TITLE} AND Clean All Models Of Current User
... model_name=${MODEL_NAME} project_title=${PRJ_TITLE} AND model_name=${SECOND_MODEL_NAME} project_title=${PRJ_TITLE}

Check warning

Code scanning / Robocop

Underscores in keyword name '{{ keyword_name }}' can be replaced with spaces Warning test

Underscores in keyword name 'model_name=${SECOND_MODEL_NAME}' can be replaced with spaces
@tarukumar tarukumar self-requested a review November 26, 2024 15:24
tarukumar
tarukumar previously approved these changes Nov 26, 2024
Copy link
Contributor

@tarukumar tarukumar left a comment

Choose a reason for hiding this comment

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

overall looks good to me. Please run the test from jenkins as well

@tarukumar tarukumar self-requested a review November 27, 2024 05:35
rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
 into fix-model-mesh

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
@rnetser rnetser changed the title [wip][ui] Fix model mesh tests and update runtime images Fix model mesh tests and update runtime images Nov 27, 2024
@rnetser rnetser requested a review from mwaykole November 27, 2024 09:45
 into fix-model-mesh

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
@tarukumar tarukumar merged commit 8cfdd75 into red-hat-data-services:master Nov 27, 2024
8 checks passed
@rnetser rnetser deleted the fix-model-mesh branch November 27, 2024 13:35
tarukumar added a commit that referenced this pull request Nov 28, 2024
* Update model mesh deploymenet from ui

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED

* add raw and serverless

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED

* add raw and serverless

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED

* fix tests

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED

* fix tests

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED

* fix tests

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED

* update images

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED

* fix tests and code

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED

* use cli project deletion

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED

* use cli project deletion

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED

* remove parenthasis

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED

Co-authored-by: Ruth Netser <[email protected]>
lenahorsley pushed a commit to lenahorsley/ods-ci that referenced this pull request Jan 17, 2025
…#2060)

* Update model mesh deploymenet from ui

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED

* add raw and serverless

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED

* add raw and serverless

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED

* fix tests

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED

* fix tests

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED

* fix tests

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED

* update images

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED

* fix tests and code

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED

* use cli project deletion

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED

* use cli project deletion

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED

* remove parenthasis

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
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.

3 participants