melosh/mal
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor: migrate database logs to observability

Browse Source
This commit is contained in:
mkelvers
2026-05-26 15:56:38 +02:00
parent e08a0e1f71
commit 96307d2979
2 changed files with 11 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
internal/database/database.go
View File

@@ -4,9 +4,9 @@ import (
"database/sql" "database/sql"
"embed" "embed"
"fmt" "fmt"
"log"
"mal/internal/config" "mal/internal/config"
"mal/internal/db" "mal/internal/db"
"mal/internal/observability"
"github.com/pressly/goose/v3" "github.com/pressly/goose/v3"
"go.uber.org/fx" "go.uber.org/fx"
@@ -42,7 +42,7 @@ func RunMigrations(sqlDB *sql.DB) error {
return fmt.Errorf("failed to set goose dialect: %w", err) return fmt.Errorf("failed to set goose dialect: %w", err)
} }
log.Println("Running database migrations...") observability.Info("db_migrations_start", "database", "", nil)
if err := goose.Up(sqlDB, "migrations"); err != nil { if err := goose.Up(sqlDB, "migrations"); err != nil {
return fmt.Errorf("failed to run migrations: %w", err) return fmt.Errorf("failed to run migrations: %w", err)
} }

11
internal/database/fixes.go
View File

@@ -4,10 +4,10 @@ import (
"context" "context"
"database/sql" "database/sql"
"fmt" "fmt"
"log"
"time" "time"
dbfixes "mal/internal/database/fixes" dbfixes "mal/internal/database/fixes"
"mal/internal/observability"
) )
func RunDataFixes(sqlDB *sql.DB) error { func RunDataFixes(sqlDB *sql.DB) error {
@@ -34,7 +34,14 @@ func RunDataFixes(sqlDB *sql.DB) error {
continue continue
} }
log.Printf("Running data fix id=%s", fix.ID) observability.Info(
"db_data_fix_start",
"database",
"",
map[string]any{
"id": fix.ID,
},
)
if err := fix.Apply(ctx, sqlDB); err != nil { if err := fix.Apply(ctx, sqlDB); err != nil {
return fmt.Errorf("data fix %s failed: %w", fix.ID, err) return fmt.Errorf("data fix %s failed: %w", fix.ID, err)
} }