convert logrus severity levels properly

Author: vkuptcov

bridges/otellogrus/hook.go

@@ -165,8 +165,24 @@ func (h *Hook) convertEntry(e *logrus.Entry) log.Record {
 	record.SetTimestamp(e.Time)
 	record.SetBody(log.StringValue(e.Message))
 
-	const sevOffset = logrus.Level(log.SeverityDebug) - logrus.DebugLevel
-	record.SetSeverity(log.Severity(e.Level + sevOffset))
+	var severity log.Severity
+	switch e.Level {
+	case logrus.PanicLevel:
+		severity = log.SeverityFatal4
+	case logrus.FatalLevel:
+		severity = log.SeverityFatal
+	case logrus.ErrorLevel:
+		severity = log.SeverityError
+	case logrus.WarnLevel:
+		severity = log.SeverityWarn
+	case logrus.InfoLevel:
+		severity = log.SeverityInfo
+	case logrus.DebugLevel:
+		severity = log.SeverityDebug
+	case logrus.TraceLevel:
+		severity = log.SeverityTrace
+	}
+	record.SetSeverity(severity)
 	record.AddAttributes(convertFields(e.Data)...)
 
 	return record

bridges/otellogrus/hook_test.go

@@ -162,7 +162,7 @@ func TestHookFire(t *testing.T) {
 
 			wantRecords: map[string][]log.Record{
 				name: {
-					buildRecord(log.StringValue(""), time.Time{}, 0, nil),
+					buildRecord(log.StringValue(""), time.Time{}, log.SeverityFatal4, nil),
 				},
 			},
 		},
@@ -173,7 +173,7 @@ func TestHookFire(t *testing.T) {
 			},
 			wantRecords: map[string][]log.Record{
 				name: {
-					buildRecord(log.StringValue(""), now, 0, nil),
+					buildRecord(log.StringValue(""), now, log.SeverityFatal4, nil),
 				},
 			},
 		},
@@ -184,7 +184,7 @@ func TestHookFire(t *testing.T) {
 			},
 			wantRecords: map[string][]log.Record{
 				name: {
-					buildRecord(log.StringValue(""), time.Time{}, log.SeverityTrace1, nil),
+					buildRecord(log.StringValue(""), time.Time{}, log.SeverityFatal, nil),
 				},
 			},
 		},
@@ -197,7 +197,7 @@ func TestHookFire(t *testing.T) {
 			},
 			wantRecords: map[string][]log.Record{
 				name: {
-					buildRecord(log.StringValue(""), time.Time{}, 0, []log.KeyValue{
+					buildRecord(log.StringValue(""), time.Time{}, log.SeverityFatal4, []log.KeyValue{
 						log.String("hello", "world"),
 					}),
 				},