Add redis resource type and Kubernetes Recipe in Bicep and Tf #13

Reshrahim · 2025-08-11T19:54:20Z

This PR adds the following

redis.yaml that defines the schema for the Redis resource type
A k8s Bicep and Terraform Recipe

Tested the Bicep and Terraform Recipe with todolist demo app

Building app.bicep...
Deploying template 'app.bicep' into environment '/planes/radius/local/resourceGroups/local/providers/applications.core/environments/local' from workspace 'local'...

Deployment In Progress...

Completed            db              Radius.Data/redisCaches
Completed            todolist        Applications.Core/applications
.                    demo            Applications.Core/containers

Deployment Complete

Resources:
    todolist        Applications.Core/applications
    demo            Applications.Core/containers
    db              Radius.Data/redisCaches

Contribution stage : Alpha level

Fixes : #7 , #8

Signed-off-by: Reshma Abdul Rahim <[email protected]>

zachcasper · 2025-09-04T03:02:26Z

data/redis/redis.yaml

@@ -0,0 +1,42 @@
+name: Radius.Data


Please update based on Secrets, Containers, and PersistentVolumes.

Updated. Please take a look. Thanks

Signed-off-by: Reshma Abdul Rahim <[email protected]>

Reshrahim · 2025-09-08T22:14:32Z

Data/redisCaches/recipes/kubernetes/bicep/kubernetes-redis.bicep

+  }
+} 
+
+resource redis 'apps/Deployment@v1' = {


Once Container Recipes are available, this will be updated to use Radius.Compute/Containers

Data/redisCaches/redisCaches.yaml

zachcasper · 2025-09-08T22:13:51Z

Data/redisCaches/redisCaches.yaml

+      param environment string
+      resource redisCache 'Radius.Data/redisCaches@2025-08-01-preview' = {
+        name: 'redisCache'
+        properties: {
+          environment: environment
+        }
+      }


Suggested change

param environment string

resource redisCache 'Radius.Data/redisCaches@2025-08-01-preview' = {

name: 'redisCache'

properties: {

environment: environment

}

}

extension radius

param environment string

resource myApplication 'Radius.Core/applications@2025-08-01-preview' = { ... }

resource redis 'Radius.Data/redisCaches@2025-08-01-preview' = {

name: 'redis'

properties: {

environment: environment

}

}

I've been making sure the examples were whole.

zachcasper · 2025-09-08T22:14:24Z

Data/redisCaches/redisCaches.yaml

+        }
+      }
+
+      When `capacity` is not set, the value is assumed to be `S`(Small) in Recipes.


This should be in the property description, not here.

Data/redisCaches/redisCaches.yaml

zachcasper · 2025-09-08T22:40:52Z

Data/redisCaches/recipes/kubernetes/bicep/kubernetes-redis.bicep

+  // This workaround is needed because the deployment engine omits Kubernetes resources from its output.
+  // This allows Kubernetes resources to be cleaned up when the resource is deleted.
+  // Once this gap is addressed, users won't need to do this.


Is there an issue to reference?

zachcasper · 2025-09-08T22:43:26Z

Data/redisCaches/recipes/kubernetes/terraform/main.tf

+  }
+}
+
+output "result" {


Is resources not required here as it is in the Bicep recipe? If so, be sure to include the same comment from the Bicep file.

zachcasper · 2025-09-08T22:44:29Z

Data/redisCaches/recipes/kubernetes/terraform/main.tf

+      host = "${kubernetes_service.redis.metadata[0].name}.${kubernetes_service.redis.metadata[0].namespace}.svc.cluster.local"
+      port = local.port


There are five read-only properties. They should all be set.

Is this a rule for Recipes that all read-only properties must be set? This Recipe implementation is similar to a alpha stage contribution not covering all the scenarios(e.g configuring TLS) and hence some properties are not set.

zachcasper · 2025-09-08T22:44:43Z

Data/redisCaches/recipes/kubernetes/bicep/kubernetes-redis.bicep

+    '/planes/kubernetes/local/namespaces/${svc.metadata.namespace}/providers/core/Service/${svc.metadata.name}'
+    '/planes/kubernetes/local/namespaces/${redis.metadata.namespace}/providers/apps/Deployment/${redis.metadata.name}'
+  ]
+  values: {


There are five read-only properties. They should all be set.

zachcasper · 2025-09-08T22:46:32Z

Data/redisCaches/recipes/kubernetes/terraform/main.tf

@@ -0,0 +1,85 @@
+terraform {


Output from tflint:

$ tflint 1 issue(s) found: Warning: terraform "required_version" attribute is required (terraform_required_version) on main.tf line 1: 1: terraform { Reference: https://github.com/terraform-linters/tflint-ruleset-terraform/blob/v0.12.0/docs/rules/terraform_required_version.md

Co-authored-by: Zach Casper <[email protected]> Signed-off-by: Reshma Abdul Rahim <[email protected]>

## Description Updated developer documentation to be aligned with the [Redis PR](#13) ## Testing 1. `rad resource-type create -f mySqlDatabases.yaml` 2. `rad resource-type show Radius.Data/mySqlDatabases` and confirm output is as expected 3. `rad bicep publish-extension -f mySqlDatabases.yaml --target radiusResources.tgz` and confirm Intellisense is as expected 4. Copy examples from rad resource-type show into a Bicep file and confirm they compile --------- Signed-off-by: Zach Casper <[email protected]>

## Description Updated developer documentation to be aligned with the [Redis PR](radius-project#13) ## Testing 1. `rad resource-type create -f mySqlDatabases.yaml` 2. `rad resource-type show Radius.Data/mySqlDatabases` and confirm output is as expected 3. `rad bicep publish-extension -f mySqlDatabases.yaml --target radiusResources.tgz` and confirm Intellisense is as expected 4. Copy examples from rad resource-type show into a Bicep file and confirm they compile --------- Signed-off-by: Zach Casper <[email protected]>

Add redis resource type and Kubernetes Recipe in Bicep and Tf

9832917

Signed-off-by: Reshma Abdul Rahim <[email protected]>

Reshrahim requested review from a team as code owners August 11, 2025 19:54

zachcasper reviewed Sep 4, 2025

View reviewed changes

Reshrahim and others added 8 commits September 8, 2025 10:03

Merge branch 'radius-project:main' into main

77c6cba

Update the definitions to latest guidance

b1e1eb0

Signed-off-by: Reshma Abdul Rahim <[email protected]>

Case insensitivity

e165f01

Signed-off-by: Reshma Abdul Rahim <[email protected]>

Remove duplicate 'data' folder

129e778

Tf Recipe

ad088fc

Signed-off-by: Reshma Abdul Rahim <[email protected]>

Update defn

d9ab418

Signed-off-by: Reshma Abdul Rahim <[email protected]>

Update host

f89fb24

Signed-off-by: Reshma Abdul Rahim <[email protected]>

Update desc

5b0670b

Signed-off-by: Reshma Abdul Rahim <[email protected]>

Reshrahim commented Sep 8, 2025

View reviewed changes

zachcasper suggested changes Sep 8, 2025

View reviewed changes

Apply suggestions from code review

11db5a8

Co-authored-by: Zach Casper <[email protected]> Signed-off-by: Reshma Abdul Rahim <[email protected]>

zachcasper mentioned this pull request Sep 25, 2025

Small changes to developer documentation #55

Merged

Merge branch 'radius-project:main' into main

b30855f

Merge branch 'radius-project:main' into main

8971eed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Add redis resource type and Kubernetes Recipe in Bicep and Tf #13

Add redis resource type and Kubernetes Recipe in Bicep and Tf #13

Reshrahim commented Aug 11, 2025

Uh oh!

zachcasper Sep 4, 2025

Uh oh!

Reshrahim Sep 8, 2025

Uh oh!

Reshrahim Sep 8, 2025

Uh oh!

Uh oh!

Uh oh!

zachcasper Sep 8, 2025

Uh oh!

zachcasper Sep 8, 2025

Uh oh!

Uh oh!

zachcasper Sep 8, 2025

Uh oh!

zachcasper Sep 8, 2025

Uh oh!

zachcasper Sep 8, 2025

Uh oh!

Reshrahim Sep 9, 2025

Uh oh!

zachcasper Sep 8, 2025

Uh oh!

zachcasper Sep 8, 2025

Uh oh!

Uh oh!

		host = "${kubernetes_service.redis.metadata[0].name}.${kubernetes_service.redis.metadata[0].namespace}.svc.cluster.local"
		port = local.port

Add redis resource type and Kubernetes Recipe in Bicep and Tf #13

Are you sure you want to change the base?

Add redis resource type and Kubernetes Recipe in Bicep and Tf #13

Conversation

Reshrahim commented Aug 11, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!