# HG changeset patch # User Tom Gottfried # Date 1565795543 -7200 # Node ID 5d7ce7f926eb988192b2fa56bd7cfd2cff25d259 # Parent 948e312e87bc7d85b028b8c4a54ae4ce5800675d Log fatal errors as errors, not warnings, in imports diff -r 948e312e87bc -r 5d7ce7f926eb pkg/imports/bn.go --- a/pkg/imports/bn.go Wed Aug 14 16:55:46 2019 +0200 +++ b/pkg/imports/bn.go Wed Aug 14 17:12:23 2019 +0200 @@ -397,7 +397,7 @@ if t, ok := fromTo[fromKey]; ok { tfrom.Set(t) } else { - feedback.Warn("Missing start date") + feedback.Error("Missing start date") return nil } @@ -568,7 +568,7 @@ } return nil }); err != nil { - feedback.Warn(pgxutils.ReadableError{Err: err}.Error()) + feedback.Error(pgxutils.ReadableError{Err: err}.Error()) return nil } diff -r 948e312e87bc -r 5d7ce7f926eb pkg/imports/fd.go --- a/pkg/imports/fd.go Wed Aug 14 16:55:46 2019 +0200 +++ b/pkg/imports/fd.go Wed Aug 14 17:12:23 2019 +0200 @@ -311,7 +311,7 @@ // ignore -> filtered by responsibility_areas continue features case err != nil: - feedback.Warn(pgxutils.ReadableError{Err: err}.Error()) + feedback.Error(pgxutils.ReadableError{Err: err}.Error()) continue features } // Store for potential later removal. diff -r 948e312e87bc -r 5d7ce7f926eb pkg/imports/gm.go --- a/pkg/imports/gm.go Wed Aug 14 16:55:46 2019 +0200 +++ b/pkg/imports/gm.go Wed Aug 14 17:12:23 2019 +0200 @@ -444,7 +444,7 @@ case err == sql.ErrNoRows: // thats expected, nothing to do case err != nil: - feedback.Warn(pgxutils.ReadableError{Err: err}.Error()) + feedback.Error(pgxutils.ReadableError{Err: err}.Error()) default: newP++ } @@ -475,7 +475,7 @@ case err == sql.ErrNoRows: // thats expected, nothing to do case err != nil: - feedback.Warn(pgxutils.ReadableError{Err: err}.Error()) + feedback.Error(pgxutils.ReadableError{Err: err}.Error()) default: newM++ } diff -r 948e312e87bc -r 5d7ce7f926eb pkg/imports/wa.go --- a/pkg/imports/wa.go Wed Aug 14 16:55:46 2019 +0200 +++ b/pkg/imports/wa.go Wed Aug 14 17:12:23 2019 +0200 @@ -247,7 +247,7 @@ outside++ // ignore -> filtered by responsibility_areas case err != nil: - feedback.Warn(pgxutils.ReadableError{Err: err}.Error()) + feedback.Error(pgxutils.ReadableError{Err: err}.Error()) default: features++ } diff -r 948e312e87bc -r 5d7ce7f926eb pkg/imports/wg.go --- a/pkg/imports/wg.go Wed Aug 14 16:55:46 2019 +0200 +++ b/pkg/imports/wg.go Wed Aug 14 17:12:23 2019 +0200 @@ -329,7 +329,7 @@ ).Scan(&isNew) switch { case err != nil: - feedback.Warn(pgxutils.ReadableError{Err: err}.Error()) + feedback.Error(pgxutils.ReadableError{Err: err}.Error()) if err2 := tx.Rollback(); err2 != nil { return nil, err2 } @@ -354,7 +354,7 @@ source, time.Time(*dr.Lastupdate), ); err != nil { - feedback.Warn(pgxutils.ReadableError{Err: err}.Error()) + feedback.Error(pgxutils.ReadableError{Err: err}.Error()) if err2 := tx.Rollback(); err2 != nil { return nil, err2 } @@ -391,7 +391,7 @@ unchanged++ continue case err2 != nil: - feedback.Warn(pgxutils.ReadableError{Err: err2}.Error()) + feedback.Error(pgxutils.ReadableError{Err: err2}.Error()) if err3 := tx.Rollback(); err3 != nil { return nil, err3 } @@ -438,7 +438,7 @@ code.String(), &validity, ); err != nil { - feedback.Warn(pgxutils.ReadableError{Err: err}.Error()) + feedback.Error(pgxutils.ReadableError{Err: err}.Error()) if err2 := tx.Rollback(); err2 != nil { return nil, err2 } @@ -486,7 +486,7 @@ string(**wl.level), int64(**wl.value), ); err != nil { - feedback.Warn(pgxutils.ReadableError{Err: err}.Error()) + feedback.Error(pgxutils.ReadableError{Err: err}.Error()) tx.Rollback() continue } diff -r 948e312e87bc -r 5d7ce7f926eb pkg/imports/wx.go --- a/pkg/imports/wx.go Wed Aug 14 16:55:46 2019 +0200 +++ b/pkg/imports/wx.go Wed Aug 14 17:12:23 2019 +0200 @@ -321,7 +321,7 @@ // ignore -> filtered by responsibility_areas return nil case err != nil: - feedback.Warn(pgxutils.ReadableError{Err: err}.Error()) + feedback.Error(pgxutils.ReadableError{Err: err}.Error()) default: *features++ }