Skip to content

Comments

feat: shared PVC for Terraform storage across applications-rp and dyn…#11191

Draft
vishwahiremat wants to merge 1 commit intofeatures/terraform-bicep-settingsfrom
vishwahiremat/terraformInstall
Draft

feat: shared PVC for Terraform storage across applications-rp and dyn…#11191
vishwahiremat wants to merge 1 commit intofeatures/terraform-bicep-settingsfrom
vishwahiremat/terraformInstall

Conversation

@vishwahiremat
Copy link
Contributor

@vishwahiremat vishwahiremat commented Feb 5, 2026

Description

This PR changes Terraform storage from pod-local emptyDir volumes to a shared PersistentVolumeClaim with ReadWriteMany access mode. This enables applications-rp and dynamic-rp to share the same Terraform binaries installed via rad terraform install.

Changes:

  • New terraform-pvc.yaml template creating a shared PVC (terraform-storage)
  • Updated both RP deployments to mount the shared PVC instead of emptyDir
  • Added NFS provisioner to Kind cluster setup for RWX support in CI
  • Updated Helm tests to validate PVC configuration

Why:
Previously, each RP had its own emptyDir volume, so rad terraform install only affected applications-rp. With a shared PVC, both RPs can access the same installed Terraform binary.

Type of change

  • This pull request adds or changes features of Radius and has an approved issue (issue link required).

Fixes: #issue_number

Contributor checklist

Please verify that the PR meets the following requirements, where applicable:

  • An overview of proposed schema changes is included in a linked GitHub issue.
    • Yes
    • Not applicable
  • A design document PR is created in the design-notes repository, if new APIs are being introduced.
    • Yes
    • Not applicable
  • The design document has been reviewed and approved by Radius maintainers/approvers.
    • Yes
    • Not applicable
  • A PR for the samples repository is created, if existing samples are affected by the changes in this PR.
    • Yes
    • Not applicable
  • A PR for the documentation repository is created, if the changes in this PR affect the documentation or any user facing updates are made.
    • Yes
    • Not applicable
  • A PR for the recipes repository is created, if existing recipes are affected by the changes in this PR.
    • Yes
    • Not applicable

…amic-rp

Signed-off-by: ytimocin <ytimocin@microsoft.com>
@vishwahiremat vishwahiremat temporarily deployed to external-contributor-approval February 5, 2026 18:35 — with GitHub Actions Inactive
@radius-functional-tests
Copy link

radius-functional-tests bot commented Feb 5, 2026

Radius functional test overview

🔍 Go to test action run

Click here to see the test run details
Name Value
Repository radius-project/radius
Commit ref 31bf832
Unique ID func6fc3ad20cb
Image tag pr-func6fc3ad20cb
  • gotestsum 1.13.0
  • KinD: v0.29.0
  • 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-func6fc3ad20cb
  • 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-func6fc3ad20cb
  • dynamic-rp test image location: ghcr.io/radius-project/dev/dynamic-rp:pr-func6fc3ad20cb
  • controller test image location: ghcr.io/radius-project/dev/controller:pr-func6fc3ad20cb
  • ucp test image location: ghcr.io/radius-project/dev/ucpd:pr-func6fc3ad20cb
  • 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 corerp-cloud functional tests...
⌛ Starting ucp-cloud functional tests...
❌ Failed to install Radius for corerp-cloud functional test. Please check the logs for more details
❌ corerp-cloud functional test failed. Please check the logs for more details
❌ Failed to install Radius for ucp-cloud functional test. Please check the logs for more details
❌ ucp-cloud functional test cancelled. Please check the logs for more details
⌛ Starting corerp-cloud functional tests...
❌ Failed to install Radius for corerp-cloud functional test. Please check the logs for more details
❌ corerp-cloud functional test failed. Please check the logs for more details

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.

2 participants