chore: fix parse typo for network telemetry (#13971)

deansheather · web-flow · commit d2b035312e0c · 2024-07-22T17:14:37.000Z
diff --git a/coderd/audit/request.go b/coderd/audit/request.go
@@ -272,16 +272,16 @@ func requireOrgID[T Auditable](ctx context.Context, id uuid.UUID, log slog.Logge
 // entry.
 func InitRequestWithCancel[T Auditable](w http.ResponseWriter, p *RequestParams) (*Request[T], func(commit bool)) {
 	req, commitF := InitRequest[T](w, p)
-	cancelled := false
+	canceled := false
 	return req, func(commit bool) {
 		// Once 'commit=false' is called, block
 		// any future commit attempts.
 		if !commit {
-			cancelled = true
+			canceled = true
 			return
 		}
-		// If it was ever cancelled, block any commits
-		if !cancelled {
+		// If it was ever canceled, block any commits
+		if !canceled {
 			commitF()
 		}
 	}
diff --git a/coderd/telemetry/telemetry.go b/coderd/telemetry/telemetry.go
@@ -1269,7 +1269,7 @@ func NetworkEventFromProto(proto *tailnetproto.TelemetryEvent) (NetworkEvent, er
 	if proto == nil {
 		return NetworkEvent{}, xerrors.New("nil event")
 	}
-	id, err := uuid.ParseBytes(proto.Id)
+	id, err := uuid.FromBytes(proto.Id)
 	if err != nil {
 		return NetworkEvent{}, xerrors.Errorf("parse id %q: %w", proto.Id, err)
 	}
diff --git a/coderd/workspaceagentsrpc.go b/coderd/workspaceagentsrpc.go
@@ -168,10 +168,17 @@ func (api *API) workspaceAgentRPC(rw http.ResponseWriter, r *http.Request) {
 }
 
 func (api *API) handleNetworkTelemetry(batch []*tailnetproto.TelemetryEvent) {
-	telemetryEvents := make([]telemetry.NetworkEvent, 0, len(batch))
+	var (
+		telemetryEvents = make([]telemetry.NetworkEvent, 0, len(batch))
+		didLogErr       = false
+	)
 	for _, pEvent := range batch {
 		tEvent, err := telemetry.NetworkEventFromProto(pEvent)
 		if err != nil {
+			if !didLogErr {
+				api.Logger.Warn(api.ctx, "error converting network telemetry event", slog.Error(err))
+				didLogErr = true
+			}
 			// Events that fail to be converted get discarded for now.
 			continue
 		}
diff --git a/tailnet/conn.go b/tailnet/conn.go
@@ -775,11 +775,9 @@ func (c *Conn) sendPingTelemetry(pr *ipnstate.PingResult) {
 
 // The returned telemetry event will not have it's status set.
 func (c *Conn) newTelemetryEvent() *proto.TelemetryEvent {
-	// Infallible
-	id, _ := c.id.MarshalBinary()
 	event := c.telemetryStore.newEvent()
 	event.ClientType = c.clientType
-	event.Id = id
+	event.Id = c.id[:]
 	event.ConnectionAge = durationpb.New(time.Since(c.createdAt))
 	return event
 }
diff --git a/tailnet/service.go b/tailnet/service.go
@@ -261,6 +261,9 @@ func NewNetworkTelemetryBatcher(clk quartz.Clock, frequency time.Duration, maxSi
 		closed:    make(chan struct{}),
 		done:      make(chan struct{}),
 	}
+	if b.batchFn == nil {
+		b.batchFn = func(batch []*proto.TelemetryEvent) {}
+	}
 	b.start()
 	return b
 }

Original file line number	Diff line number	Diff line change
`@@ -1269,7 +1269,7 @@ func NetworkEventFromProto(proto *tailnetproto.TelemetryEvent) (NetworkEvent, er`
`1269`	`1269`	`if proto == nil {`
`1270`	`1270`	`return NetworkEvent{}, xerrors.New("nil event")`
`1271`	`1271`	`}`
`1272`		`- id, err := uuid.ParseBytes(proto.Id)`
	`1272`	`+ id, err := uuid.FromBytes(proto.Id)`
`1273`	`1273`	`if err != nil {`
`1274`	`1274`	`return NetworkEvent{}, xerrors.Errorf("parse id %q: %w", proto.Id, err)`
`1275`	`1275`	`}`
Original file line number	Diff line number	Diff line change
`@@ -261,6 +261,9 @@ func NewNetworkTelemetryBatcher(clk quartz.Clock, frequency time.Duration, maxSi`
`261`	`261`	`closed: make(chan struct{}),`
`262`	`262`	`done: make(chan struct{}),`
`263`	`263`	`}`
	`264`	`+ if b.batchFn == nil {`
	`265`	`+ b.batchFn = func(batch []*proto.TelemetryEvent) {}`
	`266`	`+ }`
`264`	`267`	`b.start()`
`265`	`268`	`return b`
`266`	`269`	`}`