remove unnecessary data migration and dead DevelopmentMode config
All checks were successful
check / check (push) Successful in 1m52s
All checks were successful
check / check (push) Successful in 1m52s
- Remove retry→http data migration from migrate() — no databases exist pre-1.0 - Remove unused DevelopmentMode field and DEVELOPMENT_MODE env var from config - Remove DevelopmentMode from config log output (dead code cleanup)
This commit is contained in:
parent
25e27cc57f
commit
3588facfff
@ -34,7 +34,6 @@ type Config struct {
|
||||
DataDir string
|
||||
Debug bool
|
||||
MaintenanceMode bool
|
||||
DevelopmentMode bool
|
||||
Environment string
|
||||
MetricsPassword string
|
||||
MetricsUsername string
|
||||
@ -101,7 +100,6 @@ func New(lc fx.Lifecycle, params ConfigParams) (*Config, error) {
|
||||
DataDir: envString("DATA_DIR"),
|
||||
Debug: envBool("DEBUG", false),
|
||||
MaintenanceMode: envBool("MAINTENANCE_MODE", false),
|
||||
DevelopmentMode: envBool("DEVELOPMENT_MODE", false),
|
||||
Environment: environment,
|
||||
MetricsUsername: envString("METRICS_USERNAME"),
|
||||
MetricsPassword: envString("METRICS_PASSWORD"),
|
||||
@ -131,7 +129,6 @@ func New(lc fx.Lifecycle, params ConfigParams) (*Config, error) {
|
||||
"port", s.Port,
|
||||
"debug", s.Debug,
|
||||
"maintenanceMode", s.MaintenanceMode,
|
||||
"developmentMode", s.DevelopmentMode,
|
||||
"dataDir", s.DataDir,
|
||||
"hasSentryDSN", s.SentryDSN != "",
|
||||
"hasMetricsAuth", s.MetricsUsername != "" && s.MetricsPassword != "",
|
||||
|
||||
@ -92,16 +92,6 @@ func (d *Database) migrate() error {
|
||||
}
|
||||
d.log.Info("database migrations completed")
|
||||
|
||||
// Data migration: merge "retry" target type into "http".
|
||||
// Previously there were two separate HTTP-based target types: "http"
|
||||
// (fire-and-forget) and "retry" (with retries). Now "http" handles
|
||||
// both: max_retries=0 means fire-and-forget, max_retries>0 enables
|
||||
// retries with exponential backoff and circuit breaker.
|
||||
if err := d.db.Exec("UPDATE targets SET type = 'http' WHERE type = 'retry'").Error; err != nil {
|
||||
d.log.Error("failed to migrate retry targets to http", "error", err)
|
||||
return err
|
||||
}
|
||||
|
||||
// Check if admin user exists
|
||||
var userCount int64
|
||||
if err := d.db.Model(&User{}).Count(&userCount).Error; err != nil {
|
||||
|
||||
Loading…
Reference in New Issue
Block a user