test(e2e): fix debug mode test timing and add PVC cleanup #1554
Conversation
|
Skipping CI for Draft Pull Request. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: akurinnoy, rohanKanojia The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Thanks a lot for the fix! Do you think it fixes #1535 ? |
|
/ok-to-test |
|
New changes are detected. LGTM label has been removed. |
|
/retest |
2 similar comments
|
/retest |
|
/retest |
|
/retest |
akurinnoy
force-pushed
the
fix-e2e-test
branch
from
2dcce79 to
11a4f81
Compare
|
/retest |
Signed-off-by: Oleksii Kurinnyi <[email protected]>
- Add ControllerRuntimeClient() method to test client for accessing CRDs - Add controllerv1alpha1 to scheme for DWOC access - Add DWOC save/restore in BeforeAll/AfterAll to prevent config leaks - Use ginkgo.Ordered to ensure sequential test execution - Update copyright year to 2026 This prevents config leaks between custom init container tests and ensures clean state for each test run. Assisted-by: Claude Sonnet 4.5 Signed-off-by: Oleksii Kurinnyi <[email protected]>
akurinnoy
force-pushed
the
fix-e2e-test
branch
from
11a4f81 to
1014b99
Compare
3 participants
What does this PR do?
This PR fixes the debug mode e2e test: waits the test devworkspace for Running status instead of Starting. Also, is adds the AfterEach cleanup step to prevent PVC conflicts.
What issues does this PR fix or reference?
#1569
Is it tested? How?
PR Checklist
/test v8-devworkspace-operator-e2e, v8-che-happy-pathto trigger)v8-devworkspace-operator-e2e: DevWorkspace e2e testv8-che-happy-path: Happy path for verification integration with Che