Skip to content

Consult the council#11432

Open
sylvainsf wants to merge 12 commits intomainfrom
sylvainsf/rivendell-review
Open

Consult the council#11432
sylvainsf wants to merge 12 commits intomainfrom
sylvainsf/rivendell-review

Conversation

@sylvainsf
Copy link
Contributor

Description

Add Rivendell Council as an AI-powered code review workflow for pull requests. This integrates sylvainsf/rivendell_council as a reusable GitHub Actions workflow that convenes a fellowship of AI reviewers — each with a specialized domain — to evaluate every PR.

The council reviews PRs in parallel using GitHub Models via actions/ai-inference, requiring no external API keys. Each member scores the diff on a 1–10 scale and provides concerns and suggestions. Gandalf synthesizes all reviews and delivers a final PASS/FAIL verdict as a PR comment.

Council members:

  • 🌟 Elrond — Correctness (logic, edge cases, tests)
  • ⚔️ Gimli — Strength (security, validation, error handling)
  • 👑 Aragorn — Maintainability (structure, idioms, naming)
  • 🏹 Legolas — Performance (efficiency, resource usage)
  • 🧙 Gandalf — Arbiter (synthesizes all reviews, delivers verdict)

Changes:

  • Added .github/workflows/council-review.yml calling the reusable workflow

Type of change

  • This pull request is a minor refactor, code cleanup, test improvement, or other maintenance task and doesn't change the functionality of Radius (issue link optional).

Fixes: N/A

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

Signed-off-by: Sylvain Niles <sylvainniles@microsoft.com>
@sylvainsf sylvainsf requested review from a team as code owners March 15, 2026 21:19
Copilot AI review requested due to automatic review settings March 15, 2026 21:19
@sylvainsf sylvainsf requested a deployment to external-contributor-approval March 15, 2026 21:19 — with GitHub Actions Waiting
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds an AI-driven pull request review workflow to the Radius repository by invoking the external sylvainsf/rivendell_council reusable workflow, intended to run automated multi-reviewer analysis on PR open/sync/reopen events.

Changes:

  • Added a new GitHub Actions workflow to run Rivendell Council reviews on pull requests.
  • Configured the workflow to request PR write permissions and GitHub Models read access, and to pass a model identifier to the reusable workflow.

Copy link
Contributor

Copilot AI commented Mar 15, 2026

@sylvainsf I've opened a new pull request, #11433, to work on those changes. Once the pull request is ready, I'll request review from you.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Sylvain Niles <sylvainniles@microsoft.com>
@sylvainsf sylvainsf requested a deployment to external-contributor-approval March 15, 2026 21:32 — with GitHub Actions Waiting
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Sylvain Niles <sylvainniles@microsoft.com>
@sylvainsf sylvainsf requested a deployment to external-contributor-approval March 15, 2026 21:32 — with GitHub Actions Waiting
Copy link
Contributor

Copilot AI commented Mar 15, 2026

@sylvainsf I've opened a new pull request, #11434, to work on those changes. Once the pull request is ready, I'll request review from you.

…lures (#11434)

# Description

On `pull_request` events from forks, `GITHUB_TOKEN` is read-only — the
`council` job requesting `pull-requests: write` would fail for external
contributors. Adds a job-level guard to skip execution on fork PRs.

## Changes

- **`.github/workflows/rivendell_council.yml`**: Added `if` condition to
the `council` job:
  ```yaml
  if: github.event.pull_request.head.repo.full_name == github.repository
  ```
The job is now skipped entirely when the PR originates from a fork,
avoiding token permission errors.

## Type of change

- This pull request is a minor refactor, code cleanup, test improvement,
or other maintenance task and doesn't change the functionality of Radius
(issue link optional).

## Contributor checklist

- An overview of proposed schema changes is included in a linked GitHub
issue.
    - [ ] Yes <!-- TaskRadio schema -->
    - [x] Not applicable <!-- TaskRadio schema -->
- A design document PR is created in the [design-notes
repository](https://github.com/radius-project/design-notes/), if new
APIs are being introduced.
    - [ ] Yes <!-- TaskRadio design-pr -->
    - [x] Not applicable <!-- TaskRadio design-pr -->
- The design document has been reviewed and approved by Radius
maintainers/approvers.
    - [ ] Yes <!-- TaskRadio design-review -->
    - [x] Not applicable <!-- TaskRadio design-review -->
- A PR for the [samples
repository](https://github.com/radius-project/samples) is created, if
existing samples are affected by the changes in this PR.
    - [ ] Yes <!-- TaskRadio samples-pr -->
    - [x] Not applicable <!-- TaskRadio samples-pr -->
- A PR for the [documentation
repository](https://github.com/radius-project/docs) is created, if the
changes in this PR affect the documentation or any user facing updates
are made.
    - [ ] Yes <!-- TaskRadio docs-pr -->
    - [x] Not applicable <!-- TaskRadio docs-pr -->
- A PR for the [recipes
repository](https://github.com/radius-project/recipes) is created, if
existing recipes are affected by the changes in this PR.
    - [ ] Yes <!-- TaskRadio recipes-pr -->
    - [x] Not applicable <!-- TaskRadio recipes-pr -->

<!-- START COPILOT CODING AGENT TIPS -->
---

💬 Send tasks to Copilot coding agent from
[Slack](https://gh.io/cca-slack-docs) and
[Teams](https://gh.io/cca-teams-docs) to turn conversations into code.
Copilot posts an update in your thread when it's finished.

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: sylvainsf <540991+sylvainsf@users.noreply.github.com>
@sylvainsf sylvainsf requested a deployment to external-contributor-approval March 15, 2026 21:37 — with GitHub Actions Waiting
Signed-off-by: Sylvain Niles <sylvainniles@microsoft.com>
@sylvainsf sylvainsf deployed to external-contributor-approval March 15, 2026 21:39 — with GitHub Actions Active
@codecov
Copy link

codecov bot commented Mar 15, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 51.24%. Comparing base (c30a4e2) to head (8360908).

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #11432      +/-   ##
==========================================
- Coverage   51.24%   51.24%   -0.01%     
==========================================
  Files         699      699              
  Lines       44062    44062              
==========================================
- Hits        22580    22578       -2     
- Misses      19326    19327       +1     
- Partials     2156     2157       +1     

☔ View full report in Codecov by Sentry.
📢 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.

@github-actions
Copy link

github-actions bot commented Mar 15, 2026

Unit Tests

    2 files  ±0    415 suites  ±0   6m 41s ⏱️ -12s
4 881 tests ±0  4 879 ✅ ±0  2 💤 ±0  0 ❌ ±0 
5 783 runs  ±0  5 781 ✅ ±0  2 💤 ±0  0 ❌ ±0 

Results for commit 8360908. ± Comparison against base commit c30a4e2.

♻️ This comment has been updated with latest results.

@radius-functional-tests
Copy link

radius-functional-tests bot commented Mar 15, 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 f3e7e27
Unique ID funcbfffaeb055
Image tag pr-funcbfffaeb055
  • 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-funcbfffaeb055
  • 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-funcbfffaeb055
  • dynamic-rp test image location: ghcr.io/radius-project/dev/dynamic-rp:pr-funcbfffaeb055
  • controller test image location: ghcr.io/radius-project/dev/controller:pr-funcbfffaeb055
  • ucp test image location: ghcr.io/radius-project/dev/ucpd:pr-funcbfffaeb055
  • 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

fix to call the current version

Signed-off-by: Sylvain Niles <sylvainniles@microsoft.com>
@sylvainsf sylvainsf requested a deployment to external-contributor-approval March 15, 2026 22:41 — with GitHub Actions Waiting
this adds pinned shas for all upstream actions

Signed-off-by: Sylvain Niles <sylvainniles@microsoft.com>
@sylvainsf sylvainsf requested a deployment to external-contributor-approval March 15, 2026 22:48 — with GitHub Actions Waiting
valid model

Signed-off-by: Sylvain Niles <sylvainniles@microsoft.com>
@sylvainsf sylvainsf requested a deployment to external-contributor-approval March 15, 2026 23:03 — with GitHub Actions Waiting
Signed-off-by: Sylvain Niles <sylvainniles@microsoft.com>
@sylvainsf sylvainsf requested a deployment to external-contributor-approval March 15, 2026 23:09 — with GitHub Actions Waiting
run

Signed-off-by: Sylvain Niles <sylvainniles@microsoft.com>
run

Signed-off-by: Sylvain Niles <sylvainniles@microsoft.com>
Signed-off-by: Sylvain Niles <sylvainniles@microsoft.com>
@sylvainsf sylvainsf requested a deployment to external-contributor-approval March 15, 2026 23:21 — with GitHub Actions Waiting
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.

3 participants