search

vertica / vertica-kubernetes

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

Update upgrade reconciler to handle subclusters in sandbox #772

Closed roypaulin closed 6 months ago

roypaulin commented 6 months ago

This change will modify the upgrade reconcilers, both offline and online, to function in either the main cluster or a sandbox. Our immediate plan is to use the offline upgrade reconciler within the sandbox controller, although this will be done in a follow-on task.

roypaulin commented 6 months ago

Everything looks good. Thanks for doing the revisions. I noticed in the last run leg 9 failed related to replicated upgrade. I didn't look at it in too much detail. So, I'm not sure if its a flaky test or not. I'm approving assuming it's a flaky test or requires a small change to fix.

I think it is a flaky test, I got that yesterday and rerun the test. At wait-for-secondaries only 2/3 nodes are up. If it persists we would need to take a closer look.