test:e2e: Ensure QueryLogFile is set in spec

Author: philipgough

test/e2e/config_test.go

@@ -151,6 +151,7 @@ func TestClusterMonitorPrometheusK8Config(t *testing.T) {
 	data := fmt.Sprintf(`prometheusK8s:
   logLevel: debug
   retention: 10h
+  queryLogFile: /tmp/test.log
   tolerations:
     - operator: "Exists"
   externalLabels:
@@ -199,6 +200,10 @@ func TestClusterMonitorPrometheusK8Config(t *testing.T) {
 			name:      "assert remote write url value in set in CR",
 			assertion: assertRemoteWriteWasSet(f.Ns, crName, "https://test.remotewrite.com/api/write"),
 		},
+		{
+			name:      "assert query log file value is set and correct",
+			assertion: assertQueryLogValueEquals(f.Ns, crName, "/tmp/test.log"),
+		},
 		{
 			name:      "assert rule for Thanos sidecar exists",
 			assertion: f.AssertPrometheusRuleExists(thanosRule, f.Ns),
@@ -523,6 +528,7 @@ func TestUserWorkloadMonitorPrometheusK8Config(t *testing.T) {
   enforcedTargetLimit: 10
   logLevel: debug
   retention: 10h
+  queryLogFile: /tmp/test.log
   tolerations:
     - operator: "Exists"
   externalLabels:
@@ -577,6 +583,10 @@ func TestUserWorkloadMonitorPrometheusK8Config(t *testing.T) {
 			name:      "assert enforced target limit is configured",
 			assertion: assertEnforcedTargetLimit(10),
 		},
+		{
+			name:      "assert query log file value is set and correct",
+			assertion: assertQueryLogValueEquals(f.UserWorkloadMonitoringNs, crName, "/tmp/test.log"),
+		},
 	} {
 		t.Run(tc.name, tc.assertion)
 	}
@@ -813,3 +823,25 @@ func assertEnforcedTargetLimit(limit uint64) func(*testing.T) {
 		}
 	}
 }
+
+func assertQueryLogValueEquals(namespace, crName, value string) func(t *testing.T) {
+	return func(t *testing.T) {
+		err := framework.Poll(time.Second, time.Minute*5, func() error {
+			prom, err := f.MonitoringClient.Prometheuses(namespace).Get(context.Background(), crName, metav1.GetOptions{})
+			if err != nil {
+				t.Fatal("failed to get required prometheus cr", err)
+			}
+
+			if prom.Spec.QueryLogFile != value {
+				return fmt.Errorf(
+					"expected query log file value not found wanted '%s', got '%s'",
+					value, prom.Spec.QueryLogFile,
+				)
+			}
+			return nil
+		})
+		if err != nil {
+			t.Fatal(err)
+		}
+	}
+}