Skip to content

Commit

Permalink
Merge pull request #5042 from systeminit/fnichol/sdf-always-migrate-a…
Browse files Browse the repository at this point in the history
…udit-db

chore(sdf): always migrate `si_audit` database
  • Loading branch information
fnichol authored Nov 29, 2024
2 parents 654d7c0 + c0cbb20 commit 53aa84c
Showing 1 changed file with 3 additions and 8 deletions.
11 changes: 3 additions & 8 deletions lib/sdf-server/src/migrations.rs
Original file line number Diff line number Diff line change
Expand Up @@ -100,14 +100,9 @@ impl Migrator {
pub async fn run_migrations(self) -> MigratorResult<()> {
let span = current_span_for_instrument_at!("info");

// TODO(nick,john): once we have seeded the database successfully, we can replace this with
// error propagation.
if let Err(err) = self.migrate_audit_database().await {
warn!(
?err,
"skipping audit database migration due to error, which is currently expected"
);
}
self.migrate_audit_database()
.await
.map_err(|err| span.record_err(err))?;

self.migrate_layer_db_database()
.await
Expand Down

0 comments on commit 53aa84c

Please sign in to comment.