Update tests to remove httpRoutes since services can now be deployed as part of container rendering. #6241
Conversation
nithyatsu
changed the title
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
Test Results2 958 tests ±0 2 946 ✔️ ±0 2m 43s ⏱️ ±0s Results for commit ac5fc10. ± Comparison against base commit 51fe742. This pull request removes 2 and adds 2 tests. Note that renamed tests count towards both.♻️ This comment has been updated with latest results. |
nithyatsu
changed the title
nithyatsu
force-pushed
the
update_datastore_tests
branch
from
17f12a1 to
f980220
Compare
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
|
For the detailed report, please go to
|
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
|
For the detailed report, please go to
|
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
|
For the detailed report, please go to
|
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
|
For the detailed report, please go to
|
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
|
For the detailed report, please go to
|
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
|
For the detailed report, please go to
|
This comment has been minimized.
This comment has been minimized.
|
⌛ Building Radius and pushing container images for functional tests... ✅ Container images build succeeded |
nithyatsu
force-pushed
the
update_datastore_tests
branch
from
a59b055 to
8d94315
Compare
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
|
For the detailed report, please go to
|
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
|
For the detailed report, please go to
|
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
|
For the detailed report, please go to
|
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
|
For the detailed report, please go to
|
|
For the detailed report, please go to
|
nithyatsu
force-pushed
the
update_datastore_tests
branch
from
68ad425 to
c4c3ba5
Compare
|
For the detailed report, please go to
|
|
/ok-to-test |
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
|
/ok-to-test |
This comment has been minimized.
This comment has been minimized.
nithyatsu
force-pushed
the
update_datastore_tests
branch
from
c4c3ba5 to
4dc2d1c
Compare
|
For the detailed report, please go to
|
|
⌛ Building Radius and pushing container images for functional tests... ✅ Container images build succeeded |
This reverts commit a93854169188c65825d98fd17259016efa52bf61.
nithyatsu
force-pushed
the
update_datastore_tests
branch
from
4dc2d1c to
ac5fc10
Compare
|
For the detailed report, please go to
|
Description
We now have the ability to deploy k8s services as part of container rendering (ref #6375)
This PR updates all the radius repo tests and samples
Type of change
Fixes: #6099
Auto-generated summary
🤖 Generated by Copilot at e22a126
Summary
🌐🛠️🗑️
This pull request simplifies the service generation logic for datastores and containers in the radius project. It removes the dependency on httpRoutes resources for datastores, and allows specifying different port and target port values for containers. It updates the renderers, test cases, and test data files accordingly.
Walkthrough
render_test.goto use different container and service port numbers and add port field to container port definition (link, link, link)render.go(link)mongodb_test.go,redis_test.go, andsql_test.go(link, link, link, link, link, link)datastoresrp-rs-mongodb-manual.bicepanddatastoresrp-resources-sqldb-manual.bicep(link, link)datastoresrp-resources-redis-manual.bicep(link)datastoresrp-rs-mongodb-manual.bicep,datastoresrp-resources-redis-manual.bicep, anddatastoresrp-resources-sqldb-manual.bicepto use container name and port instead of httpRoutes hostname and port (link, link, link)datastoresrp-rs-mongodb-manual.bicep,datastoresrp-resources-redis-manual.bicep, anddatastoresrp-resources-sqldb-manual.bicep(link, link, link)