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

migration scripts are not included in package #513

Open
nuclearcat opened this issue Apr 5, 2024 · 1 comment
Open

migration scripts are not included in package #513

nuclearcat opened this issue Apr 5, 2024 · 1 comment
Labels
techdebt Something that works for now, but should be done better

Comments

@nuclearcat
Copy link
Member

I am not sure it is good idea to include them with package, but just as we are doing migration sometimes, we need to document somehow procedure how we will do that, plus if possible to automate migrations to certain extent, including backup, maybe fetching migration scripts, running them, verifying results, rolling back on failure.

@nuclearcat nuclearcat added the techdebt Something that works for now, but should be done better label Apr 5, 2024
@JenySadadia
Copy link
Collaborator

I'll take a look if I can include the migration scripts to the package with dev mode.
About the documentation, an initial version is here: https://github.com/kernelci/kernelci-api/blob/main/doc/api-details.md#migrations
It can be enhanced though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
techdebt Something that works for now, but should be done better
Projects
Status: No status
Development

No branches or pull requests

2 participants