ClusterCockpit/cc-backend
1
0
Fork 0
mirror of https://github.com/ClusterCockpit/cc-backend synced 2025-03-04 05:55:56 +01:00
Code Issues Packages Releases Activity

fix: fix svelte js race condition on metric selection change, see #335

Browse Source 
- only dispatch new data to statsTable on 'load-all'
This commit is contained in:
Christoph Kluge 2025-02-14 16:18:58 +01:00
parent f7a67c72bf
commit 1b043838ea
1 changed files with 7 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
web/frontend/src/job/Metric.svelte
View File

@ -148,16 +148,17 @@
zoomState = {...pendingZoomState} zoomState = {...pendingZoomState}
} }
// Set selected scope to min of returned scopes // On additional scope request
if (selectedScope == "load-all") { if (selectedScope == "load-all") {
selectedScope = minScope(scopes) // Push scope to statsTable (Needs to be in this case, else newly selected 'Metric.svelte' renders cause statsTable race condition)
nodeOnly = (selectedScope == "node") // "node" still only scope after load-all
}
const statsTableData = $metricData.data.singleUpdate.filter((x) => x.scope !== "node") const statsTableData = $metricData.data.singleUpdate.filter((x) => x.scope !== "node")
if (statsTableData.length > 0) { if (statsTableData.length > 0) {
dispatch("more-loaded", statsTableData); dispatch("more-loaded", statsTableData);
} }
// Set selected scope to min of returned scopes
selectedScope = minScope(scopes)
nodeOnly = (selectedScope == "node") // "node" still only scope after load-all
}
patternMatches = statsPattern.exec(selectedScope) patternMatches = statsPattern.exec(selectedScope)