Skip to content

Commit 3729eb9

Browse files
Add tracer provider
Signed-off-by: Alexandros Filios <alexandros.filios@ibm.com>
1 parent fc404ae commit 3729eb9

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

platform/fabric/core/generic/delivery/service.go

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -16,11 +16,9 @@ import (
1616
"github.com/hyperledger-labs/fabric-smart-client/platform/fabric/driver"
1717
"github.com/hyperledger-labs/fabric-smart-client/platform/view/services/hash"
1818
"github.com/hyperledger-labs/fabric-smart-client/platform/view/services/metrics"
19-
"github.com/hyperledger-labs/fabric-smart-client/platform/view/services/metrics/disabled"
2019
"github.com/hyperledger/fabric-protos-go/common"
2120
pb "github.com/hyperledger/fabric-protos-go/peer"
2221
"go.opentelemetry.io/otel/trace"
23-
"go.opentelemetry.io/otel/trace/noop"
2422
)
2523

2624
type ValidationFlags []uint8
@@ -37,8 +35,9 @@ type Service struct {
3735
transactionManager driver.TransactionManager
3836
waitForEventTimeout time.Duration
3937
acceptedHeaderTypes collections.Set[common.HeaderType]
40-
41-
deliveryService *Delivery
38+
tracerProvider trace.TracerProvider
39+
metricsProvider metrics.Provider
40+
deliveryService *Delivery
4241
}
4342

4443
func NewService(
@@ -88,6 +87,8 @@ func NewService(
8887
waitForEventTimeout: waitForEventTimeout,
8988
deliveryService: deliveryService,
9089
transactionManager: transactionManager,
90+
tracerProvider: tracerProvider,
91+
metricsProvider: metricsProvider,
9192
acceptedHeaderTypes: collections.NewSet(acceptedHeaderTypes...),
9293
}, nil
9394
}
@@ -113,8 +114,8 @@ func (c *Service) scanBlock(ctx context.Context, vault Vault, callback driver.Bl
113114
callback,
114115
vault,
115116
c.channelConfig.CommitterWaitForEventTimeout(),
116-
&noop.TracerProvider{},
117-
&disabled.Provider{},
117+
c.tracerProvider,
118+
c.metricsProvider,
118119
)
119120
if err != nil {
120121
return err

0 commit comments

Comments
 (0)