search

vertica / vertica-kubernetes

Operator, container and Helm chart to deploy Vertica in Kubernetes
Apache License 2.0
44 stars 25 forks source link

[Ready for review] Fixed e2e tests #964

Open cchen-vertica opened 6 days ago

cchen-vertica commented 6 days ago

This PR addresses issues in the daily run for leg-4 and udx.

Leg-4: I improved stability by refactoring and isolating vdb-gen, reducing the likelihood of failures. However, the root cause of the reconcile requeue not functioning in GitHub CI remains unclear. After we use our own runner, we can revisit this issue again. Additionally, I resolved K8s scrutinizer issues for other failed revive tests.

UDX: I updated the DevOps Jenkins pipeline and fixed a script issue in the e2e test. Now udx is stable.

roypaulin commented 4 days ago

Should we fix those tests before merging this? Otherwise, regular e2e may always fail.

HaoYang0000 commented 3 days ago

We should also increase the timeout of istio, maybe to 12 min? As sometimes the istio job failed due to timeout

cchen-vertica commented 1 day ago

Should we fix those tests before merging this? Otherwise, regular e2e may always fail.

Yes, I fixed the tests. The vdb-gen test is still somewhat flaky, but it’s much more stable now.

cchen-vertica commented 1 day ago

We should also increase the timeout of istio, maybe to 12 min? As sometimes the istio job failed due to timeout

istio should be handled by other PR. This one is for leg4 and udx. Also, before increasing the timeout, we should confirm that Istio is failing due to slowness. For instance, issues with vdb-gen and udx are caused by other factors, so increasing the timeout would only extend our wait time without resolving the root problems.