Phase 1: inject managed Recipe secrets through container connections - #300
Draft
willdavsmith wants to merge 3 commits into
Draft
Phase 1: inject managed Recipe secrets through container connections#300willdavsmith wants to merge 3 commits into
willdavsmith wants to merge 3 commits into
Conversation
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
Copilot-Session: 04ca0340-665a-4e7a-a907-0a0e6ea942c5 Signed-off-by: willdavsmith <willdavsmith@gmail.com>
willdavsmith
force-pushed
the
willdavsmith-secret-connection-env-vars
branch
from
August 19, 2026 22:54
1cd0fed to
9b353de
Compare
Signed-off-by: willdavsmith <willdavsmith@gmail.com>
This was referenced Aug 20, 2026
Signed-off-by: willdavsmith <willdavsmith@gmail.com>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
Phase 1 updates the Kubernetes Bicep and Terraform Container Recipes so one producer connection can inject both ordinary connection values and Recipe-managed secret references.
connection.propertiesvalues asCONNECTION_<CONNECTION>_<PROPERTY>.connection.secrets.<key>.sourceand.keymetadata through KubernetesvalueFrom.secretKeyRef.Radius.Security/secretsresources. These now generate fully uppercased names such asCONNECTION_CREDENTIALS_APIKEY; this intentional compatibility change replaces the priorenvFrombehavior that preserved data-key casing.disableDefaultEnvVars, managed-secret-over-ordinary precedence, and uppercase collision validation for secret-derived names.Dependency and scope
Producer-managed output references depend on radius-project/radius#12709 supplying resolved metadata at:
context.resource.connections.<connection>.secrets.<key>.sourcecontext.resource.connections.<connection>.secrets.<key>.keyDirect user-authored Secret connections do not depend on #12709 and retain independent live CI coverage through the existing authored connection source and resolved Secret data.
This PR intentionally excludes Phase 2 catalog adoption and coordinated live producer-output verification. READMEs, schemas, and test apps for AI/models, AI/search, Data/mongoDatabases, Data/redisCaches, Messaging/kafka, Messaging/rabbitMQ, Storage/objectStorage, and
docs/using-resource-types.mdremain at the base contract. End-to-end managed-output verification belongs in Phase 2/resource-types-verification once a Radius build containing #12709 is available; this PR does not conditionally skip that assertion or turn a missing dependency into success.Validation
git diff --checkThe earlier failed Bicep and Terraform jobs both deployed Redis successfully and then failed only at the coordinated
CONNECTION_REDIS_URLassertion because the workflow control plane does not yet exposeconnection.secrets. That assertion and its Redis adoption fixture were removed from Phase 1 rather than conditionally skipped; all unrelated recipe deployments in those jobs passed.