feat: unset query timeouts so they are defined in the backend

Signed-off-by: Gabriel Bernal <gbernal@redhat.com>

Author: jgbernalp

web/src/components/Incidents/api.ts

@@ -8,6 +8,9 @@ import {
 import { getPrometheusBasePath, buildPrometheusUrl } from '../utils';
 import { PROMETHEUS_QUERY_INTERVAL_SECONDS } from './utils';
 
+// Disable client-side timeout (-1) to let the backend control query timeouts
+const NO_TIMEOUT = -1;
+
 const MAX_URL_LENGTH = 2048;
 
 /**
@@ -136,7 +139,7 @@ export const fetchDataForIncidentsAndAlerts = async (
       } as PrometheusResponse);
     }
 
-    return consoleFetchJSON(url);
+    return consoleFetchJSON(url, 'GET', {}, NO_TIMEOUT);
   });
 
   const responses = await Promise.all(promises);

web/src/components/console/utils/safe-fetch-hook.ts

@@ -1,6 +1,9 @@
 import { useEffect, useRef } from 'react';
 import { consoleFetchJSON } from '@openshift-console/dynamic-plugin-sdk';
 
+// Disable client-side timeout (-1) to let the backend control query timeouts
+const NO_TIMEOUT = -1;
+
 export const useSafeFetch = () => {
   const controller = useRef<AbortController>();
   useEffect(() => {
@@ -9,5 +12,5 @@ export const useSafeFetch = () => {
   }, []);
 
   return <T>(url: string): Promise<T> =>
-    consoleFetchJSON(url, 'get', { signal: controller.current.signal as AbortSignal });
+    consoleFetchJSON(url, 'GET', { signal: controller.current.signal as AbortSignal }, NO_TIMEOUT);
 };

web/src/components/fetch-alerts.tsx

@@ -1,5 +1,8 @@
 import { consoleFetchJSON, PrometheusRulesResponse } from '@openshift-console/dynamic-plugin-sdk';
 
+// Disable client-side timeout (-1) to let the backend control query timeouts
+const NO_TIMEOUT = -1;
+
 // Merges Prometheus monitoring alerts with external sources
 export const fetchAlerts = async (
   prometheusURL: string,
@@ -10,7 +13,7 @@ export const fetchAlerts = async (
   namespace?: string,
 ): Promise<PrometheusRulesResponse> => {
   if (!externalAlertsFetch || externalAlertsFetch.length === 0) {
-    return consoleFetchJSON(prometheusURL);
+    return consoleFetchJSON(prometheusURL, 'GET', {}, NO_TIMEOUT);
   }
 
   const resolvedExternalAlertsSources = externalAlertsFetch.map((extensionProperties) => ({
@@ -22,7 +25,7 @@ export const fetchAlerts = async (
 
   try {
     const groups = await Promise.allSettled([
-      consoleFetchJSON(prometheusURL),
+      consoleFetchJSON(prometheusURL, 'GET', {}, NO_TIMEOUT),
       ...resolvedExternalAlertsSources.map((source) => source.fetch(namespace)),
     ]).then((results) =>
       results
@@ -39,6 +42,6 @@ export const fetchAlerts = async (
 
     return { data: { groups }, status: 'success' };
   } catch {
-    return consoleFetchJSON(prometheusURL);
+    return consoleFetchJSON(prometheusURL, 'GET', {}, NO_TIMEOUT);
   }
 };

web/src/components/proxied-fetch.ts

@@ -43,7 +43,8 @@ export const proxiedFetch = <T>(url: string, init?: RequestInitWithTimeout): Pro
     return response.json();
   });
 
-  const timeout = init?.timeout ?? 30 * 1000;
+  // Disable client-side timeout by default (-1) to let the backend control query timeouts
+  const timeout = init?.timeout ?? -1;
 
   if (timeout <= 0) {
     return fetchPromise;