mirror of https://github.com/docker/cli.git
otel: capture exit code as int64
Signed-off-by: Laura Brehm <laurabrehm@hey.com>
This commit is contained in:
parent
10b9810989
commit
cefcba9871
|
@ -3,7 +3,6 @@ package command
|
||||||
import (
|
import (
|
||||||
"context"
|
"context"
|
||||||
"fmt"
|
"fmt"
|
||||||
"strconv"
|
|
||||||
"strings"
|
"strings"
|
||||||
"time"
|
"time"
|
||||||
|
|
||||||
|
@ -110,7 +109,7 @@ func attributesFromError(err error) []attribute.KeyValue {
|
||||||
}
|
}
|
||||||
attrs = append(attrs, attribute.String("command.error.type", otelErrorType(err)))
|
attrs = append(attrs, attribute.String("command.error.type", otelErrorType(err)))
|
||||||
}
|
}
|
||||||
attrs = append(attrs, attribute.String("command.status.code", strconv.Itoa(exitCode)))
|
attrs = append(attrs, attribute.Int("command.status.code", exitCode))
|
||||||
|
|
||||||
return attrs
|
return attrs
|
||||||
}
|
}
|
||||||
|
|
|
@ -159,7 +159,7 @@ func TestAttributesFromError(t *testing.T) {
|
||||||
testName: "no error",
|
testName: "no error",
|
||||||
err: nil,
|
err: nil,
|
||||||
expected: []attribute.KeyValue{
|
expected: []attribute.KeyValue{
|
||||||
attribute.String("command.status.code", "0"),
|
attribute.Int("command.status.code", 0),
|
||||||
},
|
},
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
|
@ -167,7 +167,7 @@ func TestAttributesFromError(t *testing.T) {
|
||||||
err: statusError{StatusCode: 127},
|
err: statusError{StatusCode: 127},
|
||||||
expected: []attribute.KeyValue{
|
expected: []attribute.KeyValue{
|
||||||
attribute.String("command.error.type", "generic"),
|
attribute.String("command.error.type", "generic"),
|
||||||
attribute.String("command.status.code", "127"),
|
attribute.Int("command.status.code", 127),
|
||||||
},
|
},
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
|
@ -175,7 +175,7 @@ func TestAttributesFromError(t *testing.T) {
|
||||||
err: context.Canceled,
|
err: context.Canceled,
|
||||||
expected: []attribute.KeyValue{
|
expected: []attribute.KeyValue{
|
||||||
attribute.String("command.error.type", "canceled"),
|
attribute.String("command.error.type", "canceled"),
|
||||||
attribute.String("command.status.code", "1"),
|
attribute.Int("command.status.code", 1),
|
||||||
},
|
},
|
||||||
},
|
},
|
||||||
} {
|
} {
|
||||||
|
|
Loading…
Reference in New Issue