Register Radius.Compute and Radius.Security resource types by default #11062
+1,695
−5
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…`Radius.Security/secrets` resource types automatically on Radius installation. These types are defined in [resource-types-contrib](https://github.com/radius-project/resource-types-contrib) and previously required manual registration. Signed-off-by: Karishma Chawla <kachawla@microsoft.com>
kachawla
temporarily deployed
to
functional-tests
GitHub Actions
Inactive
kachawla
temporarily deployed
to
publish-bicep
GitHub Actions
Inactive
Update manifest file naming convention to use underscores instead of hyphens for consistency with other provider manifest files. Signed-off-by: Karishma Chawla <kachawla@microsoft.com>
kachawla
temporarily deployed
to
functional-tests
GitHub Actions
Inactive
kachawla
temporarily deployed
to
publish-bicep
GitHub Actions
Inactive
kachawla
temporarily deployed
to
functional-tests
GitHub Actions
Inactive
kachawla
temporarily deployed
to
publish-bicep
GitHub Actions
Inactive
…source types Updated the test to use API version 2025-08-01-preview for all Radius.* resource types (not just Radius.Core). This is needed because the newly added Radius.Compute and Radius.Security resource types also use the 2025-08-01-preview API version. Signed-off-by: Karishma Chawla <kachawla@microsoft.com>
kachawla
force-pushed
the
rrt-registration
branch
from
cfc49e1 to
7dc2f48
Compare
kachawla
temporarily deployed
to
functional-tests
GitHub Actions
Inactive
kachawla
temporarily deployed
to
publish-bicep
GitHub Actions
Inactive
Radius functional test overviewClick here to see the test run details
Test Status⌛ Building Radius and pushing container images for functional tests... |
zachcasper
reviewed
Jan 26, 2026
Comment on lines
+2
to
+3
| location: | ||
| global: "http://localhost:8082" |
Contributor
There was a problem hiding this comment.
Can you explain why we added location? Location is an unused and unnecessary property on our existing resource types. We specifically did not want to include them in the new resource types.
Contributor
Author
There was a problem hiding this comment.
This is routing location. I'm guessing your comment is assuming this is a part of the schema (it is not)?
Contributor
There was a problem hiding this comment.
I see. Yes, my comment was about the location property in the schema.
kachawla
requested review from
nicolejms and
nithyatsu
and removed request for
nicolejms
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Register
Radius.Compute/{containers,routes,persistentVolumes}andRadius.Security/secretsresource types automatically on Radius installation. These types are defined in resource-types-contrib and previously required manual registration.Changes
Added manifest files for both dev and self-hosted environments:
deploy/manifest/built-in-providers/{dev,self-hosted}/radius_compute.yaml- defines containers, routes, persistentVolumes types with complete OpenAPI schemasdeploy/manifest/built-in-providers/{dev,self-hosted}/radius_security.yaml- defines secrets type with complete OpenAPI schemaThe manifests include the full schema definitions from resource-types-contrib. The only difference from the upstream definitions is the addition of a
locationfield, which specifies where the resource provider is hosted (required for Radius deployment).Manifests are included in UCP image via
deploy/images/ucpd/Dockerfileand loaded automatically by the initializer service during startup. Resource types route todynamic-rpservice (http://dynamic-rp.radius-system:8082for self-hosted,http://localhost:8082for dev).Type of change
Partially Fixes: #9830
Contributor checklist
Please verify that the PR meets the following requirements, where applicable: