Add changes

Signed-off-by: FlamingSaint <[email protected]>

cmd/jaeger/internal/extension/jaegerquery/server.go

@@ -10,7 +10,6 @@ import (
 
 	"go.opentelemetry.io/collector/component"
 	"go.opentelemetry.io/collector/extension"
-	"go.uber.org/zap"
 
 	"github.com/jaegertracing/jaeger/cmd/jaeger/internal/extension/jaegerstorage"
 	queryApp "github.com/jaegertracing/jaeger/cmd/query/app"
@@ -19,7 +18,6 @@ import (
 	"github.com/jaegertracing/jaeger/pkg/telemetery"
 	"github.com/jaegertracing/jaeger/pkg/tenancy"
 	"github.com/jaegertracing/jaeger/plugin/metrics/disabled"
-	"github.com/jaegertracing/jaeger/storage"
 	"github.com/jaegertracing/jaeger/storage/metricsstore"
 )
 
@@ -142,10 +140,6 @@ func (s *server) createMetricStorage(host component.Host) (metricsstore.Reader,
 	}
 
 	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))
-		return disabled.NewMetricsReader()
-	}
 	return metricsReader, err
 }
 

scripts/spm-integration-test.sh

@@ -89,13 +89,19 @@ validate_service_metrics() {
     echo "Metric datapoints found for service '$service': " "${metric_points[@]}"
     # Check that atleast some values are non-zero after the threshold
     local non_zero_count=0
-    local expected_non_zero_count=3
+    local expected_non_zero_count=5
     local zero_count=0
-    local expected_max_zero_count=3
+    local expected_max_zero_count=4
+    local flag=false # Becomes true after first non-zero value 
     for value in "${metric_points[@]}"; do
       if [[ $(echo "$value > 0.0" | bc) == "1" ]]; then
+        flag=true
         non_zero_count=$((non_zero_count + 1))
       else
+        if [ $flag == true ]; then
+          echo "❌ ERROR: Zero values appearing after a non-zero value not expected"
+          return 1
+        fi
         zero_count=$((zero_count + 1))
       fi
 
@@ -105,7 +111,7 @@ validate_service_metrics() {
       fi
     done
     if [ $non_zero_count -lt $expected_non_zero_count ]; then
-      echo "⏳ Expecting at least 3 non-zero data points"
+      echo "⏳ Expecting at least 4 non-zero data points"
       return 1
     fi
     return 0

storage/factory.go

@@ -83,14 +83,6 @@ type ArchiveFactory interface {
 	CreateArchiveSpanWriter() (spanstore.Writer, error)
 }
 
-var (
-	// ErrMetricStorageNotConfigured can be returned by the MetricsFactory when the metric storage is not configured.
-	ErrMetricStorageNotConfigured = errors.New("Metric storage not configured")
-
-	// ErrMetricStorageNotSupported can be returned by the MetricsFactory when the metric storage is not supported by the backend.
-	ErrMetricStorageNotSupported = errors.New("Metric storage not supported")
-)
-
 // MetricsFactory defines an interface for a factory that can create implementations of different metrics storage components.
 // Implementations are also encouraged to implement plugin.Configurable interface.
 //