feat: test no metrics

.github/workflows/ci-e2e-no-metrics-tests.yml

@@ -17,7 +17,7 @@ jobs:
       - name: pull pre-built images
         run: sudo docker compose -f ci.docker-compose.yml pull
       - name: build and start proxy service and it's dependencies
-        run: METRIC_DATABASE_ENABLED=false sudo docker compose -f ci.docker-compose.yml up -d --build proxy redis
+        run: echo "METRIC_DATABASE_ENABLED=$METRIC_DATABASE_ENABLED" && METRIC_DATABASE_ENABLED=false sudo docker compose -f ci.docker-compose.yml up -d --build proxy redis
         env:
             METRIC_DATABASE_ENABLED: false
       - name: wait for proxy service to be running

ci.docker-compose.yml

@@ -22,6 +22,7 @@ services:
       dockerfile: ci.Dockerfile
     env_file: .env
     environment:
+      METRIC_DATABASE_ENABLED: "${METRIC_DATABASE_ENABLED:-true}"
       PROXY_HEIGHT_BASED_ROUTING_ENABLED: "true"
       PROXY_SHARDED_ROUTING_ENABLED: "true"
       # use public testnet as backend origin server to avoid having
@@ -32,7 +33,6 @@ services:
       # fake the shards by defining shards with existing backends
       PROXY_SHARD_BACKEND_HOST_URL_MAP: localhost:7777>10|https://evmrpcdata.internal.testnet.proxy.kava.io|20|https://evmrpcdata.internal.testnet.proxy.kava.io
       EVM_QUERY_SERVICE_URL: https://evmrpc.internal.testnet.proxy.kava.io
-      METRIC_DATABASE_ENABLED: "false"
     ports:
       - "${PROXY_HOST_PORT}:${PROXY_CONTAINER_PORT}"
       - "${TEST_UNCONFIGURED_PROXY_PORT}:${PROXY_CONTAINER_PORT}"

main_test.go

@@ -1217,6 +1217,10 @@ func cleanUpRedis(t *testing.T, redisClient *redis.Client) {
 }
 
 func cleanMetricsDb(t *testing.T, db database.PostgresClient) {
+	if shouldSkipMetrics() {
+		return
+	}
+
 	_, err := db.Exec("TRUNCATE proxied_request_metrics;")
 	require.NoError(t, err)
 }

routines/metric_partitioning_test.go

@@ -31,6 +31,10 @@ var (
 )
 
 func TestE2ETestMetricPartitioningRoutinePrefillsExpectedPartitionsAfterStartupDelay(t *testing.T) {
+	if shouldSkipMetrics() {
+		t.Skip("Skipping test because environment variable SKIP_METRICS is set to true")
+	}
+
 	// prepare
 	time.Sleep(time.Duration(MetricPartitioningRoutineDelayFirstRunSeconds) * time.Second)
 
@@ -93,3 +97,8 @@ func TestUnitTestpartitionsForPeriodReturnsExpectedNumPartitionsWhenPrefillPerio
 	assert.Nil(t, err)
 	assert.Equal(t, daysToPrefill, len(actualPartitionsForPeriod))
 }
+
+func shouldSkipMetrics() bool {
+	// Check if the environment variable SKIP_METRICS is set to "true"
+	return os.Getenv("SKIP_METRICS") == "true"
+}