New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix value in integration test #953

Closed

HaoYang0000 wants to merge 1 commit into main from debug_save_restore_point

Collaborator

HaoYang0000 commented Oct 9, 2024

The value is not setup correctly so we see error in the integration test:
kubectl patch vdb vertica-db --type='json' -p='[{

"op": "add",
"path": "/status/conditions/-",
"value": {"type": "SaveRestorePointNeeded", "status": "True"}
}]' --subresource='status'
The VerticaDB "vertica-db" is invalid:

conditions[3].lastTransitionTime: Required value
conditions[3].message: Required value
conditions[3].reason: Required value

We need to add value for the missing attribute.


          fix value in integration test

3afc057

HaoYang0000 requested review from roypaulin, cchen-vertica, fenic-fawkes, qindotguan and LiboYu2 as code owners

October 9, 2024 10:08

Collaborator Author

HaoYang0000 commented Oct 9, 2024

Duplicate with #952, closing

HaoYang0000 closed this

HaoYang0000 deleted the debug_save_restore_point branch

October 9, 2024 13:41

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

roypaulin Awaiting requested review from roypaulin roypaulin is a code owner

cchen-vertica Awaiting requested review from cchen-vertica cchen-vertica is a code owner

fenic-fawkes Awaiting requested review from fenic-fawkes fenic-fawkes is a code owner

qindotguan Awaiting requested review from qindotguan qindotguan is a code owner

LiboYu2 Awaiting requested review from LiboYu2 LiboYu2 is a code owner

Labels

None yet