tracing: make trace and span ids redactable

The trace and span ids can be used for out of band tracing. We don't
want those to be redacted.

Epic: none

Release note: None

pkg/testutils/lint/passes/redactcheck/redactcheck.go

@@ -225,6 +225,10 @@ func runAnalyzer(pass *analysis.Pass) (interface{}, error) {
 						"WorkKind":  {},
 						"QueueKind": {},
 					},
+					"github.com/cockroachdb/cockroach/pkg/util/tracing/tracingpb": {
+						"TraceID": {},
+						"SpanID":  {},
+					},
 					"github.com/cockroachdb/cockroach/pkg/util/hlc": {
 						"ClockTimestamp":  {},
 						"LegacyTimestamp": {},

pkg/util/tracing/tracingpb/recorded_span.go

@@ -22,9 +22,13 @@ const (
 // TraceID is a probabilistically-unique id, shared by all spans in a trace.
 type TraceID uint64
 
+func (TraceID) SafeValue() {}
+
 // SpanID is a probabilistically-unique span id.
 type SpanID uint64
 
+func (SpanID) SafeValue() {}
+
 // Recording represents a group of RecordedSpans rooted at a fixed root span, as
 // returned by GetRecording. Spans are sorted by StartTime.
 type Recording []RecordedSpan