alexvanin/distribution
1
0
Fork 0
forked from TrueCloudLab/distribution
Code Pull requests Activity

Merge pull request #4041 from milosgajdos/duplicate-code

Browse source 
Remove duplicate code that instruments Redis otel
This commit is contained in:
Milos Gajdos 2023-09-04 19:30:06 +01:00 committed by GitHub
commit 6787846b9d
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 0 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
registry/handlers/app.go
View file

@ -498,11 +498,6 @@ func (app *App) configureRedis(cfg *configuration.Configuration) {
app.redis = app.createPool(cfg.Redis) app.redis = app.createPool(cfg.Redis)
// Enable tracing instrumentation.
if err := redisotel.InstrumentTracing(app.redis); err != nil {
dcontext.GetLogger(app).Errorf("failed to instrument tracing on redis: %v", err)
}
// Enable metrics instrumentation. // Enable metrics instrumentation.
if err := redisotel.InstrumentMetrics(app.redis); err != nil { if err := redisotel.InstrumentMetrics(app.redis); err != nil {
dcontext.GetLogger(app).Errorf("failed to instrument metrics on redis: %v", err) dcontext.GetLogger(app).Errorf("failed to instrument metrics on redis: %v", err)