mirror of
https://github.com/SigNoz/signoz.git
synced 2026-03-27 22:50:26 +00:00
Some checks failed
build-staging / prepare (push) Has been cancelled
build-staging / js-build (push) Has been cancelled
build-staging / go-build (push) Has been cancelled
build-staging / staging (push) Has been cancelled
Release Drafter / update_release_draft (push) Has been cancelled
* feat(instrumentation): add OTel exception semantic convention log handler
Add a loghandler.Wrapper that enriches error log records with OpenTelemetry
exception semantic convention attributes (exception.type, exception.code,
exception.message, exception.stacktrace).
- Add errors.Attr() helper for standardized error logging under "exception" key
- Add exception log handler that replaces raw error attrs with structured group
- Wire exception handler into the instrumentation SDK logger chain
- Remove LogValue() from errors.base as the handler now owns structuring
* refactor: replace "error", err with errors.Attr(err) across codebase
Migrate all slog error logging from ad-hoc "error", err key-value pairs
to the standardized errors.Attr(err) helper, enabling the exception log
handler to enrich these logs with OTel semantic convention attributes.
* refactor: enforce attr-only slog style across codebase
Change sloglint from kv-only to attr-only, requiring all slog calls to
use typed attributes (slog.String, slog.Any, etc.) instead of key-value
pairs. Convert all existing kv-style slog calls in non-excluded paths.
* refactor: tighten slog.Any to specific types and standardize error attrs
- Replace slog.Any with slog.String for string values (action, key, where_clause)
- Replace slog.Any with slog.Uint64 for uint64 values (start, end, step, etc.)
- Replace slog.Any("err", err) with errors.Attr(err) in dispatcher and segment analytics
- Replace slog.Any("error", ctx.Err()) with errors.Attr in factory registry
* fix(instrumentation): use Unwrapb message for exception.message
Use the explicit error message (m) from Unwrapb instead of
foundErr.Error(), which resolves to the inner cause's message
for wrapped errors.
* feat(errors): capture stacktrace at error creation time
Store program counters ([]uintptr) in base errors at creation time
using runtime.Callers, inspired by thanos-io/thanos/pkg/errors. The
exception log handler reads the stacktrace from the error instead of
capturing at log time, showing where the error originated.
* fix(instrumentation): apply default log wrappers uniformly in NewLogger
Move correlation, filtering, and exception wrappers into NewLogger so
all call sites (including CLI loggers in cmd/) get them automatically.
* refactor(instrumentation): remove variadic wrappers from NewLogger
NewLogger no longer accepts arbitrary wrappers. The core wrappers
(correlation, filtering, exception) are hardcoded, preventing callers
from accidentally duplicating behavior.
* refactor: migrate remaining "error", <var> to errors.Attr across legacy paths
Replace all remaining "error", <variable> key-value pairs with
errors.Attr(<variable>) in pkg/query-service/ and ee/query-service/
paths that were missed in the initial migration due to non-standard
variable names (res.Err, filterErr, apiErrorObj.Err, etc).
* refactor(instrumentation): use flat exception.* keys instead of nested group
Use flat keys (exception.type, exception.code, exception.message,
exception.stacktrace) instead of a nested slog.Group in the exception
log handler.
121 lines
3.7 KiB
Go
121 lines
3.7 KiB
Go
package sqlmigrator
|
|
|
|
import (
|
|
"context"
|
|
"log/slog"
|
|
"time"
|
|
|
|
"github.com/SigNoz/signoz/pkg/errors"
|
|
|
|
"github.com/uptrace/bun/migrate"
|
|
|
|
"github.com/SigNoz/signoz/pkg/factory"
|
|
"github.com/SigNoz/signoz/pkg/sqlstore"
|
|
)
|
|
|
|
var (
|
|
migrationTableName string = "migration"
|
|
migrationLockTableName string = "migration_lock"
|
|
)
|
|
|
|
type migrator struct {
|
|
settings factory.ScopedProviderSettings
|
|
config Config
|
|
migrator *migrate.Migrator
|
|
dialect string
|
|
}
|
|
|
|
func New(ctx context.Context, providerSettings factory.ProviderSettings, sqlstore sqlstore.SQLStore, migrations *migrate.Migrations, config Config) SQLMigrator {
|
|
return &migrator{
|
|
migrator: migrate.NewMigrator(
|
|
sqlstore.BunDB(),
|
|
migrations,
|
|
migrate.WithTableName(migrationTableName),
|
|
migrate.WithLocksTableName(migrationLockTableName),
|
|
// This is to ensure that the migration is marked as applied only on success. If the migration fails, no entry is made in the migration table
|
|
// and the migration will be retried.
|
|
migrate.WithMarkAppliedOnSuccess(true),
|
|
),
|
|
settings: factory.NewScopedProviderSettings(providerSettings, "github.com/SigNoz/signoz/pkg/sqlmigrator"),
|
|
config: config,
|
|
dialect: sqlstore.BunDB().Dialect().Name().String(),
|
|
}
|
|
}
|
|
|
|
func (migrator *migrator) Migrate(ctx context.Context) error {
|
|
migrator.settings.Logger().InfoContext(ctx, "starting sqlstore migrations", slog.String("dialect", migrator.dialect))
|
|
if err := migrator.migrator.Init(ctx); err != nil {
|
|
return err
|
|
}
|
|
|
|
if err := migrator.Lock(ctx); err != nil {
|
|
return err
|
|
}
|
|
|
|
defer migrator.migrator.Unlock(ctx) //nolint:errcheck
|
|
|
|
group, err := migrator.migrator.Migrate(ctx)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
|
|
if group.IsZero() {
|
|
migrator.settings.Logger().InfoContext(ctx, "no new migrations to run (database is up to date)", slog.String("dialect", migrator.dialect))
|
|
return nil
|
|
}
|
|
|
|
migrator.settings.Logger().InfoContext(ctx, "migrated to", slog.String("group", group.String()), slog.String("dialect", migrator.dialect))
|
|
return nil
|
|
}
|
|
|
|
func (migrator *migrator) Rollback(ctx context.Context) error {
|
|
if err := migrator.Lock(ctx); err != nil {
|
|
return err
|
|
}
|
|
defer migrator.migrator.Unlock(ctx) //nolint:errcheck
|
|
|
|
group, err := migrator.migrator.Rollback(ctx)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
|
|
if group.IsZero() {
|
|
migrator.settings.Logger().InfoContext(ctx, "no groups to roll back", slog.String("dialect", migrator.dialect))
|
|
return nil
|
|
}
|
|
|
|
migrator.settings.Logger().InfoContext(ctx, "rolled back", slog.String("group", group.String()), slog.String("dialect", migrator.dialect))
|
|
return nil
|
|
}
|
|
|
|
func (migrator *migrator) Lock(ctx context.Context) error {
|
|
if err := migrator.migrator.Lock(ctx); err == nil {
|
|
migrator.settings.Logger().InfoContext(ctx, "acquired migration lock", slog.String("dialect", migrator.dialect))
|
|
return nil
|
|
}
|
|
|
|
timer := time.NewTimer(migrator.config.Lock.Timeout)
|
|
defer timer.Stop()
|
|
|
|
ticker := time.NewTicker(migrator.config.Lock.Interval)
|
|
defer ticker.Stop()
|
|
|
|
for {
|
|
select {
|
|
case <-timer.C:
|
|
err := errors.New(errors.TypeTimeout, errors.CodeTimeout, "timed out waiting for lock")
|
|
migrator.settings.Logger().ErrorContext(ctx, "cannot acquire lock", errors.Attr(err), slog.String("lock_timeout", migrator.config.Lock.Timeout.String()), slog.String("dialect", migrator.dialect))
|
|
return err
|
|
case <-ticker.C:
|
|
var err error
|
|
if err = migrator.migrator.Lock(ctx); err == nil {
|
|
migrator.settings.Logger().InfoContext(ctx, "acquired migration lock", slog.String("dialect", migrator.dialect))
|
|
return nil
|
|
}
|
|
migrator.settings.Logger().ErrorContext(ctx, "attempt to acquire lock failed", errors.Attr(err), slog.String("lock_interval", migrator.config.Lock.Interval.String()), slog.String("dialect", migrator.dialect))
|
|
case <-ctx.Done():
|
|
return ctx.Err()
|
|
}
|
|
}
|
|
}
|