Loki/Prometheus: Improve facetting message in label browser (#40998)

* Chore: Improve Loki/Prometheus facetting message in label browser

* Chore: Remove uneeded console log and comment
This commit is contained in:
Devin Smith
2021-10-28 16:19:54 +02:00
committed by GitHub
parent 05f22e5c7d
commit 8869df2ffe
2 changed files with 2 additions and 6 deletions

View File

@@ -361,9 +361,7 @@ export class UnthemedLokiLabelBrowser extends React.Component<BrowserProps, Brow
return;
}
if (Object.keys(possibleLabels).length === 0) {
// Sometimes the backend does not return a valid set
console.error('No results for label combination, but should not occur.');
this.setState({ error: `Facetting failed for ${selector}` });
this.setState({ error: `Empty results, no matching label for ${selector}` });
return;
}
const labels: SelectableLabel[] = facetLabels(this.state.labels, possibleLabels, lastFacetted);

View File

@@ -420,9 +420,7 @@ export class UnthemedPrometheusMetricsBrowser extends React.Component<BrowserPro
return;
}
if (Object.keys(possibleLabels).length === 0) {
// Sometimes the backend does not return a valid set
console.error('No results for label combination, but should not occur.');
this.setState({ error: `Facetting failed for ${selector}` });
this.setState({ error: `Empty results, no matching label for ${selector}` });
return;
}
const labels: SelectableLabel[] = facetLabels(this.state.labels, possibleLabels, lastFacetted);