Skip to content

Add PostgreSQL to the default Kubernetes Recipe Pack - #12755

Merged
willdavsmith merged 5 commits into
radius-project:mainfrom
willdavsmith:willdavsmith-add-default-postgresql-recipe
Aug 25, 2026
Merged

Add PostgreSQL to the default Kubernetes Recipe Pack#12755
willdavsmith merged 5 commits into
radius-project:mainfrom
willdavsmith:willdavsmith-add-default-postgresql-recipe

Conversation

@willdavsmith

@willdavsmith willdavsmith commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add Radius.Data/postgreSqlDatabases to the default Recipe Pack created by rad install kubernetes
  • source the existing contrib Kubernetes Bicep Recipe from ghcr.io/radius-project/kube-recipes/postgresqldatabases using the standard channel-aware tag resolution
  • add a focused corerp-noncloud E2E test that deploys PostgreSQL without an explicit Recipe and validates the Recipe-provisioned Pod, Deployment, and Service

Fixes: radius-project/resource-types-contrib#276

Testing

  • go test ./pkg/cli/recipepack ./test/validation
  • go test ./test/functional-portable/corerp/noncloud/resources -run '^$'
  • ~/.rad/bin/bicep build test/functional-portable/corerp/noncloud/resources/testdata/corerp-resources-postgresqldatabase.bicep --stdout
  • gofmt
  • git diff --check

The targeted E2E requires the PR CI Kubernetes environment and is covered by the corerp-noncloud functional test job.

Copilot AI lite review requested due to automatic review settings August 20, 2026 21:23
@willdavsmith
willdavsmith requested review from a team as code owners August 20, 2026 21:23

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the CLI’s default Kubernetes recipe pack (created by rad install kubernetes) to include the Radius.Data/postgreSqlDatabases core type, sourcing the recipe from the shared ghcr.io/radius-project/kube-recipes registry using the existing channel-aware tag resolution logic.

Changes:

  • Add Radius.Data/postgreSqlDatabases to the core default recipe definitions with an OCI source of .../postgresqldatabases:<resolved-tag>.
  • Update unit tests to expect the additional default core recipe entry and validate it is present in the generated default recipe pack resource.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
pkg/cli/recipepack/recipepack.go Adds PostgreSQL database recipe entry to the core default recipe definitions with channel-aware tag resolution.
pkg/cli/recipepack/recipepack_test.go Updates tests for the new core recipe count/type list and asserts the PostgreSQL recipe is included in the default recipe pack resource.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread pkg/cli/recipepack/recipepack_test.go Outdated
@codecov

codecov Bot commented Aug 20, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 27.27273% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 59.90%. Comparing base (921e55c) to head (13df16c).

Files with missing lines Patch % Lines
test/validation/k8s.go 0.00% 8 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #12755      +/-   ##
==========================================
- Coverage   59.91%   59.90%   -0.02%     
==========================================
  Files         775      775              
  Lines       46305    46316      +11     
==========================================
- Hits        27745    27744       -1     
- Misses      18560    18572      +12     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@willdavsmith
willdavsmith force-pushed the willdavsmith-add-default-postgresql-recipe branch from 18a126f to d1606b0 Compare August 24, 2026 17:30
Signed-off-by: willdavsmith <willdavsmith@gmail.com>
Signed-off-by: willdavsmith <willdavsmith@gmail.com>
@willdavsmith
willdavsmith force-pushed the willdavsmith-add-default-postgresql-recipe branch from d1606b0 to 71f6ba8 Compare August 24, 2026 17:36
Comment thread pkg/cli/recipepack/recipepack_test.go Outdated
Signed-off-by: willdavsmith <willdavsmith@gmail.com>
lakshmimsft
lakshmimsft previously approved these changes Aug 24, 2026
Signed-off-by: Will Smith <willdavsmith@gmail.com>
Signed-off-by: willdavsmith <willdavsmith@gmail.com>
@willdavsmith
willdavsmith enabled auto-merge August 25, 2026 16:19
@radius-functional-tests

radius-functional-tests Bot commented Aug 25, 2026

Copy link
Copy Markdown

Radius functional test overview

🔍 Go to test action run

Click here to see the test run details
Name Value
Repository willdavsmith/radius
Commit ref 13df16c
Unique ID funcbf7b8c18fa
Image tag pr-funcbf7b8c18fa
  • Dapr: 1.14.4
  • Azure KeyVault CSI driver: 1.4.2
  • Azure Workload identity webhook: 1.3.0
  • Bicep recipe location ghcr.io/radius-project/dev/test/testrecipes/test-bicep-recipes/<name>:pr-funcbf7b8c18fa
  • Terraform recipe location http://tf-module-server.radius-test-tf-module-server.svc.cluster.local/<name>.zip (in cluster)
  • applications-rp test image location: ghcr.io/radius-project/dev/applications-rp:pr-funcbf7b8c18fa
  • dynamic-rp test image location: ghcr.io/radius-project/dev/dynamic-rp:pr-funcbf7b8c18fa
  • controller test image location: ghcr.io/radius-project/dev/controller:pr-funcbf7b8c18fa
  • ucp test image location: ghcr.io/radius-project/dev/ucpd:pr-funcbf7b8c18fa
  • deployment-engine test image location: ghcr.io/radius-project/deployment-engine:latest

Test Status

⌛ Building Radius and pushing container images for functional tests...
✅ Container images build succeeded
⌛ Publishing Bicep Recipes for functional tests...
✅ Recipe publishing succeeded
⌛ Starting ucp-cloud functional tests...
⌛ Starting corerp-cloud functional tests...
✅ ucp-cloud functional tests succeeded
✅ corerp-cloud functional tests succeeded

@willdavsmith
willdavsmith added this pull request to the merge queue Aug 25, 2026
Merged via the queue into radius-project:main with commit d76e986 Aug 25, 2026
77 of 78 checks passed
@willdavsmith
willdavsmith deleted the willdavsmith-add-default-postgresql-recipe branch August 25, 2026 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: Radius.Data/postgreSqlDatabases is missing from the default Kubernetes recipe pack

3 participants