fix load balancing e2e test#8352
Conversation
✅ Deploy Preview for cerulean-figolla-1f9435 canceled.
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #8352 +/- ##
=======================================
Coverage 73.83% 73.83%
=======================================
Files 242 242
Lines 37064 37064
=======================================
Hits 27365 27365
Misses 7767 7767
Partials 1932 1932 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Looks like the load balancing tests are still failing. |
yeah, no idea why this happen. |
|
Should we disable these tests for now? They're blocking CI. |
|
let's disabled it. |
Signed-off-by: zirain <zirain2009@gmail.com>
Signed-off-by: zirain <zirain2009@gmail.com>
|
Hi @Zirian — thanks for taking care of this. Just FYI, in the community meeting, the conclusion was to give this a bit more time to resolve properly rather than disabling the tests, since this is an important feature that users rely on. |
|
I have no idea why this happen, let me close this for someone know more. |
#8349 didn't fix it at all
This PR refactor the e2e tests:
readinessProbetoapp-backend-lbdeployment to make sure all the pods ready before run traffic tests.