Add read-only local e2e test mode with backplane support#512
Add read-only local e2e test mode with backplane support#512nephomaniac wants to merge 1 commit into
Conversation
|
/label tide/merge-method-squash |
|
Warning Review limit reached
More reviews will be available in 28 minutes and 59 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: Repository YAML (base), Central YAML (inherited) Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (3)
✨ 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 |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: nephomaniac The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
Holding during review + merge timing |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #512 +/- ##
=======================================
Coverage 68.29% 68.29%
=======================================
Files 8 8
Lines 1088 1088
=======================================
Hits 743 743
Misses 315 315
Partials 30 30 🚀 New features to boost your workflow:
|
Add Ginkgo labels (read-only/mutating) to all test specs, a wrapper script for running tests locally, and backplane impersonation support. - Label all tests as read-only or mutating for filtering - Add test/e2e/run-local.sh wrapper (auto-detects backplane, builds binary, filters by label) - Add test/e2e/lint-labels.sh to enforce labels on new tests - Support E2E_IMPERSONATE_USER for backplane cluster-admin impersonation - Support E2E_READ_ONLY to make Prometheus client non-fatal (metric tests skip when prom unavailable due to restricted RBAC) Pipeline behavior is unchanged: no env vars set, no label filter, all tests run with strict assertions. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
3f3f087 to
2763cf9
Compare
|
PR needs rebase. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
@nephomaniac: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
Summary
read-only/mutating) to all test specs for filteringtest/e2e/run-local.shwrapper script for running tests locally against live clusterstest/e2e/lint-labels.shto enforce labels on new testsE2E_IMPERSONATE_USERfor backplane cluster-admin impersonationE2E_READ_ONLYto make Prometheus client non-fatal (metric tests skip when prom unavailable)Pipeline behavior is unchanged: no env vars set, no label filter, all tests run with strict assertions.
Usage
Test plan
lint-labels.shpasses🤖 Generated with Claude Code