-
Notifications
You must be signed in to change notification settings - Fork 12
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
atlasmigration: store migrations.tar.gz
in k8s' secrets
#175
Conversation
migration.tar
in k8s' secretsmigrations.tar
in k8s' secrets
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't we delete the previous version of the secret?
Also should secrets be deleted when the migration resource is deleted ?
Absolute yes.
I'm not sure when we need to delete the old version, because if we keep only single version, we only need to delete it when the migration resource was deleted. If we store more than one version, we may delete the old one when a new deployment was success. But, what if the latest deployment is a |
27f0df7
to
a594be7
Compare
migrations.tar
in k8s' secretsmigrations.tar.gz
in k8s' secrets
f30ddf4
to
0a5b2f1
Compare
Chain of down migrations is an interesting use case. E.g: Re: failed down migration, or aborted down migration, you never replace the stored state, since the state is still the version before you rolled back. Sounds OK ? |
This need to be check/update to expected in the PR #167; for now, we just save the dir-state after successful reconcile, whatever flow it did. |
Co-authored-by: Dat. Ba Dao <datdao@ariga.io>
f65b12e
to
aa19cea
Compare
No description provided.