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

Added FIL model REST protocol test on triton for Modelmesh #2038

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

Conversation

rpancham
Copy link
Contributor

No description provided.

@rpancham rpancham added needs testing Needs to be tested in Jenkins do not merge Do not merge this yet please labels Nov 15, 2024
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}

Check warning

Code scanning / Robocop

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

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

Robot Results

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

Copy link

sonarcloud bot commented Nov 19, 2024

@@ -150,6 +152,41 @@
... AND
... Clean All Models Of Current User

Test FIL Model Rest Inference Via UI (Triton on Modelmesh)

Check warning

Code scanning / Robocop

Test case '{{ test_name }}' is too long ({{ test_length }}/{{ allowed_length }}) Warning test

Test case 'Test FIL Model Rest Inference Via UI (Triton on Modelmesh)' is too long (33/20)
@@ -150,6 +152,41 @@
... AND
... Clean All Models Of Current User

Test FIL Model Rest Inference Via UI (Triton on Modelmesh)

Check warning

Code scanning / Robocop

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

Test case 'Test FIL Model Rest Inference Via UI (Triton on Modelmesh)' has too many keywords inside (14/10)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge Do not merge this yet please needs testing Needs to be tested in Jenkins
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant