search

vertica / vertica-kubernetes

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

Applied the latest vclusterOps library #842

Closed cchen-vertica closed 4 months ago

cchen-vertica commented 4 months ago

The latest vclusterOps improves the promote_sandbox API to clean communal storage after sandbox promotion. This PR simply updates vclusterOps library to clean communal storage in the online upgrade.

chinhtranvan commented 4 months ago

I'm fine with these changes; however, I left some comments on the vcluster side. I still think we should add an NMA health op to fail promote_sandbox earlier

chinhtranvan commented 4 months ago

Approved! But you need to pull the latest vclusterops that add the NMA health check

cchen-vertica commented 4 months ago

Approved! But you need to pull the latest vclusterops that add the NMA health check

Sure, will update vclusterOps.