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

Smoke Test failure - Name fix for Runtime template #2103

Merged
merged 5 commits into from
Dec 3, 2024

Conversation

Raghul-M
Copy link
Contributor

@Raghul-M Raghul-M commented Dec 2, 2024

No description provided.

@Raghul-M Raghul-M requested a review from ConorOM1 December 2, 2024 13:03
Copy link
Contributor

github-actions bot commented Dec 2, 2024

Robot Results

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

ConorOM1
ConorOM1 previously approved these changes Dec 2, 2024
@@ -5,14 +5,15 @@ Resource ../../Resources/Page/ODH/ODHDashboard/ODHDashboardSettingsRunt
Resource ../../Resources/Page/ODH/ODHDashboard/ODHDataScienceProject/DataConnections.resource
Resource ../../Resources/Page/ODH/ODHDashboard/ODHModelServing.resource
Suite Setup Custom Serving Runtime Suite Setup
Suite Teardown Custom Serving Runtime Suite Teardown
#Suite Teardown Custom Serving Runtime Suite Teardown
Copy link
Contributor

Choose a reason for hiding this comment

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

Why teardown is commented ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I forgot to uncomment it , let me uncomment it now

@tarukumar tarukumar enabled auto-merge (squash) December 3, 2024 08:49
ConorOM1
ConorOM1 previously approved these changes Dec 3, 2024
@@ -25,7 +26,7 @@ Verify RHODS Admins Can Import A Custom Serving Runtime Template By Uploading A
Open Dashboard Settings settings_page=Serving runtimes
Upload Serving Runtime Template runtime_filepath=${OVMS_RUNTIME_FILEPATH}
... serving_platform=multi runtime_protocol=gRPC
Serving Runtime Template Should Be Listed displayed_name=${UPLOADED_OVMS_DISPLAYED_NAME}
Serving Runtime Template Should Be Listed displayed_name=${UPLOADED_OVMS_XPATH_NAME}
Copy link
Contributor

Choose a reason for hiding this comment

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

Where is the displayed_name=${UPLOADED_OVMS_XPATH_NAME varible name i f9 not see in the suite

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed it

@Raghul-M Raghul-M requested a review from ConorOM1 December 3, 2024 08:54
@Raghul-M Raghul-M requested a review from tarukumar December 3, 2024 08:55
Copy link

sonarqubecloud bot commented Dec 3, 2024

@tarukumar tarukumar merged commit 40d3509 into red-hat-data-services:master Dec 3, 2024
9 checks passed
Raghul-M added a commit to Raghul-M/ods-ci that referenced this pull request Dec 10, 2024
…ices#2103)

* Name fix for Runtime template

* Name fix for Runtime YAML template

* uncommented teardown

---------

Co-authored-by: Tarun Kumar <takumar@redhat.com>
kobihk added a commit that referenced this pull request Dec 10, 2024
lenahorsley pushed a commit to lenahorsley/ods-ci that referenced this pull request Jan 17, 2025
…ices#2103)

* Name fix for Runtime template

* Name fix for Runtime YAML template

* uncommented teardown

---------

Co-authored-by: Tarun Kumar <takumar@redhat.com>
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