search

vertica / vertica-kubernetes

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

Implement controller logic for VerticaRestorePointsQuery #693

Closed chinhtranvan closed 8 months ago

chinhtranvan commented 8 months ago

As we discussed, this task is divided into two parts. This PR completed the first part, which involved calling the vclusterops API to perform the restore points query. A follow-up task to copy the result of the query into the CRs status fields will be processed later

chinhtranvan commented 8 months ago

@spilchen All comments have been addressed. As we discussed in the scrum, I will wait for your changes to be updated and merged into 'vnext' in order to resolve the failures in the e2e tests

chinhtranvan commented 8 months ago

Thanks for doing the revisions. vnext has been merged up. You should be able to pull that into your branch for the e2e tests to succeed.

Thank you for informing me. I observed that the scorecard test failed when you merged 'main' into 'vnext' https://github.com/vertica/vertica-kubernetes/actions/runs/7788337652. The error message reads: 'Unable to find image 'registry:2' locally. Docker: Error response from daemon: unauthorized: authentication required.' It appears that we are missing the image

spilchen commented 8 months ago

Thanks for doing the revisions. vnext has been merged up. You should be able to pull that into your branch for the e2e tests to succeed.

Thank you for informing me. I observed that the scorecard test failed when you merged 'main' into 'vnext' https://github.com/vertica/vertica-kubernetes/actions/runs/7788337652. The error message reads: 'Unable to find image 'registry:2' locally. Docker: Error response from daemon: unauthorized: authentication required.' It appears that we are missing the image

That's not an image we build. So, I'm guessing its a flaky network connection. I'll rerun that job once the workflow is done.