Skip to content

Add CI coverage for database.enabled=true install #12227

Closed

Description

@sylvainsf

Overview

PR #12214 added database.enabled=true support (PostgreSQL-backed control plane for Repo Radius / rad startup / rad shutdown). During review, two blocking bugs in the database-enabled path (a non-existent mirror image tag → ImagePullBackOff, and missing per-RP GRANTs → permission denied for table resources) reached the PR because no CI job exercises database.enabled=true — the default functional suite installs with database.enabled=false.

PR #12214 added cheap regression guards via helm template assertions (image reference + GRANT statements), but those only validate rendering, not an actual install-and-run.

Acceptance criteria

  • A CI job installs Radius with database.enabled=true on a kind cluster and confirms the control plane reaches a healthy state (pods Ready, no ImagePullBackOff, no permission denied crash).
  • Ideally exercises a minimal deploy so the PostgreSQL store is actually written and read.
  • Runs on PRs that touch the Helm chart database templates and/or the database provider code.

Additional context

Activity

  1. added
    maintenanceIssue is a non-user-facing task like updating tests, improving automation, etc..
    on Jun 24, 2026
  2. radius-triage commented on Jun 24, 2026

    @radius-triage

    👋 @sylvainsf Thanks for filing this issue.

    A project maintainer will review this issue and get back to you soon.

    We also welcome community contributions! If you would like to pick this item up sooner and submit a pull request, please visit our contribution guidelines and assign this to yourself by commenting "/assign" on this issue.

    For more information on our triage process please visit our triage overview

  3. radius-triage commented on Jun 29, 2026

    @radius-triage

    👍 We've reviewed this issue and have agreed to add it to our backlog. Please subscribe to this issue for notifications, we'll provide updates when we pick it up.

    We also welcome community contributions! If you would like to pick this item up sooner and submit a pull request, please visit our contribution guidelines and assign this to yourself by commenting "/assign" on this issue.

    For more information on our triage process please visit our triage overview

  4. nicolejms commented on Jun 29, 2026

    @nicolejms
    Contributor

    bringing into next sprint. Sylvain has fix in his branch.

  5. added a commit that references this issue on Aug 25, 2026
    14736f4
  6. added a commit that references this issue on Aug 26, 2026
    4222a87
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

maintenanceIssue is a non-user-facing task like updating tests, improving automation, etc..triagedThis issue has been reviewed and triaged

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions