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

Automation tests for pytorch model gRPC RHOAIENG-9053 for triton #1834

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

rpancham
Copy link
Contributor

@rpancham rpancham commented Sep 20, 2024

Added Automation Test for RHOAIENG-11561

@@ -151,6 +155,52 @@
... Clean All Models Of Current User
... AND
... Delete Serving Runtime Template From CLI displayed_name=triton-kserve-grpc

Check warning

Code scanning / Robocop

Trailing whitespace at the end of line Warning test

Trailing whitespace at the end of line
@@ -151,6 +155,52 @@
... Clean All Models Of Current User
... AND
... Delete Serving Runtime Template From CLI displayed_name=triton-kserve-grpc

Test PYTORCH Model Grpc Inference Via UI (Triton on Kserve) # robocop: off=too-long-test-case

Check warning

Code scanning / Robocop

Test case '{{ test_name }}' has too many keywords inside ({{ keyword_count }}/{{ max_allowed_count }}) Warning test

Test case 'Test PYTORCH Model Grpc Inference Via UI (Triton on Kserve)' has too many keywords inside (24/10)
... token=${TRUE}
Wait For Pods To Be Ready label_selector=serving.kserve.io/inferenceservice=${PYTORCH_MODEL_LABEL}
... namespace=${PRJ_TITLE}
${EXPECTED_INFERENCE_GRPC_OUTPUT_PYTORCH}= Load Json File file_path=${EXPECTED_INFERENCE_GRPC_OUTPUT_FILE_PYTORCH}

Check warning

Code scanning / Robocop

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

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

github-actions bot commented Sep 20, 2024

Robot Results

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

@rpancham
Copy link
Contributor Author

Verified with Jenkins Build 432

Log ${EXPECTED_INFERENCE_GRPC_OUTPUT_PYTORCH}
Open Model Serving Home Page
${host_url}= Get Model Route Via UI model_name=${PYTORCH_MODEL_NAME}
${host}= Evaluate re.search(r"${PATTERN}", r"${host_url}").group(1) re
Copy link
Contributor

Choose a reason for hiding this comment

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

please create a kw for this one given that it's been re-used.
One question, why do we need this? What's returning from "Get Model Route Via UI"? I don't remember we had issues in the past about this

Copy link
Contributor

Choose a reason for hiding this comment

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

So, the Get Model Route Via UI keyword returns the whole URL. We just need the domain name, which is why we used a regex pattern. We will create it as a keyword

Copy link
Contributor

Choose a reason for hiding this comment

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

IMO you should also report this to Dashboard team because the UI should report the right URL based on the protocol

Copy link
Contributor

Choose a reason for hiding this comment

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

Informed @bdattoma

@rpancham
Copy link
Contributor Author

Verified with Jenkins Build 465

Raghul-M
Raghul-M previously approved these changes Sep 26, 2024
tarukumar
tarukumar previously approved these changes Sep 27, 2024
Log ${inference_output}
${inference_output}= Evaluate json.dumps(${inference_output})
Log ${inference_output}
${result} ${list}= Inference Comparison ${EXPECTED_INFERENCE_GRPC_OUTPUT_PYTORCH} ${inference_output}
Copy link
Contributor

Choose a reason for hiding this comment

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

iirc this keyword does not fail directly, but you should add a check on the ${result}. Check how it is used in Verify Model Inference keyword.

Copy link

sonarcloud bot commented Sep 30, 2024

@rpancham rpancham dismissed stale reviews from tarukumar and Raghul-M via bd5e1c5 December 10, 2024 11:18
Copy link

sonarcloud bot commented Dec 10, 2024

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.

4 participants