You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
ci: revert Swatinem/rust-cache to allow-listed v2.9.2 - #2582
Since #2555 merged (2026-11-06, 13:52 UTC), every run of the Integration Tests and NeMo Relay plugin workflows has failed at startup, both on develop and on every PR. #2555 moved Swatinem/rust-cache from 6323deb (v2.9.2) to d1527c8 (v2.9.3). The repository's Actions policy only lets an action from outside GitHub run if it is on an allow-list. That list permits rust-cache at specific commits, including 6323deb but not d1527c8, so GitHub won't start either workflow.
This PR points rust-cache back to v2.9.2 in integration-tests.yml and relay-plugin.yml. Both files are identical to their state before #2555. v2.9.3 only fixes cache-timestamp pruning and changes a log message, so nothing is lost by going back.
Nobody noticed because a run that fails at startup doesn't create a check run. #2555's own run on 3 November failed this way, but nothing showed on the PR, and Integration Tests isn't a required check. As a result, #2565, #2566, #2567, #2573 and #2581 merged without integration tests. The develop run that follows this merge will cover them.
Follow-up: before we can take v2.9.3, the allow-list needs Swatinem/rust-cache@d1527c8f67ef9af357290475d4af4b0fab8f12e0 added. Until then, close Dependabot's PR when it proposes the bump again.
Related Issue(s)
Fixes #<issue_number>
Issue assignee: @
Verification
6323deb102c322ba6fcbdcafc7e3dddab59af2b6 is the upstream v2.9.2 tag commit.
I checked every action used under .github/ against the repository's allow-list (gh api repos/NVIDIA-NeMo/Guardrails/actions/permissions/selected-actions). rust-cache@6323deb matches an entry; rust-cache@d1527c8 matches none. Both affected workflows use rust-cache, and the workflows that don't use it still pass.
uv run --locked pre-commit run --files .github/workflows/integration-tests.yml .github/workflows/relay-plugin.yml: passed, including zizmor.
The Integration Tests and NeMo Relay plugin workflows should start and pass on this PR. Full Tests doesn't run because it ignores .github/**.
Not checked: the organization-level Actions policy (I don't have access) and GitHub's exact refusal message, which a startup failure doesn't expose through the API. The diagnosis rests on the timing and the allow-list contents.
AI Assistance
No AI tools were used.
AI tools were used; a human reviewed and can explain every change (tool: Claude Code).
This restores the allow-listed cache action version in two workflows so they can start. I found no behavioral risk in the change itself.
Architecture risk: Low · up to 71745
The changed surface does not map to a changed system, dependency edge, entrypoint, or external dependency.
Changed systems: None identified.
Architecture concerns
No architecture-level concerns identified.
Review details
Before / after behavior
observed — Modified behavior in .github/workflows/integration-tests.yml: The Cargo cache action pin changes from the v2.9.3 commit to the v2.9.2 commit; the job’s cache configuration is unchanged.
observed — Modified behavior in .github/workflows/relay-plugin.yml: The Cargo cache action reference changed from the commit labeled v2.9.3 to the commit labeled v2.9.2.
Pre-merge checks: 6
Check name
Status
Explanation
Description Check
Check skipped - CodeRabbit’s high-level summary is enabled.
Title check
The title "ci: revert Swatinem/rust-cache to allow-listed v2.9.2" matches the main change. The PR pins Swatinem/rust-cache back to commit 6323deb (v2.9.2) in integration-tests.yml and relay-plugin.yml…
Docstring Coverage
No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check
Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check
Check skipped because no linked issues were found for this pull request.
Test Results For Major Changes
The PR makes a minor change to the Rust cache action pin in two workflows. It does not change application numerics, convergence, or performance. The PR description also reports pre-commit results.
Full details: Title check
Explanation
The title "ci: revert Swatinem/rust-cache to allow-listed v2.9.2" matches the main change. The PR pins Swatinem/rust-cache back to commit 6323deb (v2.9.2) in integration-tests.yml and relay-plugin.yml, replacing d1527c8 (v2.9.3). The title names the action, the version, and the reason (the allow-list). The change is small and the title describes it accurately. The title is short, specific, and follows the conventional commit style.
Full details: Docstring Coverage
Explanation
No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.)
Finishing Touches
Generate unit tests (beta)
Commit to this branch
Create a new PR
Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Reviewing files that changed from the base of the PR and between c550266 and 71745af.
Files selected for processing: 2
.github/workflows/integration-tests.yml
.github/workflows/relay-plugin.yml
Included review availability: This review used your included allowance. Your plan provides up to 12 included reviews per hour; 11 remain after this review.
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
size: XSstatus: triagedTriaged by a maintainer; eligible for automated review (CodeRabbit/Greptile).
1 participant
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.
Description
Since #2555 merged (2026-11-06, 13:52 UTC), every run of the Integration Tests and NeMo Relay plugin workflows has failed at startup, both on
developand on every PR. #2555 movedSwatinem/rust-cachefrom6323deb(v2.9.2) tod1527c8(v2.9.3). The repository's Actions policy only lets an action from outside GitHub run if it is on an allow-list. That list permitsrust-cacheat specific commits, including6323debbut notd1527c8, so GitHub won't start either workflow.This PR points
rust-cacheback to v2.9.2 inintegration-tests.ymlandrelay-plugin.yml. Both files are identical to their state before #2555. v2.9.3 only fixes cache-timestamp pruning and changes a log message, so nothing is lost by going back.Nobody noticed because a run that fails at startup doesn't create a check run. #2555's own run on 3 November failed this way, but nothing showed on the PR, and Integration Tests isn't a required check. As a result, #2565, #2566, #2567, #2573 and #2581 merged without integration tests. The
developrun that follows this merge will cover them.Follow-up: before we can take v2.9.3, the allow-list needs
Swatinem/rust-cache@d1527c8f67ef9af357290475d4af4b0fab8f12e0added. Until then, close Dependabot's PR when it proposes the bump again.Related Issue(s)
Verification
6323deb102c322ba6fcbdcafc7e3dddab59af2b6is the upstreamv2.9.2tag commit..github/against the repository's allow-list (gh api repos/NVIDIA-NeMo/Guardrails/actions/permissions/selected-actions).rust-cache@6323debmatches an entry;rust-cache@d1527c8matches none. Both affected workflows userust-cache, and the workflows that don't use it still pass.uv run --locked pre-commit run --files .github/workflows/integration-tests.yml .github/workflows/relay-plugin.yml: passed, including zizmor..github/**.AI Assistance
Checklist
Summary by CodeRabbit