fix: ad dmissing resampleConfig handling to scope select

ClusterCockpit · Oct 8, 2024 · ed991de · ed991de
1 parent 322e161
commit ed991de
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/web/frontend/src/job/Metric.svelte b/web/frontend/src/job/Metric.svelte
@@ -110,23 +110,24 @@
   const selectedMetrics = [metricName]
 
   $: if (selectedScope || pendingResolution) {
-    if (!selectedResolution) {
-      // Skips reactive data load on init
+
+    if (resampleConfig && !selectedResolution) {
+      // Skips reactive data load on init || Only if resampling is enabled
       selectedResolution = Number(pendingResolution)
 
     } else {
       if (selectedScope == "load-all") {
         selectedScopes = [...scopes, "socket", "core", "accelerator"]
       }
 
-      if (pendingResolution) {
+      if (resampleConfig && pendingResolution) {
         selectedResolution = Number(pendingResolution)
       }
 
       metricData = queryStore({
         client: client,
         query: subQuery,
-        variables: { dbid, selectedMetrics, selectedScopes, selectedResolution },
+        variables: { dbid, selectedMetrics, selectedScopes, selectedResolution: (resampleConfig ? selectedResolution : 0) },
         // Never user network-only: causes reactive load-loop!
       });