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

chore(sdf): always migrate si_audit database #5042

Merged
merged 1 commit into from
Nov 29, 2024

Conversation

fnichol
Copy link
Contributor

@fnichol fnichol commented Nov 29, 2024

This change removes the "attempt to migrate and log errors" behavior when migrating the si_audit database. With this change a migration is performed, identical to migrations for the si and si_layer_cache databases. When an error is encountered, it early-returns with the error.

This change removes the "attempt to migrate and log errors" behavior
when migrating the `si_audit` database. With this change a migration is
performed, identical to migrations for the `si` and `si_layer_cache`
databases. When an error is encountered, it early-returns with the
error.

Signed-off-by: Fletcher Nichol <[email protected]>
@github-actions github-actions bot added the A-sdf Area: Primary backend API service [Rust] label Nov 29, 2024
Copy link
Contributor

@johnrwatson johnrwatson left a comment

Choose a reason for hiding this comment

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

I really liked this way of shipping safely

@fnichol
Copy link
Contributor Author

fnichol commented Nov 29, 2024

We're good to revert this try-it-and-log-it behvior as we've confirmed that we're migrating successfully in production

@fnichol fnichol added this pull request to the merge queue Nov 29, 2024
Merged via the queue into main with commit 53aa84c Nov 29, 2024
8 checks passed
@fnichol fnichol deleted the fnichol/sdf-always-migrate-audit-db branch November 29, 2024 20:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-sdf Area: Primary backend API service [Rust]
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants