Conversation
|
Warning Review limit reached
More reviews will be available in 43 minutes and 9 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughHidden review stack artifactWalkthroughThe GitHub Actions workflow ChangesDownstream Workflow Reference Update
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Updates from 303e716 to fcbfd6a which adds Patch 5 to the runner container — wraps stop_display_manager() in try/except so the cleanup no longer raises CalledProcessError (no systemd in container) and exits 1, masking all 50 passing smoke test results. Without this fix, E2E smoke always exits code 1 even when all scenarios pass. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…continue-on-error)
…e_scenario) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Includes all qecore-headless container patches for images where gnome-session runs as a different UID (lts, dakota, bazzite, gnomeos): - Patch 4: pgrep UID fallback - Patch 4b: no-PID env seed from container os.environ - Patch 6: continue instead of exiting on empty env dict - Patch 8: seed full container os.environ on IOError reading /proc Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
Superseded by fresh PR with updated SHA 20fb066 |
Pull request was closed
Bumps testsuite SHA to
fcbfd6a(was303e716) which adds all e2e workflow fixes including the critical Patch 5 that wrapsstop_display_manager()in try/except.Without this fix, E2E smoke always exits code 1 (CalledProcessError on
sudo systemctl stop gdminside the container — no systemd PID 1), masking all 50 passing scenarios.Fixes included since
25c9ad0(previous pin):--apply-livefallback on ostree imagesbluefin-nvidiaimages to latest working kernel, monthly #327: wrap stop_display_manager() in try/except (critical — fixes exit code 1)This is the minimum required to get green E2E smoke runs in all consumer repos.
Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com
Summary by CodeRabbit