Skip to content

Commit

Permalink
Fixes
Browse files Browse the repository at this point in the history
Signed-off-by: Yuri Shkuro <[email protected]>
  • Loading branch information
yurishkuro authored Jul 20, 2024
1 parent f105bc9 commit 0cdb5ca
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
1 change: 0 additions & 1 deletion cmd/jaeger/internal/extension/jaegerquery/server.go
Original file line number Diff line number Diff line change
Expand Up @@ -141,7 +141,6 @@ func (s *server) createMetricStorage(host component.Host) (metricsstore.Reader,
return nil, fmt.Errorf("cannot find metrics storage factory: %w", err)

Check warning on line 141 in cmd/jaeger/internal/extension/jaegerquery/server.go

View check run for this annotation

Codecov / codecov/patch

cmd/jaeger/internal/extension/jaegerquery/server.go#L139-L141

Added lines #L139 - L141 were not covered by tests
}

// metricsReader, ok := opts.InitMetricStorage(mf, s.telset.Logger)
metricsReader, err := mf.CreateMetricsReader()
if errors.Is(err, storage.ErrMetricStorageNotConfigured) || errors.Is(err, storage.ErrMetricStorageNotSupported) {
s.telset.Logger.Info("Metric storage not created", zap.Error(err))
Expand Down
2 changes: 1 addition & 1 deletion scripts/spm-integration-test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ if [ "$BINARY" == "jaeger" ]; then
compose_file=docker-compose/monitor/docker-compose-v2.yml
fi

timeout=300
timeout=600
end_time=$((SECONDS + timeout))
success="false"

Expand Down

0 comments on commit 0cdb5ca

Please sign in to comment.