Skip to content
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.

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

Add restore parameters to v1 API #640

Merged
merged 6 commits into from
Dec 20, 2023
Merged

Add restore parameters to v1 API #640

merged 6 commits into from
Dec 20, 2023

Conversation

jizhuoyu
Copy link
Collaborator

@jizhuoyu jizhuoyu commented Dec 18, 2023

  • add RestorePointPolicy type in Spec
    • Archive must be specified if a restore is intended
    • one of ID and Index must be specified (Index assumed to be 1-based)
  • webhook rules to validate RestorePointPolicy
  • revivedb_reconciler checks whether restore is supported given the server version and deployment method
    • must use vclusterops and has server version greater than v24.2.0

@jizhuoyu jizhuoyu self-assigned this Dec 18, 2023
@jizhuoyu jizhuoyu changed the title Add API for restore db Add restore parameters to v1 API Dec 18, 2023
Copy link
Collaborator

@roypaulin roypaulin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see this is still a draft but given that you have requested my review I am adding a few comments.

api/v1/verticadb_webhook.go Outdated Show resolved Hide resolved
api/v1/verticadb_types.go Show resolved Hide resolved
api/v1/verticadb_types.go Outdated Show resolved Hide resolved
api/v1/verticadb_types.go Outdated Show resolved Hide resolved
api/v1/helpers.go Show resolved Hide resolved
pkg/controllers/vdb/revivedb_reconciler.go Outdated Show resolved Hide resolved
pkg/version/version.go Outdated Show resolved Hide resolved
api/v1/version.go Outdated Show resolved Hide resolved
api/v1/verticadb_types.go Outdated Show resolved Hide resolved
pkg/controllers/vdb/revivedb_reconciler.go Outdated Show resolved Hide resolved
pkg/controllers/vdb/revivedb_reconciler.go Outdated Show resolved Hide resolved
api/v1/verticadb_types.go Outdated Show resolved Hide resolved
pkg/controllers/vdb/revivedb_reconciler.go Outdated Show resolved Hide resolved
@jizhuoyu jizhuoyu marked this pull request as ready for review December 19, 2023 05:14
api/v1/verticadb_types.go Show resolved Hide resolved
api/v1/helpers.go Outdated Show resolved Hide resolved
api/v1/helpers.go Outdated Show resolved Hide resolved
pkg/version/version.go Outdated Show resolved Hide resolved
Copy link
Collaborator

@roypaulin roypaulin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for applying all the comments.

@jizhuoyu jizhuoyu merged commit d10d15b into vnext Dec 20, 2023
27 checks passed
@jizhuoyu jizhuoyu deleted the zji/restore-params-api branch December 20, 2023 15:38
chinhtranvan pushed a commit that referenced this pull request Jan 27, 2024
- add `RestorePointPolicy` type in `Spec`
  - `Archive` must be specified if a restore is intended
- one of `ID` and `Index` must be specified (`Index` assumed to be
1-based)
- webhook rules to validate `RestorePointPolicy`
- revivedb_reconciler checks whether restore is supported given the
server version and deployment method
  - must use vclusterops and has server version greater than v24.2.0

---------

Co-authored-by: Roy Paulin <[email protected]>
chinhtranvan pushed a commit that referenced this pull request Jan 27, 2024
- add `RestorePointPolicy` type in `Spec`
  - `Archive` must be specified if a restore is intended
- one of `ID` and `Index` must be specified (`Index` assumed to be
1-based)
- webhook rules to validate `RestorePointPolicy`
- revivedb_reconciler checks whether restore is supported given the
server version and deployment method
  - must use vclusterops and has server version greater than v24.2.0

---------

Co-authored-by: Roy Paulin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants