ROX-29771: Ensure Sensor is up before e2e tests#17502
Conversation
|
This change is part of the following stack: Change managed by git-spice. |
|
Skipping CI for Draft Pull Request. |
|
Images are ready for the commit at a44e392. To use with deploy scripts, first |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #17502 +/- ##
=======================================
Coverage 48.67% 48.67%
=======================================
Files 2723 2723
Lines 202878 202878
=======================================
Hits 98752 98752
Misses 96359 96359
Partials 7767 7767
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
/test gke-nongroovy-e2e-tests gke-upgrade-tests |
|
/retest-times 10 ocp-4-12-nongroovy-e2e-tests |
|
/test ocp-4-12-nongroovy-e2e-tests |
|
/test ocp-4-19-nongroovy-e2e-tests ocp-4-18-nongroovy-e2e-tests ocp-4-12-nongroovy-e2e-tests gke-nongroovy-e2e-tests |
|
/test ocp-4-12-nongroovy-e2e-tests |
4 similar comments
|
/test ocp-4-12-nongroovy-e2e-tests |
|
/test ocp-4-12-nongroovy-e2e-tests |
|
/test ocp-4-12-nongroovy-e2e-tests |
|
/test ocp-4-12-nongroovy-e2e-tests |
|
/test ocp-4-12-nongroovy-e2e-tests |
Description
Add Sensor health checks to
TestPodandTestContainerInstancesto prevent test failures caused by insufficient recovery time after Sensor restarts.Problem
Tests that depend on process event collection (TestPod, TestContainerInstances) can fail when they start immediately after a previous test restarts Sensor. In the analyzed failure:
Solution
Added
waitForSensorHealthy()function that:This is called at the beginning of both
TestPodandTestContainerInstances, following the same pattern used inTestDelegatedScanning.waitForHealthyCentralSensorConn().User-facing documentation
Testing and quality
Automated testing
How I validated my change
ocp-4-XX-nongroovy-e2e-testsmultiple timesExpected CI behavior:
If tests still fail in CI, the health check timeouts may need adjustment or additional diagnostics may be needed to understand Sensor recovery time.
AI-assisted development
waitForSensorHealthy()function and health check calls