fix: check alert rule queries are all disabled if at least one query is set

pkg/query-service/rules/api_params.go

@@ -168,18 +168,49 @@ func isValidLabelValue(v string) bool {
 	return utf8.ValidString(v)
 }
 
+func isAllQueriesDisabled(compositeQuery *v3.CompositeQuery) bool {
+	if compositeQuery == nil {
+		return false
+	}
+	if compositeQuery.BuilderQueries == nil && compositeQuery.PromQueries == nil && compositeQuery.ClickHouseQueries == nil {
+		return false
+	}
+	for _, query := range compositeQuery.BuilderQueries {
+		if !query.Disabled {
+			return false
+		}
+	}
+	for _, query := range compositeQuery.PromQueries {
+		if !query.Disabled {
+			return false
+		}
+	}
+	for _, query := range compositeQuery.ClickHouseQueries {
+		if !query.Disabled {
+			return false
+		}
+	}
+	return true
+}
+
 func (r *PostableRule) Validate() error {
 
 	var errs []error
 
 	if r.RuleCondition == nil {
 		errs = append(errs, errors.Errorf("rule condition is required"))
+		// will get panic if we try to access CompositeQuery, so return here
+		return multierr.Combine(errs...)
 	} else {
 		if r.RuleCondition.CompositeQuery == nil {
 			errs = append(errs, errors.Errorf("composite metric query is required"))
 		}
 	}
 
+	if isAllQueriesDisabled(r.RuleCondition.CompositeQuery) {
+		errs = append(errs, errors.Errorf("all queries are disabled in rule condition"))
+	}
+
 	if r.RuleType == RuleTypeThreshold {
 		if r.RuleCondition.Target == nil {
 			errs = append(errs, errors.Errorf("rule condition missing the threshold"))

pkg/query-service/rules/api_params_test.go

@@ -0,0 +1,70 @@
+package rules
+
+import (
+	"testing"
+
+	v3 "go.signoz.io/signoz/pkg/query-service/model/v3"
+)
+
+func TestIsAllQueriesDisabled(t *testing.T) {
+	// Test case for all queries disabled
+	compositeQuery := &v3.CompositeQuery{
+		BuilderQueries: map[string]*v3.BuilderQuery{
+			"query1": {
+				Disabled: true,
+			},
+			"query2": {
+				Disabled: true,
+			},
+		},
+		PromQueries: map[string]*v3.PromQuery{
+			"query3": {
+				Disabled: true,
+			},
+			"query4": {
+				Disabled: true,
+			},
+		},
+		ClickHouseQueries: map[string]*v3.ClickHouseQuery{
+			"query5": {
+				Disabled: true,
+			},
+			"query6": {
+				Disabled: true,
+			},
+		},
+	}
+
+	testCases := []*v3.CompositeQuery{
+		compositeQuery,
+		nil,
+		&v3.CompositeQuery{},
+		&v3.CompositeQuery{
+			BuilderQueries: map[string]*v3.BuilderQuery{
+				"query1": {
+					Disabled: true,
+				},
+				"query2": {
+					Disabled: false,
+				},
+			},
+		},
+		&v3.CompositeQuery{
+			PromQueries: map[string]*v3.PromQuery{
+				"query3": {
+					Disabled: false,
+				},
+			},
+		},
+	}
+
+	expectedResult := []bool{true, false, false, false, false}
+
+	for index, compositeQuery := range testCases {
+		expected := expectedResult[index]
+		actual := isAllQueriesDisabled(compositeQuery)
+		if actual != expected {
+			t.Errorf("Expected %v, but got %v", expected, actual)
+		}
+	}
+}