search

vertica / vertica-kubernetes

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

Add status reconciler to sandbox controller #793

Closed roypaulin closed 5 months ago

roypaulin commented 5 months ago

The status reconciler in vdb controller was removing a subcluster's status when the subcluster is in a sandbox. Now it will keep unchanged the status of subclusters that do not belong to the passed in sandbox. To do that, it finds all the subclusters regardless of their cluster.

roypaulin commented 5 months ago

Are there any e2e test changes we can do now that we can rely on the status fields across sandboxes?

I can update the sandbox test case

spilchen commented 5 months ago

@roypaulin a follow-on PR for this status change is that we want .status.subclusters[].detail[].addedToDB to control if a pod has a database. This is needed by @cchen-vertica to properly start a node after unsandboxing. We can discuss more in today's scrum.