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 sandbox option in vrep #795

Closed chinhtranvan closed 5 months ago

chinhtranvan commented 5 months ago

This PR incorporates the latest changes from the vCluster API to update the sandbox option in vrep

spilchen commented 5 months ago

Can I assume that without this change, replication from a source sandbox would not be possible? Even though the operator provides the source IP for a sandbox, vclusterOps couldn't utilize the source IP without recognizing that it originated from a sandbox.

spilchen commented 5 months ago

I have no issues with this change, assuming my understanding is correct. You should coordinate with @jizhuoyu as you both are changing the same test (see #786)

chinhtranvan commented 5 months ago

Can I assume that without this change, replication from a source sandbox would not be possible? Even though the operator provides the source IP for a sandbox, vclusterOps couldn't utilize the source IP without recognizing that it originated from a sandbox.

Yes, your assumption is correct, vclusterOps needs to identify that the source IP belongs to a sandbox in order to function properly