Merge #4178

bors[bot] · Guitarheroua · web-flow · commit 10204df9bbf1 · 2023-04-10T18:51:39.000Z
4178: [Observer] Fixed issue with incorrect observer logs r=SaveTheRbtz a=Guitarheroua #3040 ### Context Previously observer logs and metrics were marked as "access". Changed them to display correctly as "observer". Co-authored-by: Andriy Slisarchuk <andriyslisarchuk@gmail.com> Co-authored-by: Andrii Slisarchuk <Guitarheroua@users.noreply.github.com>
diff --git a/cmd/observer/node_builder/observer_builder.go b/cmd/observer/node_builder/observer_builder.go
@@ -566,7 +566,7 @@ func NewFlowObserverServiceBuilder(opts ...Option) *ObserverServiceBuilder {
 	}
 	anb := &ObserverServiceBuilder{
 		ObserverServiceConfig:   config,
-		FlowNodeBuilder:         cmd.FlowNode(flow.RoleAccess.String()),
+		FlowNodeBuilder:         cmd.FlowNode("observer"),
 		FinalizationDistributor: pubsub.NewFinalizationDistributor(),
 	}
 	anb.FinalizationDistributor.AddConsumer(notifications.NewSlashingViolationsConsumer(anb.Logger))

Original file line number	Diff line number	Diff line change
`@@ -566,7 +566,7 @@ func NewFlowObserverServiceBuilder(opts ...Option) *ObserverServiceBuilder {`
`566`	`566`	`}`
`567`	`567`	`anb := &ObserverServiceBuilder{`
`568`	`568`	`ObserverServiceConfig: config,`
`569`		`- FlowNodeBuilder: cmd.FlowNode(flow.RoleAccess.String()),`
	`569`	`+ FlowNodeBuilder: cmd.FlowNode("observer"),`
`570`	`570`	`FinalizationDistributor: pubsub.NewFinalizationDistributor(),`
`571`	`571`	`}`
`572`	`572`	`anb.FinalizationDistributor.AddConsumer(notifications.NewSlashingViolationsConsumer(anb.Logger))`